Display table header groups at the top of each page

third_party/WebKit/Source/core/paint/TablePainter.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/TablePainter.h"
 
 #include "core/layout/LayoutTable.h"
 #include "core/layout/LayoutTableSection.h"
+#include "core/layout/LayoutView.h"
 #include "core/style/CollapsedBorderValue.h"
 #include "core/paint/BoxClipper.h"
 #include "core/paint/BoxPainter.h"
 #include "core/paint/LayoutObjectDrawingRecorder.h"
 #include "core/paint/ObjectPainter.h"
 #include "core/paint/PaintInfo.h"
 #include "core/paint/ScopeRecorder.h"
 #include "core/paint/TableSectionPainter.h"
 
 namespace blink {
 
+void TablePainter::paintRepeatingHeaderGroup(const PaintInfo& paintInfo, const LayoutPoint& paintOffset, LayoutBox* child, const CollapsedBorderValue& currentBorderValue, ItemToPaint itemToPaint)

[Xianzhu, 2016/05/31 02:11:33]

Pass paintInfoForDescendants from caller, then you can avoid
paintInfo.forDescendants()) in this function. 

[Xianzhu, 2016/06/01 19:19:39]

s/LayoutBox* child/const LayoutTableSection& headerSection/

+{
+    if (m_layoutTable.header() != child)
+        return;
+    if (child->getPaginationBreakability() == LayoutTableBoxComponent::AllowAnyBreaks)

[Xianzhu, 2016/05/31 02:11:33]

Use LayoutBox::AllowAnyBreaks.

+        return;
+    LayoutUnit printedPageLogicalHeight = child->view() ? child->view()->pageLogicalHeight() : LayoutUnit();
+    LayoutUnit pageHeight = std::max(printedPageLogicalHeight, m_layoutTable.flowThreadContainingBlock() ? m_layoutTable.pageLogicalHeightForOffset(LayoutUnit()) : LayoutUnit());
+    if (!pageHeight)
+        return;
+
+    int pages = static_cast<int>(ceilf(m_layoutTable.logicalHeight() / pageHeight));
+    LayoutPoint paginationOffset = paintOffset;
+    paginationOffset += LayoutPoint(0, pageHeight - m_layoutTable.pageLogicalOffset());
+    for (int i = 1; i < pages; i++) {
+        LayoutPoint childPoint = m_layoutTable.flipForWritingModeForChild(child, paginationOffset);
+        if (itemToPaint == PaintCollapsedBorders) {
+            LayoutTableSection* section = toLayoutTableSection(child);
+            TableSectionPainter(*section).paintCollapsedBorders(paintInfo.forDescendants(), childPoint, currentBorderValue);

[Xianzhu, 2016/06/01 19:19:39]

Move the common code (LayoutTableSection* section = ...; TableSectionPainter
sectionPainter(*section)) out of the loop.

+        } else {
+            child->paint(paintInfo.forDescendants(), childPoint);
+        }
+        paginationOffset += LayoutPoint(0, pageHeight);
+    }

[Xianzhu, 2016/05/31 02:11:34]

We paint each page as a layer fragment in PaintLayerPainter in a loop, so the
above loop is redundant. Here you should paint for one page only.

[rhogan, 2016/06/01 18:39:46]

Right, but table headers don't have a layer - so the nearest object that will
paint a layer is the flow thread and it won't be able to inspect for table
header groups when collecting fragments. So this is the only place I can think
of for repeating the table headers across each page.

Maybe I'm missing something?

[Xianzhu, 2016/06/01 19:19:39]

I think this is the right place (and TableSectionPainter is more appropriate,
see comments below).

My comment was about performance. In the above loop, normally only 1 iteration
will actually paint into the cull rect of paintInfo because the cull rect is for
one page/fragment.

However, to make the code not strictly coupled to PaintLayerPainter, you can
keep the loop, but with modified starting and ending value according to the cull
rect to avoid unnecessary iterations.

+}
+
 void TablePainter::paintObject(const PaintInfo& paintInfo, const LayoutPoint& paintOffset)
 {
     PaintPhase paintPhase = paintInfo.phase;
 
     if (shouldPaintSelfBlockBackground(paintPhase)) {
         paintBoxDecorationBackground(paintInfo, paintOffset);
         if (paintPhase == PaintPhaseSelfBlockBackgroundOnly)
             return;
     }
 
     if (paintPhase == PaintPhaseMask) {
         paintMask(paintInfo, paintOffset);
         return;
     }
 
     if (paintPhase != PaintPhaseSelfOutlineOnly) {
         PaintInfo paintInfoForDescendants = paintInfo.forDescendants();
 
         for (LayoutObject* child = m_layoutTable.firstChild(); child; child = child->nextSibling()) {
             if (child->isBox() && !toLayoutBox(child)->hasSelfPaintingLayer() && (child->isTableSection() || child->isTableCaption())) {
                 LayoutPoint childPoint = m_layoutTable.flipForWritingModeForChild(toLayoutBox(child), paintOffset);
                 child->paint(paintInfoForDescendants, childPoint);
+                paintRepeatingHeaderGroup(paintInfo, paintOffset, toLayoutBox(child), CollapsedBorderValue(), PaintChild);

[Xianzhu, 2016/06/01 19:19:39]

This should be in TableSectionPainter::paint(). With this patch you'll miss the
case that the header section has self-painting layer. It may look like:

   TableSectionPainter::paint()
   {
       orignalPaint();
       if (m_layoutTableSection == m_layoutTableSection.table()->header())
           paintRepeatingHeaderGroup();
   }


(Fwiw, TableSectionPainter::paint has two callsites:
1. From TablePainter if the section has no self-painting layer;
2. From PaintLayerPainter if the section has a self-painting layer.)

             }
         }
 
         if (m_layoutTable.collapseBorders() && shouldPaintDescendantBlockBackgrounds(paintPhase) && m_layoutTable.style()->visibility() == VISIBLE) {
             // Using our cached sorted styles, we then do individual passes,
             // painting each style of border from lowest precedence to highest precedence.
             LayoutTable::CollapsedBorderValues collapsedBorders = m_layoutTable.collapsedBorders();
             size_t count = collapsedBorders.size();
             for (size_t i = 0; i < count; ++i) {
                 for (LayoutTableSection* section = m_layoutTable.bottomSection(); section; section = m_layoutTable.sectionAbove(section)) {
                     LayoutPoint childPoint = m_layoutTable.flipForWritingModeForChild(section, paintOffset);
                     TableSectionPainter(*section).paintCollapsedBorders(paintInfoForDescendants, childPoint, collapsedBorders[i]);
+                    paintRepeatingHeaderGroup(paintInfo, paintOffset, section, collapsedBorders[i], PaintCollapsedBorders);
                 }
             }
         }
     }
 
     if (shouldPaintSelfOutline(paintPhase))
         ObjectPainter(m_layoutTable).paintOutline(paintInfo, paintOffset);
 }
 
 void TablePainter::paintBoxDecorationBackground(const PaintInfo& paintInfo, const LayoutPoint& paintOffset)
@@ skipping 15 matching lines @@
         return;
 
     LayoutRect rect(paintOffset, m_layoutTable.size());
     m_layoutTable.subtractCaptionRect(rect);
 
     LayoutObjectDrawingRecorder recorder(paintInfo.context, m_layoutTable, paintInfo.phase, rect);
     BoxPainter(m_layoutTable).paintMaskImages(paintInfo, rect);
 }
 
 } // namespace blink