CSSValue Immediates: Change CSSValue to an object instead of a pointer

Source/core/animation/DeferredLegacyStyleInterpolation.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/DeferredLegacyStyleInterpolation.h"
 
 #include "core/animation/ElementAnimations.h"
 #include "core/animation/css/CSSAnimatableValueFactory.h"
 #include "core/css/CSSBasicShapes.h"
@@ skipping 10 matching lines @@
 namespace blink {
 
 void DeferredLegacyStyleInterpolation::apply(StyleResolverState& state) const
 {
     if (m_outdated || !state.element()->elementAnimations() || !state.element()->elementAnimations()->isAnimationStyleChange()) {
         RefPtrWillBeRawPtr<AnimatableValue> startAnimatableValue;
         RefPtrWillBeRawPtr<AnimatableValue> endAnimatableValue;
 
         // Snapshot underlying values for neutral keyframes first because non-neutral keyframes will mutate the StyleResolverState.
         if (!m_endCSSValue) {
-            endAnimatableValue = StyleResolver::createAnimatableValueSnapshot(state, m_id, m_endCSSValue.get());
-            startAnimatableValue = StyleResolver::createAnimatableValueSnapshot(state, m_id, m_startCSSValue.get());
+            endAnimatableValue = StyleResolver::createAnimatableValueSnapshot(state, m_id, m_endCSSValue);
+            startAnimatableValue = StyleResolver::createAnimatableValueSnapshot(state, m_id, m_startCSSValue);
         } else {
-            startAnimatableValue = StyleResolver::createAnimatableValueSnapshot(state, m_id, m_startCSSValue.get());
-            endAnimatableValue = StyleResolver::createAnimatableValueSnapshot(state, m_id, m_endCSSValue.get());
+            startAnimatableValue = StyleResolver::createAnimatableValueSnapshot(state, m_id, m_startCSSValue);
+            endAnimatableValue = StyleResolver::createAnimatableValueSnapshot(state, m_id, m_endCSSValue);
         }
 
         m_innerInterpolation = LegacyStyleInterpolation::create(startAnimatableValue, endAnimatableValue, m_id);
         m_outdated = false;
     }
 
     m_innerInterpolation->interpolate(m_cachedIteration, m_cachedFraction);
     m_innerInterpolation->apply(state);
 }
 
@@ skipping 84 matching lines @@
 
 bool DeferredLegacyStyleInterpolation::interpolationRequiresStyleResolve(const CSSSVGDocumentValue& documentValue)
 {
     return true;
 }
 
 bool DeferredLegacyStyleInterpolation::interpolationRequiresStyleResolve(const CSSValueList& valueList)
 {
     size_t length = valueList.length();
     for (size_t index = 0; index < length; ++index) {
-        if (interpolationRequiresStyleResolve(*valueList.item(index)))
+        if (interpolationRequiresStyleResolve(valueList.item(index)))
             return true;
     }
     return false;
 }
 
 bool DeferredLegacyStyleInterpolation::interpolationRequiresStyleResolve(const CSSBasicShape& shape)
 {
     // FIXME: Should determine the specific shape, and inspect the members.
     return false;
 }
 
 DEFINE_TRACE(DeferredLegacyStyleInterpolation)
 {
     visitor->trace(m_startCSSValue);
     visitor->trace(m_endCSSValue);
     visitor->trace(m_innerInterpolation);
     StyleInterpolation::trace(visitor);
 }
 
 }