Invalidate collapsed borders using currentColor when color changes

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

Index: third_party/WebKit/Source/core/layout/LayoutTableBoxComponent.cpp
diff --git a/third_party/WebKit/Source/core/layout/LayoutTableBoxComponent.cpp b/third_party/WebKit/Source/core/layout/LayoutTableBoxComponent.cpp
index b0e06e8778e98768917c88495e6dc416739615c6..947d3eba920a21bc7f0fbcc3c7ea83bd2c07a0e3 100644
--- a/third_party/WebKit/Source/core/layout/LayoutTableBoxComponent.cpp
+++ b/third_party/WebKit/Source/core/layout/LayoutTableBoxComponent.cpp
@@ -10,6 +10,17 @@
 
 namespace blink {
 
+void LayoutTableBoxComponent::InvalidateCollapsedBordersOnStyleChange(
+    const LayoutObject& table_part,
+    LayoutTable& table,
+    const StyleDifference& diff,
+    const ComputedStyle& old_style) {

[Xianzhu, 2017/04/24 21:09:22]

This doesn't include the !table->SelfNeedsLayout() and similar conditions in
StyleDidChange() methods because
- the conditions were inconsistent;
- the new code is simpler and the extra cost is trivial;
- in the future we can avoid invalidating collapsed borders on layout change,
but only on border style change and NeedsSectionRecalc instead.

+  if (old_style.Border() != table_part.StyleRef().Border() ||
+      (diff.TextDecorationOrColorChanged() &&
+       table_part.StyleRef().HasBorderColorReferencingCurrentColor()))
+    table.InvalidateCollapsedBorders();
+}
+
 bool LayoutTableBoxComponent::DoCellsHaveDirtyWidth(
     const LayoutObject& table_part,
     const LayoutTable& table,