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

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

 /*
  * (C) 1999-2003 Lars Knoll (knoll@kde.org)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2012, 2013 Apple Inc.
  * All rights reserved.
  * Copyright (C) 2011 Research In Motion Limited. All rights reserved.
  * Copyright (C) 2013 Intel Corporation. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
@@ skipping 278 matching lines @@
   return propertyAt(foundPropertyIndex).shorthandID();
 }
 
 bool StylePropertySet::isPropertyImplicit(CSSPropertyID propertyID) const {
   int foundPropertyIndex = findPropertyIndex(propertyID);
   if (foundPropertyIndex == -1)
     return false;
   return propertyAt(foundPropertyIndex).isImplicit();
 }
 
-bool MutableStylePropertySet::setProperty(
+MutableStylePropertySet::SetResult MutableStylePropertySet::setProperty(
     CSSPropertyID unresolvedProperty,
     const String& value,
     bool important,
     StyleSheetContents* contextStyleSheet) {
   DCHECK_GE(unresolvedProperty, firstCSSProperty);
 
   // Setting the value to an empty string just removes the property in both IE
   // and Gecko. Setting it to null seems to produce less consistent results, but
   // we treat it just the same.
-  if (value.isEmpty())
-    return removeProperty(resolveCSSPropertyID(unresolvedProperty));
+  if (value.isEmpty()) {
+    bool didParse = true;
+    bool didChange = removeProperty(resolveCSSPropertyID(unresolvedProperty));
+    return SetResult{didParse, didChange};
+  }
 
   // When replacing an existing property value, this moves the property to the
   // end of the list. Firefox preserves the position, and MSIE moves the
   // property to the beginning.
   return CSSParser::parseValue(this, unresolvedProperty, value, important,
                                contextStyleSheet);
 }
 
-bool MutableStylePropertySet::setProperty(
+MutableStylePropertySet::SetResult MutableStylePropertySet::setProperty(
     const AtomicString& customPropertyName,
     const String& value,
     bool important,
     StyleSheetContents* contextStyleSheet,
     bool isAnimationTainted) {
-  if (value.isEmpty())
-    return removeProperty(customPropertyName);
+  if (value.isEmpty()) {
+    bool didParse = true;
+    bool didChange = removeProperty(customPropertyName);
+    return MutableStylePropertySet::SetResult{didParse, didChange};
+  }
   return CSSParser::parseValueForCustomProperty(this, customPropertyName, value,
                                                 important, contextStyleSheet,
                                                 isAnimationTainted);
 }
 
 void MutableStylePropertySet::setProperty(CSSPropertyID propertyID,
                                           const CSSValue& value,
                                           bool important) {
   StylePropertyShorthand shorthand = shorthandForProperty(propertyID);
   if (!shorthand.length()) {
@@ skipping 265 matching lines @@
 
 MutableStylePropertySet* MutableStylePropertySet::create(
     const CSSProperty* properties,
     unsigned count) {
   return new MutableStylePropertySet(properties, count);
 }
 
 DEFINE_TRACE(CSSLazyPropertyParser) {}
 
 }  // namespace blink