Title: [209243] trunk/LayoutTests
Revision
209243
Author
[email protected]
Date
2016-12-02 10:05:33 -0800 (Fri, 02 Dec 2016)

Log Message

[CSS Parser] Fix invalid gradients test
https://bugs.webkit.org/show_bug.cgi?id=165310

Reviewed by Zalan Bujtas.

* fast/gradients/generated-gradients.html:
The content property here is using invalid syntax. It's supposed to be
space-separated, not comma-separated. The old parser was finding the first
value and then bailing when it saw the comma, but not rejecting. The new parser
properly rejected. To keep the test working, just drop all the extra invalid
gradients and keep the first one.

Modified Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (209242 => 209243)


--- trunk/LayoutTests/ChangeLog	2016-12-02 17:55:32 UTC (rev 209242)
+++ trunk/LayoutTests/ChangeLog	2016-12-02 18:05:33 UTC (rev 209243)
@@ -1,5 +1,19 @@
 2016-12-02  Dave Hyatt  <[email protected]>
 
+        [CSS Parser] Fix invalid gradients test
+        https://bugs.webkit.org/show_bug.cgi?id=165310
+
+        Reviewed by Zalan Bujtas.
+
+        * fast/gradients/generated-gradients.html:
+        The content property here is using invalid syntax. It's supposed to be
+        space-separated, not comma-separated. The old parser was finding the first
+        value and then bailing when it saw the comma, but not rejecting. The new parser
+        properly rejected. To keep the test working, just drop all the extra invalid
+        gradients and keep the first one.
+
+2016-12-02  Dave Hyatt  <[email protected]>
+
         [CSS Parser] Make sure margin and font set the implicit flag properly
         https://bugs.webkit.org/show_bug.cgi?id=165306
 

Modified: trunk/LayoutTests/fast/gradients/generated-gradients.html (209242 => 209243)


--- trunk/LayoutTests/fast/gradients/generated-gradients.html	2016-12-02 17:55:32 UTC (rev 209242)
+++ trunk/LayoutTests/fast/gradients/generated-gradients.html	2016-12-02 18:05:33 UTC (rev 209243)
@@ -2,10 +2,7 @@
 <style>
 div { }
 .radial::before { width:150px; height:150px; border:2px solid black;
-           content: -webkit-gradient(radial, 45 45, 10, 52 50, 30, from(#A7D30C), to(rgba(1,159,98,0)), color-stop(90%, #019F62)),
-                       -webkit-gradient(radial, 105 105, 20, 112 120, 50, from(#ff5f98), to(rgba(255,1,136,0)), color-stop(75%, #ff0188)),
-                       -webkit-gradient(radial, 95 15, 15, 102 20, 40, from(#00c9ff), to(rgba(0,201,255,0)), color-stop(80%, #00b5e2)),
-                       -webkit-gradient(radial, 0 150, 50, 0 140, 90, from(#f4f201), to(rgba(228, 199,0,0)), color-stop(80%, #e4c700));
+           content: -webkit-gradient(radial, 45 45, 10, 52 50, 30, from(#A7D30C), to(rgba(1,159,98,0)), color-stop(90%, #019F62));
             display: block;
         }
 .linear::after { width:130px; height:130px; border:2px solid black; 
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to