Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(387)

Unified Diff: third_party/WebKit/Source/core/layout/LayoutGrid.h

Issue 2536703002: [css-grid] Pass Grid as argument to the items' placements methods (Closed)
Patch Set: Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/core/layout/LayoutGrid.h
diff --git a/third_party/WebKit/Source/core/layout/LayoutGrid.h b/third_party/WebKit/Source/core/layout/LayoutGrid.h
index 3fe38083260ddb7b5642c9fff26cad6f4219bc61..93063df7ba57b3f671e7826b9d1e4796bc6c36ea 100644
--- a/third_party/WebKit/Source/core/layout/LayoutGrid.h
+++ b/third_party/WebKit/Source/core/layout/LayoutGrid.h
@@ -146,17 +146,20 @@ class LayoutGrid final : public LayoutBlock {
std::unique_ptr<OrderedTrackIndexSet> computeEmptyTracksForAutoRepeat(
GridTrackSizingDirection) const;
- void placeItemsOnGrid(SizingOperation);
- void populateExplicitGridAndOrderIterator();
+ class Grid;
+ void placeItemsOnGrid(Grid&, SizingOperation);
+ void populateExplicitGridAndOrderIterator(Grid&) const;
std::unique_ptr<GridArea> createEmptyGridAreaAtSpecifiedPositionsOutsideGrid(
const LayoutBox&,
GridTrackSizingDirection,
const GridSpan& specifiedPositions) const;
- void placeSpecifiedMajorAxisItemsOnGrid(const Vector<LayoutBox*>&);
- void placeAutoMajorAxisItemsOnGrid(const Vector<LayoutBox*>&);
+ void placeSpecifiedMajorAxisItemsOnGrid(Grid&,
+ const Vector<LayoutBox*>&) const;
+ void placeAutoMajorAxisItemsOnGrid(Grid&, const Vector<LayoutBox*>&) const;
void placeAutoMajorAxisItemOnGrid(
+ Grid&,
LayoutBox&,
- std::pair<size_t, size_t>& autoPlacementCursor);
+ std::pair<size_t, size_t>& autoPlacementCursor) const;
GridTrackSizingDirection autoPlacementMajorAxisDirection() const;
GridTrackSizingDirection autoPlacementMinorAxisDirection() const;

Powered by Google App Engine
This is Rietveld 408576698