Migrate the remaining CSS properties interpolable as Lengths to LengthInterpolationType

Source/core/animation/LengthStyleInterpolation.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 "config.h"
 #include "core/animation/LengthStyleInterpolation.h"
 
+#include "core/animation/LengthPropertyFunctions.h"
 #include "core/animation/css/CSSAnimatableValueFactory.h"
 #include "core/css/CSSCalculationValue.h"
 #include "core/css/resolver/StyleBuilder.h"
 #include "core/css/resolver/StyleResolverState.h"
 #include "platform/CalculationValue.h"
 
 namespace blink {
 
-namespace {
-
-bool pixelsForKeyword(CSSPropertyID property, CSSValueID valueID, double& result)
-{
-    switch (property) {
-    case CSSPropertyBorderBottomWidth:
-    case CSSPropertyBorderLeftWidth:
-    case CSSPropertyBorderRightWidth:
-    case CSSPropertyBorderTopWidth:
-    case CSSPropertyWebkitColumnRuleWidth:
-    case CSSPropertyOutlineWidth:
-        if (valueID == CSSValueThin) {
-            result = 1;
-            return true;
-        }
-        if (valueID == CSSValueMedium) {
-            result = 3;
-            return true;
-        }
-        if (valueID == CSSValueThick) {
-            result = 5;
-            return true;
-        }
-        return false;
-    case CSSPropertyLetterSpacing:
-    case CSSPropertyWordSpacing:
-        if (valueID == CSSValueNormal) {
-            result = 0;
-            return true;
-        }
-        return false;
-    default:
-        return false;
-    }
-}
-
-} // namespace
-
 bool LengthStyleInterpolation::canCreateFrom(const CSSValue& value, CSSPropertyID property)
 {
     if (!value.isPrimitiveValue())
         return false;
 
     const CSSPrimitiveValue& primitiveValue = toCSSPrimitiveValue(value);
-    if (primitiveValue.isCalculated())
-        return true;
-
-    if (primitiveValue.isValueID()) {
-        CSSValueID valueID = primitiveValue.getValueID();
-        double pixels;
-        return pixelsForKeyword(property, valueID, pixels);
-    }
-
     return primitiveValue.isLength() || primitiveValue.isPercentage() || primitiveValue.isCalculatedPercentageWithLength();
 }
 
 PassOwnPtrWillBeRawPtr<InterpolableValue> LengthStyleInterpolation::toInterpolableValue(const CSSValue& value, CSSPropertyID id)
 {
     ASSERT(canCreateFrom(value, id));
     OwnPtrWillBeRawPtr<InterpolableList> listOfValuesAndTypes = InterpolableList::create(2);
     OwnPtrWillBeRawPtr<InterpolableList> listOfValues = InterpolableList::create(CSSPrimitiveValue::LengthUnitTypeCount);
     OwnPtrWillBeRawPtr<InterpolableList> listOfTypes = InterpolableList::create(CSSPrimitiveValue::LengthUnitTypeCount);
 
     const CSSPrimitiveValue& primitive = toCSSPrimitiveValue(value);
 
     CSSLengthArray arrayOfValues;
     CSSPrimitiveValue::CSSLengthTypeArray arrayOfTypes;
     for (size_t i = 0; i < CSSPrimitiveValue::LengthUnitTypeCount; i++)
         arrayOfValues.append(0);
-
     arrayOfTypes.ensureSize(CSSPrimitiveValue::LengthUnitTypeCount);
-    if (primitive.isValueID()) {
-        CSSValueID valueID = primitive.getValueID();
-        double pixels;
-        pixelsForKeyword(id, valueID, pixels);
-        arrayOfTypes.set(CSSPrimitiveValue::UnitTypePixels);
-        arrayOfValues[CSSPrimitiveValue::UnitTypePixels] = pixels;
-    } else {
-        primitive.accumulateLengthArray(arrayOfValues, arrayOfTypes);
-    }
+    primitive.accumulateLengthArray(arrayOfValues, arrayOfTypes);
 
     for (size_t i = 0; i < CSSPrimitiveValue::LengthUnitTypeCount; i++) {
         listOfValues->set(i, InterpolableNumber::create(arrayOfValues.at(i)));
         listOfTypes->set(i, InterpolableNumber::create(arrayOfTypes.get(i)));
     }
 
     listOfValuesAndTypes->set(0, listOfValues.release());
     listOfValuesAndTypes->set(1, listOfTypes.release());
 
     return listOfValuesAndTypes.release();
 }
 
-bool LengthStyleInterpolation::isPixelsOrPercentOnly(const InterpolableValue& value)
+namespace {
+
+bool isPixelsOrPercentOnly(const InterpolableValue& value)
 {
     const InterpolableList& types = *toInterpolableList(toInterpolableList(value).get(1));
     bool result = false;
     for (size_t i = 0; i < CSSPrimitiveValue::LengthUnitTypeCount; i++) {
         bool typeIsPresent = toInterpolableNumber(types.get(i))->value();
         if (i == CSSPrimitiveValue::UnitTypePixels)
             result |= typeIsPresent;
         else if (i == CSSPrimitiveValue::UnitTypePercentage)
             result |= typeIsPresent;
         else if (typeIsPresent)
             return false;
     }
     return result;
 }
 
-LengthStyleInterpolation::LengthSetter LengthStyleInterpolation::lengthSetterForProperty(CSSPropertyID property)
-{
-    switch (property) {
-    case CSSPropertyBottom:
-        return &ComputedStyle::setBottom;
-    case CSSPropertyCx:
-        return &ComputedStyle::setCx;
-    case CSSPropertyCy:
-        return &ComputedStyle::setCy;
-    case CSSPropertyFlexBasis:
-        return &ComputedStyle::setFlexBasis;
-    case CSSPropertyHeight:
-        return &ComputedStyle::setHeight;
-    case CSSPropertyLeft:
-        return &ComputedStyle::setLeft;
-    case CSSPropertyLineHeight:
-        return &ComputedStyle::setLineHeight;
-    case CSSPropertyMarginBottom:
-        return &ComputedStyle::setMarginBottom;
-    case CSSPropertyMarginLeft:
-        return &ComputedStyle::setMarginLeft;
-    case CSSPropertyMarginRight:
-        return &ComputedStyle::setMarginRight;
-    case CSSPropertyMarginTop:
-        return &ComputedStyle::setMarginTop;
-    case CSSPropertyMaxHeight:
-        return &ComputedStyle::setMaxHeight;
-    case CSSPropertyMaxWidth:
-        return &ComputedStyle::setMaxWidth;
-    case CSSPropertyMinHeight:
-        return &ComputedStyle::setMinHeight;
-    case CSSPropertyMinWidth:
-        return &ComputedStyle::setMinWidth;
-    case CSSPropertyMotionOffset:
-        return &ComputedStyle::setMotionOffset;
-    case CSSPropertyPaddingBottom:
-        return &ComputedStyle::setPaddingBottom;
-    case CSSPropertyPaddingLeft:
-        return &ComputedStyle::setPaddingLeft;
-    case CSSPropertyPaddingRight:
-        return &ComputedStyle::setPaddingRight;
-    case CSSPropertyPaddingTop:
-        return &ComputedStyle::setPaddingTop;
-    case CSSPropertyR:
-        return &ComputedStyle::setR;
-    case CSSPropertyRx:
-        return &ComputedStyle::setRx;
-    case CSSPropertyRy:
-        return &ComputedStyle::setRy;
-    case CSSPropertyRight:
-        return &ComputedStyle::setRight;
-    case CSSPropertyShapeMargin:
-        return &ComputedStyle::setShapeMargin;
-    case CSSPropertyStrokeDashoffset:
-        return &ComputedStyle::setStrokeDashOffset;
-    case CSSPropertyTop:
-        return &ComputedStyle::setTop;
-    case CSSPropertyWidth:
-        return &ComputedStyle::setWidth;
-    case CSSPropertyWebkitPerspectiveOriginX:
-        return &ComputedStyle::setPerspectiveOriginX;
-    case CSSPropertyWebkitPerspectiveOriginY:
-        return &ComputedStyle::setPerspectiveOriginY;
-    case CSSPropertyWebkitTransformOriginX:
-        return &ComputedStyle::setTransformOriginX;
-    case CSSPropertyWebkitTransformOriginY:
-        return &ComputedStyle::setTransformOriginY;
-    case CSSPropertyX:
-        return &ComputedStyle::setX;
-    case CSSPropertyY:
-        return &ComputedStyle::setY;
-    // These properties don't have a ComputedStyle setter with the signature void(*)(const Length&).
-    case CSSPropertyBaselineShift:
-    case CSSPropertyBorderBottomWidth:
-    case CSSPropertyBorderLeftWidth:
-    case CSSPropertyBorderRightWidth:
-    case CSSPropertyBorderTopWidth:
-    case CSSPropertyFontSize:
-    case CSSPropertyLetterSpacing:
-    case CSSPropertyOutlineOffset:
-    case CSSPropertyOutlineWidth:
-    case CSSPropertyPerspective:
-    case CSSPropertyStrokeWidth:
-    case CSSPropertyVerticalAlign:
-    case CSSPropertyWebkitBorderHorizontalSpacing:
-    case CSSPropertyWebkitBorderVerticalSpacing:
-    case CSSPropertyWebkitColumnGap:
-    case CSSPropertyWebkitColumnRuleWidth:
-    case CSSPropertyWebkitColumnWidth:
-    case CSSPropertyWebkitTransformOriginZ:
-    case CSSPropertyWordSpacing:
-        return nullptr;
-    default:
-        ASSERT_NOT_REACHED();
-        return nullptr;
-    }
-}
-
-namespace {
-
 static CSSPrimitiveValue::UnitType toUnitType(int lengthUnitType)
 {
     return static_cast<CSSPrimitiveValue::UnitType>(CSSPrimitiveValue::lengthUnitTypeToUnitType(static_cast<CSSPrimitiveValue::LengthUnitType>(lengthUnitType)));
 }
 
 static PassRefPtrWillBeRawPtr<CSSCalcExpressionNode> constructCalcExpression(const InterpolableList* list)
 {
     const InterpolableList* listOfValues = toInterpolableList(list->get(0));
     const InterpolableList* listOfTypes = toInterpolableList(list->get(1));
     RefPtrWillBeRawPtr<CSSCalcExpressionNode> expression = nullptr;
@@ skipping 69 matching lines @@
                 return CSSPrimitiveValue::create(value, toUnitType(i));
             }
         }
         ASSERT_NOT_REACHED();
     default:
         ValueRange valueRange = (range == RangeNonNegative) ? ValueRangeNonNegative : ValueRangeAll;
         return CSSPrimitiveValue::create(CSSCalcValue::create(constructCalcExpression(listOfValuesAndTypes), valueRange));
     }
 }
 
-void LengthStyleInterpolation::applyInterpolableValue(CSSPropertyID property, const InterpolableValue& value, InterpolationRange range, StyleResolverState& state, LengthSetter lengthSetter)
+void LengthStyleInterpolation::applyInterpolableValue(CSSPropertyID property, const InterpolableValue& value, InterpolationRange range, StyleResolverState& state)
 {
-    if (lengthSetter && isPixelsOrPercentOnly(value)) {
-        (state.style()->*lengthSetter)(lengthFromInterpolableValue(value, range, state.style()->effectiveZoom()));
+    if (isPixelsOrPercentOnly(value)) {
+        Length length = lengthFromInterpolableValue(value, range, state.style()->effectiveZoom());
+        if (LengthPropertyFunctions::setLength(property, *state.style(), length)) {
 #if ENABLE(ASSERT)

[dstockwell, 2015/08/31 05:33:40]

Seeing this the second time, I think this assert should have a comment.

[alancutter (OOO until 2018), 2015/08/31 07:15:30]

Done.

-        RefPtrWillBeRawPtr<AnimatableValue> before = CSSAnimatableValueFactory::create(property, *state.style());
-        StyleBuilder::applyProperty(property, state, fromInterpolableValue(value, range).get());
-        RefPtrWillBeRawPtr<AnimatableValue> after = CSSAnimatableValueFactory::create(property, *state.style());
-        ASSERT(before->equals(*after));
+            RefPtrWillBeRawPtr<AnimatableValue> before = CSSAnimatableValueFactory::create(property, *state.style());
+            StyleBuilder::applyProperty(property, state, fromInterpolableValue(value, range).get());
+            RefPtrWillBeRawPtr<AnimatableValue> after = CSSAnimatableValueFactory::create(property, *state.style());
+            ASSERT(before->equals(*after));
 #endif
-    } else {
-        StyleBuilder::applyProperty(property, state, fromInterpolableValue(value, range).get());
+            return;
+        }
     }
+    StyleBuilder::applyProperty(property, state, fromInterpolableValue(value, range).get());
 }
 
 void LengthStyleInterpolation::apply(StyleResolverState& state) const
 {
-    applyInterpolableValue(m_id, *m_cachedValue, m_range, state, m_lengthSetter);
+    applyInterpolableValue(m_id, *m_cachedValue, m_range, state);
 }
 
 DEFINE_TRACE(LengthStyleInterpolation)
 {
     StyleInterpolation::trace(visitor);
 }
 
 }