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

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

Issue 2356183002: Stay put at the top of the current page when inserting a forced break. (Closed)
Patch Set: 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/LayoutBlock.h
diff --git a/third_party/WebKit/Source/core/layout/LayoutBlock.h b/third_party/WebKit/Source/core/layout/LayoutBlock.h
index 723e398ea0100c31e8f05f150a6bbb571c3d99f3..07df0816b3e15ad1d8fec3cd7860f8d89531baef 100644
--- a/third_party/WebKit/Source/core/layout/LayoutBlock.h
+++ b/third_party/WebKit/Source/core/layout/LayoutBlock.h
@@ -380,15 +380,7 @@ protected:
bool isPageLogicalHeightKnown(LayoutUnit logicalOffset) const { return pageLogicalHeightForOffset(logicalOffset); }
// Returns the logical offset at the top of the next page, for a given offset.
- //
- // If the given offset is at a page boundary, using AssociateWithLatterPage as PageBoundaryRule
- // will move us one page ahead (since the offset is at the top of the "current" page). Using
- // AssociateWithFormerPage instead will keep us where we are (since the offset is at the bottom
- // of the "current" page, which is exactly the same offset as the top offset on the next page).
- //
- // For a page height of 800px, AssociateWithLatterPage will return 1600 if the value passed in
- // is 800. AssociateWithFormerPage will simply return 800.
- LayoutUnit nextPageLogicalTop(LayoutUnit logicalOffset, PageBoundaryRule) const;
+ LayoutUnit nextPageLogicalTop(LayoutUnit logicalOffset) const;
// Paginated content inside this block was laid out.
// |logicalBottomOffsetAfterPagination| is the logical bottom offset of the child content after

Powered by Google App Engine
This is Rietveld 408576698