Update our treatment of repeating headers in tables

third_party/WebKit/Source/core/layout/LayoutTableRow.cpp

 /*
  * Copyright (C) 1997 Martin Jones (mjones@kde.org)
  *           (C) 1997 Torben Weis (weis@kde.org)
  *           (C) 1998 Waldo Bastian (bastian@kde.org)
  *           (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  * Copyright (C) 2003, 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2013
 *                Apple Inc.
  *               All rights reserved.
  *
@@ skipping 315 matching lines @@
       cell->VisualOverflowRectForPropagation();
   cell_visual_overflow_rect.Move(cell_row_offset);
   AddContentsVisualOverflow(cell_visual_overflow_rect);
 
   LayoutRect cell_layout_overflow_rect =
       cell->LayoutOverflowRectForPropagation();
   cell_layout_overflow_rect.Move(cell_row_offset);
   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

[mstensho (USE GERRIT), 2017/05/22 10:40:44]

That bug resolved by this CL?

[rhogan, 2017/05/22 18:06:04]

Yes, updated.

-  if (RowIndex())
-    return false;
-  LayoutTableSection* header = Table()->Header();
-  return header && Table()->SectionAbove(Section()) == header &&
-         header->GetPaginationBreakability() != kAllowAnyBreaks;
-}
-
 }  // namespace blink