[CSS Grid Layout] Support dots sequences in grid-template-areas

Source/core/css/parser/CSSPropertyParser.cpp

 /*
  * Copyright (C) 2003 Lars Knoll (knoll@kde.org)
  * Copyright (C) 2005 Allan Sandfeld Jensen (kde@carewolf.com)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011, 2012 Apple Inc. All rights reserved.
  * Copyright (C) 2007 Nicholas Shanks <webkit@nickshanks.com>
  * Copyright (C) 2008 Eric Seidel <eric@webkit.org>
  * Copyright (C) 2009 Torch Mobile Inc. All rights reserved. (http://www.torchmobile.com/)
  * Copyright (C) 2012 Adobe Systems Incorporated. All rights reserved.
  * Copyright (C) 2012 Intel Corporation. All rights reserved.
  *
@@ skipping 3621 matching lines @@
 
         return cssValuePool().createValue(flexValue, CSSPrimitiveValue::CSS_FR);
     }
 
     if (!validUnit(currentValue, FNonNeg | FLength | FPercent))
         return nullptr;
 
     return createPrimitiveNumericValue(currentValue);
 }
 
+static bool containsOnlyDots(const String& string)
+{
+    ASSERT(!string.isEmpty());
+    for (unsigned i = 0; i < string.length(); ++i) {
+        if (string.characterAt(i) != '.')
+            return false;
+    }
+    return true;

[svillar, 2015/06/05 10:38:09]

I think it's better to do like LayoutText::containsOnlyWhitespace() does and get
the StringImpl to loop over and perform the checks. Also note that the use of
characterAt() is not recommended (see WTFString.h).

[Manuel Rego, 2015/06/05 11:20:10]

Thanks for the pointer, I've followed a similar approach
to LayoutText::containsOnlyWhitespace().

+}
+
 bool CSSPropertyParser::parseGridTemplateAreasRow(NamedGridAreaMap& gridAreaMap, const size_t rowCount, size_t& columnCount)
 {
     CSSParserValue* currentValue = m_valueList->current();
     if (!currentValue || currentValue->unit != CSSPrimitiveValue::CSS_STRING)
         return false;
 
     String gridRowNames = currentValue->string;
     if (gridRowNames.isEmpty() || gridRowNames.containsOnlyWhitespace())
         return false;
 
     Vector<String> columnNames;
     gridRowNames.split(' ', columnNames);
 
     if (!columnCount) {
         columnCount = columnNames.size();
         ASSERT(columnCount);
     } else if (columnCount != columnNames.size()) {
         // The declaration is invalid is all the rows don't have the number of columns.
         return false;
     }
 
     for (size_t currentCol = 0; currentCol < columnCount; ++currentCol) {
         const String& gridAreaName = columnNames[currentCol];
 
         // Unamed areas are always valid (we consider them to be 1x1).
-        if (gridAreaName == ".")
+        if (containsOnlyDots(gridAreaName))
             continue;
 
         // We handle several grid areas with the same name at once to simplify the validation code.
         size_t lookAheadCol;
         for (lookAheadCol = currentCol; lookAheadCol < (columnCount - 1); ++lookAheadCol) {
             if (columnNames[lookAheadCol + 1] != gridAreaName)
                 break;
         }
 
         NamedGridAreaMap::iterator gridAreaIt = gridAreaMap.find(gridAreaName);
@@ skipping 4729 matching lines @@
         }
     }
 
     if (!list->length())
         return nullptr;
 
     return list.release();
 }
 
 } // namespace blink