Offset repeating theads correctly when two tables adjoin at a page border

third_party/WebKit/Source/core/paint/TableSectionPainter.cpp

Index: third_party/WebKit/Source/core/paint/TableSectionPainter.cpp
diff --git a/third_party/WebKit/Source/core/paint/TableSectionPainter.cpp b/third_party/WebKit/Source/core/paint/TableSectionPainter.cpp
index 90aebb1aacbc75d845cc8b03c0f5959c377af307..923902c8e243b0af42263911eb9a9fb04005c896 100644
--- a/third_party/WebKit/Source/core/paint/TableSectionPainter.cpp
+++ b/third_party/WebKit/Source/core/paint/TableSectionPainter.cpp
@@ -45,7 +45,9 @@ void TableSectionPainter::paintRepeatingHeaderGroup(const PaintInfo& paintInfo,
     LayoutUnit pageHeight = table->pageLogicalHeightForOffset(LayoutUnit());
 
     // Move paginationOffset to the top of the next page.
-    LayoutUnit offsetToNextPage = pageHeight - intMod(table->pageLogicalOffset(), pageHeight);
+    // The header may have a pagination strut before it so we need to account for that when establishing its position.
+    LayoutUnit headerGroupOffset = table->pageLogicalOffset() + m_layoutTableSection.paginationStrutForRow(m_layoutTableSection.firstRow(), table->pageLogicalOffset());

[mstensho (USE GERRIT), 2016/08/10 09:42:48]

m_layoutTableSection.firstRow() may be nullptr.

What's the difference between calculating the pagination strut for the first
row, and simply using its logicalTop(), BTW?

[rhogan, 2016/08/10 18:05:14]

We don't propagate the strut to the logical top of the row or the section, we
just add it to the logicalHeight() of the row. We don't even store it like we do
for LayoutBox. I'm thinking it might be cleaner to just store it on the
paginationStrut() on the LayoutTableRow rather than calculate it like this. We
only need to call on it when the section is a header group but I guess it would
be less confusing to store it in all cases.

+    LayoutUnit offsetToNextPage = pageHeight - intMod(headerGroupOffset, pageHeight);
     paginationOffset.move(0, offsetToNextPage);
     // Now move paginationOffset to the top of the page the cull rect starts on.
     if (paintInfo.cullRect().m_rect.y() > paginationOffset.y())