Web Animations: Use of hyphens is no longer supported

third_party/WebKit/Source/core/animation/AnimationInputHelpers.cpp

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "core/animation/AnimationInputHelpers.h"
 
 #include "core/SVGNames.h"
 #include "core/css/CSSValueList.h"
 #include "core/css/parser/CSSParser.h"
 #include "core/css/resolver/CSSToStyleMap.h"
@@ skipping 13 matching lines @@
 }
 
 static String removeSVGPrefix(const String& property)
 {
     ASSERT(isSVGPrefixed(property));
     return property.substring(kSVGPrefixLength);
 }
 
 CSSPropertyID AnimationInputHelpers::keyframeAttributeToCSSProperty(const String& property, const Document& document)
 {
-    // Disallow prefixed properties.
-    if (property[0] == '-' || isASCIIUpper(property[0]))
+    // Disallow hyphenated and prefixed properties.
+    if (property.find('-') != kNotFound) {

[alancutter (OOO until 2018), 2016/03/10 01:26:50]

Does this logic need to move? This seems more expensive without any reason for
it.

[Eric Willigers, 2016/03/10 01:56:04]

Moved back.

+        if (cssPropertyID(property) != CSSPropertyInvalid)
+            Deprecation::countDeprecation(document, UseCounter::WebAnimationHyphenatedProperty);
+        return CSSPropertyInvalid;
+    }
+    if (isASCIIUpper(property[0]))
         return CSSPropertyInvalid;
     if (property == "cssFloat")
         return CSSPropertyFloat;
+
     StringBuilder builder;
     for (size_t i = 0; i < property.length(); ++i) {
-        if (property[i] == '-')
-            Deprecation::countDeprecation(document, UseCounter::WebAnimationHyphenatedProperty);
         if (isASCIIUpper(property[i]))
             builder.append('-');
         builder.append(property[i]);
     }
     return cssPropertyID(builder.toString());
 }
 
 CSSPropertyID AnimationInputHelpers::keyframeAttributeToPresentationAttribute(const String& property, const Element& element)
 {
     if (!RuntimeEnabledFeatures::webAnimationsSVGEnabled() || !element.isSVGElement() || !isSVGPrefixed(property))
@@ skipping 153 matching lines @@
         ASSERT(!value || value->isCSSWideKeyword());
         return nullptr;
     }
     CSSValueList* valueList = toCSSValueList(value.get());
     if (valueList->length() > 1)
         return nullptr;
     return CSSToStyleMap::mapAnimationTimingFunction(*valueList->item(0), true);
 }
 
 } // namespace blink