Open Bug 1328056 Opened 8 years ago Updated 2 years ago

Sound indicator blinks on Youtube when I switch to tab

Categories

(Core :: Audio/Video: Playback, defect, P3)

defect

Tracking

()

People

(Reporter: arni2033, Unassigned)

References

Details

(Keywords: regression)

>>> My Info: Win7_64, Nightly 49, 32bit, ID 20160526082509 STR_1: 0.(new versions) Open about:config, set "browser.tabs.delayHidingAudioPlayingIconMS" to 0 1. Open https://www.youtube.com/watch?v=mD3v1B_aXw0 2. Open http://example.org in a new selected tab 3. Wait more than 10 seconds 4. Click on the tab from Step 1 AR: Sound indicator in the tab from Step 1 briefly disappears and appears again ER: Sound indicator should be constantly displayed (assuming you performed STR_1 in less than 141s) This is regression from bug 1224973. Regression range: > https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=13e4cb81d4eb1c4996daf34c68301ca8a8ab0183&tochange=2fe467985f04512869ca7daa1a2db286642ce127 Note: See bug 1239372 comment 18, bug 1239372 comment 20: > > IMMO, the platform audio notification should be reflected the audio playing > > state precisely. If we want to do some delay, it might be implemented in > > front-end code for each different platform. > > > > Eg. The media control interface on Fennec is also depend on the platform > > audio notification to decide when it should be show/hide. And I don't want > > to delay the hiding time for that. > > Yes, you are right. We shouldn't lie about when the audio stops. The delay > should remain a front-end feature.@ Dan Glastonbury :kamidphish (PTO 19-Dec-16 to 13-Jan-17): It seems that this is a regresion caused by your change. Please have a look.
No longer blocks: 1277113
Component: Untriaged → Audio/Video: Playback
Product: Firefox → Core
Jared - this doesn't look like a platform issue. Can you shed some light on this?
Flags: needinfo?(jaws)
There is discussion about this in bug 1337492. It appears related to how SVG graphics are loaded and painted.
Flags: needinfo?(jaws)
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.