[CSS Grid Layout] Use gridRowCount() and gridColumnCount() when possible
authorrego@igalia.com <rego@igalia.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 23 May 2014 20:31:21 +0000 (20:31 +0000)
committerrego@igalia.com <rego@igalia.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 23 May 2014 20:31:21 +0000 (20:31 +0000)
https://bugs.webkit.org/show_bug.cgi?id=133213

Reviewed by Sergio Villar Senin.

Use gridRowCount() and gridColumnCount() instead of m_grid.size() and
m_grid[0].size() to make the code more descriptive and easier to
understand.

No new tests (No change in behavior).

* rendering/RenderGrid.cpp:
(WebCore::RenderGrid::growGrid):
(WebCore::RenderGrid::placeAutoMajorAxisItemOnGrid):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@169277 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/rendering/RenderGrid.cpp

index 6e9f429..736a37d 100644 (file)
@@ -1,3 +1,20 @@
+2014-05-23  Manuel Rego Casasnovas  <rego@igalia.com>
+
+        [CSS Grid Layout] Use gridRowCount() and gridColumnCount() when possible
+        https://bugs.webkit.org/show_bug.cgi?id=133213
+
+        Reviewed by Sergio Villar Senin.
+
+        Use gridRowCount() and gridColumnCount() instead of m_grid.size() and
+        m_grid[0].size() to make the code more descriptive and easier to
+        understand.
+
+        No new tests (No change in behavior).
+
+        * rendering/RenderGrid.cpp:
+        (WebCore::RenderGrid::growGrid):
+        (WebCore::RenderGrid::placeAutoMajorAxisItemOnGrid):
+
 2014-05-23  Alexey Proskuryakov  <ap@apple.com>
 
         REGRESSION (r153917): Chart for the Category Explorer within Quicken Essentials
index 8cd1564..da36c29 100644 (file)
@@ -630,13 +630,13 @@ bool RenderGrid::tracksAreWiderThanMinTrackBreadth(GridTrackSizingDirection dire
 void RenderGrid::growGrid(GridTrackSizingDirection direction)
 {
     if (direction == ForColumns) {
-        const size_t oldColumnSize = m_grid[0].size();
-        for (size_t row = 0; row < m_grid.size(); ++row)
+        const size_t oldColumnSize = gridColumnCount();
+        for (size_t row = 0; row < gridRowCount(); ++row)
             m_grid[row].grow(oldColumnSize + 1);
     } else {
-        const size_t oldRowSize = m_grid.size();
+        const size_t oldRowSize = gridRowCount();
         m_grid.grow(oldRowSize + 1);
-        m_grid[oldRowSize].grow(m_grid[0].size());
+        m_grid[oldRowSize].grow(gridColumnCount());
     }
 }
 
@@ -776,8 +776,8 @@ void RenderGrid::placeAutoMajorAxisItemOnGrid(RenderBox* gridItem)
     }
 
     // We didn't find an empty grid area so we need to create an extra major axis line and insert our gridItem in it.
-    const size_t columnIndex = (autoPlacementMajorAxisDirection() == ForColumns) ? m_grid[0].size() : minorAxisIndex;
-    const size_t rowIndex = (autoPlacementMajorAxisDirection() == ForColumns) ? minorAxisIndex : m_grid.size();
+    const size_t columnIndex = (autoPlacementMajorAxisDirection() == ForColumns) ? gridColumnCount() : minorAxisIndex;
+    const size_t rowIndex = (autoPlacementMajorAxisDirection() == ForColumns) ? minorAxisIndex : gridRowCount();
     growGrid(autoPlacementMajorAxisDirection());
     insertItemIntoGrid(gridItem, rowIndex, columnIndex);
 }