Use surface contents scale from effect tree (3)

cc/layers/render_surface_unittest.cc

Index: cc/layers/render_surface_unittest.cc
diff --git a/cc/layers/render_surface_unittest.cc b/cc/layers/render_surface_unittest.cc
index 25569faa633c91b8ff752673025fd29d314198a8..483bf4c8182bfdeadbd9625d60a85086da3e11c9 100644
--- a/cc/layers/render_surface_unittest.cc
+++ b/cc/layers/render_surface_unittest.cc
@@ -13,6 +13,7 @@
 #include "cc/test/mock_occlusion_tracker.h"
 #include "cc/test/test_shared_bitmap_manager.h"
 #include "cc/test/test_task_graph_runner.h"
+#include "cc/trees/draw_property_utils.h"
 #include "cc/trees/layer_tree_impl.h"
 #include "cc/trees/single_thread_proxy.h"
 #include "testing/gmock/include/gmock/gmock.h"
@@ -105,6 +106,10 @@ TEST(RenderSurfaceTest, SanityCheckSurfaceCreatesCorrectSharedQuadState) {
   root_layer->test_properties()->AddChild(std::move(owning_layer));
   host_impl.active_tree()->SetRootLayerForTesting(std::move(root_layer));
   host_impl.active_tree()->BuildLayerListAndPropertyTreesForTesting();
+  EXPECT_TRUE(
+      host_impl.active_tree()->property_trees()->effect_tree.needs_update());
+  draw_property_utils::ComputeEffects(

[ajuma, 2016/07/19 21:59:58]

Would it make sense to call active_tree()->UpdateDrawProperties() here instead?

[jaydasika, 2016/07/19 22:56:11]

Done.

+      &(host_impl.active_tree()->property_trees()->effect_tree));
 
   gfx::Rect content_rect(0, 0, 50, 50);
   gfx::Rect clip_rect(5, 5, 40, 40);