Repeat footers in paginated context

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"
 #include "core/paint/BoxPainter.h"
 #include "core/paint/LayoutObjectDrawingRecorder.h"
 #include "core/paint/ObjectPainter.h"
 #include "core/paint/PaintInfo.h"
 #include "core/paint/TableCellPainter.h"
 #include "core/paint/TableRowPainter.h"
 #include <algorithm>
 
 namespace blink {
 
+void TableSectionPainter::PaintRepeatingFooterGroup(

[mstensho (USE GERRIT), 2017/04/24 11:57:16]

Please match the order of declaration. There PaintRepeatingFooterGroup() comes
after PaintRepeatingHeaderGroup().

+    const PaintInfo& paint_info,
+    const LayoutPoint& paint_offset,
+    const CollapsedBorderValue& current_border_value,
+    ItemToPaint item_to_paint) {
+  if (!layout_table_section_.IsRepeatingFooterGroup())
+    return;
+  LayoutTable* table = layout_table_section_.Table();
+  LayoutRect sections_rect(LayoutPoint(), table->Size());
+  table->SubtractCaptionRect(sections_rect, true);

[mstensho (USE GERRIT), 2017/04/24 11:57:16]

Why do we want to keep the top captions?

+  LayoutUnit total_height_of_rows =
+      sections_rect.Height() - table->OuterBorderAfter();

[mstensho (USE GERRIT), 2017/04/24 11:57:16]

Mixing physical (Height()) and logical (OuterBorderAfter()) dimensions. Might as
well use OuterBorderBottom() perhaps, since this code already seems to assume
that writing mode isn't twisted.

+  total_height_of_rows -= (layout_table_section_.LogicalHeight() -
+                           layout_table_section_.FirstRow()->PaginationStrut());

[mstensho (USE GERRIT), 2017/04/24 11:57:16]

Will there ever be a pagination strut here? Seems wrong. That would mean that
the first row of the repeating header has been pushed to the next fragmentainer.

+  LayoutPoint top_offset = paint_offset;
+  top_offset.Move(0, -total_height_of_rows.ToInt());
+  IntRect cull_rect = paint_info.GetCullRect().rect_;
+  cull_rect.Move(0, -total_height_of_rows.ToInt());
+  for (unsigned page = 0;
+       page < table->PositionOfRepeatingFooterGroupOnPage().size(); page++) {
+    LayoutPoint pagination_offset = top_offset;
+    int footer_position =
+        table->GetPositionOfRepeatingFooterGroupOnPage(page).ToInt();
+    if (!footer_position)
+      continue;
+    pagination_offset.Move(0, footer_position);
+    if (item_to_paint == kPaintCollapsedBorders) {
+      PaintCollapsedSectionBorders(paint_info, pagination_offset,
+                                   current_border_value);
+    } else {
+      PaintSection(paint_info, pagination_offset);
+    }
+  }
+}
+
 void TableSectionPainter::PaintRepeatingHeaderGroup(
     const PaintInfo& paint_info,
     const LayoutPoint& paint_offset,
     const CollapsedBorderValue& current_border_value,
     ItemToPaint item_to_paint) {
   if (!layout_table_section_.IsRepeatingHeaderGroup())
     return;
 
   LayoutTable* table = layout_table_section_.Table();
   LayoutPoint pagination_offset = paint_offset;
   LayoutUnit page_height = table->PageLogicalHeightForOffset(LayoutUnit());
 
   LayoutUnit header_group_offset = table->BlockOffsetToFirstRepeatableHeader();
   // The header may have a pagination strut before it so we need to account for
   // that when establishing its position.
+  LayoutUnit first_row_offset;
   if (LayoutTableRow* row = layout_table_section_.FirstRow())
-    header_group_offset += row->PaginationStrut();
+    first_row_offset = row->PaginationStrut();
+  header_group_offset += first_row_offset;
   LayoutUnit offset_to_next_page =
       page_height - IntMod(header_group_offset, page_height);
-  // Move paginationOffset to the top of the next page.
+  // Move pagination_offset to the top of the next page.
   pagination_offset.Move(LayoutUnit(), offset_to_next_page);
-  // Now move paginationOffset to the top of the page the cull rect starts on.
+  // Now move pagination_offset to the top of the page the cull rect starts on.
   if (paint_info.GetCullRect().rect_.Y() > pagination_offset.Y()) {
     pagination_offset.Move(LayoutUnit(),
                            page_height * ((paint_info.GetCullRect().rect_.Y() -
                                            pagination_offset.Y()) /
                                           page_height)
                                              .ToInt());
   }
 
   // We only want to consider pages where we going to paint a row, so exclude
   // captions and border spacing from the table.
   LayoutRect sections_rect(LayoutPoint(), table->Size());
   table->SubtractCaptionRect(sections_rect);
   LayoutUnit total_height_of_rows =
       sections_rect.Height() - table->VBorderSpacing();
   LayoutUnit bottom_bound =
       std::min(LayoutUnit(paint_info.GetCullRect().rect_.MaxY()),
-               paint_offset.Y() + total_height_of_rows);
+               (paint_offset.Y() - first_row_offset) + total_height_of_rows);

[mstensho (USE GERRIT), 2017/04/24 11:57:16]

No need for the parentheses.

 
   while (pagination_offset.Y() < bottom_bound) {
     if (item_to_paint == kPaintCollapsedBorders) {
       PaintCollapsedSectionBorders(paint_info, pagination_offset,
                                    current_border_value);
     } else {
       PaintSection(paint_info, pagination_offset);
     }
     pagination_offset.Move(0, page_height.ToInt());
   }
 }
 
 void TableSectionPainter::Paint(const PaintInfo& paint_info,
                                 const LayoutPoint& paint_offset) {
   ObjectPainter(layout_table_section_)
       .CheckPaintOffset(paint_info, paint_offset);
   PaintSection(paint_info, paint_offset);
   LayoutTable* table = layout_table_section_.Table();
   if (table->Header() == layout_table_section_)
     PaintRepeatingHeaderGroup(paint_info, paint_offset, CollapsedBorderValue(),
                               kPaintSection);
+  if (table->Footer() == layout_table_section_) {

[mstensho (USE GERRIT), 2017/04/24 11:57:16]

else if

+    PaintRepeatingFooterGroup(paint_info, paint_offset, CollapsedBorderValue(),
+                              kPaintSection);
+  }
 }
 
 void TableSectionPainter::PaintSection(const PaintInfo& paint_info,
                                        const LayoutPoint& paint_offset) {
   DCHECK(!layout_table_section_.NeedsLayout());
   // avoid crashing on bugs that cause us to paint with dirty layout
   if (layout_table_section_.NeedsLayout())
     return;
 
   unsigned total_rows = layout_table_section_.NumRows();
@@ skipping 37 matching lines @@
 
 void TableSectionPainter::PaintCollapsedBorders(
     const PaintInfo& paint_info,
     const LayoutPoint& paint_offset,
     const CollapsedBorderValue& current_border_value) {
   PaintCollapsedSectionBorders(paint_info, paint_offset, current_border_value);
   LayoutTable* table = layout_table_section_.Table();
   if (table->Header() == layout_table_section_)
     PaintRepeatingHeaderGroup(paint_info, paint_offset, current_border_value,
                               kPaintCollapsedBorders);
+  if (table->Footer() == layout_table_section_) {

[mstensho (USE GERRIT), 2017/04/24 11:57:16]

else if

+    PaintRepeatingFooterGroup(paint_info, paint_offset, current_border_value,
+                              kPaintCollapsedBorders);
+  }
 }
 
 void TableSectionPainter::PaintCollapsedSectionBorders(
     const PaintInfo& paint_info,
     const LayoutPoint& paint_offset,
     const CollapsedBorderValue& current_border_value) {
   if (!layout_table_section_.NumRows() ||
       !layout_table_section_.Table()->EffectiveColumns().size())
     return;
 
@@ skipping 248 matching lines @@
                                     const PaintInfo& paint_info_for_cells,
                                     const LayoutPoint& paint_offset) {
   if (!cell.HasSelfPaintingLayer() && !cell.Row()->HasSelfPaintingLayer()) {
     LayoutPoint cell_point =
         layout_table_section_.FlipForWritingModeForChild(&cell, paint_offset);
     cell.Paint(paint_info_for_cells, cell_point);
   }
 }
 
 }  // namespace blink