[css-tables] Set table and cell widths dirty when section border 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 b0ca3aaea220adc1d37963604f08dfeed2533c95..cf9d1855c860b4c3cfcd3d74a02b8eeb7bfc788d 100644
--- a/third_party/WebKit/Source/core/layout/LayoutTableBoxComponent.cpp
+++ b/third_party/WebKit/Source/core/layout/LayoutTableBoxComponent.cpp
@@ -25,4 +25,12 @@ void LayoutTableBoxComponent::imageChanged(WrappedImagePtr, const IntRect*)
     m_backgroundChangedSinceLastPaintInvalidation = true;
 }
 
+bool LayoutTableBoxComponent::borderWidthChanged(const ComputedStyle* oldStyle, const ComputedStyle* newStyle)
+{
+    return oldStyle->borderLeftWidth() != newStyle->borderLeftWidth()

[eae, 2016/07/01 22:20:25]

"return oldStyle->border() != newStyle->border()" will compare the BorderData
objects. It will return true if the style changes in addition to the width but
triggering layout in that case doesn't seem too bad. Especially not if it
simplifies the logic.

Another option would be to add a sizeEquals method to BorderData and using that.

return return oldStyle->border().sizeEquals(newStyle->border());

[dgrogan, 2016/07/06 23:10:29]

I like this option... keep the logic about borders in the BorderData class.

+        || oldStyle->borderTopWidth() != newStyle->borderTopWidth()
+        || oldStyle->borderRightWidth() != newStyle->borderRightWidth()
+        || oldStyle->borderBottomWidth() != newStyle->borderBottomWidth();
+}
+
 } // namespace blink