Closed Bug 962560 Opened 7 years ago Closed 5 years ago
Audio volume UI resets to full volume after toggling the screen size of a video
User agents:  Mozilla/5.0 (Windows NT 6.1; WOW64; rv:28.0) Gecko/20100101 Firefox/28.0  Mozilla/5.0 (Windows NT 6.3; WOW64; rv:28.0) Gecko/20100101 Firefox/28.0  Mozilla/5.0 (X11; Linux i686; rv:28.0) Gecko/20100101 Firefox/28.0  Mozilla/5.0 (X11; Linux x86_64; rv:28.0) Gecko/20100101 Firefox/28.0  Mozilla/5.0 (Machintosh; Intel Mac OS X 10.9; rv:28.0) Gecko/20100101 Firefox/28.0 Reproducible on: Windows 7 64-bit , Windows 8 64-bit , Ubuntu 12.04 32-bit , Ubuntu 13.10 64-bit  and Mac OS X 10.9  platforms, using: - the latest Aurora (Build ID: 20140122004004) - the latest Nightly (Build ID: 20140122030521) Steps to reproduce: 1. Open one of the VP9-encoded video samples attached to Bug 949525 * e.g. https://bug949525.bugzilla.mozilla.org/attachment.cgi?id=8346897 2. Adjust the audio volume of the video using the controls bar 3. Change the video's window size to full screen using the associated button 4. Change the video's screen size back to the default one Expected results: The audio volume remains as configured at Step 2 no matter how many times the user changes the video's screen size. Actual results: In terms of actual sound, the audio volume of the video remains the same, but the UI from the controls bar displays full volume instead of actual one.
I'm guessing this isn't VP9 specific, but a videocontrols issue.
Component: Video/Audio → Video/Audio Controls
Product: Core → Toolkit
Assignee: nobody → jaws
Status: NEW → ASSIGNED
Attachment #8749800 - Flags: review?(gijskruitbosch+bugs)
Comment on attachment 8749800 [details] [diff] [review] Patch Review of attachment 8749800 [details] [diff] [review]: ----------------------------------------------------------------- Nice find, thanks! ::: toolkit/content/widgets/videocontrols.xml @@ +412,5 @@ > this.controlBar.hidden = true; > this.adjustControlSize(); > + > + // Must compute _volumeControlWidth first since the > + // slider implementation requires it. very-pedantic-nit: Because this comment is at the second thing out of the 2 order-dependent things we're doing, I think it makes more sense if it says something like: "Can only update the volume controls once we've computed _volumeControlWidth, since the volume slider implementation depends on it." Or something.
Attachment #8749800 - Flags: review?(gijskruitbosch+bugs) → review+
Still doesn't apply cleanly to inbound.
Duplicate of this bug: 1223413
Duplicate of this bug: 1246566
You need to log in before you can comment on or make changes to this bug.