Implement left property animation on InvalidatableStyleInterpolation

Source/core/animation/InvalidatableStyleInterpolation.cpp

 // Copyright 2015 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/InvalidatableStyleInterpolation.h"
 
 #include "core/animation/StringKeyframe.h"
+#include "core/css/resolver/StyleResolverState.h"
 
 namespace blink {
 
 InvalidatableStyleInterpolation::InvalidatableStyleInterpolation(
     const Vector<const InterpolationType*>& interpolationTypes,
     const CSSPropertySpecificKeyframe& startKeyframe,
     const CSSPropertySpecificKeyframe& endKeyframe)
     : StyleInterpolation(nullptr, nullptr, interpolationTypes.first()->property())
     , m_interpolationTypes(interpolationTypes)
     , m_startKeyframe(startKeyframe)
     , m_endKeyframe(endKeyframe)
 {
-    maybeCachePairwiseConversion(nullptr);
+    maybeCachePairwiseConversion(nullptr, nullptr);
     interpolate(0, 0);
 }
 
-bool InvalidatableStyleInterpolation::maybeCachePairwiseConversion(const StyleResolverState* state) const
+bool InvalidatableStyleInterpolation::maybeCachePairwiseConversion(const StyleResolverState* state, const InterpolationValue* underlyingValue) const
 {
     for (const auto& interpolationType : m_interpolationTypes) {
+        if ((m_startKeyframe.isNeutral() || m_endKeyframe.isNeutral()) && (!underlyingValue || underlyingValue->type() != *interpolationType))
+            continue;
         OwnPtrWillBeRawPtr<PairwisePrimitiveInterpolation> pairwiseConversion = interpolationType->maybeConvertPairwise(m_startKeyframe, m_endKeyframe, state, m_conversionCheckers);
         if (pairwiseConversion) {
             m_cachedValue = pairwiseConversion->initialValue();
             m_cachedConversion = pairwiseConversion.release();
             return true;
         }
     }
     return false;
 }
 
 void InvalidatableStyleInterpolation::interpolate(int, double fraction)
 {
     m_currentFraction = fraction;
     if (m_cachedConversion)
-        m_cachedConversion->interpolate(fraction, m_cachedValue);
+        m_cachedConversion->interpolateValue(fraction, m_cachedValue);
     // We defer the interpolation to ensureValidInterpolation() if m_cachedConversion is null.
 }
 
-PassOwnPtrWillBeRawPtr<InterpolationValue> InvalidatableStyleInterpolation::convertSingleKeyframe(const CSSPropertySpecificKeyframe& keyframe, const StyleResolverState& state) const
+PassOwnPtrWillBeRawPtr<InterpolationValue> InvalidatableStyleInterpolation::convertSingleKeyframe(const CSSPropertySpecificKeyframe& keyframe, const StyleResolverState& state, const InterpolationValue* underlyingValue) const
 {
+    if (keyframe.isNeutral() && !underlyingValue)
+        return nullptr;
     for (const auto& interpolationType : m_interpolationTypes) {
+        if (keyframe.isNeutral() && underlyingValue->type() != *interpolationType)
+            continue;
         OwnPtrWillBeRawPtr<InterpolationValue> result = interpolationType->maybeConvertSingle(keyframe, &state, m_conversionCheckers);
         if (result)
             return result.release();
     }
-    ASSERT_NOT_REACHED();
+    ASSERT(keyframe.isNeutral());
     return nullptr;
 }
 
-bool InvalidatableStyleInterpolation::isCacheValid(const StyleResolverState& state) const
+PassOwnPtrWillBeRawPtr<InterpolationValue> InvalidatableStyleInterpolation::maybeConvertUnderlyingValue(const StyleResolverState& state) const
 {
+    for (const auto& interpolationType : m_interpolationTypes) {
+        OwnPtrWillBeRawPtr<InterpolationValue> result = interpolationType->maybeConvertUnderlyingValue(state);
+        if (result)
+            return result.release();
+    }
+    return nullptr;
+}
+
+bool InvalidatableStyleInterpolation::dependsOnUnderlyingValue() const
+{
+    return (m_startKeyframe.underlyingFraction() != 0 && m_currentFraction != 1) || (m_endKeyframe.underlyingFraction() != 0 && m_currentFraction != 0);
+}
+
+bool InvalidatableStyleInterpolation::isNeutralKeyframeActive() const
+{
+    return (m_startKeyframe.isNeutral() && m_currentFraction != 1) || (m_endKeyframe.isNeutral() && m_currentFraction != 0);
+}
+
+bool InvalidatableStyleInterpolation::isCacheValid(const StyleResolverState& state, const InterpolationValue* underlyingValue) const
+{
+    if (isNeutralKeyframeActive()) {
+        if (m_cachedConversion->isFlip())
+            return false;
+        // Pairwise interpolation can never happen between different InterpolationTypes, neutral values always represent the underlying value.
+        if (!underlyingValue || !m_cachedValue || m_cachedValue->type() != underlyingValue->type())
+            return false;
+    }
     for (const auto& checker : m_conversionCheckers) {
         if (!checker->isValid(state))
             return false;
     }
     return true;
 }
 
-void InvalidatableStyleInterpolation::ensureValidInterpolation(const StyleResolverState& state) const
+void InvalidatableStyleInterpolation::ensureValidInterpolation(const StyleResolverState& state, const InterpolationValue* underlyingValue) const
 {
-    if (m_cachedConversion && isCacheValid(state))
+    if (m_cachedConversion && isCacheValid(state, underlyingValue))
         return;
     m_conversionCheckers.clear();
-    if (!maybeCachePairwiseConversion(&state)) {
+    if (!maybeCachePairwiseConversion(&state, underlyingValue)) {
         m_cachedConversion = FlipPrimitiveInterpolation::create(
-            convertSingleKeyframe(m_startKeyframe, state),
-            convertSingleKeyframe(m_endKeyframe, state));
+            convertSingleKeyframe(m_startKeyframe, state, underlyingValue),
+            convertSingleKeyframe(m_endKeyframe, state, underlyingValue));
     }
-    m_cachedConversion->interpolate(m_currentFraction, m_cachedValue);
+    m_cachedConversion->interpolateValue(m_currentFraction, m_cachedValue);
+}
+
+void InvalidatableStyleInterpolation::setFlagIfInheritUsed(StyleResolverState& state) const
+{
+    if ((m_startKeyframe.value() && m_startKeyframe.value()->isInheritedValue())
+        || (m_endKeyframe.value() && m_endKeyframe.value()->isInheritedValue())) {
+        state.parentStyle()->setHasExplicitlyInheritedProperties();

[dstockwell, 2015/07/02 23:57:46]

Are you sure that parentStyle is never null here?

[alancutter (OOO until 2018), 2015/07/03 01:38:22]

Nope, added check.

+    }
 }
 
 void InvalidatableStyleInterpolation::apply(StyleResolverState& state) const
 {
-    ensureValidInterpolation(state);
-    m_cachedValue->type().apply(m_cachedValue->interpolableValue(), m_cachedValue->nonInterpolableValue(), state);
+    OwnPtrWillBeRawPtr<InterpolationValue> underlyingValue = dependsOnUnderlyingValue() ? maybeConvertUnderlyingValue(state) : nullptr;
+    ensureValidInterpolation(state, underlyingValue.get());
+    if (!m_cachedValue)
+        return;
+    const InterpolableValue* appliedInterpolableValue = &m_cachedValue->interpolableValue();
+    if (underlyingValue && m_cachedValue->type() == underlyingValue->type()) {
+        double underlyingFraction = m_cachedConversion->interpolateUnderlyingFraction(m_startKeyframe.underlyingFraction(), m_endKeyframe.underlyingFraction(), m_currentFraction);
+        underlyingValue->interpolableValue().scaleAndAdd(underlyingFraction, m_cachedValue->interpolableValue());
+        appliedInterpolableValue = &underlyingValue->interpolableValue();
+    }
+    m_cachedValue->type().apply(*appliedInterpolableValue, m_cachedValue->nonInterpolableValue(), state);
+    setFlagIfInheritUsed(state);
 }
 
 } // namespace blink