Paint collapsed borders of a table as one display item

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

Index: third_party/WebKit/Source/core/paint/PaintInvalidator.cpp
diff --git a/third_party/WebKit/Source/core/paint/PaintInvalidator.cpp b/third_party/WebKit/Source/core/paint/PaintInvalidator.cpp
index 18181096b12946f1b6250fee2c54a311780b5c3b..18d9ae30343b2d3a0562396bc400b30da70e4292 100644
--- a/third_party/WebKit/Source/core/paint/PaintInvalidator.cpp
+++ b/third_party/WebKit/Source/core/paint/PaintInvalidator.cpp
@@ -209,11 +209,8 @@ void PaintInvalidator::updatePaintingLayer(const LayoutObject& object,
     context.paintingLayer->setNeedsPaintPhaseDescendantBlockBackgrounds();
   }
 
-  if (object.isTable()) {
-    const LayoutTable& table = toLayoutTable(object);
-    if (table.collapseBorders() && !table.collapsedBorders().isEmpty())
-      context.paintingLayer->setNeedsPaintPhaseDescendantBlockBackgrounds();
-  }
+  if (object.isTable() && toLayoutTable(object).hasCollapsedBorders())

[chrishtr, 2016/11/03 21:16:29]

The old code was just inaccurate I guess?

[Xianzhu, 2016/11/03 21:26:55]

They are equivalent. The new function LayoutTable::hasCollapsedBorders() checks
the same condition.

+    context.paintingLayer->setNeedsPaintPhaseDescendantBlockBackgrounds();
 }
 
 namespace {