Bug 120553 – Volume slider value should be 0 when audio is muted

• Status: RESOLVED
• Resolution: FIXED
• Component: New Bugs

[Xabier Rodríguez Calvar]

Volume slider value should be 0 when audio is muted

[Xabier Rodríguez Calvar]

Created attachment 210167 [details]
Patch

Set the slider value to 0 when audio is muted.

[Eric Carlson]

Comment on attachment 210167 [details]
Patch

This looks OK, but it should be testable.

[Xabier Rodríguez Calvar]

(In reply to comment #2)
> (From update of attachment 210167 [details])
> This looks OK, but it should be testable.

I'll write a test as soon as I can. Thanks!

[Xabier Rodríguez Calvar]

Created attachment 210213 [details]
Patch

Added test.

[Build Bot]

Comment on attachment 210213 [details]
Patch

Attachment 210213 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.appspot.com/results/1679229

New failing tests:
media/video-volume-slider.html
media/volume-bar-empty-when-muted.html

[Build Bot]

Created attachment 210218 [details]
Archive of layout-test-results from webkit-ews-08 for mac-mountainlion

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: webkit-ews-08  Port: mac-mountainlion  Platform: Mac OS X 10.8.4

[Eric Carlson]

Comment on attachment 210213 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=210213&action=review

> Source/WebCore/html/shadow/MediaControls.cpp:111
> -            m_volumeSlider->setVolume(m_mediaController->volume());
> +            setSliderVolume();

It looks like you need to make this same change to MediaControlsApple::reset. I assume this is what makes the new test fail on the Mac bots.

> LayoutTests/media/video-volume-slider.html:42
> +    volume slider is implemented and mouse over the right bottom corner of<br>

Nice catch. Don't the results for this test also need to be updated?

[Build Bot]

Comment on attachment 210213 [details]
Patch

Attachment 210213 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.appspot.com/results/1642400

New failing tests:
media/video-volume-slider.html
media/volume-bar-empty-when-muted.html

[Build Bot]

Created attachment 210220 [details]
Archive of layout-test-results from webkit-ews-03 for mac-mountainlion

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: webkit-ews-03  Port: mac-mountainlion  Platform: Mac OS X 10.8.4

[Build Bot]

Comment on attachment 210213 [details]
Patch

Attachment 210213 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.appspot.com/results/1603458

New failing tests:
editing/unsupported-content/table-delete-003.html
editing/unsupported-content/list-type-before.html
media/video-volume-slider.html
editing/unsupported-content/table-type-before.html
editing/unsupported-content/list-delete-003.html
media/volume-bar-empty-when-muted.html
editing/unsupported-content/list-delete-001.html
editing/unsupported-content/list-type-after.html
editing/unsupported-content/table-delete-002.html
editing/unsupported-content/table-type-after.html
editing/unsupported-content/table-delete-001.html

[Build Bot]

Created attachment 210222 [details]
Archive of layout-test-results from webkit-ews-09 for mac-mountainlion-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: webkit-ews-09  Port: mac-mountainlion-wk2  Platform: Mac OS X 10.8.4

[Xabier Rodríguez Calvar]

(In reply to comment #7)
> > LayoutTests/media/video-volume-slider.html:42
> > +    volume slider is implemented and mouse over the right bottom corner of<br>
> 
> Nice catch. Don't the results for this test also need to be updated?

Yes, they do. I think I'll file another bug and fix it unreviewed. I'll CC you anyway.

[Xabier Rodríguez Calvar]

Created attachment 210233 [details]
Patch

Fixed Apple problem, updated Blackberry and removed the typo that shall be corrected in another bug.

[Eric Carlson]

Comment on attachment 210233 [details]
Patch

Thanks!

[WebKit Commit Bot]

Comment on attachment 210233 [details]
Patch

Clearing flags on attachment: 210233

Committed r154944: <http://trac.webkit.org/changeset/154944>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.