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

Unified Diff: third_party/WebKit/Source/core/layout/LayoutTableRow.cpp

Issue 2882043002: Update our treatment of repeating headers in tables (Closed)
Patch Set: bug 720620 Created 3 years, 7 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/LayoutTableRow.cpp
diff --git a/third_party/WebKit/Source/core/layout/LayoutTableRow.cpp b/third_party/WebKit/Source/core/layout/LayoutTableRow.cpp
index 9f89aaaaa044c17475756308e90889cdd9b28b2b..4b68ff68b116917e258cab993078ada7c4a9b953 100644
--- a/third_party/WebKit/Source/core/layout/LayoutTableRow.cpp
+++ b/third_party/WebKit/Source/core/layout/LayoutTableRow.cpp
@@ -333,18 +333,4 @@ void LayoutTableRow::AddOverflowFromCell(const LayoutTableCell* cell) {
AddLayoutOverflow(cell_layout_overflow_rect);
}
-bool LayoutTableRow::IsFirstRowInSectionAfterHeader() const {
- // If there isn't room on the page for at least one content row after the
- // header group, then we won't repeat the header on each page.
- // https://drafts.csswg.org/css-tables-3/#repeated-headers reads like
- // it wants us to drop headers on only the pages that a single row
- // won't fit but we avoid the complexity of that reading until it
- // is clarified. Tracked by crbug.com/675904
- if (RowIndex())
- return false;
- LayoutTableSection* header = Table()->Header();
- return header && Table()->SectionAbove(Section()) == header &&
- header->GetPaginationBreakability() != kAllowAnyBreaks;
-}
-
} // namespace blink
« no previous file with comments | « third_party/WebKit/Source/core/layout/LayoutTableRow.h ('k') | third_party/WebKit/Source/core/layout/LayoutTableSection.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698