2009-12-14 Eric Carlson <eric.carlson@apple.com>
authoreric.carlson@apple.com <eric.carlson@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 14 Dec 2009 16:16:57 +0000 (16:16 +0000)
committereric.carlson@apple.com <eric.carlson@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 14 Dec 2009 16:16:57 +0000 (16:16 +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-14  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/accessibility/media-element-expected.txt:
        * platform/mac-leopard/media/audio-controls-rendering-expected.checksum:
        * platform/mac-leopard/media/audio-controls-rendering-expected.txt:
        * platform/mac-leopard/media/controls-after-reload-expected.checksum: Added.
        * platform/mac-leopard/media/controls-after-reload-expected.png: Added.
        * 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.checksum: Added.
        * platform/mac-leopard/media/controls-strict-expected.png: Added.
        * platform/mac-leopard/media/controls-strict-expected.txt:
        * platform/mac-leopard/media/controls-strict-mode-expected.txt: Added.
        * platform/mac-leopard/media/controls-styling-expected.checksum: Added.
        * platform/mac-leopard/media/controls-styling-expected.png: Added.
        * 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.txt:
        * platform/mac-leopard/media/video-display-toggle-expected.checksum:
        * platform/mac-leopard/media/video-display-toggle-expected.txt:
        * platform/mac-leopard/media/video-no-audio-expected.checksum: Added.
        * platform/mac-leopard/media/video-no-audio-expected.png: Added.
        * 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.txt:

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

27 files changed:
LayoutTests/ChangeLog
LayoutTests/platform/mac-leopard/Skipped
LayoutTests/platform/mac-leopard/accessibility/media-element-expected.txt
LayoutTests/platform/mac-leopard/media/audio-controls-rendering-expected.checksum
LayoutTests/platform/mac-leopard/media/audio-controls-rendering-expected.txt
LayoutTests/platform/mac-leopard/media/controls-after-reload-expected.checksum [new file with mode: 0644]
LayoutTests/platform/mac-leopard/media/controls-after-reload-expected.png [new file with mode: 0644]
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.checksum [new file with mode: 0644]
LayoutTests/platform/mac-leopard/media/controls-strict-expected.png [new file with mode: 0644]
LayoutTests/platform/mac-leopard/media/controls-strict-expected.txt
LayoutTests/platform/mac-leopard/media/controls-strict-mode-expected.txt [new file with mode: 0644]
LayoutTests/platform/mac-leopard/media/controls-styling-expected.checksum [new file with mode: 0644]
LayoutTests/platform/mac-leopard/media/controls-styling-expected.png [new file with mode: 0644]
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.txt
LayoutTests/platform/mac-leopard/media/video-display-toggle-expected.checksum
LayoutTests/platform/mac-leopard/media/video-display-toggle-expected.txt
LayoutTests/platform/mac-leopard/media/video-no-audio-expected.checksum [new file with mode: 0644]
LayoutTests/platform/mac-leopard/media/video-no-audio-expected.png [new file with mode: 0644]
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.txt
WebCore/ChangeLog
WebCore/rendering/RenderThemeMac.mm

index a579a1c..30f21a1 100644 (file)
@@ -1,3 +1,36 @@
+2009-12-14  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/accessibility/media-element-expected.txt:
+        * platform/mac-leopard/media/audio-controls-rendering-expected.checksum:
+        * platform/mac-leopard/media/audio-controls-rendering-expected.txt:
+        * platform/mac-leopard/media/controls-after-reload-expected.checksum: Added.
+        * platform/mac-leopard/media/controls-after-reload-expected.png: Added.
+        * 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.checksum: Added.
+        * platform/mac-leopard/media/controls-strict-expected.png: Added.
+        * platform/mac-leopard/media/controls-strict-expected.txt:
+        * platform/mac-leopard/media/controls-strict-mode-expected.txt: Added.
+        * platform/mac-leopard/media/controls-styling-expected.checksum: Added.
+        * platform/mac-leopard/media/controls-styling-expected.png: Added.
+        * 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.txt:
+        * platform/mac-leopard/media/video-display-toggle-expected.checksum:
+        * platform/mac-leopard/media/video-display-toggle-expected.txt:
+        * platform/mac-leopard/media/video-no-audio-expected.checksum: Added.
+        * platform/mac-leopard/media/video-no-audio-expected.png: Added.
+        * 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.txt:
+
 2009-12-14  Philippe Normand  <pnormand@igalia.com>
 
         Rubber-stamped by Gustavo Noronha Silva.
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 c54bede..7eacfba 100644 (file)
@@ -8,10 +8,18 @@ State at 'load' event:
     role: AXRole: AXToolbar
 
 
+        description: AXDescription: back 30 seconds
+        role: AXRole: AXButton
+
+
         description: AXDescription: play
         role: AXRole: AXButton
 
 
+        description: AXDescription: elapsed time
+        role: AXRole: AXStaticText
+
+
         description: AXDescription: 
         role: AXRole: AXSlider
 
@@ -20,12 +28,8 @@ State at 'load' event:
             role: AXRole: AXValueIndicator
 
 
-        description: AXDescription: fast reverse
-        role: AXRole: AXButton
-
-
-        description: AXDescription: fast forward
-        role: AXRole: AXButton
+        description: AXDescription: remaining time
+        role: AXRole: AXStaticText
 
 
         description: AXDescription: mute
index 77de6be..d7ab43a 100644 (file)
@@ -6,46 +6,63 @@ layer at (0,0) size 800x600
       RenderBlock {P} at (0,0) size 784x18
         RenderText {#text} at (0,0) size 153x18
           text run at (0,0) width 153: "Test controls placement."
-      RenderBlock (anonymous) at (0,34) size 784x86
-        RenderMedia {AUDIO} at (0,0) size 200x16 [bgcolor=#0000FF]
-        RenderBR {BR} at (200,16) size 0x0
-        RenderBR {BR} at (0,16) size 0x18
-        RenderMedia {AUDIO} at (0,34) size 320x16 [bgcolor=#0000FF]
-        RenderBR {BR} at (320,50) size 0x0
-        RenderBR {BR} at (0,50) size 0x18
+      RenderBlock (anonymous) at (0,34) size 784x104
+        RenderMedia {AUDIO} at (0,0) size 200x25 [bgcolor=#0000FF]
+        RenderBR {BR} at (200,25) size 0x0
+        RenderBR {BR} at (0,25) size 0x18
+        RenderMedia {AUDIO} at (0,43) size 320x25 [bgcolor=#0000FF]
+        RenderBR {BR} at (320,68) size 0x0
         RenderBR {BR} at (0,68) size 0x18
-layer at (8,42) size 200x16
-  RenderBlock (relative positioned) {DIV} at (0,0) size 200x16
-layer at (8,42) size 200x16
-  RenderFlexibleBox (positioned) {DIV} at (0,0) size 200x16
-    RenderButton {INPUT} at (0,0) size 16x16
-    RenderFlexibleBox {DIV} at (16,0) size 136x16
-      RenderSlider {INPUT} at (0,0) size 136x16
-        RenderBlock {DIV} at (2,1) size 13x14
-    RenderButton {INPUT} at (152,0) size 16x16
-    RenderButton {INPUT} at (168,0) size 16x16
-    RenderButton {INPUT} at (184,0) size 16x16
-layer at (8,76) size 320x16
-  RenderBlock (relative positioned) {DIV} at (0,0) size 320x16
-layer at (8,76) size 320x16
-  RenderFlexibleBox (positioned) {DIV} at (0,0) 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,110) size 320x100
-  RenderMedia {AUDIO} at (8,110) size 320x100 [bgcolor=#0000FF]
-layer at (8,110) size 320x100
+        RenderBR {BR} at (0,86) size 0x18
+layer at (8,42) size 200x25
+  RenderBlock (relative positioned) {DIV} at (0,0) size 200x25
+layer at (8,42) size 200x25
+  RenderFlexibleBox (positioned) {DIV} at (0,0) size 200x25
+    RenderButton {INPUT} at (6,3) size 18x18
+    RenderButton {INPUT} at (32,4) size 16x16
+    RenderFlexibleBox {DIV} at (49,1) size 126x22
+      RenderSlider {INPUT} at (0,5) size 126x13
+        RenderBlock {DIV} at (0,2) size 10x8
+    RenderButton {INPUT} at (177,5) size 15x14
+layer at (8,85) size 320x25
+  RenderBlock (relative positioned) {DIV} at (0,0) size 320x25
+layer at (8,85) size 320x25
+  RenderFlexibleBox (positioned) {DIV} at (0,0) 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,92) 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,92) 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:07"
+layer at (8,128) size 320x100
+  RenderMedia {AUDIO} at (8,128) size 320x100 [bgcolor=#0000FF]
+layer at (8,128) size 320x100
   RenderBlock (relative positioned) {DIV} at (0,0) size 320x100
-layer at (8,194) size 320x16
-  RenderFlexibleBox (positioned) {DIV} at (0,84) 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,203) size 320x25
+  RenderFlexibleBox (positioned) {DIV} at (0,75) 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,210) 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,210) 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:07"
diff --git a/LayoutTests/platform/mac-leopard/media/controls-after-reload-expected.checksum b/LayoutTests/platform/mac-leopard/media/controls-after-reload-expected.checksum
new file mode 100644 (file)
index 0000000..0677211
--- /dev/null
@@ -0,0 +1 @@
+1273306fd2815b4c06a2c49634621bd6
\ No newline at end of file
diff --git a/LayoutTests/platform/mac-leopard/media/controls-after-reload-expected.png b/LayoutTests/platform/mac-leopard/media/controls-after-reload-expected.png
new file mode 100644 (file)
index 0000000..5ad3160
Binary files /dev/null and b/LayoutTests/platform/mac-leopard/media/controls-after-reload-expected.png differ
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
 
diff --git a/LayoutTests/platform/mac-leopard/media/controls-strict-expected.checksum b/LayoutTests/platform/mac-leopard/media/controls-strict-expected.checksum
new file mode 100644 (file)
index 0000000..713f622
--- /dev/null
@@ -0,0 +1 @@
+4f2430c865a33db87ed098279109288e
\ No newline at end of file
diff --git a/LayoutTests/platform/mac-leopard/media/controls-strict-expected.png b/LayoutTests/platform/mac-leopard/media/controls-strict-expected.png
new file mode 100644 (file)
index 0000000..bcab490
Binary files /dev/null and b/LayoutTests/platform/mac-leopard/media/controls-strict-expected.png differ
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"
diff --git a/LayoutTests/platform/mac-leopard/media/controls-strict-mode-expected.txt b/LayoutTests/platform/mac-leopard/media/controls-strict-mode-expected.txt
new file mode 100644 (file)
index 0000000..fbd0592
--- /dev/null
@@ -0,0 +1,5 @@
+layer at (0,0) size 800x600
+  RenderView at (0,0) size 800x600
+layer at (0,0) size 800x600
+  RenderBlock {HTML} at (0,0) size 800x600
+    RenderBody {BODY} at (8,8) size 784x584
diff --git a/LayoutTests/platform/mac-leopard/media/controls-styling-expected.checksum b/LayoutTests/platform/mac-leopard/media/controls-styling-expected.checksum
new file mode 100644 (file)
index 0000000..754fe1e
--- /dev/null
@@ -0,0 +1 @@
+a2d4f10efd0372bf429cd96e2ca8de96
\ No newline at end of file
diff --git a/LayoutTests/platform/mac-leopard/media/controls-styling-expected.png b/LayoutTests/platform/mac-leopard/media/controls-styling-expected.png
new file mode 100644 (file)
index 0000000..8392d66
Binary files /dev/null and b/LayoutTests/platform/mac-leopard/media/controls-styling-expected.png differ
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 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 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"
diff --git a/LayoutTests/platform/mac-leopard/media/video-no-audio-expected.checksum b/LayoutTests/platform/mac-leopard/media/video-no-audio-expected.checksum
new file mode 100644 (file)
index 0000000..8cb46a6
--- /dev/null
@@ -0,0 +1 @@
+ff94259493306a136e4509ba2aa98349
\ No newline at end of file
diff --git a/LayoutTests/platform/mac-leopard/media/video-no-audio-expected.png b/LayoutTests/platform/mac-leopard/media/video-no-audio-expected.png
new file mode 100644 (file)
index 0000000..208d478
Binary files /dev/null and b/LayoutTests/platform/mac-leopard/media/video-no-audio-expected.png differ
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 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 f303804..25f866d 100644 (file)
@@ -1,3 +1,14 @@
+2009-12-14  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-14  Adam Roben  <aroben@apple.com>
 
         Move QuartzCore-related build settings into a vsprops file
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