Store border-*-color on SurroundData in ComputedStyle not BorderColorAndStyle

third_party/WebKit/Source/core/style/ComputedStyle.cpp

Index: third_party/WebKit/Source/core/style/ComputedStyle.cpp
diff --git a/third_party/WebKit/Source/core/style/ComputedStyle.cpp b/third_party/WebKit/Source/core/style/ComputedStyle.cpp
index 76494dc4ec00c89ce559781ab36aa1741870862d..a1baa78129a8f39146fbad36b115bf8229c0f8fb 100644
--- a/third_party/WebKit/Source/core/style/ComputedStyle.cpp
+++ b/third_party/WebKit/Source/core/style/ComputedStyle.cpp
@@ -886,7 +886,8 @@ bool ComputedStyle::DiffNeedsPaintInvalidationObject(
       PrintColorAdjust() != other.PrintColorAdjust() ||
       InsideLink() != other.InsideLink() ||
       !Border().VisuallyEqual(other.Border()) || !RadiiEqual(other) ||
-      !BorderSizeEquals(other) || *background_data_ != *other.background_data_)
+      !BorderColorEquals(other) || !BorderSizeEquals(other) ||

[shend, 2017/05/11 00:41:13]

Is this necessary? Originally, we only compared Border with VisualEqual,
RadiiEqual and SizeEqual.

[nainar, 2017/05/11 01:04:50]

This is needed because VisuallyEqual used the BorderValue::operator== which
checks color and color_is_current_color.

+      *background_data_ != *other.background_data_)
     return true;
 
   if (rare_inherited_data_.Get() != other.rare_inherited_data_.Get()) {