[css-grid] grid-auto-flow|row should take a <track-size>+

third_party/WebKit/Source/core/css/ComputedStyleCSSValueMapping.cpp

 /*
  * Copyright (C) 2004 Zack Rusin <zack@kde.org>
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011, 2012 Apple Inc. All rights reserved.
  * Copyright (C) 2007 Alexey Proskuryakov <ap@webkit.org>
  * Copyright (C) 2007 Nicholas Shanks <webkit@nickshanks.com>
  * Copyright (C) 2011 Sencha, Inc. All rights reserved.
  * Copyright (C) 2015 Google Inc. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Lesser General Public
@@ skipping 800 matching lines @@
 {
     if (collector.isEmpty())
         return;
 
     CSSGridLineNamesValue* lineNames = CSSGridLineNamesValue::create();
     collector.collectLineNamesForIndex(*lineNames, i);
     if (lineNames->length())
         list.append(*lineNames);
 }
 
+static CSSValue* valueForGridTrackSizeList(GridTrackSizingDirection direction, const ComputedStyle& style)
+{
+    const Vector<GridTrackSize>& autoTrackSizes = direction == ForColumns ? style.gridAutoColumns() : style.gridAutoRows();
+
+    CSSValueList* list = CSSValueList::createSpaceSeparated();
+    for (auto& trackSize : autoTrackSizes)
+        list->append(*specifiedValueForGridTrackSize(trackSize, style));
+    return list;
+}
+
 static CSSValue* valueForGridTrackList(GridTrackSizingDirection direction, const LayoutObject* layoutObject, const ComputedStyle& style)
 {
     bool isRowAxis = direction == ForColumns;
     const Vector<GridTrackSize>& trackSizes = isRowAxis ? style.gridTemplateColumns() : style.gridTemplateRows();
     const Vector<GridTrackSize>& autoRepeatTrackSizes = isRowAxis ? style.gridAutoRepeatColumns() : style.gridAutoRepeatRows();
     bool isLayoutGrid = layoutObject && layoutObject->isLayoutGrid();
 
     // Handle the 'none' case.
     bool trackListIsEmpty = trackSizes.isEmpty() && autoRepeatTrackSizes.isEmpty();
     if (isLayoutGrid && trackListIsEmpty) {
@@ skipping 1129 matching lines @@
             list->append(*CSSPrimitiveValue::createIdentifier(CSSValueDense));
             break;
         default:
             // Do nothing.
             break;
         }
 
         return list;
     }
     // Specs mention that getComputedStyle() should return the used value of the property instead of the computed
-    // one for grid-definition-{rows|columns} but not for the grid-auto-{rows|columns} as things like
+    // one for grid-template-{rows|columns} but not for the grid-auto-{rows|columns} as things like
     // grid-auto-columns: 2fr; cannot be resolved to a value in pixels as the '2fr' means very different things
     // depending on the size of the explicit grid or the number of implicit tracks added to the grid. See
     // http://lists.w3.org/Archives/Public/www-style/2013Nov/0014.html
     case CSSPropertyGridAutoColumns:
-        return specifiedValueForGridTrackSize(style.gridAutoColumns(), style);
+        return valueForGridTrackSizeList(ForColumns, style);
     case CSSPropertyGridAutoRows:
-        return specifiedValueForGridTrackSize(style.gridAutoRows(), style);
+        return valueForGridTrackSizeList(ForRows, style);
 
     case CSSPropertyGridTemplateColumns:
         return valueForGridTrackList(ForColumns, layoutObject, style);
     case CSSPropertyGridTemplateRows:
         return valueForGridTrackList(ForRows, layoutObject, style);
 
     case CSSPropertyGridColumnStart:
         return valueForGridPosition(style.gridColumnStart());
     case CSSPropertyGridColumnEnd:
         return valueForGridPosition(style.gridColumnEnd());
@@ skipping 1018 matching lines @@
     case CSSPropertyAll:
         return nullptr;
     default:
         break;
     }
     ASSERT_NOT_REACHED();
     return nullptr;
 }
 
 } // namespace blink