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

third_party/WebKit/Source/core/css/DOMWindowCSS.cpp

Index: third_party/WebKit/Source/core/css/DOMWindowCSS.cpp
diff --git a/third_party/WebKit/Source/core/css/DOMWindowCSS.cpp b/third_party/WebKit/Source/core/css/DOMWindowCSS.cpp
index 9c0e4681442cecd1482894cc47d1255fce588ec2..171ec70270f3167ed8f4d90e131db0dcbcc687cc 100644
--- a/third_party/WebKit/Source/core/css/DOMWindowCSS.cpp
+++ b/third_party/WebKit/Source/core/css/DOMWindowCSS.cpp
@@ -47,7 +47,8 @@ bool DOMWindowCSS::supports(const String& property, const String& value) {
         MutableStylePropertySet::create(HTMLStandardMode);
     bool isAnimationTainted = false;
     return CSSParser::parseValueForCustomProperty(
-        dummyStyle, "--valid", value, false, nullptr, isAnimationTainted);
+               dummyStyle, "--valid", value, false, nullptr, isAnimationTainted)
+        .didChange;

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

We might as well change these to didParse.

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

Done.

   }
 
   ASSERT(CSSPropertyMetadata::isEnabledProperty(unresolvedProperty));
@@ -55,7 +56,8 @@ bool DOMWindowCSS::supports(const String& property, const String& value) {
   // This will return false when !important is present
   MutableStylePropertySet* dummyStyle =
       MutableStylePropertySet::create(HTMLStandardMode);
-  return CSSParser::parseValue(dummyStyle, unresolvedProperty, value, false, 0);
+  return CSSParser::parseValue(dummyStyle, unresolvedProperty, value, false, 0)
+      .didChange;
 }
 
 bool DOMWindowCSS::supports(const String& conditionText) {