Animation: Fix loss of type information when interpolating value of 0

Source/core/animation/StringKeyframe.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/StringKeyframe.h"
 
 #include "core/animation/ColorStyleInterpolation.h"
 #include "core/animation/DefaultStyleInterpolation.h"
 #include "core/animation/DeferredLegacyStyleInterpolation.h"
@@ skipping 119 matching lines @@
     case CSSPropertyColor:
     case CSSPropertyFloodColor:
     case CSSPropertyLightingColor:
     case CSSPropertyOutlineColor:
     case CSSPropertyStopColor:
     case CSSPropertyTextDecorationColor:
     case CSSPropertyWebkitColumnRuleColor:
     case CSSPropertyWebkitTextStrokeColor:
         if (ColorStyleInterpolation::canCreateFrom(*fromCSSValue) && ColorStyleInterpolation::canCreateFrom(*toCSSValue))
             return ColorStyleInterpolation::create(*fromCSSValue, *toCSSValue, property);
+    case CSSPropertyBorderBottomLeftRadius:
+    case CSSPropertyBorderBottomRightRadius:
+    case CSSPropertyBorderTopLeftRadius:
+    case CSSPropertyBorderTopRightRadius:
+    case CSSPropertyPerspectiveOrigin:

[Eric Willigers, 2014/12/19 05:05:52]

PerspectiveOrigin can be negative:
MDN - https://developer.mozilla.org/en-US/docs/Web/CSS/perspective-origin
Experiment - http://jsfiddle.net/ericwilligers/ud5aado3/

[evemj (not active), 2014/12/20 05:57:45]

Done.

+        range = ValueRangeNonNegative;
+        // Fall through
+    case CSSPropertyObjectPosition:
+        if (LengthPairStyleInterpolation::canCreateFrom(*fromCSSValue) && LengthPairStyleInterpolation::canCreateFrom(*toCSSValue))
+            return LengthPairStyleInterpolation::create(*fromCSSValue, *toCSSValue, property, range);
+        break;
     default:
         break;
     }
 
     if (DeferredLegacyStyleInterpolation::interpolationRequiresStyleResolve(*fromCSSValue) || DeferredLegacyStyleInterpolation::interpolationRequiresStyleResolve(*toCSSValue))
         return DeferredLegacyStyleInterpolation::create(fromCSSValue, toCSSValue, property);
 
     // FIXME: Remove the use of AnimatableValues, RenderStyles and Elements here.
     // FIXME: Remove this cache
     ASSERT(element);
@@ skipping 19 matching lines @@
 }
 
 void StringKeyframe::PropertySpecificKeyframe::trace(Visitor* visitor)
 {
     visitor->trace(m_value);
     visitor->trace(m_animatableValueCache);
     Keyframe::PropertySpecificKeyframe::trace(visitor);
 }
 
 }