cc: Compute target space transform dynamically

cc/layers/layer.cc

Index: cc/layers/layer.cc
diff --git a/cc/layers/layer.cc b/cc/layers/layer.cc
index e0e09b5d630c48532175ed6f7c0177344743c140..461b7f3eba8d3634f1fe69b94b388c9a27a19a36 100644
--- a/cc/layers/layer.cc
+++ b/cc/layers/layer.cc
@@ -981,6 +981,15 @@ int Layer::effect_tree_index() const {
   return effect_tree_index_;
 }
 
+int Layer::render_target_effect_tree_index() const {
+  EffectNode* effect_node =
+      layer_tree_host_->property_trees()->effect_tree.Node(effect_tree_index_);
+  if (effect_node->render_surface)

[ajuma, 2016/07/22 18:29:40]

I think this is always null on the main thread side. So I think this function is
currently always returning the root effect node's id. Did you mean to use
has_render_surface here (on the main thread side it makes sense to do that,
since there's no resourceless software mode to worry about). It also suggests
that the remaining tests that depend on this logic aren't doing interesting
things with surfaces.

[sunxd, 2016/07/22 20:25:03]

Done.

+    return effect_node->id;
+  else
+    return effect_node->target_id;
+}
+
 void Layer::SetScrollTreeIndex(int index) {
   DCHECK(IsPropertyChangeAllowed());
   if (scroll_tree_index_ == index)