cc: Compute animation scale on demand

cc/trees/property_tree_builder.cc

Index: cc/trees/property_tree_builder.cc
diff --git a/cc/trees/property_tree_builder.cc b/cc/trees/property_tree_builder.cc
index c86dde4330a5d4983c9389c8167cc069069ef8f9..7f35dd96ef481d61b0bf8f8ddcb38da5e3e2bdc9 100644
--- a/cc/trees/property_tree_builder.cc
+++ b/cc/trees/property_tree_builder.cc
@@ -558,17 +558,6 @@ bool AddTransformNodeIfNeeded(
   node->data.has_potential_animation = has_potentially_animated_transform;
   node->data.is_currently_animating = layer->TransformIsAnimating();
   if (has_potentially_animated_transform) {
-    float maximum_animation_target_scale = 0.f;
-    if (layer->MaximumTargetScale(&maximum_animation_target_scale)) {
-      node->data.local_maximum_animation_target_scale =
-          maximum_animation_target_scale;
-    }
-
-    float starting_animation_scale = 0.f;
-    if (layer->AnimationStartScale(&starting_animation_scale)) {
-      node->data.local_starting_animation_scale = starting_animation_scale;
-    }
-
     node->data.has_only_translation_animations =
         layer->HasOnlyTranslationTransforms();
   }
@@ -1393,6 +1382,7 @@ void PropertyTreeBuilder::BuildPropertyTrees(
       outer_viewport_scroll_layer, overscroll_elasticity_layer,
       elastic_overscroll, page_scale_factor, device_scale_factor, viewport,
       device_transform, property_trees, color);
+  property_trees->ResetCachedData();

[jaydasika, 2016/06/28 23:08:40]

Should this also be in the Layer version of the function ? Or maybe, put it
inside BuildPropertyTreesTopLevelInternal so that all clearing code is in one
place ?

[sunxd, 2016/06/30 15:29:44]

The only place where animation scales get used is in PictureLayerImpl in
production code, and ResetCachedData is called when pushing property trees from
main to impl. This line here is only for testing. Maybe when we decide to move
other stuff from property trees to the cached data structure, we need to reset
the cache on main thread, but I would still stick to this currently.

 }
 
 }  // namespace cc