Animation: Add CSSPropertyClip and CSSPropertyBorderImageSlice to StringKeyframe

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/ConstantStyleInterpolation.h"
 #include "core/animation/DeferredLegacyStyleInterpolation.h"
@@ skipping 228 matching lines @@
         // FIXME: AnimatableShadow incorrectly animates between inset and non-inset values so it will never indicate it needs default interpolation
         if (ShadowStyleInterpolation::usesDefaultStyleInterpolation(*fromCSSValue, *toCSSValue)) {
             forceDefaultInterpolation = true;
             break;
         }
 
         // FIXME: Handle interpolation from/to none, unspecified color values
         fallBackToLegacy = true;
 
         break;
+
     }
 
-    case CSSPropertyWebkitMaskBoxImageSlice:
-        if (LengthBoxStyleInterpolation::matchingFill(*toCSSValue, *fromCSSValue) && LengthBoxStyleInterpolation::canCreateFrom(*fromCSSValue) && LengthStyleInterpolation::canCreateFrom(*toCSSValue))
-            return LengthBoxStyleInterpolation::createFromBorderImageSlice(*fromCSSValue, *toCSSValue, property);
-        break;
     case CSSPropertyTransform: {
         if (TransformStyleInterpolation::fallBackToLegacy(*fromCSSValue, *toCSSValue)) {
             // FIXME: Handle matrices and interpolating to/from CSSValueNone from/to CSSValueList
             fallBackToLegacy = true;
             break;
         }
         RefPtrWillBeRawPtr<Interpolation> interpolation = TransformStyleInterpolation::maybeCreateFrom(*fromCSSValue, *toCSSValue, property);
         if (interpolation)
             return interpolation.release();
         break;
     }
 
+
+    case CSSPropertyClip:
+    case CSSPropertyBorderImageSlice:
+    case CSSPropertyWebkitMaskBoxImageSlice:
+        {

[Eric Willigers, 2015/02/11 03:56:32]

Move brace to end of previous line.

[jadeg, 2015/02/12 05:06:00]

Done.

+            RefPtrWillBeRawPtr<Interpolation> interpolation = LengthBoxStyleInterpolation::maybeCreateFrom(fromCSSValue, toCSSValue, property);
+            if (interpolation)
+                return interpolation.release();
+            break;
+        }

[Eric Willigers, 2015/02/11 03:56:32]

This brace and the preceding lines can be unindented.

[jadeg, 2015/02/12 05:06:00]

Done.

     case CSSPropertyStrokeWidth:
         range = RangeNonNegative;
         // Fall through
     case CSSPropertyBaselineShift:
     case CSSPropertyStrokeDashoffset: {
         RefPtrWillBeRawPtr<Interpolation> interpolation = SVGLengthStyleInterpolation::maybeCreate(*fromCSSValue, *toCSSValue, property, range);
         if (interpolation)
             return interpolation.release();
 
         // We use default interpolation for
@@ skipping 58 matching lines @@
 }
 
 void StringKeyframe::PropertySpecificKeyframe::trace(Visitor* visitor)
 {
     visitor->trace(m_value);
     visitor->trace(m_animatableValueCache);
     Keyframe::PropertySpecificKeyframe::trace(visitor);
 }
 
 }