Title: [209347] trunk/LayoutTests
Revision
209347
Author
hy...@apple.com
Date
2016-12-05 13:56:38 -0800 (Mon, 05 Dec 2016)

Log Message

[CSS Parser] Fix column tests with illegal syntax
https://bugs.webkit.org/show_bug.cgi?id=165411

Reviewed by Zalan Bujtas.

Patch column-span layout tests that use invalid syntax
to use the spec-compliant syntax instead (none instead of 1).

* fast/multicol/newmulticol/remove-spanner4.html:
* fast/multicol/newmulticol/remove-spanner5.html:
* fast/multicol/newmulticol/remove-spanner6.html:
* fast/multicol/span/span-as-immediate-child-property-removal.html:

Modified Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (209346 => 209347)


--- trunk/LayoutTests/ChangeLog	2016-12-05 21:31:23 UTC (rev 209346)
+++ trunk/LayoutTests/ChangeLog	2016-12-05 21:56:38 UTC (rev 209347)
@@ -1,5 +1,20 @@
 2016-12-05  Dave Hyatt  <hy...@apple.com>
 
+        [CSS Parser] Fix column tests with illegal syntax
+        https://bugs.webkit.org/show_bug.cgi?id=165411
+
+        Reviewed by Zalan Bujtas.
+
+        Patch column-span layout tests that use invalid syntax
+        to use the spec-compliant syntax instead (none instead of 1).
+
+        * fast/multicol/newmulticol/remove-spanner4.html:
+        * fast/multicol/newmulticol/remove-spanner5.html:
+        * fast/multicol/newmulticol/remove-spanner6.html:
+        * fast/multicol/span/span-as-immediate-child-property-removal.html:
+
+2016-12-05  Dave Hyatt  <hy...@apple.com>
+
         [CSS Parser] Avoid a few more tests
         https://bugs.webkit.org/show_bug.cgi?id=165404
 

Modified: trunk/LayoutTests/fast/multicol/newmulticol/remove-spanner4.html (209346 => 209347)


--- trunk/LayoutTests/fast/multicol/newmulticol/remove-spanner4.html	2016-12-05 21:31:23 UTC (rev 209346)
+++ trunk/LayoutTests/fast/multicol/newmulticol/remove-spanner4.html	2016-12-05 21:56:38 UTC (rev 209347)
@@ -6,8 +6,8 @@
             function test() {
                 var elm = document.getElementById('elm');
                 elm.offsetTop; // trigger layout
-                elm.style.webkitColumnSpan = '1';
-                elm.style.columnSpan = '1';
+                elm.style.webkitColumnSpan = 'none';
+                elm.style.columnSpan = 'none';
             }
         </script>
     </head>

Modified: trunk/LayoutTests/fast/multicol/newmulticol/remove-spanner5.html (209346 => 209347)


--- trunk/LayoutTests/fast/multicol/newmulticol/remove-spanner5.html	2016-12-05 21:31:23 UTC (rev 209346)
+++ trunk/LayoutTests/fast/multicol/newmulticol/remove-spanner5.html	2016-12-05 21:56:38 UTC (rev 209347)
@@ -6,8 +6,8 @@
             function test() {
                 var elm = document.getElementById('elm');
                 elm.offsetTop; // trigger layout
-                elm.style.webkitColumnSpan = '1';
-                elm.style.columnSpan = '1';
+                elm.style.webkitColumnSpan = 'none';
+                elm.style.columnSpan = 'none';
             }
         </script>
     </head>

Modified: trunk/LayoutTests/fast/multicol/newmulticol/remove-spanner6.html (209346 => 209347)


--- trunk/LayoutTests/fast/multicol/newmulticol/remove-spanner6.html	2016-12-05 21:31:23 UTC (rev 209346)
+++ trunk/LayoutTests/fast/multicol/newmulticol/remove-spanner6.html	2016-12-05 21:56:38 UTC (rev 209347)
@@ -6,8 +6,8 @@
             function test() {
                 var elm = document.getElementById('elm');
                 elm.offsetTop; // trigger layout
-                elm.style.webkitColumnSpan = '1';
-                elm.style.columnSpan = '1';
+                elm.style.webkitColumnSpan = 'none';
+                elm.style.columnSpan = 'none';
             }
         </script>
     </head>

Modified: trunk/LayoutTests/fast/multicol/span/span-as-immediate-child-property-removal.html (209346 => 209347)


--- trunk/LayoutTests/fast/multicol/span/span-as-immediate-child-property-removal.html	2016-12-05 21:31:23 UTC (rev 209346)
+++ trunk/LayoutTests/fast/multicol/span/span-as-immediate-child-property-removal.html	2016-12-05 21:56:38 UTC (rev 209347)
@@ -132,7 +132,7 @@
 <script>
 spanners = document.getElementsByTagName('h2');
 for (i = 0; i < spanners.length; i++) {
-    spanners[i].style.webkitColumnSpan = 1;
+    spanners[i].style.webkitColumnSpan = 'none';
 }
 </script>
 
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to