Unreviewed, rolling out r156656.

http://trac.webkit.org/changeset/156656
https://bugs.webkit.org/show_bug.cgi?id=122111

Broke several media tests (Requested by ap on #webkit).

Source/WebCore:

* Modules/mediacontrols/mediaControlsApple.js:
(Controller):
(Controller.prototype.addVideoListeners):
(Controller.prototype.removeVideoListeners):

LayoutTests:

* platform/mac/TestExpectations:
* platform/mac/media/video-no-audio-expected.txt:

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@156661 268f45cc-cd09-0410-ab3c-d52691b4dbfc
parent 60edc2c7
2013-09-30 Commit Queue <commit-queue@webkit.org>
Unreviewed, rolling out r156656.
http://trac.webkit.org/changeset/156656
https://bugs.webkit.org/show_bug.cgi?id=122111
Broke several media tests (Requested by ap on #webkit).
* platform/mac/TestExpectations:
* platform/mac/media/video-no-audio-expected.txt:
2013-09-30 Krzysztof Czech <k.czech@samsung.com>
[EFL] Changed expectation for accessibility/file-upload-button-stringvalue.html after r156532
......@@ -1358,6 +1358,7 @@ webkit.org/b/122021 media/video-controls-toggling.html
webkit.org/b/122021 media/video-controls-visible-exiting-fullscreen.html
webkit.org/b/122021 media/video-display-toggle.html
webkit.org/b/122021 media/video-empty-source.html
webkit.org/b/122021 media/video-no-audio.html
webkit.org/b/122021 media/video-rtl.html
webkit.org/b/122021 media/video-volume-slider.html
webkit.org/b/122021 media/volume-bar-empty-when-muted.html
......
......@@ -14,28 +14,22 @@ layer at (8,42) size 352x288
RenderFlexibleBox {DIV} at (0,0) size 352x288
layer at (8,305) size 352x25
RenderFlexibleBox {DIV} at (0,263) size 352x25
layer at (14,309) size 16x16
RenderButton {BUTTON} at (6,4) size 16x16 [color=#FFFFFF]
layer at (40,310) size 16x16
RenderButton {BUTTON} at (32,5) size 16x16 [color=#FFFFFF]
layer at (57,312) size 273x11
RenderFlexibleBox {DIV} at (49,7) size 273x11
RenderButton {INPUT} at (6,3) size 18x18
RenderButton {INPUT} at (32,4) size 16x16
RenderFlexibleBox {DIV} at (49,5) size 273x15
RenderSlider {INPUT} at (45,2) size 183x13
RenderFlexibleBox {DIV} at (0,0) size 183x13
RenderBlock {DIV} at (0,2) size 183x8
RenderBlock {DIV} at (0,0) size 10x8
RenderButton {INPUT} at (329,4) size 16x16
RenderBlock {DIV} at (322,12) size 0x0
layer at (57,312) size 45x11
RenderFlexibleBox {DIV} at (0,0) size 45x11 [color=#FFFFFF]
RenderFlexibleBox {DIV} at (0,2) size 45x11 [color=#FFFFFF]
RenderBlock (anonymous) at (9,0) size 26x11
RenderText {#text} at (0,0) size 26x11
text run at (0,0) width 26: "00:00"
layer at (285,312) size 45x11
RenderFlexibleBox {DIV} at (228,0) size 45x11 [color=#FFFFFF]
RenderFlexibleBox {DIV} at (228,2) size 45x11 [color=#FFFFFF]
RenderBlock (anonymous) at (7,0) size 31x11
RenderText {#text} at (0,0) size 31x11
text run at (0,0) width 31: "-00:09"
layer at (104,313) size 179x9
RenderFlexibleBox {DIV} at (47,1) size 179x9
RenderSlider {INPUT} at (0,0) size 179x9 [color=#909090] [bgcolor=#000000]
RenderFlexibleBox {DIV} at (0,0) size 179x9
RenderBlock {DIV} at (0,1) size 179x6
layer at (104,314) size 6x6
RenderBlock {DIV} at (0,0) size 6x6 [bgcolor=#FFFFFF]
layer at (337,310) size 16x16
RenderButton {BUTTON} at (329,5) size 16x16 [color=#FFFFFF]
2013-09-30 Commit Queue <commit-queue@webkit.org>
Unreviewed, rolling out r156656.
http://trac.webkit.org/changeset/156656
https://bugs.webkit.org/show_bug.cgi?id=122111
Broke several media tests (Requested by ap on #webkit).
* Modules/mediacontrols/mediaControlsApple.js:
(Controller):
(Controller.prototype.addVideoListeners):
(Controller.prototype.removeVideoListeners):
2013-09-29 Sam Weinig <sam@webkit.org>
Convert a bunch of OwnPtrs on Page to std::unique_ptrs
......@@ -25,7 +25,6 @@ function Controller(root, video, host)
this.updateCaptionButton();
this.updateCaptionContainer();
this.updateVolume();
this.updateHasAudio();
};
/* Enums */
......@@ -113,11 +112,6 @@ Controller.prototype = {
this.listenFor(this.video.textTracks, 'addtrack', this.handleTextTrackAdd);
this.listenFor(this.video.textTracks, 'removetrack', this.handleTextTrackRemove);
/* audio tracks */
this.listenFor(this.video.audioTracks, 'change', this.updateHasAudio);
this.listenFor(this.video.audioTracks, 'addtrack', this.updateHasAudio);
this.listenFor(this.video.audioTracks, 'removetrack', this.updateHasAudio);
/* controls attribute */
this.controlsObserver = new MutationObserver(this.handleControlsChange.bind(this));
this.controlsObserver.observe(this.video, { attributes: true, attributeFilter: ['controls'] });
......@@ -134,11 +128,6 @@ Controller.prototype = {
this.stopListeningFor(this.video.textTracks, 'addtrack', this.handleTextTrackAdd);
this.stopListeningFor(this.video.textTracks, 'removetrack', this.handleTextTrackRemove);
/* audio tracks */
this.stopListeningFor(this.video.audioTracks, 'change', this.updateHasAudio);
this.stopListeningFor(this.video.audioTracks, 'addtrack', this.updateHasAudio);
this.stopListeningFor(this.video.audioTracks, 'removetrack', this.updateHasAudio);
/* controls attribute */
this.controlsObserver.disconnect();
delete(this.controlsObserver);
......@@ -925,14 +914,6 @@ Controller.prototype = {
delete this.captionMenuItems;
},
updateHasAudio: function()
{
if (this.video.audioTracks.length)
this.controls.muteBox.classList.remove(this.ClassNames.hidden);
else
this.controls.muteBox.classList.add(this.ClassNames.hidden);
},
updateVolume: function()
{
if (this.video.muted || !this.video.volume) {
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment