Fix paint invalidation rects for repeating paginated table headers.

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

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "core/paint/TableSectionPainter.h"
 
 #include "core/layout/LayoutTableCell.h"
 #include "core/layout/LayoutTableCol.h"
 #include "core/layout/LayoutTableRow.h"
 #include "core/paint/BoxClipper.h"
@@ skipping 19 matching lines @@
     if (row > dirtiedRows.start() && m_layoutTableSection.primaryCellAt(row - 1, column) == cell)
         return nullptr;
     // We have painted (row, column) when painting (row, column -1).
     if (column > dirtiedColumns.start() && m_layoutTableSection.primaryCellAt(row, column - 1) == cell)
         return nullptr;
     return cell;
 }
 
 void TableSectionPainter::paintRepeatingHeaderGroup(const PaintInfo& paintInfo, const LayoutPoint& paintOffset, const CollapsedBorderValue& currentBorderValue, ItemToPaint itemToPaint)
 {
-    if (m_layoutTableSection.getPaginationBreakability() == LayoutBox::AllowAnyBreaks)
-        return;
-    // TODO(rhogan): Should we paint a header repeatedly if it's self-painting?
-    if (m_layoutTableSection.hasSelfPaintingLayer())
-        return;
-    LayoutTable* table = m_layoutTableSection.table();
-    LayoutUnit pageHeight = table->pageLogicalHeightForOffset(LayoutUnit());
-    if (!pageHeight)
+    if (!m_layoutTableSection.hasRepeatingHeaderGroup())
         return;
 
+    LayoutTable* table = m_layoutTableSection.table();
     LayoutPoint paginationOffset = paintOffset;
+    LayoutUnit pageHeight = table->pageLogicalHeightForOffset(LayoutUnit());
     // Move paginationOffset to the top of the second page.
     paginationOffset.move(0, pageHeight - table->pageLogicalOffset());
     // Now move paginationOffset to the top of the page the cull rect starts on.
     if (paintInfo.cullRect().m_rect.y() > paginationOffset.y())
         paginationOffset.move(0, pageHeight * static_cast<int>((paintInfo.cullRect().m_rect.y() - paginationOffset.y()) / pageHeight));
     LayoutUnit bottomBound = std::min(LayoutUnit(paintInfo.cullRect().m_rect.maxY()), paintOffset.y() + table->logicalHeight());
     while (paginationOffset.y() < bottomBound) {
         LayoutPoint nestedOffset = paginationOffset + LayoutPoint(0, m_layoutTableSection.offsetForRepeatingHeader());
         if (itemToPaint == PaintCollapsedBorders)
             paintCollapsedSectionBorders(paintInfo, nestedOffset, currentBorderValue);
@@ skipping 238 matching lines @@
     DisplayItem::Type type = shadowStyle == Normal ? DisplayItem::TableSectionBoxShadowNormal : DisplayItem::TableSectionBoxShadowInset;
     if (LayoutObjectDrawingRecorder::useCachedDrawingIfPossible(paintInfo.context, m_layoutTableSection, type))
         return;
 
     LayoutRect bounds = BoxPainter(m_layoutTableSection).boundsForDrawingRecorder(paintOffset);
     LayoutObjectDrawingRecorder recorder(paintInfo.context, m_layoutTableSection, type, bounds);
     BoxPainter::paintBoxShadow(paintInfo, LayoutRect(paintOffset, m_layoutTableSection.size()), m_layoutTableSection.styleRef(), shadowStyle);
 }
 
 } // namespace blink