Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1879)

Unified Diff: third_party/WebKit/Source/core/layout/LayoutTableBoxComponent.cpp

Issue 2869043002: Store border-*-color on SurroundData in ComputedStyle not BorderColorAndStyle (Closed)
Patch Set: Always return VisitedDependantColor(CSSPropertyID) Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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 b8c5b92157a3238e3777371fc5eb110e224fe7d3..9e506a38c7279071d1108cec18e48df691446766 100644
--- a/third_party/WebKit/Source/core/layout/LayoutTableBoxComponent.cpp
+++ b/third_party/WebKit/Source/core/layout/LayoutTableBoxComponent.cpp
@@ -18,6 +18,26 @@ void LayoutTableBoxComponent::InvalidateCollapsedBordersOnStyleChange(
if (!table.ShouldCollapseBorders())
return;
if (old_style.Border() != table_part.StyleRef().Border() ||
+ (old_style.VisitedDependentColor(CSSPropertyBorderLeftColor) !=
+ table_part.StyleRef().VisitedDependentColor(
+ CSSPropertyBorderLeftColor) ||
+ old_style.VisitedDependentColor(CSSPropertyBorderRightColor) !=
+ table_part.StyleRef().VisitedDependentColor(
+ CSSPropertyBorderRightColor) ||
+ old_style.VisitedDependentColor(CSSPropertyBorderTopColor) !=
+ table_part.StyleRef().VisitedDependentColor(
+ CSSPropertyBorderTopColor) ||
+ old_style.VisitedDependentColor(CSSPropertyBorderBottomColor) !=
+ table_part.StyleRef().VisitedDependentColor(
+ CSSPropertyBorderBottomColor)) ||
+ (old_style.BorderLeftColorIsCurrentColor() !=
+ table_part.StyleRef().BorderLeftColorIsCurrentColor() ||
+ old_style.BorderRightColorIsCurrentColor() !=
+ table_part.StyleRef().BorderRightColorIsCurrentColor() ||
+ old_style.BorderTopColorIsCurrentColor() !=
+ table_part.StyleRef().BorderTopColorIsCurrentColor() ||
+ old_style.BorderBottomColorIsCurrentColor() !=
+ table_part.StyleRef().BorderBottomColorIsCurrentColor()) ||
!old_style.RadiiEqual(table_part.StyleRef()) ||
!old_style.BorderSizeEquals(table_part.StyleRef()) ||
(diff.TextDecorationOrColorChanged() &&

Powered by Google App Engine
This is Rietveld 408576698