cc: Move Layer Id to Node Map to Individual Property Tree Private

cc/trees/layer_tree_host.cc

Index: cc/trees/layer_tree_host.cc
diff --git a/cc/trees/layer_tree_host.cc b/cc/trees/layer_tree_host.cc
index 4e6238ac9960364f0edad3913c05e2a834be1917..bb9d57a88dcc79886f37c5bfd662c6f62f09b6fb 100644
--- a/cc/trees/layer_tree_host.cc
+++ b/cc/trees/layer_tree_host.cc
@@ -1240,10 +1240,9 @@ void LayerTreeHost::SetElementOpacityMutated(ElementId element_id,
   DCHECK_LE(opacity, 1.f);
   layer->OnOpacityAnimated(opacity);
 
-  if (property_trees_.IsInIdToIndexMap(PropertyTrees::TreeType::EFFECT,
-                                       layer->id())) {
+  if (property_trees_.effect_tree.FindNodeFromId(layer->id())) {

[ajuma, 2017/03/16 17:15:31]

Since we're not using the node, this could FindNodeIndexFromId instead, or we
could keep the result for FindNodeFromId and skip the call to Node below.

     DCHECK_EQ(layer->effect_tree_index(),
-              property_trees_.layer_id_to_effect_node_index[layer->id()]);
+              property_trees_.effect_tree.FindNodeIndexFromId(layer->id()));
     EffectNode* node =
         property_trees_.effect_tree.Node(layer->effect_tree_index());
     if (node->opacity == opacity)
@@ -1264,10 +1263,9 @@ void LayerTreeHost::SetElementTransformMutated(
   DCHECK(layer);
   layer->OnTransformAnimated(transform);
 
-  if (property_trees_.IsInIdToIndexMap(PropertyTrees::TreeType::TRANSFORM,
-                                       layer->id())) {
+  if (property_trees_.transform_tree.FindNodeFromId(layer->id())) {

[ajuma, 2017/03/16 17:15:31]

Here too.

     DCHECK_EQ(layer->transform_tree_index(),
-              property_trees_.layer_id_to_transform_node_index[layer->id()]);
+              property_trees_.transform_tree.FindNodeIndexFromId(layer->id()));
     TransformNode* node =
         property_trees_.transform_tree.Node(layer->transform_tree_index());
     if (node->local == transform)