[CSS Grid Layout] Fix grid-template-areas parsing to avoid spaces
[WebKit-https.git] / Source / WebCore / ChangeLog
index 6e34b42..4a014aa 100644 (file)
@@ -1,3 +1,31 @@
+2015-06-12  Manuel Rego Casasnovas  <rego@igalia.com>
+
+        [CSS Grid Layout] Fix grid-template-areas parsing to avoid spaces
+        https://bugs.webkit.org/show_bug.cgi?id=145860
+
+        Reviewed by Sergio Villar Senin.
+
+        The spec doesn't require to have spaces between unnamed and named areas
+        in grid-template-areas syntax. But spaces are currently required in our
+        code.
+
+        This was confirmed in the CSS WG mailing list:
+        https://lists.w3.org/Archives/Public/www-style/2015May/0239.html
+
+        This patch updates grid-template-areas parsing to allow the possibility
+        of removing spaces between unnamed and named areas.
+
+        Added new cases to fast/css-grid-layout/grid-template-areas-get-set.html.
+
+        * css/CSSParser.cpp:
+        (WebCore::parseGridTemplateAreasColumnNames): New helper method to
+        determine the column names split by white spaces or dots.
+        (WebCore::CSSParser::parseGridTemplateAreasRow): Use the new helper
+        method to get the column names.
+        (WebCore::containsOnlyDots): Deleted. Not needed anymore as
+        parseGridTemplateAreasColumnNames() is using a single dot for unnamed
+        grid areas (despite of being defined with 1 or more dots).
+
 2015-06-12  Eric Carlson  <eric.carlson@apple.com>
 
         [Mac] AirPlay menu button doesn't always show on page load