2009-12-12 Eric Carlson <eric.carlson@apple.com>
authoreric.carlson@apple.com <eric.carlson@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 13 Dec 2009 03:28:42 +0000 (03:28 +0000)
committereric.carlson@apple.com <eric.carlson@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 13 Dec 2009 03:28:42 +0000 (03:28 +0000)
        Reviewed by Dan Bernstein.

        <rdar://problem/6293979> Adopt shared control drawing for <video> controls on Leopard and Windows

        Use MediaControllerThemeQuickTime on Leopard.

        * rendering/RenderThemeMac.mm:
        (WebCore::mediaControllerTheme):

2009-12-12  Eric Carlson  <eric.carlson@apple.com>

        Reviewed by Dan Bernstein.

        <rdar://problem/6293979> Adopt shared control drawing for <video> controls on Leopard and Windows

        Regenerate layout and pixel test results for the new UI.

        * platform/mac-leopard/Skipped:
        * platform/mac-leopard/media/controls-after-reload-expected.txt:
        * platform/mac-leopard/media/controls-drag-timebar-expected.txt:
        * platform/mac-leopard/media/controls-strict-expected.txt:
        * platform/mac-leopard/media/controls-styling-expected.txt:
        * platform/mac-leopard/media/video-controls-rendering-expected.checksum:
        * platform/mac-leopard/media/video-controls-rendering-expected.png:
        * platform/mac-leopard/media/video-controls-rendering-expected.txt:
        * platform/mac-leopard/media/video-display-toggle-expected.checksum:
        * platform/mac-leopard/media/video-display-toggle-expected.png:
        * platform/mac-leopard/media/video-display-toggle-expected.txt:
        * platform/mac-leopard/media/video-no-audio-expected.txt:
        * platform/mac-leopard/media/video-zoom-controls-expected.checksum:
        * platform/mac-leopard/media/video-zoom-controls-expected.png:
        * platform/mac-leopard/media/video-zoom-controls-expected.txt:
        * platform/mac/media/audio-controls-rendering-expected.checksum:
        * platform/mac/media/audio-controls-rendering-expected.png:
        * platform/mac/media/video-zoom-expected.checksum:
        * platform/mac/media/video-zoom-expected.png:

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

22 files changed:
LayoutTests/ChangeLog
LayoutTests/platform/mac-leopard/Skipped
LayoutTests/platform/mac-leopard/media/controls-after-reload-expected.txt
LayoutTests/platform/mac-leopard/media/controls-drag-timebar-expected.txt
LayoutTests/platform/mac-leopard/media/controls-strict-expected.txt
LayoutTests/platform/mac-leopard/media/controls-styling-expected.txt
LayoutTests/platform/mac-leopard/media/video-controls-rendering-expected.checksum
LayoutTests/platform/mac-leopard/media/video-controls-rendering-expected.png
LayoutTests/platform/mac-leopard/media/video-controls-rendering-expected.txt
LayoutTests/platform/mac-leopard/media/video-display-toggle-expected.checksum
LayoutTests/platform/mac-leopard/media/video-display-toggle-expected.png
LayoutTests/platform/mac-leopard/media/video-display-toggle-expected.txt
LayoutTests/platform/mac-leopard/media/video-no-audio-expected.txt
LayoutTests/platform/mac-leopard/media/video-zoom-controls-expected.checksum
LayoutTests/platform/mac-leopard/media/video-zoom-controls-expected.png
LayoutTests/platform/mac-leopard/media/video-zoom-controls-expected.txt
LayoutTests/platform/mac/media/audio-controls-rendering-expected.checksum
LayoutTests/platform/mac/media/audio-controls-rendering-expected.png
LayoutTests/platform/mac/media/video-zoom-expected.checksum
LayoutTests/platform/mac/media/video-zoom-expected.png
WebCore/ChangeLog
WebCore/rendering/RenderThemeMac.mm

index d431ef9..f772a3d 100644 (file)
@@ -1,3 +1,31 @@
+2009-12-12  Eric Carlson  <eric.carlson@apple.com>
+
+        Reviewed by Dan Bernstein.
+
+        <rdar://problem/6293979> Adopt shared control drawing for <video> controls on Leopard and Windows
+
+        Regenerate layout and pixel test results for the new UI.
+
+        * platform/mac-leopard/Skipped:
+        * platform/mac-leopard/media/controls-after-reload-expected.txt:
+        * platform/mac-leopard/media/controls-drag-timebar-expected.txt:
+        * platform/mac-leopard/media/controls-strict-expected.txt:
+        * platform/mac-leopard/media/controls-styling-expected.txt:
+        * platform/mac-leopard/media/video-controls-rendering-expected.checksum:
+        * platform/mac-leopard/media/video-controls-rendering-expected.png:
+        * platform/mac-leopard/media/video-controls-rendering-expected.txt:
+        * platform/mac-leopard/media/video-display-toggle-expected.checksum:
+        * platform/mac-leopard/media/video-display-toggle-expected.png:
+        * platform/mac-leopard/media/video-display-toggle-expected.txt:
+        * platform/mac-leopard/media/video-no-audio-expected.txt:
+        * platform/mac-leopard/media/video-zoom-controls-expected.checksum:
+        * platform/mac-leopard/media/video-zoom-controls-expected.png:
+        * platform/mac-leopard/media/video-zoom-controls-expected.txt:
+        * platform/mac/media/audio-controls-rendering-expected.checksum:
+        * platform/mac/media/audio-controls-rendering-expected.png:
+        * platform/mac/media/video-zoom-expected.checksum:
+        * platform/mac/media/video-zoom-expected.png:
+
 2009-12-12  Kenneth Russell  <kbr@google.com>
 
         Reviewed by Oliver Hunt.
index 5e370e0..2bb4933 100644 (file)
@@ -33,3 +33,11 @@ media/media-can-play-ogg.html
 
 # This test relies on a JS object getting GC'd, which seems to be less reliable on Leopard.
 fast/workers/dedicated-worker-lifecycle.html
+
+# Disable media tests that depend on controller size/position until
+# <rdar://problem/6502936> Update run-webkit-tests to deal with new media controller look
+# has been fixed
+media/audio-delete-while-slider-thumb-clicked.html
+media/audio-delete-while-step-button-clicked.html
+media/video-controls-visible-audio-only.html
+media/video-controls-zoomed.html
index 923fac3..1904766 100644 (file)
@@ -14,12 +14,21 @@ layer at (8,42) size 320x240
   RenderVideo {VIDEO} at (0,0) size 320x240
 layer at (8,42) size 320x240
   RenderBlock (relative positioned) {DIV} at (0,0) 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 256x16
-      RenderSlider {INPUT} at (0,0) size 256x16
-        RenderBlock {DIV} at (2,1) size 13x14
-    RenderButton {INPUT} at (272,0) size 16x16
-    RenderButton {INPUT} at (288,0) size 16x16
-    RenderButton {INPUT} at (304,0) size 16x16
+layer at (8,257) size 320x25
+  RenderFlexibleBox (positioned) {DIV} at (0,215) size 320x25
+    RenderButton {INPUT} at (6,3) size 18x18
+    RenderButton {INPUT} at (32,4) size 16x16
+    RenderFlexibleBox {DIV} at (49,5) size 246x15
+      RenderSlider {INPUT} at (45,2) size 156x13
+        RenderBlock {DIV} at (0,2) size 10x8
+    RenderButton {INPUT} at (297,5) size 15x14
+layer at (57,264) size 45x11
+  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 (258,264) size 45x11
+  RenderFlexibleBox {DIV} at (201,2) size 45x11 [color=#FFFFFF]
+    RenderBlock (anonymous) at (6,0) size 32x11
+      RenderText {#text} at (0,0) size 32x11
+        text run at (0,0) width 32: "-00:06"
index 3231085..1fdf1ad 100644 (file)
@@ -4,10 +4,10 @@ RUN(video.autoplay = true)
 RUN(video.src = 'content/test.mp4')
 EVENT(playing)
 EVENT(seeked)
-Time: 2.1
+Time: 0.3
 EVENT(seeked)
-Time: 2.6
+Time: 1.1
 EVENT(seeked)
-Time: 3.1
+Time: 1.9
 END OF TEST
 
index 5296d55..94db021 100644 (file)
@@ -14,12 +14,21 @@ layer at (8,50) size 320x240
   RenderVideo {VIDEO} at (0,0) size 320x240
 layer at (8,50) size 320x240
   RenderBlock (relative positioned) {DIV} at (0,0) 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 256x16
-      RenderSlider {INPUT} at (0,0) size 256x16
-        RenderBlock {DIV} at (2,1) size 13x14
-    RenderButton {INPUT} at (272,0) size 16x16
-    RenderButton {INPUT} at (288,0) size 16x16
-    RenderButton {INPUT} at (304,0) size 16x16
+layer at (8,265) size 320x25
+  RenderFlexibleBox (positioned) {DIV} at (0,215) size 320x25
+    RenderButton {INPUT} at (6,3) size 18x18
+    RenderButton {INPUT} at (32,4) size 16x16
+    RenderFlexibleBox {DIV} at (49,5) size 246x15
+      RenderSlider {INPUT} at (45,2) size 156x13
+        RenderBlock {DIV} at (0,2) size 10x8
+    RenderButton {INPUT} at (297,5) size 15x14
+layer at (57,272) size 45x11
+  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 (258,272) size 45x11
+  RenderFlexibleBox {DIV} at (201,2) size 45x11 [color=#FFFFFF]
+    RenderBlock (anonymous) at (6,0) size 32x11
+      RenderText {#text} at (0,0) size 32x11
+        text run at (0,0) width 32: "-00:06"
index fd24373..890e91e 100644 (file)
@@ -18,23 +18,41 @@ layer at (8,282) size 320x240
   RenderVideo {VIDEO} at (0,0) size 320x240
 layer at (18,42) size 320x240
   RenderBlock (relative positioned) {DIV} at (0,0) 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 256x16
-      RenderSlider {INPUT} at (0,0) size 256x16
-        RenderBlock {DIV} at (2,1) size 13x14
-    RenderButton {INPUT} at (272,0) size 16x16
-    RenderButton {INPUT} at (288,0) size 16x16
-    RenderButton {INPUT} at (304,0) size 16x16
+layer at (18,257) size 320x25
+  RenderFlexibleBox (positioned) {DIV} at (0,215) size 320x25
+    RenderButton {INPUT} at (6,3) size 18x18
+    RenderButton {INPUT} at (32,4) size 16x16
+    RenderFlexibleBox {DIV} at (49,5) size 246x15
+      RenderSlider {INPUT} at (45,2) size 156x13
+        RenderBlock {DIV} at (0,2) size 10x8
+    RenderButton {INPUT} at (297,5) size 15x14
+layer at (67,264) size 45x11
+  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 (268,264) size 45x11
+  RenderFlexibleBox {DIV} at (201,2) size 45x11 [color=#FFFFFF]
+    RenderBlock (anonymous) at (6,0) size 32x11
+      RenderText {#text} at (0,0) size 32x11
+        text run at (0,0) width 32: "-00:06"
 layer at (8,282) size 320x240
   RenderBlock (relative positioned) {DIV} at (0,0) 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 256x16
-      RenderSlider {INPUT} at (0,0) size 256x16
-        RenderBlock {DIV} at (2,1) size 13x14
-    RenderButton {INPUT} at (272,0) size 16x16
-    RenderButton {INPUT} at (288,0) size 16x16
-    RenderButton {INPUT} at (304,0) size 16x16
+layer at (8,497) size 320x25
+  RenderFlexibleBox (positioned) {DIV} at (0,215) size 320x25
+    RenderButton {INPUT} at (6,3) size 18x18
+    RenderButton {INPUT} at (32,4) size 16x16
+    RenderFlexibleBox {DIV} at (49,5) size 246x15
+      RenderSlider {INPUT} at (45,2) size 156x13
+        RenderBlock {DIV} at (0,2) size 10x8
+    RenderButton {INPUT} at (297,5) size 15x14
+layer at (57,504) size 45x11
+  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 (258,504) size 45x11
+  RenderFlexibleBox {DIV} at (201,2) size 45x11 [color=#FFFFFF]
+    RenderBlock (anonymous) at (6,0) size 32x11
+      RenderText {#text} at (0,0) size 32x11
+        text run at (0,0) width 32: "-00:06"
index 437d47f..5b4bff2 100644 (file)
Binary files a/LayoutTests/platform/mac-leopard/media/video-controls-rendering-expected.png and b/LayoutTests/platform/mac-leopard/media/video-controls-rendering-expected.png differ
index fff11e1..787d870 100644 (file)
@@ -17,36 +17,63 @@ layer at (8,282) size 320x240
   RenderVideo {VIDEO} at (0,0) size 320x240
 layer at (8,42) size 320x240
   RenderBlock (relative positioned) {DIV} at (0,0) 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 256x16
-      RenderSlider {INPUT} at (0,0) size 256x16
-        RenderBlock {DIV} at (2,1) size 13x14
-    RenderButton {INPUT} at (272,0) size 16x16
-    RenderButton {INPUT} at (288,0) size 16x16
-    RenderButton {INPUT} at (304,0) size 16x16
+layer at (8,257) size 320x25
+  RenderFlexibleBox (positioned) {DIV} at (0,215) size 320x25
+    RenderButton {INPUT} at (6,3) size 18x18
+    RenderButton {INPUT} at (32,4) size 16x16
+    RenderFlexibleBox {DIV} at (49,5) size 246x15
+      RenderSlider {INPUT} at (45,2) size 156x13
+        RenderBlock {DIV} at (0,2) size 10x8
+    RenderButton {INPUT} at (297,5) size 15x14
+layer at (57,264) size 45x11
+  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 (258,264) size 45x11
+  RenderFlexibleBox {DIV} at (201,2) size 45x11 [color=#FFFFFF]
+    RenderBlock (anonymous) at (6,0) size 32x11
+      RenderText {#text} at (0,0) size 32x11
+        text run at (0,0) width 32: "-00:06"
 layer at (8,282) size 320x240
   RenderBlock (relative positioned) {DIV} at (0,0) 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 256x16
-      RenderSlider {INPUT} at (0,0) size 256x16
-        RenderBlock {DIV} at (2,1) size 13x14
-    RenderButton {INPUT} at (272,0) size 16x16
-    RenderButton {INPUT} at (288,0) size 16x16
-    RenderButton {INPUT} at (304,0) size 16x16
+layer at (8,497) size 320x25
+  RenderFlexibleBox (positioned) {DIV} at (0,215) size 320x25
+    RenderButton {INPUT} at (6,3) size 18x18
+    RenderButton {INPUT} at (32,4) size 16x16
+    RenderFlexibleBox {DIV} at (49,5) size 246x15
+      RenderSlider {INPUT} at (45,2) size 156x13
+        RenderBlock {DIV} at (0,2) size 10x8
+    RenderButton {INPUT} at (297,5) size 15x14
+layer at (57,504) size 45x11
+  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 (258,504) size 45x11
+  RenderFlexibleBox {DIV} at (201,2) size 45x11 [color=#FFFFFF]
+    RenderBlock (anonymous) at (6,0) size 32x11
+      RenderText {#text} at (0,0) size 32x11
+        text run at (0,0) width 32: "-00:06"
 layer at (8,522) size 320x240
   RenderVideo {VIDEO} at (8,522) size 320x240
 layer at (8,522) size 320x240
   RenderBlock (relative positioned) {DIV} at (0,0) 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 256x16
-      RenderSlider {INPUT} at (0,0) size 256x16
-        RenderBlock {DIV} at (2,1) size 13x14
-    RenderButton {INPUT} at (272,0) size 16x16
-    RenderButton {INPUT} at (288,0) size 16x16
-    RenderButton {INPUT} at (304,0) size 16x16
+layer at (8,737) size 320x25
+  RenderFlexibleBox (positioned) {DIV} at (0,215) size 320x25
+    RenderButton {INPUT} at (6,3) size 18x18
+    RenderButton {INPUT} at (32,4) size 16x16
+    RenderFlexibleBox {DIV} at (49,5) size 246x15
+      RenderSlider {INPUT} at (45,2) size 156x13
+        RenderBlock {DIV} at (0,2) size 10x8
+    RenderButton {INPUT} at (297,5) size 15x14
+layer at (57,744) size 45x11
+  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 (258,744) size 45x11
+  RenderFlexibleBox {DIV} at (201,2) size 45x11 [color=#FFFFFF]
+    RenderBlock (anonymous) at (6,0) size 32x11
+      RenderText {#text} at (0,0) size 32x11
+        text run at (0,0) width 32: "-00:06"
index 4a50deb..9577ec2 100644 (file)
Binary files a/LayoutTests/platform/mac-leopard/media/video-display-toggle-expected.png and b/LayoutTests/platform/mac-leopard/media/video-display-toggle-expected.png differ
index a9baa00..72242e0 100644 (file)
@@ -13,12 +13,21 @@ layer at (8,26) size 320x240
   RenderVideo {VIDEO} at (0,18) size 320x240
 layer at (8,26) size 320x240
   RenderBlock (relative positioned) {DIV} at (0,0) 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 256x16
-      RenderSlider {INPUT} at (0,0) size 256x16
-        RenderBlock {DIV} at (2,1) size 13x14
-    RenderButton {INPUT} at (272,0) size 16x16
-    RenderButton {INPUT} at (288,0) size 16x16
-    RenderButton {INPUT} at (304,0) size 16x16
+layer at (8,241) size 320x25
+  RenderFlexibleBox (positioned) {DIV} at (0,215) size 320x25
+    RenderButton {INPUT} at (6,3) size 18x18
+    RenderButton {INPUT} at (32,4) size 16x16
+    RenderFlexibleBox {DIV} at (49,5) size 246x15
+      RenderSlider {INPUT} at (45,2) size 156x13
+        RenderBlock {DIV} at (0,2) size 10x8
+    RenderButton {INPUT} at (297,5) size 15x14
+layer at (57,248) size 45x11
+  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 (258,248) size 45x11
+  RenderFlexibleBox {DIV} at (201,2) size 45x11 [color=#FFFFFF]
+    RenderBlock (anonymous) at (6,0) size 32x11
+      RenderText {#text} at (0,0) size 32x11
+        text run at (0,0) width 32: "-00:06"
index f0872fc..e042007 100644 (file)
@@ -14,11 +14,20 @@ layer at (8,42) size 640x480
   RenderVideo {VIDEO} at (0,0) size 640x480
 layer at (8,42) size 640x480
   RenderBlock (relative positioned) {DIV} at (0,0) size 640x480
-layer at (8,506) size 640x16
-  RenderFlexibleBox (positioned) {DIV} at (0,464) size 640x16
-    RenderButton {INPUT} at (0,0) size 16x16
-    RenderFlexibleBox {DIV} at (16,0) size 592x16
-      RenderSlider {INPUT} at (0,0) size 592x16
-        RenderBlock {DIV} at (2,1) size 13x14
-    RenderButton {INPUT} at (608,0) size 16x16
-    RenderButton {INPUT} at (624,0) size 16x16
+layer at (8,497) size 640x25
+  RenderFlexibleBox (positioned) {DIV} at (0,455) size 640x25
+    RenderButton {INPUT} at (6,3) size 18x18
+    RenderButton {INPUT} at (32,4) size 16x16
+    RenderFlexibleBox {DIV} at (49,5) size 591x15
+      RenderSlider {INPUT} at (45,2) size 501x13
+        RenderBlock {DIV} at (0,2) size 10x8
+layer at (57,504) size 45x11
+  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 (603,504) size 45x11
+  RenderFlexibleBox {DIV} at (546,2) size 45x11 [color=#FFFFFF]
+    RenderBlock (anonymous) at (6,0) size 32x11
+      RenderText {#text} at (0,0) size 32x11
+        text run at (0,0) width 32: "-00:09"
index 5845579..57eebb8 100644 (file)
Binary files a/LayoutTests/platform/mac-leopard/media/video-zoom-controls-expected.png and b/LayoutTests/platform/mac-leopard/media/video-zoom-controls-expected.png differ
index 93618ac..b9b79a7 100644 (file)
@@ -10,25 +10,23 @@ layer at (57,85) size 240x180
   RenderVideo {VIDEO} at (45,73) size 240x180
 layer at (57,85) size 240x180
   RenderBlock (relative positioned) {DIV} at (0,0) size 240x180
-layer at (57,241) size 240x24
-  RenderFlexibleBox (positioned) {DIV} at (0,156) size 240x24
-    RenderButton {INPUT} at (0,0) size 24x24
-    RenderFlexibleBox {DIV} at (24,0) size 144x24
-      RenderSlider {INPUT} at (0,0) size 144x24
-        RenderBlock {DIV} at (3,1) size 19x21
-    RenderButton {INPUT} at (168,0) size 24x24
-    RenderButton {INPUT} at (192,0) size 24x24
-    RenderButton {INPUT} at (216,0) size 24x24
+layer at (57,228) size 240x37
+  RenderFlexibleBox (positioned) {DIV} at (0,143) size 240x37
+    RenderButton {INPUT} at (9,4) size 27x27
+    RenderButton {INPUT} at (48,6) size 24x24
+    RenderFlexibleBox {DIV} at (73,1) size 130x34
+      RenderSlider {INPUT} at (0,9) size 130x19
+        RenderBlock {DIV} at (0,3) size 15x12
+    RenderButton {INPUT} at (206,8) size 22x21
 layer at (57,310) size 240x180
   RenderVideo {VIDEO} at (45,298) size 240x180
 layer at (57,310) size 240x180
   RenderBlock (relative positioned) {DIV} at (0,0) size 240x180
-layer at (57,466) size 240x24
-  RenderFlexibleBox (positioned) {DIV} at (0,156) size 240x24
-    RenderButton {INPUT} at (0,0) size 24x24
-    RenderFlexibleBox {DIV} at (24,0) size 144x24
-      RenderSlider {INPUT} at (0,0) size 144x24
-        RenderBlock {DIV} at (3,1) size 19x21
-    RenderButton {INPUT} at (168,0) size 24x24
-    RenderButton {INPUT} at (192,0) size 24x24
-    RenderButton {INPUT} at (216,0) size 24x24
+layer at (57,453) size 240x37
+  RenderFlexibleBox (positioned) {DIV} at (0,143) size 240x37
+    RenderButton {INPUT} at (9,4) size 27x27
+    RenderButton {INPUT} at (48,6) size 24x24
+    RenderFlexibleBox {DIV} at (73,1) size 130x34
+      RenderSlider {INPUT} at (0,9) size 130x19
+        RenderBlock {DIV} at (0,3) size 15x12
+    RenderButton {INPUT} at (206,8) size 22x21
index 59ebedb..120d085 100644 (file)
Binary files a/LayoutTests/platform/mac/media/audio-controls-rendering-expected.png and b/LayoutTests/platform/mac/media/audio-controls-rendering-expected.png differ
index e84d520..ecfe7ab 100644 (file)
@@ -1 +1 @@
-03db042acc21df075462370d8c8a1f39
\ No newline at end of file
+a21b801b42e9444b7097d969e214e933
\ No newline at end of file
index 3617d59..d1bcbfa 100644 (file)
Binary files a/LayoutTests/platform/mac/media/video-zoom-expected.png and b/LayoutTests/platform/mac/media/video-zoom-expected.png differ
index 87c7858..a9ce7f4 100644 (file)
@@ -1,3 +1,14 @@
+2009-12-12  Eric Carlson  <eric.carlson@apple.com>
+
+        Reviewed by Dan Bernstein.
+
+        <rdar://problem/6293979> Adopt shared control drawing for <video> controls on Leopard and Windows
+
+        Use MediaControllerThemeQuickTime on Leopard.
+
+        * rendering/RenderThemeMac.mm:
+        (WebCore::mediaControllerTheme):
+
 2009-12-12  Kenneth Russell  <kbr@google.com>
 
         Reviewed by Oliver Hunt.
index 6304947..ddb538b 100644 (file)
@@ -1434,13 +1434,10 @@ static int mediaControllerTheme()
 
     controllerTheme = MediaControllerThemeClassic;
 
-    if (!wkMediaControllerThemeAvailable(MediaControllerThemeQuickTime))
-        return controllerTheme;
-
     Boolean validKey;
     Boolean useQTMediaUIPref = CFPreferencesGetAppBooleanValue(CFSTR("UseQuickTimeMediaUI"), CFSTR("com.apple.WebCore"), &validKey);
 
-#if !defined(BUILDING_ON_TIGER) && !defined(BUILDING_ON_LEOPARD)
+#if !defined(BUILDING_ON_TIGER)
     if (validKey && !useQTMediaUIPref)
         return controllerTheme;
 #else