2010-05-21 Jer Noble <jer.noble@apple.com>
authorjer.noble@apple.com <jer.noble@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 21 May 2010 22:19:23 +0000 (22:19 +0000)
committerjer.noble@apple.com <jer.noble@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 21 May 2010 22:19:23 +0000 (22:19 +0000)
        Reviewed by Anders Carlsson.

        Disable full-screen video on Windows
        https://bugs.webkit.org/show_bug.cgi?id=39506
        rdar://problem/8012516

        Disable full-screen video through MediaPlayerPrivateVisualContext since
        we no longer use GWorlds and QuickTime's built in full-screen support.

        Media layout tests updated with new expected results.

        * platform/graphics/win/MediaPlayerPrivateQuickTimeVisualContext.cpp:
        (WebCore::MediaPlayerPrivateQuickTimeVisualContext::supportsFullscreen):
2010-05-21  Jer Noble  <jer.noble@apple.com>

        Reviewed by Anders Carlsson.

        Disable full-screen video on Windows
        https://bugs.webkit.org/show_bug.cgi?id=39506
        rdar://problem/8012516

        Disable full-screen video through MediaPlayerPrivateVisualContext since
        we no longer use GWorlds and QuickTime's built in full-screen support.

        Media layout tests updated with new expected results.

        * platform/win/media/controls-after-reload-expected.txt:
        * platform/win/media/controls-drag-timebar-expected.txt:
        * platform/win/media/controls-strict-expected.txt:
        * platform/win/media/controls-styling-expected.txt:
        * platform/win/media/video-controls-rendering-expected.txt:
        * platform/win/media/video-display-toggle-expected.txt:
        * platform/win/media/video-no-audio-expected.txt:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@59966 268f45cc-cd09-0410-ab3c-d52691b4dbfc

LayoutTests/ChangeLog
LayoutTests/platform/win/media/controls-after-reload-expected.txt
LayoutTests/platform/win/media/controls-drag-timebar-expected.txt
LayoutTests/platform/win/media/controls-strict-expected.txt
LayoutTests/platform/win/media/controls-styling-expected.txt
LayoutTests/platform/win/media/video-controls-rendering-expected.txt
LayoutTests/platform/win/media/video-display-toggle-expected.txt
LayoutTests/platform/win/media/video-no-audio-expected.txt
WebCore/ChangeLog
WebCore/platform/graphics/win/MediaPlayerPrivateQuickTimeVisualContext.cpp

index bb3f744fba1e4ddab2ff078b1a711c2ef5e10853..9305bd37a345e6ac0283065f36941d47b62c24a1 100644 (file)
@@ -1,3 +1,24 @@
+2010-05-21  Jer Noble  <jer.noble@apple.com>
+
+        Reviewed by Anders Carlsson.
+
+        Disable full-screen video on Windows
+        https://bugs.webkit.org/show_bug.cgi?id=39506
+        rdar://problem/8012516
+
+        Disable full-screen video through MediaPlayerPrivateVisualContext since
+        we no longer use GWorlds and QuickTime's built in full-screen support.
+        
+        Media layout tests updated with new expected results.        
+
+        * platform/win/media/controls-after-reload-expected.txt:
+        * platform/win/media/controls-drag-timebar-expected.txt:
+        * platform/win/media/controls-strict-expected.txt:
+        * platform/win/media/controls-styling-expected.txt:
+        * platform/win/media/video-controls-rendering-expected.txt:
+        * platform/win/media/video-display-toggle-expected.txt:
+        * platform/win/media/video-no-audio-expected.txt:
+
 2010-05-21  Simon Fraser  <simon.fraser@apple.com>
 
         Reviewed by Dan Bernstein.
index a433cf63a01ffeaf92d009096d6e7c57a7593c54..923fac32fcfca52a789b8f59d25ead38a018472c 100644 (file)
@@ -17,10 +17,9 @@ layer at (8,42) size 320x240
 layer at (8,266) size 320x16
   RenderFlexibleBox (positioned) {DIV} at (0,224) size 320x16
     RenderButton {INPUT} at (0,0) size 16x16
-    RenderFlexibleBox {DIV} at (16,0) size 240x16
-      RenderSlider {INPUT} at (0,0) size 240x16
+    RenderFlexibleBox {DIV} at (16,0) size 256x16
+      RenderSlider {INPUT} at (0,0) size 256x16
         RenderBlock {DIV} at (2,1) size 13x14
-    RenderButton {INPUT} at (256,0) size 16x16
     RenderButton {INPUT} at (272,0) size 16x16
     RenderButton {INPUT} at (288,0) size 16x16
     RenderButton {INPUT} at (304,0) size 16x16
index b7dbe1d4751b66bea71d69f71f11c1143c4017cf..323108508fe1416a7464f0f1f4dd3c2b3b72bf94 100644 (file)
@@ -4,10 +4,10 @@ RUN(video.autoplay = true)
 RUN(video.src = 'content/test.mp4')
 EVENT(playing)
 EVENT(seeked)
-Time: 2.3
+Time: 2.1
 EVENT(seeked)
-Time: 2.8
+Time: 2.6
 EVENT(seeked)
-Time: 3.4
+Time: 3.1
 END OF TEST
 
index 45db871420d6506f0dcbc727cc7abc4ef9f55c6e..5296d5593e54bb5c333c2ffdf41b5c5f2a9ee1bc 100644 (file)
@@ -17,10 +17,9 @@ layer at (8,50) size 320x240
 layer at (8,274) size 320x16
   RenderFlexibleBox (positioned) {DIV} at (0,224) size 320x16
     RenderButton {INPUT} at (0,0) size 16x16
-    RenderFlexibleBox {DIV} at (16,0) size 240x16
-      RenderSlider {INPUT} at (0,0) size 240x16
+    RenderFlexibleBox {DIV} at (16,0) size 256x16
+      RenderSlider {INPUT} at (0,0) size 256x16
         RenderBlock {DIV} at (2,1) size 13x14
-    RenderButton {INPUT} at (256,0) size 16x16
     RenderButton {INPUT} at (272,0) size 16x16
     RenderButton {INPUT} at (288,0) size 16x16
     RenderButton {INPUT} at (304,0) size 16x16
index 507123f4ef4215fd5087bec23be03b07450e49b8..fd24373b502566d41fe5995583a8a2af8f12f346 100644 (file)
@@ -21,10 +21,9 @@ layer at (18,42) size 320x240
 layer at (18,266) size 320x16
   RenderFlexibleBox (positioned) {DIV} at (0,224) size 320x16
     RenderButton {INPUT} at (0,0) size 16x16
-    RenderFlexibleBox {DIV} at (16,0) size 240x16
-      RenderSlider {INPUT} at (0,0) size 240x16
+    RenderFlexibleBox {DIV} at (16,0) size 256x16
+      RenderSlider {INPUT} at (0,0) size 256x16
         RenderBlock {DIV} at (2,1) size 13x14
-    RenderButton {INPUT} at (256,0) size 16x16
     RenderButton {INPUT} at (272,0) size 16x16
     RenderButton {INPUT} at (288,0) size 16x16
     RenderButton {INPUT} at (304,0) size 16x16
@@ -33,10 +32,9 @@ layer at (8,282) size 320x240
 layer at (8,506) size 320x16
   RenderFlexibleBox (positioned) {DIV} at (0,224) size 320x16
     RenderButton {INPUT} at (0,0) size 16x16
-    RenderFlexibleBox {DIV} at (16,0) size 240x16
-      RenderSlider {INPUT} at (0,0) size 240x16
+    RenderFlexibleBox {DIV} at (16,0) size 256x16
+      RenderSlider {INPUT} at (0,0) size 256x16
         RenderBlock {DIV} at (2,1) size 13x14
-    RenderButton {INPUT} at (256,0) size 16x16
     RenderButton {INPUT} at (272,0) size 16x16
     RenderButton {INPUT} at (288,0) size 16x16
     RenderButton {INPUT} at (304,0) size 16x16
index 791807ed69de920033fe6f6027a68dacf6404419..fff11e122c248c748c42b7c6a9f4e09e9507b92c 100644 (file)
@@ -20,10 +20,9 @@ layer at (8,42) size 320x240
 layer at (8,266) size 320x16
   RenderFlexibleBox (positioned) {DIV} at (0,224) size 320x16
     RenderButton {INPUT} at (0,0) size 16x16
-    RenderFlexibleBox {DIV} at (16,0) size 240x16
-      RenderSlider {INPUT} at (0,0) size 240x16
+    RenderFlexibleBox {DIV} at (16,0) size 256x16
+      RenderSlider {INPUT} at (0,0) size 256x16
         RenderBlock {DIV} at (2,1) size 13x14
-    RenderButton {INPUT} at (256,0) size 16x16
     RenderButton {INPUT} at (272,0) size 16x16
     RenderButton {INPUT} at (288,0) size 16x16
     RenderButton {INPUT} at (304,0) size 16x16
@@ -32,10 +31,9 @@ layer at (8,282) size 320x240
 layer at (8,506) size 320x16
   RenderFlexibleBox (positioned) {DIV} at (0,224) size 320x16
     RenderButton {INPUT} at (0,0) size 16x16
-    RenderFlexibleBox {DIV} at (16,0) size 240x16
-      RenderSlider {INPUT} at (0,0) size 240x16
+    RenderFlexibleBox {DIV} at (16,0) size 256x16
+      RenderSlider {INPUT} at (0,0) size 256x16
         RenderBlock {DIV} at (2,1) size 13x14
-    RenderButton {INPUT} at (256,0) size 16x16
     RenderButton {INPUT} at (272,0) size 16x16
     RenderButton {INPUT} at (288,0) size 16x16
     RenderButton {INPUT} at (304,0) size 16x16
@@ -46,10 +44,9 @@ layer at (8,522) size 320x240
 layer at (8,746) size 320x16
   RenderFlexibleBox (positioned) {DIV} at (0,224) size 320x16
     RenderButton {INPUT} at (0,0) size 16x16
-    RenderFlexibleBox {DIV} at (16,0) size 240x16
-      RenderSlider {INPUT} at (0,0) size 240x16
+    RenderFlexibleBox {DIV} at (16,0) size 256x16
+      RenderSlider {INPUT} at (0,0) size 256x16
         RenderBlock {DIV} at (2,1) size 13x14
-    RenderButton {INPUT} at (256,0) size 16x16
     RenderButton {INPUT} at (272,0) size 16x16
     RenderButton {INPUT} at (288,0) size 16x16
     RenderButton {INPUT} at (304,0) size 16x16
index a4f8539d16d3c74ac6a98b1cc00b7ae93bb70bed..a9baa000fa65b1b18018f6556dada257995fc4a1 100644 (file)
@@ -16,10 +16,9 @@ layer at (8,26) size 320x240
 layer at (8,250) size 320x16
   RenderFlexibleBox (positioned) {DIV} at (0,224) size 320x16
     RenderButton {INPUT} at (0,0) size 16x16
-    RenderFlexibleBox {DIV} at (16,0) size 240x16
-      RenderSlider {INPUT} at (0,0) size 240x16
+    RenderFlexibleBox {DIV} at (16,0) size 256x16
+      RenderSlider {INPUT} at (0,0) size 256x16
         RenderBlock {DIV} at (2,1) size 13x14
-    RenderButton {INPUT} at (256,0) size 16x16
     RenderButton {INPUT} at (272,0) size 16x16
     RenderButton {INPUT} at (288,0) size 16x16
     RenderButton {INPUT} at (304,0) size 16x16
index 099b4642b8ba2e5fc092bb3352b93e213f3d9aee..1de7c85e527f84ed257def2cb5280a957e36f489 100644 (file)
@@ -17,9 +17,8 @@ layer at (8,42) size 352x288
 layer at (8,314) size 352x16
   RenderFlexibleBox (positioned) {DIV} at (0,272) size 352x16
     RenderButton {INPUT} at (0,0) size 16x16
-    RenderFlexibleBox {DIV} at (16,0) size 288x16
-      RenderSlider {INPUT} at (0,0) size 288x16
+    RenderFlexibleBox {DIV} at (16,0) size 304x16
+      RenderSlider {INPUT} at (0,0) size 304x16
         RenderBlock {DIV} at (2,1) size 13x14
-    RenderButton {INPUT} at (304,0) size 16x16
     RenderButton {INPUT} at (320,0) size 16x16
     RenderButton {INPUT} at (336,0) size 16x16
index 4e4fde85a164a56cbd1185453a88df5f216570a3..42cf4af230adda253eab580d4c7e10f51c30ed93 100644 (file)
@@ -1,3 +1,19 @@
+2010-05-21  Jer Noble  <jer.noble@apple.com>
+
+        Reviewed by Anders Carlsson.
+
+        Disable full-screen video on Windows
+        https://bugs.webkit.org/show_bug.cgi?id=39506
+        rdar://problem/8012516
+        
+        Disable full-screen video through MediaPlayerPrivateVisualContext since
+        we no longer use GWorlds and QuickTime's built in full-screen support.
+        
+        Media layout tests updated with new expected results.        
+        
+        * platform/graphics/win/MediaPlayerPrivateQuickTimeVisualContext.cpp:
+        (WebCore::MediaPlayerPrivateQuickTimeVisualContext::supportsFullscreen):
+
 2010-05-21  Simon Fraser  <simon.fraser@apple.com>
 
         Reviewed by Dan Bernstein.
index 6782c5aac64ca7e7e916dfe4eebe3cf3ada771e1..e1d457f13a3ec5aed4ad7332fdc06a6fa32fbefd 100644 (file)
@@ -150,7 +150,7 @@ MediaPlayerPrivateQuickTimeVisualContext::~MediaPlayerPrivateQuickTimeVisualCont
 
 bool MediaPlayerPrivateQuickTimeVisualContext::supportsFullscreen() const
 {
-    return true;
+    return false;
 }
 
 PlatformMedia MediaPlayerPrivateQuickTimeVisualContext::platformMedia() const