[CSS Grid Layout] Remove old FIXME in RenderGrid::placeItemsOnGrid()
authorrego@igalia.com <rego@igalia.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 23 Feb 2015 10:28:47 +0000 (10:28 +0000)
committerrego@igalia.com <rego@igalia.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 23 Feb 2015 10:28:47 +0000 (10:28 +0000)
https://bugs.webkit.org/show_bug.cgi?id=141899

Reviewed by Sergio Villar Senin.

There was a FIXME about not re-resolving positions if the grid is grown during
auto-placement. But as the spec has changed [1] and you can only refer to the
explicit grid (even if you use for example grid-column: 1 / -1). It seems that
we can get rid of this FIXME as it's not a causing a problem anymore.

[1] http://dev.w3.org/csswg/css-grid/#grid-placement-int

No new tests (no change in behavior).

* rendering/RenderGrid.cpp:
(WebCore::RenderGrid::placeItemsOnGrid): Remove FIXME.

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

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

index 40e3562..3755811 100644 (file)
@@ -1,3 +1,22 @@
+2015-02-23  Manuel Rego Casasnovas  <rego@igalia.com>
+
+        [CSS Grid Layout] Remove old FIXME in RenderGrid::placeItemsOnGrid()
+        https://bugs.webkit.org/show_bug.cgi?id=141899
+
+        Reviewed by Sergio Villar Senin.
+
+        There was a FIXME about not re-resolving positions if the grid is grown during
+        auto-placement. But as the spec has changed [1] and you can only refer to the
+        explicit grid (even if you use for example grid-column: 1 / -1). It seems that
+        we can get rid of this FIXME as it's not a causing a problem anymore.
+
+        [1] http://dev.w3.org/csswg/css-grid/#grid-placement-int
+
+        No new tests (no change in behavior).
+
+        * rendering/RenderGrid.cpp:
+        (WebCore::RenderGrid::placeItemsOnGrid): Remove FIXME.
+
 2015-02-22  Gyuyoung Kim  <gyuyoung.kim@samsung.com>
 
         Move std::unique_ptr<>|make_unique<> to WebCore/page/scrolling
index 867b3e3..4ca6bd8 100644 (file)
@@ -808,8 +808,6 @@ void RenderGrid::placeItemsOnGrid()
     Vector<RenderBox*> autoMajorAxisAutoGridItems;
     Vector<RenderBox*> specifiedMajorAxisAutoGridItems;
     for (RenderBox* child = m_orderIterator.first(); child; child = m_orderIterator.next()) {
-        // FIXME: We never re-resolve positions if the grid is grown during auto-placement which may lead auto / <integer>
-        // positions to not match the author's intent. The specification is unclear on what should be done in this case.
         std::unique_ptr<GridSpan> rowPositions = GridResolvedPosition::resolveGridPositionsFromStyle(style(), *child, ForRows);
         std::unique_ptr<GridSpan> columnPositions = GridResolvedPosition::resolveGridPositionsFromStyle(style(), *child, ForColumns);
         if (!rowPositions || !columnPositions) {