CC: Add curve (whole animation) timing function to compositor animations.

cc/animation/keyframed_animation_curve.cc

 // Copyright 2012 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 <algorithm>
 
 #include "cc/animation/keyframed_animation_curve.h"
 #include "ui/gfx/animation/tween.h"
 #include "ui/gfx/box_f.h"
 
@@ skipping 11 matching lines @@
       if (keyframe->Time() < keyframes[i]->Time()) {
         keyframes.insert(keyframes.begin() + i, keyframe.Pass());
         return;
       }
     }
   }
 
   keyframes.push_back(keyframe.Pass());
 }
 
-template <class Keyframes>
-float GetProgress(double t, size_t i, const Keyframes& keyframes) {
-  float progress =
-      static_cast<float>((t - keyframes[i]->Time()) /
-                         (keyframes[i + 1]->Time() - keyframes[i]->Time()));
+// Assumes that (*keyframes).front()->Time() < t < (*keyframes).back()-Time().

[samli, 2014/09/24 07:55:32]

Use a DCHECK to assert this assumption

[ikilpatrick, 2014/09/25 04:33:25]

Acknowledged.

+template <typename KeyframeType>

[samli, 2014/09/24 07:55:32]

Is there a reason for renaming from Keyframes to KeyframeType?

[ikilpatrick, 2014/09/25 04:33:25]

A lot of this came from deleted GetCurveValue below which had KeyframeType
instead.

Which version is more idiomatic?

[samli, 2014/09/25 07:03:48]

Keyframe IMO

+double GetProgress(const ScopedPtrVector<KeyframeType>* keyframes,

[samli, 2014/09/24 07:55:32]

Can you separate the curve timing function logic to the keyframe timing
function? It's a little confusing now that some of the logic has been shifted
around. eg i being created, passed in and out.

[samli, 2014/09/24 07:55:32]

Use references instead

[ikilpatrick, 2014/09/25 04:33:25]

Done. PTAL.

[ikilpatrick, 2014/09/25 04:33:25]

Done.

[samli, 2014/09/25 07:03:48]

Looks good

+                   const scoped_ptr<TimingFunction>* timing_function,
+                   double time,
+                   size_t* i) {
+  double t = time;

[samli, 2014/09/24 07:55:32]

Unnecessary variable

[ikilpatrick, 2014/09/25 04:33:25]

Done.

+  double end_time = (*keyframes).back()->Time();
 
-  if (keyframes[i]->timing_function())
-    progress = keyframes[i]->timing_function()->GetValue(progress);
+  // Apply the curve timing function.
+  if (*timing_function) {
+    double start_time = (*keyframes).front()->Time();
+    double duration = end_time - start_time;
+    double animationProgress = (t - start_time) / duration;
+
+    t = (*timing_function)->GetValue(animationProgress) * duration + start_time;
+  }
+
+  // May have exceeded the end time of the last keyframe.
+  if (t >= end_time)
+    return 1.;
+
+  // Find the active keyframe.
+  *i = 0;
+  for (; *i < keyframes->size() - 1; ++*i) {
+    if (t < (*keyframes)[*i + 1]->Time())
+      break;
+  }
+
+  double progress = (t - (*keyframes)[*i]->Time()) /
+                    ((*keyframes)[*i + 1]->Time() - (*keyframes)[*i]->Time());
+
+  if ((*keyframes)[*i]->timing_function())
+    // Clamp progress as timing functions only valid for inputs [0,1].
+    progress = (*keyframes)[*i]->timing_function()->GetValue(
+        std::min(std::max(progress, 0.), 1.));

[samli, 2014/09/24 07:55:32]

No need for 0., just use 0

[ikilpatrick, 2014/09/25 04:33:25]

Need to use 0. as template won't match.
"deduced conflicting types for parameter '_Tp' ('double' vs. 'int')"

Need to compare like with like.

[samli, 2014/09/25 07:03:48]

Ah. I think the correct style for double is 0.0

+
   return progress;
 }
 
 scoped_ptr<TimingFunction> CloneTimingFunction(
     const TimingFunction* timing_function) {
   DCHECK(timing_function);
   scoped_ptr<AnimationCurve> curve(timing_function->Clone());
   return scoped_ptr<TimingFunction>(
       static_cast<TimingFunction*>(curve.release()));
 }
@@ skipping 132 matching lines @@
 
 double KeyframedColorAnimationCurve::Duration() const {
   return keyframes_.back()->Time() - keyframes_.front()->Time();
 }
 
 scoped_ptr<AnimationCurve> KeyframedColorAnimationCurve::Clone() const {
   scoped_ptr<KeyframedColorAnimationCurve> to_return(
       KeyframedColorAnimationCurve::Create());
   for (size_t i = 0; i < keyframes_.size(); ++i)
     to_return->AddKeyframe(keyframes_[i]->Clone());
+
+  if (timing_function_)
+    to_return->SetTimingFunction(CloneTimingFunction(timing_function_.get()));
+
   return to_return.PassAs<AnimationCurve>();
 }
 
 SkColor KeyframedColorAnimationCurve::GetValue(double t) const {
   if (t <= keyframes_.front()->Time())
     return keyframes_.front()->Value();
 
   if (t >= keyframes_.back()->Time())
     return keyframes_.back()->Value();
 
   size_t i = 0;
-  for (; i < keyframes_.size() - 1; ++i) {
-    if (t < keyframes_[i + 1]->Time())
-      break;
-  }
+  double progress = GetProgress(&keyframes_, &timing_function_, t, &i);
 
-  float progress = GetProgress(t, i, keyframes_);
+  if (progress >= 1.)
+    return keyframes_.back()->Value();
 
   return gfx::Tween::ColorValueBetween(
       progress, keyframes_[i]->Value(), keyframes_[i + 1]->Value());
 }
 
 // KeyframedFloatAnimationCurve
 
 scoped_ptr<KeyframedFloatAnimationCurve> KeyframedFloatAnimationCurve::
     Create() {
   return make_scoped_ptr(new KeyframedFloatAnimationCurve);
@@ skipping 10 matching lines @@
 
 double KeyframedFloatAnimationCurve::Duration() const {
   return keyframes_.back()->Time() - keyframes_.front()->Time();
 }
 
 scoped_ptr<AnimationCurve> KeyframedFloatAnimationCurve::Clone() const {
   scoped_ptr<KeyframedFloatAnimationCurve> to_return(
       KeyframedFloatAnimationCurve::Create());
   for (size_t i = 0; i < keyframes_.size(); ++i)
     to_return->AddKeyframe(keyframes_[i]->Clone());
+
+  if (timing_function_)
+    to_return->SetTimingFunction(CloneTimingFunction(timing_function_.get()));
+
   return to_return.PassAs<AnimationCurve>();
 }
 
 float KeyframedFloatAnimationCurve::GetValue(double t) const {
   if (t <= keyframes_.front()->Time())
     return keyframes_.front()->Value();
 
   if (t >= keyframes_.back()->Time())
     return keyframes_.back()->Value();
 
   size_t i = 0;
-  for (; i < keyframes_.size() - 1; ++i) {
-    if (t < keyframes_[i+1]->Time())
-      break;
-  }
+  double progress = GetProgress(&keyframes_, &timing_function_, t, &i);
 
-  float progress = GetProgress(t, i, keyframes_);
+  if (progress >= 1.)
+    return keyframes_.back()->Value();
 
   return keyframes_[i]->Value() +
       (keyframes_[i+1]->Value() - keyframes_[i]->Value()) * progress;
 }
 
 scoped_ptr<KeyframedTransformAnimationCurve> KeyframedTransformAnimationCurve::
     Create() {
   return make_scoped_ptr(new KeyframedTransformAnimationCurve);
 }
 
 KeyframedTransformAnimationCurve::KeyframedTransformAnimationCurve() {}
 
 KeyframedTransformAnimationCurve::~KeyframedTransformAnimationCurve() {}
 
 void KeyframedTransformAnimationCurve::AddKeyframe(
     scoped_ptr<TransformKeyframe> keyframe) {
   InsertKeyframe(keyframe.Pass(), keyframes_);
 }
 
 double KeyframedTransformAnimationCurve::Duration() const {
   return keyframes_.back()->Time() - keyframes_.front()->Time();
 }
 
 scoped_ptr<AnimationCurve> KeyframedTransformAnimationCurve::Clone() const {
   scoped_ptr<KeyframedTransformAnimationCurve> to_return(
       KeyframedTransformAnimationCurve::Create());
   for (size_t i = 0; i < keyframes_.size(); ++i)
     to_return->AddKeyframe(keyframes_[i]->Clone());
+
+  if (timing_function_)
+    to_return->SetTimingFunction(CloneTimingFunction(timing_function_.get()));
+
   return to_return.PassAs<AnimationCurve>();
 }
 
-// Assumes that (*keyframes).front()->Time() < t < (*keyframes).back()-Time().
-template<typename ValueType, typename KeyframeType>
-static ValueType GetCurveValue(const ScopedPtrVector<KeyframeType>* keyframes,
-                               double t) {
-  size_t i = 0;
-  for (; i < keyframes->size() - 1; ++i) {
-    if (t < (*keyframes)[i+1]->Time())
-      break;
-  }
-
-  double progress = (t - (*keyframes)[i]->Time()) /
-                    ((*keyframes)[i+1]->Time() - (*keyframes)[i]->Time());
-
-  if ((*keyframes)[i]->timing_function())
-    progress = (*keyframes)[i]->timing_function()->GetValue(progress);
-
-  return (*keyframes)[i+1]->Value().Blend((*keyframes)[i]->Value(), progress);
-}
-
 gfx::Transform KeyframedTransformAnimationCurve::GetValue(double t) const {
   if (t <= keyframes_.front()->Time())
     return keyframes_.front()->Value().Apply();
 
   if (t >= keyframes_.back()->Time())
     return keyframes_.back()->Value().Apply();
 
-  return GetCurveValue<gfx::Transform, TransformKeyframe>(&keyframes_, t);
+  size_t i = 0;
+  double progress = GetProgress(&keyframes_, &timing_function_, t, &i);
+
+  if (progress >= 1.)
+    return keyframes_.back()->Value().Apply();
+
+  return keyframes_[i + 1]->Value().Blend(keyframes_[i]->Value(), progress);
 }
 
 bool KeyframedTransformAnimationCurve::AnimatedBoundsForBox(
     const gfx::BoxF& box,
     gfx::BoxF* bounds) const {
   DCHECK_GE(keyframes_.size(), 2ul);
   *bounds = gfx::BoxF();
   for (size_t i = 0; i < keyframes_.size() - 1; ++i) {
     gfx::BoxF bounds_for_step;
     float min_progress = 0.0;
@@ skipping 65 matching lines @@
 
 double KeyframedFilterAnimationCurve::Duration() const {
   return keyframes_.back()->Time() - keyframes_.front()->Time();
 }
 
 scoped_ptr<AnimationCurve> KeyframedFilterAnimationCurve::Clone() const {
   scoped_ptr<KeyframedFilterAnimationCurve> to_return(
       KeyframedFilterAnimationCurve::Create());
   for (size_t i = 0; i < keyframes_.size(); ++i)
     to_return->AddKeyframe(keyframes_[i]->Clone());
+
+  if (timing_function_)
+    to_return->SetTimingFunction(CloneTimingFunction(timing_function_.get()));
+
   return to_return.PassAs<AnimationCurve>();
 }
 
 FilterOperations KeyframedFilterAnimationCurve::GetValue(double t) const {
   if (t <= keyframes_.front()->Time())
     return keyframes_.front()->Value();
 
   if (t >= keyframes_.back()->Time())
     return keyframes_.back()->Value();
 
-  return GetCurveValue<FilterOperations, FilterKeyframe>(&keyframes_, t);
+  size_t i = 0;
+  double progress = GetProgress(&keyframes_, &timing_function_, t, &i);
+
+  if (progress >= 1.)
+    return keyframes_.back()->Value();
+
+  return keyframes_[i + 1]->Value().Blend(keyframes_[i]->Value(), progress);
 }
 
 bool KeyframedFilterAnimationCurve::HasFilterThatMovesPixels() const {
   for (size_t i = 0; i < keyframes_.size(); ++i) {
     if (keyframes_[i]->Value().HasFilterThatMovesPixels()) {
       return true;
     }
   }
   return false;
 }
 
 }  // namespace cc