[css-ui] Add support for caret-color property

third_party/WebKit/Source/core/css/resolver/StyleBuilderConverter.cpp

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
  * copyright notice, this list of conditions and the following disclaimer
  * in the documentation and/or other materials provided with the
  * distribution.
  *     * Neither the name of Google Inc. nor the names of its
@@ skipping 1085 matching lines @@
   for (size_t i = 0; i < length; ++i) {
     array->append(convertLength(state, toCSSPrimitiveValue(dashes.item(i))));
   }
 
   return array.release();
 }
 
 StyleColor StyleBuilderConverter::convertStyleColor(StyleResolverState& state,
                                                     const CSSValue& value,
                                                     bool forVisitedLink) {
-  if (value.isIdentifierValue() &&
-      toCSSIdentifierValue(value).getValueID() == CSSValueCurrentcolor)
-    return StyleColor::currentColor();
+  if (value.isIdentifierValue()) {
+    if (toCSSIdentifierValue(value).getValueID() == CSSValueCurrentcolor)

[Timothy Loh, 2016/11/28 03:11:21]

Not sure why this changed.

[Manuel Rego, 2016/11/28 10:47:55]

This is a leftover from the previous version of the patch, sorry.

+      return StyleColor::currentColor();
+  }
   return state.document().textLinkColors().colorFromCSSValue(value, Color(),
                                                              forVisitedLink);
 }
+
+StyleAutoColor StyleBuilderConverter::convertStyleAutoColor(
+    StyleResolverState& state,
+    const CSSValue& value,
+    bool forVisitedLink) {
+  if (value.isIdentifierValue()) {

[Timothy Loh, 2016/11/28 03:11:21]

if (value.isIdentifierValue() && toCSSIdentifierValue(value).getValueID() ==
CSSValueAuto)
  return StyleAutoColor::autoColor();
return convertStyleColor(...);

[Manuel Rego, 2016/11/28 10:47:55]

Ok.

+    if (toCSSIdentifierValue(value).getValueID() == CSSValueCurrentcolor)
+      return StyleAutoColor::currentColor();
+    if (toCSSIdentifierValue(value).getValueID() == CSSValueAuto)
+      return StyleAutoColor::autoColor();
+  }
+  return state.document().textLinkColors().colorFromCSSValue(value, Color(),
+                                                             forVisitedLink);
+}
 
 float StyleBuilderConverter::convertTextStrokeWidth(StyleResolverState& state,
                                                     const CSSValue& value) {
   if (value.isIdentifierValue() && toCSSIdentifierValue(value).getValueID()) {
     float multiplier = convertLineWidth<float>(state, value);
     return CSSPrimitiveValue::create(multiplier / 48,
                                      CSSPrimitiveValue::UnitType::Ems)
         ->computeLength<float>(state.cssToLengthConversionData());
   }
   return toCSSPrimitiveValue(value).computeLength<float>(
@@ skipping 168 matching lines @@
       // Instead of the actual zoom, use 1 to avoid potential rounding errors
       Length length = primitiveValue.convertToLength(
           state.cssToLengthConversionData().copyWithAdjustedZoom(1));
       return *CSSPrimitiveValue::create(length, 1);
     }
   }
   return value;
 }
 
 }  // namespace blink