Don't update column position in StyleDidChange.

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

Index: third_party/WebKit/Source/core/layout/LayoutTable.cpp
diff --git a/third_party/WebKit/Source/core/layout/LayoutTable.cpp b/third_party/WebKit/Source/core/layout/LayoutTable.cpp
index 39a9e325d72617e5e36baa46c59167bf1473ebe1..e7dfbd5d600d8bd73e9b0dd773b26d280807bc38 100644
--- a/third_party/WebKit/Source/core/layout/LayoutTable.cpp
+++ b/third_party/WebKit/Source/core/layout/LayoutTable.cpp
@@ -83,7 +83,6 @@ void LayoutTable::StyleDidChange(StyleDifference diff,
   // In the collapsed border model, there is no cell spacing.
   h_spacing_ = ShouldCollapseBorders() ? 0 : Style()->HorizontalBorderSpacing();

[mstensho (USE GERRIT), 2017/05/04 13:37:52]

Furthermore, why are we updating h_spacing_ and v_spacing_ at this point? Can't
it wait until UpdateLayout()?

[rune, 2017/05/04 20:32:03]

That sounds like a good idea, assuming LayoutTable::UpdateLayout is always
called when the new h/v_border_spacing_ needs to be updated.

   v_spacing_ = ShouldCollapseBorders() ? 0 : Style()->VerticalBorderSpacing();
-  effective_column_positions_[0] = h_spacing_;
 
   if (!table_layout_ ||
       Style()->IsFixedTableLayout() != old_fixed_table_layout) {