[SCM] WebKit Debian packaging branch, webkit-1.2, updated. upstream/1.1.90-6072-g9a69373

eric.carlson at apple.com eric.carlson at apple.com
Thu Apr 8 00:34:23 UTC 2010


The following commit has been merged in the webkit-1.2 branch:
commit 1b924ac60d36e959126b2bc72662fc22ba9d86e8
Author: eric.carlson at apple.com <eric.carlson at apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date:   Sun Dec 13 03:28:42 2009 +0000

    2009-12-12  Eric Carlson  <eric.carlson at 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  Eric Carlson  <eric.carlson at 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: http://svn.webkit.org/repository/webkit/trunk@52059 268f45cc-cd09-0410-ab3c-d52691b4dbfc

diff --git a/LayoutTests/ChangeLog b/LayoutTests/ChangeLog
index d431ef9..f772a3d 100644
--- a/LayoutTests/ChangeLog
+++ b/LayoutTests/ChangeLog
@@ -1,3 +1,31 @@
+2009-12-12  Eric Carlson  <eric.carlson at 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 at google.com>
 
         Reviewed by Oliver Hunt.
diff --git a/LayoutTests/platform/mac-leopard/Skipped b/LayoutTests/platform/mac-leopard/Skipped
index 5e370e0..2bb4933 100644
--- a/LayoutTests/platform/mac-leopard/Skipped
+++ b/LayoutTests/platform/mac-leopard/Skipped
@@ -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
diff --git a/LayoutTests/platform/mac-leopard/media/controls-after-reload-expected.txt b/LayoutTests/platform/mac-leopard/media/controls-after-reload-expected.txt
index 923fac3..1904766 100644
--- a/LayoutTests/platform/mac-leopard/media/controls-after-reload-expected.txt
+++ b/LayoutTests/platform/mac-leopard/media/controls-after-reload-expected.txt
@@ -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"
diff --git a/LayoutTests/platform/mac-leopard/media/controls-drag-timebar-expected.txt b/LayoutTests/platform/mac-leopard/media/controls-drag-timebar-expected.txt
index 3231085..1fdf1ad 100644
--- a/LayoutTests/platform/mac-leopard/media/controls-drag-timebar-expected.txt
+++ b/LayoutTests/platform/mac-leopard/media/controls-drag-timebar-expected.txt
@@ -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.txt b/LayoutTests/platform/mac-leopard/media/controls-strict-expected.txt
index 5296d55..94db021 100644
--- a/LayoutTests/platform/mac-leopard/media/controls-strict-expected.txt
+++ b/LayoutTests/platform/mac-leopard/media/controls-strict-expected.txt
@@ -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-styling-expected.txt b/LayoutTests/platform/mac-leopard/media/controls-styling-expected.txt
index fd24373..890e91e 100644
--- a/LayoutTests/platform/mac-leopard/media/controls-styling-expected.txt
+++ b/LayoutTests/platform/mac-leopard/media/controls-styling-expected.txt
@@ -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"
diff --git a/LayoutTests/platform/mac-leopard/media/video-controls-rendering-expected.checksum b/LayoutTests/platform/mac-leopard/media/video-controls-rendering-expected.checksum
index f95b6d9..c0cd921 100644
--- a/LayoutTests/platform/mac-leopard/media/video-controls-rendering-expected.checksum
+++ b/LayoutTests/platform/mac-leopard/media/video-controls-rendering-expected.checksum
@@ -1 +1 @@
-bf317da4b87ce6fed4dc9f516467697d
\ No newline at end of file
+64d308758277aa61c61875a25e85d480
\ No newline at end of file
diff --git a/LayoutTests/platform/mac-leopard/media/video-controls-rendering-expected.png b/LayoutTests/platform/mac-leopard/media/video-controls-rendering-expected.png
index 437d47f..5b4bff2 100644
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
diff --git a/LayoutTests/platform/mac-leopard/media/video-controls-rendering-expected.txt b/LayoutTests/platform/mac-leopard/media/video-controls-rendering-expected.txt
index fff11e1..787d870 100644
--- a/LayoutTests/platform/mac-leopard/media/video-controls-rendering-expected.txt
+++ b/LayoutTests/platform/mac-leopard/media/video-controls-rendering-expected.txt
@@ -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"
diff --git a/LayoutTests/platform/mac-leopard/media/video-display-toggle-expected.checksum b/LayoutTests/platform/mac-leopard/media/video-display-toggle-expected.checksum
index 5f01a8c..07729ae 100644
--- a/LayoutTests/platform/mac-leopard/media/video-display-toggle-expected.checksum
+++ b/LayoutTests/platform/mac-leopard/media/video-display-toggle-expected.checksum
@@ -1 +1 @@
-5311c5ec0c96c170daf516be906e8c37
\ No newline at end of file
+17ee6d9a2205cf9f242bd7aeee4e003d
\ No newline at end of file
diff --git a/LayoutTests/platform/mac-leopard/media/video-display-toggle-expected.png b/LayoutTests/platform/mac-leopard/media/video-display-toggle-expected.png
index 4a50deb..9577ec2 100644
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
diff --git a/LayoutTests/platform/mac-leopard/media/video-display-toggle-expected.txt b/LayoutTests/platform/mac-leopard/media/video-display-toggle-expected.txt
index a9baa00..72242e0 100644
--- a/LayoutTests/platform/mac-leopard/media/video-display-toggle-expected.txt
+++ b/LayoutTests/platform/mac-leopard/media/video-display-toggle-expected.txt
@@ -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.txt b/LayoutTests/platform/mac-leopard/media/video-no-audio-expected.txt
index f0872fc..e042007 100644
--- a/LayoutTests/platform/mac-leopard/media/video-no-audio-expected.txt
+++ b/LayoutTests/platform/mac-leopard/media/video-no-audio-expected.txt
@@ -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"
diff --git a/LayoutTests/platform/mac-leopard/media/video-zoom-controls-expected.checksum b/LayoutTests/platform/mac-leopard/media/video-zoom-controls-expected.checksum
index 5410607..34318d9 100644
--- a/LayoutTests/platform/mac-leopard/media/video-zoom-controls-expected.checksum
+++ b/LayoutTests/platform/mac-leopard/media/video-zoom-controls-expected.checksum
@@ -1 +1 @@
-eee3bf7eb7e80387520c20a3f7fa3fb1
\ No newline at end of file
+ab0418171dc006bc21c5848e7c474bf8
\ No newline at end of file
diff --git a/LayoutTests/platform/mac-leopard/media/video-zoom-controls-expected.png b/LayoutTests/platform/mac-leopard/media/video-zoom-controls-expected.png
index 5845579..57eebb8 100644
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
diff --git a/LayoutTests/platform/mac-leopard/media/video-zoom-controls-expected.txt b/LayoutTests/platform/mac-leopard/media/video-zoom-controls-expected.txt
index 93618ac..b9b79a7 100644
--- a/LayoutTests/platform/mac-leopard/media/video-zoom-controls-expected.txt
+++ b/LayoutTests/platform/mac-leopard/media/video-zoom-controls-expected.txt
@@ -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
diff --git a/LayoutTests/platform/mac/media/audio-controls-rendering-expected.checksum b/LayoutTests/platform/mac/media/audio-controls-rendering-expected.checksum
index 42d1ef7..2e1a032 100644
--- a/LayoutTests/platform/mac/media/audio-controls-rendering-expected.checksum
+++ b/LayoutTests/platform/mac/media/audio-controls-rendering-expected.checksum
@@ -1 +1 @@
-cd78dfa514aedff4e81f0a43408d688e
\ No newline at end of file
+23d92921001dcac799a7bf70535f0723
\ No newline at end of file
diff --git a/LayoutTests/platform/mac/media/audio-controls-rendering-expected.png b/LayoutTests/platform/mac/media/audio-controls-rendering-expected.png
index 59ebedb..120d085 100644
Binary files a/LayoutTests/platform/mac/media/audio-controls-rendering-expected.png and b/LayoutTests/platform/mac/media/audio-controls-rendering-expected.png differ
diff --git a/LayoutTests/platform/mac/media/video-zoom-expected.checksum b/LayoutTests/platform/mac/media/video-zoom-expected.checksum
index e84d520..ecfe7ab 100644
--- a/LayoutTests/platform/mac/media/video-zoom-expected.checksum
+++ b/LayoutTests/platform/mac/media/video-zoom-expected.checksum
@@ -1 +1 @@
-03db042acc21df075462370d8c8a1f39
\ No newline at end of file
+a21b801b42e9444b7097d969e214e933
\ No newline at end of file
diff --git a/LayoutTests/platform/mac/media/video-zoom-expected.png b/LayoutTests/platform/mac/media/video-zoom-expected.png
index 3617d59..d1bcbfa 100644
Binary files a/LayoutTests/platform/mac/media/video-zoom-expected.png and b/LayoutTests/platform/mac/media/video-zoom-expected.png differ
diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog
index 87c7858..a9ce7f4 100644
--- a/WebCore/ChangeLog
+++ b/WebCore/ChangeLog
@@ -1,3 +1,14 @@
+2009-12-12  Eric Carlson  <eric.carlson at 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 at google.com>
 
         Reviewed by Oliver Hunt.
diff --git a/WebCore/rendering/RenderThemeMac.mm b/WebCore/rendering/RenderThemeMac.mm
index 6304947..ddb538b 100644
--- a/WebCore/rendering/RenderThemeMac.mm
+++ b/WebCore/rendering/RenderThemeMac.mm
@@ -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

-- 
WebKit Debian packaging



More information about the Pkg-webkit-commits mailing list