Emit console warning when element.animate() keyframe value fails to parse

third_party/WebKit/Source/core/css/parser/CSSParser.cpp

Index: third_party/WebKit/Source/core/css/parser/CSSParser.cpp
diff --git a/third_party/WebKit/Source/core/css/parser/CSSParser.cpp b/third_party/WebKit/Source/core/css/parser/CSSParser.cpp
index ee592d990f4cc5cde6f3f1148268e0945e0e3541..6cd6f93bf05c610093346934ae7b828b28774710 100644
--- a/third_party/WebKit/Source/core/css/parser/CSSParser.cpp
+++ b/third_party/WebKit/Source/core/css/parser/CSSParser.cpp
@@ -7,7 +7,6 @@
 #include "core/css/CSSColorValue.h"
 #include "core/css/CSSKeyframeRule.h"
 #include "core/css/StyleColor.h"
-#include "core/css/StylePropertySet.h"
 #include "core/css/StyleRule.h"
 #include "core/css/StyleSheetContents.h"
 #include "core/css/parser/CSSParserFastPaths.h"
@@ -76,20 +75,28 @@ void CSSParser::parseSheetForInspector(const CSSParserContext& context,
                                                     observer);
 }
 
-bool CSSParser::parseValue(MutableStylePropertySet* declaration,
-                           CSSPropertyID unresolvedProperty,
-                           const String& string,
-                           bool important,
-                           StyleSheetContents* styleSheet) {
-  if (string.isEmpty())
-    return false;
+MutableStylePropertySet::SetResult CSSParser::parseValue(
+    MutableStylePropertySet* declaration,
+    CSSPropertyID unresolvedProperty,
+    const String& string,
+    bool important,
+    StyleSheetContents* styleSheet) {
+  if (string.isEmpty()) {
+    bool didParse = true;

[Timothy Loh, 2016/11/25 01:40:59]

false probably

[alancutter (OOO until 2018), 2016/11/25 11:59:26]

Done.

[alancutter (OOO until 2018), 2016/11/25 11:59:26]

Done.

+    bool didChange = false;
+    return MutableStylePropertySet::SetResult{didParse, didChange};
+  }
+
   CSSPropertyID resolvedProperty = resolveCSSPropertyID(unresolvedProperty);
   CSSParserMode parserMode = declaration->cssParserMode();
   CSSValue* value =
       CSSParserFastPaths::maybeParseValue(resolvedProperty, string, parserMode);
-  if (value)
-    return declaration->setProperty(
+  if (value) {
+    bool didParse = true;
+    bool didChange = declaration->setProperty(
         CSSProperty(resolvedProperty, *value, important));
+    return MutableStylePropertySet::SetResult{didParse, didChange};
+  }
   CSSParserContext context(parserMode, nullptr);
   if (styleSheet) {
     context = styleSheet->parserContext();
@@ -99,7 +106,7 @@ bool CSSParser::parseValue(MutableStylePropertySet* declaration,
                     context);
 }
 
-bool CSSParser::parseValueForCustomProperty(
+MutableStylePropertySet::SetResult CSSParser::parseValueForCustomProperty(
     MutableStylePropertySet* declaration,
     const AtomicString& propertyName,
     const String& value,
@@ -107,8 +114,11 @@ bool CSSParser::parseValueForCustomProperty(
     StyleSheetContents* styleSheet,
     bool isAnimationTainted) {
   DCHECK(CSSVariableParser::isValidVariableName(propertyName));
-  if (value.isEmpty())
-    return false;
+  if (value.isEmpty()) {
+    bool didParse = true;

[Timothy Loh, 2016/11/25 01:40:59]

false

+    bool didChange = false;
+    return MutableStylePropertySet::SetResult{didParse, didChange};
+  }
   CSSParserMode parserMode = declaration->cssParserMode();
   CSSParserContext context(parserMode, nullptr);
   if (styleSheet) {
@@ -124,11 +134,12 @@ ImmutableStylePropertySet* CSSParser::parseCustomPropertySet(
   return CSSParserImpl::parseCustomPropertySet(range);
 }
 
-bool CSSParser::parseValue(MutableStylePropertySet* declaration,
-                           CSSPropertyID unresolvedProperty,
-                           const String& string,
-                           bool important,
-                           const CSSParserContext& context) {
+MutableStylePropertySet::SetResult CSSParser::parseValue(
+    MutableStylePropertySet* declaration,
+    CSSPropertyID unresolvedProperty,
+    const String& string,
+    bool important,
+    const CSSParserContext& context) {
   return CSSParserImpl::parseValue(declaration, unresolvedProperty, string,
                                    important, context);
 }