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

Unified Diff: third_party/WebKit/Source/core/css/CSSPropertyEquality.cpp

Issue 2520873002: [css-ui] Add support for caret-color property (Closed)
Patch Set: Fix tests that were listing all the CSS properties Created 4 years, 1 month 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/css/CSSPropertyEquality.cpp
diff --git a/third_party/WebKit/Source/core/css/CSSPropertyEquality.cpp b/third_party/WebKit/Source/core/css/CSSPropertyEquality.cpp
index 1ab54cd0d103898397946039e58a8788a004835d..be8e6f2822b03ae77a137b3b391171e98a6d5c87 100644
--- a/third_party/WebKit/Source/core/css/CSSPropertyEquality.cpp
+++ b/third_party/WebKit/Source/core/css/CSSPropertyEquality.cpp
@@ -116,6 +116,9 @@ bool CSSPropertyEquality::propertiesEqual(CSSPropertyID prop,
return a.bottom() == b.bottom();
case CSSPropertyBoxShadow:
return dataEquivalent(a.boxShadow(), b.boxShadow());
+ case CSSPropertyCaretColor:
+ return a.caretColor() == b.caretColor() &&
+ a.visitedLinkCaretColor() == b.visitedLinkCaretColor();
yosin_UTC9 2016/11/22 02:25:10 Why do we need to handle "visited link caret color
Manuel Rego 2016/11/22 09:36:00 It's not only color, all the other color propertie
case CSSPropertyClip:
return a.clip() == b.clip();
case CSSPropertyColor:

Powered by Google App Engine
This is Rietveld 408576698