Handle direction control in compositor Animations

cc/animation/animation_unittest.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 "cc/animation/animation.h"
 
 #include "cc/test/animation_test_common.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
@@ skipping 31 matching lines @@
 }
 
 TEST(AnimationTest, TrimTimeInfiniteIterations) {
   scoped_ptr<Animation> anim(CreateAnimation(-1));
   EXPECT_EQ(0, anim->TrimTimeToCurrentIteration(0.0));
   EXPECT_EQ(0.5, anim->TrimTimeToCurrentIteration(0.5));
   EXPECT_EQ(0, anim->TrimTimeToCurrentIteration(1.0));
   EXPECT_EQ(0.5, anim->TrimTimeToCurrentIteration(1.5));
 }
 
-TEST(AnimationTest, TrimTimeAlternating) {
+TEST(AnimationTest, TrimTimeReverse) {

[mithro-old, 2014/03/03 06:30:52]

This suggestion is **optional**, it doesn't block landing this patch.

It would be nice to see a simple test which showed TrimTimeReverse/Alternative
interacting with pause time, start time and offset. 

   scoped_ptr<Animation> anim(CreateAnimation(-1));
-  anim->set_alternates_direction(true);
+  anim->set_direction(Animation::Reverse);
+  EXPECT_EQ(1.0, anim->TrimTimeToCurrentIteration(0));
+  EXPECT_EQ(0.75, anim->TrimTimeToCurrentIteration(0.25));
+  EXPECT_EQ(0.5, anim->TrimTimeToCurrentIteration(0.5));
+  EXPECT_EQ(0.25, anim->TrimTimeToCurrentIteration(0.75));
+  EXPECT_EQ(1.0, anim->TrimTimeToCurrentIteration(1.0));
+  EXPECT_EQ(0.75, anim->TrimTimeToCurrentIteration(1.25));
+}
+
+TEST(AnimationTest, TrimTimeAlternate) {
+  scoped_ptr<Animation> anim(CreateAnimation(-1));
+  anim->set_direction(Animation::Alternate);
   EXPECT_EQ(0, anim->TrimTimeToCurrentIteration(0.0));
+  EXPECT_EQ(0.25, anim->TrimTimeToCurrentIteration(0.25));
   EXPECT_EQ(0.5, anim->TrimTimeToCurrentIteration(0.5));
+  EXPECT_EQ(0.75, anim->TrimTimeToCurrentIteration(0.75));
   EXPECT_EQ(1, anim->TrimTimeToCurrentIteration(1.0));
   EXPECT_EQ(0.75, anim->TrimTimeToCurrentIteration(1.25));
 }
 
+TEST(AnimationTest, TrimTimeAlternateReverse) {
+  scoped_ptr<Animation> anim(CreateAnimation(-1));
+  anim->set_direction(Animation::AlternateReverse);
+  EXPECT_EQ(1.0, anim->TrimTimeToCurrentIteration(0.0));
+  EXPECT_EQ(0.75, anim->TrimTimeToCurrentIteration(0.25));
+  EXPECT_EQ(0.5, anim->TrimTimeToCurrentIteration(0.5));
+  EXPECT_EQ(0.25, anim->TrimTimeToCurrentIteration(0.75));
+  EXPECT_EQ(0.0, anim->TrimTimeToCurrentIteration(1.0));
+  EXPECT_EQ(0.25, anim->TrimTimeToCurrentIteration(1.25));
+}
+
 TEST(AnimationTest, TrimTimeStartTime) {
   scoped_ptr<Animation> anim(CreateAnimation(1));
   anim->set_start_time(4);
   EXPECT_EQ(0, anim->TrimTimeToCurrentIteration(0.0));
   EXPECT_EQ(0, anim->TrimTimeToCurrentIteration(4.0));
   EXPECT_EQ(0.5, anim->TrimTimeToCurrentIteration(4.5));
   EXPECT_EQ(1, anim->TrimTimeToCurrentIteration(5.0));
   EXPECT_EQ(1, anim->TrimTimeToCurrentIteration(6.0));
 }
 
@@ skipping 183 matching lines @@
   EXPECT_EQ(Animation::Paused, anim->run_state());
   anim->SetRunState(Animation::Running, 0.0);
   EXPECT_EQ(Animation::Paused, anim->run_state());
   anim->Resume(0);
   anim->SetRunState(Animation::Running, 0.0);
   EXPECT_EQ(Animation::Running, anim->run_state());
 }
 
 }  // namespace
 }  // namespace cc