Bug 122049 – REGRESSION(r156546): media/video-no-audio.html broken

• Status: RESOLVED
• Resolution: CONFIGURATION CHANGED
• Component: Media

[Jer Noble]

REGRESSION(r156546): media/video-no-audio.html broken

[Jer Noble]

Created attachment 212877 [details]
Patch

[Jer Noble]

Tests will continue to fail until the fix for bug #122021 is landed.

[Jer Noble]

Rather, till bug #122043 is landed.

[Build Bot]

Comment on attachment 212877 [details]
Patch

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

New failing tests:
media/controls-strict.html
media/controls-styling-strict.html
media/audio-controls-rendering.html
media/click-volume-bar-not-pausing.html
fast/hidpi/video-controls-in-hidpi.html
media/controls-without-preload.html
media/media-controls-clone.html
fast/layers/video-layer.html
media/controls-after-reload.html

[Build Bot]

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

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

[Build Bot]

Comment on attachment 212877 [details]
Patch

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

New failing tests:
media/controls-strict.html
media/controls-styling-strict.html
media/audio-controls-rendering.html
media/click-volume-bar-not-pausing.html
fast/hidpi/video-controls-in-hidpi.html
media/controls-without-preload.html
media/media-controls-clone.html
fast/layers/video-layer.html
media/controls-after-reload.html

[Build Bot]

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

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

[Build Bot]

Comment on attachment 212877 [details]
Patch

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

New failing tests:
media/controls-strict.html
media/controls-styling-strict.html
media/audio-controls-rendering.html
media/click-volume-bar-not-pausing.html
fast/hidpi/video-controls-in-hidpi.html
media/controls-without-preload.html
media/media-controls-clone.html
fast/layers/video-layer.html
media/controls-after-reload.html

[Build Bot]

Created attachment 212881 [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.5

[Eric Carlson]

Comment on attachment 212877 [details]
Patch

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

> Source/WebCore/Modules/mediacontrols/mediaControlsApple.js:474
> +    handleAudioTrackChange: function(event)
> +    {
> +        this.updateHasAudio();
> +    },
> +
> +    handleAudioTrackAdd: function(event)
> +    {
> +        this.updateHasAudio();
> +    },
> +
> +    handleAudioTrackRemove: function(event)
> +    {
> +        this.updateHasAudio();
> +    },

Why not have a singe event handler for all three events, so each of these does the same thing?

[Jer Noble]

(In reply to comment #10)
> (From update of attachment 212877 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=212877&action=review
> 
> > Source/WebCore/Modules/mediacontrols/mediaControlsApple.js:474
> > +    handleAudioTrackChange: function(event)
> > +    {
> > +        this.updateHasAudio();
> > +    },
> > +
> > +    handleAudioTrackAdd: function(event)
> > +    {
> > +        this.updateHasAudio();
> > +    },
> > +
> > +    handleAudioTrackRemove: function(event)
> > +    {
> > +        this.updateHasAudio();
> > +    },
> 
> Why not have a singe event handler for all three events, so each of these does the same thing?

Sure; that just means registering with updateHasAudio, rather than each individual function.

[Jer Noble]

Committed r156656: <http://trac.webkit.org/changeset/156656>

[Alexey Proskuryakov]

This broke several other tests, and Jer is not on IRC, so I'm rolling out.

[WebKit Commit Bot]

Re-opened since this is blocked by bug 122111

[Alexey Proskuryakov]

Rolled out in <http://trac.webkit.org/changeset/156661>.

Broken tests: <http://build.webkit.org/results/Apple%20MountainLion%20Release%20WK1%20(Tests)/r156656%20(13365)/results.html>. Not sure about the one that timed out - it never failed before, but it's not clear how it could possibly be affected.

[Radar WebKit Bug Importer]

<rdar://problem/62727154>

[Ahmad Saleem]

This test seems to be removed in following commit: https://github.com/WebKit/WebKit/commit/a39bd42ad87d82a3bea0fa6eac107376a973cf7c

Nothing to do here.