Let table row and section's self visual overflow cover whole collapsed borders

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 284 matching lines @@
   // Table row paints its background behind cells. If the cell spans multiple
   // rows, the row's visual rect should be expanded to cover the cell.
   // Here don't check background existence to avoid requirement to invalidate
   // overflow on change of background existence.
   if (cell->RowSpan() > 1) {
     LayoutRect cell_background_rect = cell->FrameRect();
     cell_background_rect.MoveBy(-Location());
     AddSelfVisualOverflow(cell_background_rect);
   }
 
+  // The cell and the row share the section's coordinate system. However
+  // the visual overflow should be determined in the coordinate system of
+  // the row, that's why we shift the rects by cell_row_offset below.
+  LayoutSize cell_row_offset = cell->Location() - Location();
+
+  // Let the row's self visual overflow cover the cell's whole collapsed
+  // borders. This ensures correct raster invalidation on row border style
+  // change.
+  if (const auto* collapsed_borders = cell->GetCollapsedBorderValues()) {
+    LayoutRect collapsed_border_rect =
+        cell->RectForOverflowPropagation(collapsed_borders->LocalVisualRect());
+    collapsed_border_rect.Move(cell_row_offset);
+    AddSelfVisualOverflow(collapsed_border_rect);
+  }
+
   // Should propagate cell's overflow to row if the cell has row span or has
   // overflow.
   if (cell->RowSpan() == 1 && !cell->HasOverflowModel())
     return;
 
-  // The cell and the row share the section's coordinate system. However
-  // the visual overflow should be determined in the coordinate system of
-  // the row, that's why we shift the rects by cell_row_offset below.
-  LayoutSize cell_row_offset = cell->Location() - Location();
-
   LayoutRect cell_visual_overflow_rect =
       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
   if (RowIndex())
     return false;
   LayoutTableSection* header = Table()->Header();
   return header && Table()->SectionAbove(Section()) == header &&
          header->GetPaginationBreakability() != kAllowAnyBreaks;
 }
 
 }  // namespace blink