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

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

Issue 2360913004: Support for multiple block fragments in getClientRects(). (Closed)
Patch Set: fast/overflow/overflow-height-float-not-removed-crash3.html crashed because saturated LayoutUnits caused zero height Created 4 years, 3 months 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/MultiColumnFragmentainerGroup.h
diff --git a/third_party/WebKit/Source/core/layout/MultiColumnFragmentainerGroup.h b/third_party/WebKit/Source/core/layout/MultiColumnFragmentainerGroup.h
index 343c8df31e7f31fcfe940378e79304fa4406e281..fcf5a21235a364abd10c085d69eb0671abae8ca8 100644
--- a/third_party/WebKit/Source/core/layout/MultiColumnFragmentainerGroup.h
+++ b/third_party/WebKit/Source/core/layout/MultiColumnFragmentainerGroup.h
@@ -67,6 +67,7 @@ public:
LayoutPoint visualPointToFlowThreadPoint(const LayoutPoint& visualPoint) const;
LayoutRect fragmentsBoundingBox(const LayoutRect& boundingBoxInFlowThread) const;
+ LayoutRect flowThreadPortionRectAt(unsigned columnIndex) const;
LayoutRect flowThreadPortionOverflowRectAt(unsigned columnIndex) const;
// Get the first and the last column intersecting the specified block range.
@@ -93,7 +94,6 @@ private:
LayoutRect columnRectAt(unsigned columnIndex) const;
LayoutUnit logicalTopInFlowThreadAt(unsigned columnIndex) const { return m_logicalTopInFlowThread + columnIndex * m_columnHeight; }
- LayoutRect flowThreadPortionRectAt(unsigned columnIndex) const;
// Return the column that the specified visual point belongs to. Only the coordinate on the
// column progression axis is relevant. Every point belongs to a column, even if said point is

Powered by Google App Engine
This is Rietveld 408576698