Two tests, which include data uri images, need to be changed and rebaselined since...
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 8 Jan 2015 02:17:51 +0000 (02:17 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 8 Jan 2015 02:17:51 +0000 (02:17 +0000)
https://bugs.webkit.org/show_bug.cgi?id=140199.

Patch by Said Abou-Hallawa <sabouhallawa@apple.com> on 2015-01-07
Reviewed by Simon Fraser.

* compositing/tiling/huge-layer-img-expected.txt:
* compositing/tiling/huge-layer-img.html:
* platform/mac/compositing/tiling/huge-layer-img-expected.txt:
This test draws a huge image on a canvas and then gets the data uri from the canvas and
set it as the source of an <img> tag. We need to rebaseline the expected results since
the original ones were incorrect. The test was getting the layer tree too early. Before
the image data was loaded, a tiled layer was created. After the image data is loaded, we
could have figured out that the image is a simple one and we could have removed the tiling.
The fix is to change the test to delay capturing the layer tree only after making sure
that the image data is loaded.

* fast/forms/basic-buttons.html:
* platform/mac-mountainlion/fast/forms/basic-buttons-expected.txt:
* platform/mac/fast/forms/basic-buttons-expected.png:
* platform/mac/fast/forms/basic-buttons-expected.txt:
This test includes a data uri image just to not include a file for the image by itself.
We need to rebaseline the expected results since the original ones were incorrect. The
test was assuming that after the <img> tag is added to the HTML body, the image size can
be retrieved immediately. Because this assumption is incorrect in general, we were getting
empty sizes for all the the data uri images. The fix is to change the test such that it
prints the sizes of the images only after making sure that all the images are loaded.

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

LayoutTests/ChangeLog
LayoutTests/compositing/tiling/huge-layer-img-expected.txt
LayoutTests/compositing/tiling/huge-layer-img.html
LayoutTests/fast/forms/basic-buttons.html
LayoutTests/platform/mac-mountainlion/fast/forms/basic-buttons-expected.txt
LayoutTests/platform/mac/compositing/tiling/huge-layer-img-expected.txt
LayoutTests/platform/mac/fast/forms/basic-buttons-expected.png
LayoutTests/platform/mac/fast/forms/basic-buttons-expected.txt

index 647c44c..73e86c9 100644 (file)
@@ -1,3 +1,32 @@
+2015-01-07  Said Abou-Hallawa  <sabouhallawa@apple.com>
+
+        Two tests, which include data uri images, need to be changed and rebaselined since the expected results are incorrect.
+        https://bugs.webkit.org/show_bug.cgi?id=140199.
+
+        Reviewed by Simon Fraser.
+
+        * compositing/tiling/huge-layer-img-expected.txt:
+        * compositing/tiling/huge-layer-img.html:
+        * platform/mac/compositing/tiling/huge-layer-img-expected.txt:
+        This test draws a huge image on a canvas and then gets the data uri from the canvas and
+        set it as the source of an <img> tag. We need to rebaseline the expected results since
+        the original ones were incorrect. The test was getting the layer tree too early. Before
+        the image data was loaded, a tiled layer was created. After the image data is loaded, we
+        could have figured out that the image is a simple one and we could have removed the tiling.
+        The fix is to change the test to delay capturing the layer tree only after making sure
+        that the image data is loaded.
+        
+        * fast/forms/basic-buttons.html:
+        * platform/mac-mountainlion/fast/forms/basic-buttons-expected.txt:
+        * platform/mac/fast/forms/basic-buttons-expected.png:
+        * platform/mac/fast/forms/basic-buttons-expected.txt:
+        This test includes a data uri image just to not include a file for the image by itself. 
+        We need to rebaseline the expected results since the original ones were incorrect. The
+        test was assuming that after the <img> tag is added to the HTML body, the image size can
+        be retrieved immediately. Because this assumption is incorrect in general, we were getting
+        empty sizes for all the the data uri images. The fix is to change the test such that it
+        prints the sizes of the images only after making sure that all the images are loaded.
+
 2015-01-07  Brent Fulgham  <bfulgham@apple.com>
 
         [Win] Rebaseline windows fast/text tests.
index 09b01a8..2de81b3 100644 (file)
@@ -9,7 +9,6 @@
         (GraphicsLayer
           (position 8.00 8.00)
           (bounds 500.00 20000.00)
-          (drawsContent 1)
         )
       )
     )
index f29fc3a..a92481a 100644 (file)
     }
     </style>
     <script type="text/javascript" charset="utf-8">
-        if (window.testRunner)
+        if (window.testRunner) {
             testRunner.dumpAsText(true);
+            testRunner.waitUntilDone();
+        }
 
         function testOnLoad()
         {
             context.fillStyle = "red";
             context.fillRect(50, 50, 200, 200);
             img.src = canvas.toDataURL();
-
-            if (window.testRunner)
+        }
+        
+        function onImageLoad()
+        {
+            if (window.testRunner) {
                 document.getElementById('layers').innerHTML = window.internals.layerTreeAsText(document);
+                testRunner.notifyDone();
+            }
         }
 
         window.addEventListener('load', testOnLoad, false);
@@ -41,7 +48,7 @@
         There should be a red box within the yellow box.
         This tests that we can support very large composited image layers.
       -->
-        <img id="container">
+        <img onload="onImageLoad()" id="container">
         <pre id="layers">Layer tree appears here in DRT.</pre>
   </body>
 </html>
index 6b78def..535240b 100644 (file)
@@ -1,4 +1,5 @@
-<!DocType html><html> 
+<!DocType html>
+<html> 
 <body> 
 
 Tests for basic button rendering. Creates a table with seven columns and seven rows. <BR>
@@ -9,6 +10,9 @@ with text ("foo") and then uses six different paddings to make sure each of the
 <script>
 var fooImage = '<img src="data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAAAoAAAAKCAYAAACNMs+9AAAABGdBTUEAALGPC/xhBQAAAAlwSFlzAAALEwAACxMBAJqcGAAAAAd0SU1FB9YGARc5KB0XV+IAAAAddEVYdENvbW1lbnQAQ3JlYXRlZCB3aXRoIFRoZSBHSU1Q72QlbgAAAF1JREFUGNO9zL0NglAAxPEfdLTs4BZM4DIO4C7OwQg2JoQ9LE1exdlYvBBeZ7jqch9//q1uH4TLzw4d6+ErXMMcXuHWxId3KOETnnXXV6MJpcq2MLaI97CER3N0vr4MkhoXe0rZigAAAABJRU5ErkJggg==" alt="Red dot" />';
 
+if (window.testRunner)
+    testRunner.waitUntilDone();
+
 function getRowHtml(style) 
 {
      return '<tr><td>' + (style || '(default)') + '</td>' +
@@ -31,7 +35,7 @@ document.body.innerHTML += '<table cellspacing=0><tbody>' +
      getRowHtml('padding: 20px') +
      '</tbody></table>';
 
-function printSize(tagname, cell
+function printSize(tagname) 
 {
     var buttons = document.getElementsByTagName(tagname);
     for (var i = 0; i < buttons.length; i++) {
@@ -48,8 +52,16 @@ function printSize(tagname, cell)
     }
 }
 
-printSize('button');
-printSize('input');
+function testOnLoad()
+{
+    printSize('button');
+    printSize('input');
+    if (window.testRunner)
+        testRunner.notifyDone();
+}
+
+window.addEventListener('load', testOnLoad, false);
+
 </script>
 
 </body> 
index 9426151..7b79e23 100644 (file)
@@ -42,7 +42,7 @@ layer at (0,0) size 800x318
                   RenderImage {IMG} at (0,1) size 10x10
             RenderTableCell {TD} at (229,22) size 157x20 [r=1 c=2 rs=1 cs=1]
               RenderText {#text} at (1,1) size 104x18
-                text run at (1,1) width 104: "(18, 16) (18, 12)"
+                text run at (1,1) width 104: "(18, 26) (18, 22)"
             RenderTableCell {TD} at (386,20) size 132x24 [r=1 c=3 rs=1 cs=1]
               RenderButton {INPUT} at (3,3) size 34x18 [bgcolor=#C0C0C0]
                 RenderBlock (anonymous) at (8,2) size 18x13
@@ -60,8 +60,8 @@ layer at (0,0) size 800x318
                 RenderBlock (anonymous) at (2,1) size 10x13
                   RenderImage {IMG} at (0,1) size 10x10
             RenderTableCell {TD} at (229,46) size 157x20 [r=2 c=2 rs=1 cs=1]
-              RenderText {#text} at (1,1) size 88x18
-                text run at (1,1) width 88: "(15, 4) (15, 0)"
+              RenderText {#text} at (1,1) size 104x18
+                text run at (1,1) width 104: "(15, 14) (15, 10)"
             RenderTableCell {TD} at (386,44) size 132x24 [r=2 c=3 rs=1 cs=1]
               RenderButton {INPUT} at (3,3) size 34x18 [bgcolor=#C0C0C0]
                 RenderBlock (anonymous) at (8,2) size 18x13
@@ -80,7 +80,7 @@ layer at (0,0) size 800x318
                   RenderImage {IMG} at (0,1) size 10x10
             RenderTableCell {TD} at (229,73) size 157x20 [r=3 c=2 rs=1 cs=1]
               RenderText {#text} at (1,1) size 104x18
-                text run at (1,1) width 104: "(25, 16) (25, 12)"
+                text run at (1,1) width 104: "(25, 26) (25, 22)"
             RenderTableCell {TD} at (386,71) size 132x24 [r=3 c=3 rs=1 cs=1]
               RenderButton {INPUT} at (3,3) size 34x18 [bgcolor=#C0C0C0]
                 RenderBlock (anonymous) at (8,2) size 18x13
@@ -98,8 +98,8 @@ layer at (0,0) size 800x318
                 RenderBlock (anonymous) at (4,2) size 10x13
                   RenderImage {IMG} at (0,1) size 10x10
             RenderTableCell {TD} at (229,100) size 157x20 [r=4 c=2 rs=1 cs=1]
-              RenderText {#text} at (1,1) size 88x18
-                text run at (1,1) width 88: "(17, 8) (17, 4)"
+              RenderText {#text} at (1,1) size 104x18
+                text run at (1,1) width 104: "(17, 18) (17, 14)"
             RenderTableCell {TD} at (386,98) size 132x24 [r=4 c=3 rs=1 cs=1]
               RenderButton {INPUT} at (3,3) size 34x18 [bgcolor=#C0C0C0]
                 RenderBlock (anonymous) at (8,2) size 18x13
@@ -118,7 +118,7 @@ layer at (0,0) size 800x318
                   RenderImage {IMG} at (0,1) size 10x10
             RenderTableCell {TD} at (229,124) size 157x20 [r=5 c=2 rs=1 cs=1]
               RenderText {#text} at (1,1) size 104x18
-                text run at (1,1) width 104: "(18, 16) (18, 12)"
+                text run at (1,1) width 104: "(18, 26) (18, 22)"
             RenderTableCell {TD} at (386,122) size 132x24 [r=5 c=3 rs=1 cs=1]
               RenderButton {INPUT} at (3,3) size 34x18 [bgcolor=#C0C0C0]
                 RenderBlock (anonymous) at (8,2) size 18x13
@@ -137,7 +137,7 @@ layer at (0,0) size 800x318
                   RenderImage {IMG} at (0,1) size 10x10
             RenderTableCell {TD} at (229,148) size 157x20 [r=6 c=2 rs=1 cs=1]
               RenderText {#text} at (1,1) size 104x18
-                text run at (1,1) width 104: "(19, 18) (19, 14)"
+                text run at (1,1) width 104: "(19, 28) (19, 24)"
             RenderTableCell {TD} at (386,146) size 132x24 [r=6 c=3 rs=1 cs=1]
               RenderButton {INPUT} at (3,3) size 34x18 [bgcolor=#C0C0C0]
                 RenderBlock (anonymous) at (8,2) size 18x13
@@ -156,7 +156,7 @@ layer at (0,0) size 800x318
                   RenderImage {IMG} at (0,1) size 10x10
             RenderTableCell {TD} at (229,190) size 157x20 [r=7 c=2 rs=1 cs=1]
               RenderText {#text} at (1,1) size 104x18
-                text run at (1,1) width 104: "(53, 44) (53, 40)"
+                text run at (1,1) width 104: "(53, 54) (53, 50)"
             RenderTableCell {TD} at (386,188) size 132x24 [r=7 c=3 rs=1 cs=1]
               RenderButton {INPUT} at (3,3) size 34x18 [bgcolor=#C0C0C0]
                 RenderBlock (anonymous) at (8,2) size 18x13
index c10b9d9..49f6bb0 100644 (file)
@@ -10,8 +10,6 @@
         (GraphicsLayer
           (position 8.00 8.00)
           (bounds 500.00 20000.00)
-          (usingTiledLayer 1)
-          (drawsContent 1)
         )
       )
     )
index 57ef327..e8b579a 100644 (file)
Binary files a/LayoutTests/platform/mac/fast/forms/basic-buttons-expected.png and b/LayoutTests/platform/mac/fast/forms/basic-buttons-expected.png differ
index af424c2..8fcc32c 100644 (file)
@@ -42,7 +42,7 @@ layer at (0,0) size 800x318
                   RenderImage {IMG} at (0,1) size 10x10
             RenderTableCell {TD} at (231,22) size 161x20 [r=1 c=2 rs=1 cs=1]
               RenderText {#text} at (1,1) size 106x18
-                text run at (1,1) width 106: "(18, 16) (18, 12)"
+                text run at (1,1) width 106: "(18, 26) (18, 22)"
             RenderTableCell {TD} at (392,20) size 135x24 [r=1 c=3 rs=1 cs=1]
               RenderButton {INPUT} at (3,3) size 34x18 [bgcolor=#C0C0C0]
                 RenderBlock (anonymous) at (8,2) size 18x13
@@ -60,8 +60,8 @@ layer at (0,0) size 800x318
                 RenderBlock (anonymous) at (2,1) size 10x13
                   RenderImage {IMG} at (0,1) size 10x10
             RenderTableCell {TD} at (231,46) size 161x20 [r=2 c=2 rs=1 cs=1]
-              RenderText {#text} at (1,1) size 90x18
-                text run at (1,1) width 90: "(15, 4) (15, 0)"
+              RenderText {#text} at (1,1) size 106x18
+                text run at (1,1) width 106: "(15, 14) (15, 10)"
             RenderTableCell {TD} at (392,44) size 135x24 [r=2 c=3 rs=1 cs=1]
               RenderButton {INPUT} at (3,3) size 34x18 [bgcolor=#C0C0C0]
                 RenderBlock (anonymous) at (8,2) size 18x13
@@ -80,7 +80,7 @@ layer at (0,0) size 800x318
                   RenderImage {IMG} at (0,1) size 10x10
             RenderTableCell {TD} at (231,73) size 161x20 [r=3 c=2 rs=1 cs=1]
               RenderText {#text} at (1,1) size 106x18
-                text run at (1,1) width 106: "(25, 16) (25, 12)"
+                text run at (1,1) width 106: "(25, 26) (25, 22)"
             RenderTableCell {TD} at (392,71) size 135x24 [r=3 c=3 rs=1 cs=1]
               RenderButton {INPUT} at (3,3) size 34x18 [bgcolor=#C0C0C0]
                 RenderBlock (anonymous) at (8,2) size 18x13
@@ -98,8 +98,8 @@ layer at (0,0) size 800x318
                 RenderBlock (anonymous) at (4,2) size 10x13
                   RenderImage {IMG} at (0,1) size 10x10
             RenderTableCell {TD} at (231,100) size 161x20 [r=4 c=2 rs=1 cs=1]
-              RenderText {#text} at (1,1) size 90x18
-                text run at (1,1) width 90: "(17, 8) (17, 4)"
+              RenderText {#text} at (1,1) size 106x18
+                text run at (1,1) width 106: "(17, 18) (17, 14)"
             RenderTableCell {TD} at (392,98) size 135x24 [r=4 c=3 rs=1 cs=1]
               RenderButton {INPUT} at (3,3) size 34x18 [bgcolor=#C0C0C0]
                 RenderBlock (anonymous) at (8,2) size 18x13
@@ -118,7 +118,7 @@ layer at (0,0) size 800x318
                   RenderImage {IMG} at (0,1) size 10x10
             RenderTableCell {TD} at (231,124) size 161x20 [r=5 c=2 rs=1 cs=1]
               RenderText {#text} at (1,1) size 106x18
-                text run at (1,1) width 106: "(18, 16) (18, 12)"
+                text run at (1,1) width 106: "(18, 26) (18, 22)"
             RenderTableCell {TD} at (392,122) size 135x24 [r=5 c=3 rs=1 cs=1]
               RenderButton {INPUT} at (3,3) size 34x18 [bgcolor=#C0C0C0]
                 RenderBlock (anonymous) at (8,2) size 18x13
@@ -137,7 +137,7 @@ layer at (0,0) size 800x318
                   RenderImage {IMG} at (0,1) size 10x10
             RenderTableCell {TD} at (231,148) size 161x20 [r=6 c=2 rs=1 cs=1]
               RenderText {#text} at (1,1) size 106x18
-                text run at (1,1) width 106: "(19, 18) (19, 14)"
+                text run at (1,1) width 106: "(19, 28) (19, 24)"
             RenderTableCell {TD} at (392,146) size 135x24 [r=6 c=3 rs=1 cs=1]
               RenderButton {INPUT} at (3,3) size 34x18 [bgcolor=#C0C0C0]
                 RenderBlock (anonymous) at (8,2) size 18x13
@@ -156,7 +156,7 @@ layer at (0,0) size 800x318
                   RenderImage {IMG} at (0,1) size 10x10
             RenderTableCell {TD} at (231,190) size 161x20 [r=7 c=2 rs=1 cs=1]
               RenderText {#text} at (1,1) size 106x18
-                text run at (1,1) width 106: "(53, 44) (53, 40)"
+                text run at (1,1) width 106: "(53, 54) (53, 50)"
             RenderTableCell {TD} at (392,188) size 135x24 [r=7 c=3 rs=1 cs=1]
               RenderButton {INPUT} at (3,3) size 34x18 [bgcolor=#C0C0C0]
                 RenderBlock (anonymous) at (8,2) size 18x13