[Typed OM] Add support for properties which take numbers in CSSProperties.in

third_party/WebKit/Source/core/css/cssom/StyleValueFactory.cpp

 // Copyright 2016 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/css/cssom/StyleValueFactory.h"
 
 #include "core/css/CSSValue.h"
+#include "core/css/cssom/CSSNumberValue.h"
 #include "core/css/cssom/CSSSimpleLength.h"
 #include "core/css/cssom/CSSStyleValue.h"
 #include "core/css/cssom/CSSTransformValue.h"
 #include "core/css/cssom/CSSUnsupportedStyleValue.h"
 
 namespace blink {
 
+namespace {
+
+CSSStyleValue* styleValueForProperty(CSSPropertyID propertyID, const CSSValue& value)
+{
+    switch (propertyID) {
+    case CSSPropertyTransform:
+        return CSSTransformValue::fromCSSValue(value);
+    default:
+        // TODO(meade): Implement other complex properties.
+        break;
+    }
+
+    if (value.isPrimitiveValue()) {
+        const CSSPrimitiveValue& primitiveValue = toCSSPrimitiveValue(value);
+        if (primitiveValue.isLength() && !primitiveValue.isCalculated())
+            return CSSSimpleLength::create(primitiveValue.getDoubleValue(), primitiveValue.typeWithCalcResolved());
+        if (primitiveValue.isNumber())
+            return CSSNumberValue::create(primitiveValue.getDoubleValue());
+    }
+
+    return nullptr;
+}
+
+CSSStyleValueVector unsupportedCSSValue(const CSSValue& value)
+{
+    CSSStyleValueVector styleValueVector;
+    styleValueVector.append(CSSUnsupportedStyleValue::create(value.cssText()));
+    return styleValueVector;
+}
+
+} // namespace
+
 CSSStyleValueVector StyleValueFactory::cssValueToStyleValueVector(CSSPropertyID propertyID, const CSSValue& value)
 {
     CSSStyleValueVector styleValueVector;
-
-    if (value.isPrimitiveValue()) {
-        const CSSPrimitiveValue& primitiveValue = toCSSPrimitiveValue(value);
-        if (primitiveValue.isLength() && !primitiveValue.isCalculated()) {
-            styleValueVector.append(CSSSimpleLength::create(primitiveValue.getDoubleValue(), primitiveValue.typeWithCalcResolved()));
-            return styleValueVector;
-        }
+    CSSStyleValue* styleValue = styleValueForProperty(propertyID, value);
+    if (styleValue) {
+        styleValueVector.append(styleValue);
+        return styleValueVector;
     }
 
-    CSSStyleValue* styleValue = nullptr;
-    switch (propertyID) {
-    case CSSPropertyTransform:
-        styleValue  = CSSTransformValue::fromCSSValue(value);
-        if (styleValue)
-            styleValueVector.append(styleValue);
-        return styleValueVector;
-    default:
-        // TODO(meade): Implement the rest.
-        break;
+    if (!value.isValueList()) {
+        return unsupportedCSSValue(value);
     }
 
-    styleValueVector.append(CSSUnsupportedStyleValue::create(value.cssText()));
+    // If it's a list, we can try it as a list valued property.
+    const CSSValueList& cssValueList = toCSSValueList(value);
+    for (const Member<const CSSValue> innerValue : cssValueList) {
+        styleValue = styleValueForProperty(propertyID, *innerValue);
+        if (!styleValue) {
+            return unsupportedCSSValue(value);
+        }
+        styleValueVector.append(styleValue);
+    }
     return styleValueVector;
 }
 
 } // namespace blink