cc: Make PictureLayerImpl use a better choice for animated raster scale.

cc/animation/keyframed_animation_curve_unittest.cc

Index: cc/animation/keyframed_animation_curve_unittest.cc
diff --git a/cc/animation/keyframed_animation_curve_unittest.cc b/cc/animation/keyframed_animation_curve_unittest.cc
index 9ce1fdbac014c1a2f89f742441b7467f648e31b2..14c8aec39a09d6fa798cca19f3715006ff7b664e 100644
--- a/cc/animation/keyframed_animation_curve_unittest.cc
+++ b/cc/animation/keyframed_animation_curve_unittest.cc
@@ -523,7 +523,7 @@ TEST(KeyframedAnimationCurveTest, MaximumScale) {
       1.0, operations1, EaseTimingFunction::Create()));
 
   float maximum_scale = 0.f;
-  EXPECT_TRUE(curve->MaximumScale(&maximum_scale));
+  EXPECT_TRUE(curve->MaximumTargetScale(&maximum_scale));
   EXPECT_EQ(3.f, maximum_scale);
 
   TransformOperations operations2;
@@ -531,7 +531,7 @@ TEST(KeyframedAnimationCurveTest, MaximumScale) {
   curve->AddKeyframe(TransformKeyframe::Create(
       2.0, operations2, EaseTimingFunction::Create()));
 
-  EXPECT_TRUE(curve->MaximumScale(&maximum_scale));
+  EXPECT_TRUE(curve->MaximumTargetScale(&maximum_scale));
   EXPECT_EQ(6.f, maximum_scale);
 
   TransformOperations operations3;
@@ -539,7 +539,21 @@ TEST(KeyframedAnimationCurveTest, MaximumScale) {
   curve->AddKeyframe(TransformKeyframe::Create(
       3.0, operations3, EaseTimingFunction::Create()));
 
-  EXPECT_FALSE(curve->MaximumScale(&maximum_scale));
+  EXPECT_FALSE(curve->MaximumTargetScale(&maximum_scale));
+
+  // The original scale is not used in computing the max.
+  scoped_ptr<KeyframedTransformAnimationCurve> curve2(
+      KeyframedTransformAnimationCurve::Create());
+
+  TransformOperations operations4;
+  curve2->AddKeyframe(TransformKeyframe::Create(
+      0.0, operations4, EaseTimingFunction::Create()));
+  operations4.AppendScale(0.5f, 0.3f, 0.8f);

[vmpstr, 2014/10/10 00:50:49]

Can you make that one -0.8f? (just to ensure that we're grabbing the abs scale)

[danakj, 2014/10/10 01:00:19]

Sure, I think did this somewhere else too.. but done!

+  curve2->AddKeyframe(TransformKeyframe::Create(
+      1.0, operations4, EaseTimingFunction::Create()));
+
+  EXPECT_TRUE(curve2->MaximumTargetScale(&maximum_scale));
+  EXPECT_EQ(0.8f, maximum_scale);
 }
 
 // Tests that an animation with a curve timing function works as expected.