cc: Calculate visible rect inside non root copy request

cc/trees/layer_tree_host_common_unittest.cc

Index: cc/trees/layer_tree_host_common_unittest.cc
diff --git a/cc/trees/layer_tree_host_common_unittest.cc b/cc/trees/layer_tree_host_common_unittest.cc
index 316545f3426620726a8ed336fb7a4bc8d8446277..91aedbfa880617155ee88861bd1bb7baa89cc6d6 100644
--- a/cc/trees/layer_tree_host_common_unittest.cc
+++ b/cc/trees/layer_tree_host_common_unittest.cc
@@ -5769,6 +5769,63 @@ TEST_F(LayerTreeHostCommonTest, ClippedOutCopyRequest) {
             root_layer->render_surface()->layer_list().at(0)->id());
 }
 
+TEST_F(LayerTreeHostCommonTest, VisibleRectInNonRootCopyRequest) {
+  FakeImplTaskRunnerProvider task_runner_provider;
+  TestSharedBitmapManager shared_bitmap_manager;
+  TestTaskGraphRunner task_graph_runner;
+  FakeLayerTreeHostImpl host_impl(&task_runner_provider, &shared_bitmap_manager,
+                                  &task_graph_runner);
+  host_impl.CreatePendingTree();

[ajuma, 2016/06/14 15:09:53]

Is this needed? It looks like the test is only using the active tree.

[weiliangc, 2016/06/15 12:27:57]

Removed.

+  const gfx::Transform identity_matrix;
+
+  LayerImpl* root = root_layer();
+  SetLayerPropertiesForTesting(root, identity_matrix, gfx::Point3F(),
+                               gfx::PointF(), gfx::Size(50, 50), true, false,
+                               true);
+  root->SetDrawsContent(true);
+  root->SetMasksToBounds(true);
+
+  LayerImpl* copy_layer = AddChild<LayerImpl>(root);
+  SetLayerPropertiesForTesting(copy_layer, identity_matrix, gfx::Point3F(),
+                               gfx::PointF(), gfx::Size(100, 100), true, false,
+                               true);
+  copy_layer->SetDrawsContent(true);

[ajuma, 2016/06/14 15:25:17]

It be worth including a test where the copy_layer itself masks to bounds, to
show that clip is applied when computing visible rects.

[weiliangc, 2016/06/15 12:27:57]

Done.

+
+  LayerImpl* copy_child = AddChild<LayerImpl>(copy_layer);
+  SetLayerPropertiesForTesting(copy_child, identity_matrix, gfx::Point3F(),
+                               gfx::PointF(40, 40), gfx::Size(20, 20), true,
+                               false, false);
+  copy_child->SetDrawsContent(true);
+
+  LayerImpl* copy_clip = AddChild<LayerImpl>(copy_layer);
+  SetLayerPropertiesForTesting(copy_clip, identity_matrix, gfx::Point3F(),
+                               gfx::PointF(), gfx::Size(55, 55), true, false,
+                               false);
+  copy_clip->SetMasksToBounds(true);
+
+  LayerImpl* copy_clipped_child = AddChild<LayerImpl>(copy_clip);
+  SetLayerPropertiesForTesting(copy_clipped_child, identity_matrix,
+                               gfx::Point3F(), gfx::PointF(40, 40),
+                               gfx::Size(20, 20), true, false, false);
+  copy_clipped_child->SetDrawsContent(true);
+
+  LayerImpl* copy_surface = AddChild<LayerImpl>(copy_clip);
+  SetLayerPropertiesForTesting(copy_surface, identity_matrix, gfx::Point3F(),
+                               gfx::PointF(45, 45), gfx::Size(20, 20), true,
+                               false, true);
+  copy_surface->SetDrawsContent(true);
+
+  copy_layer->test_properties()->copy_requests.push_back(
+      CopyOutputRequest::CreateRequest(base::Bind(&EmptyCopyOutputCallback)));
+
+  ExecuteCalculateDrawProperties(root);
+
+  EXPECT_EQ(gfx::Rect(100, 100), copy_layer->visible_layer_rect());
+  EXPECT_EQ(gfx::Rect(20, 20), copy_child->visible_layer_rect());
+  EXPECT_EQ(gfx::Rect(15, 15), copy_clipped_child->visible_layer_rect());
+  EXPECT_EQ(gfx::Rect(10, 10), copy_surface->visible_layer_rect());
+}
+
 TEST_F(LayerTreeHostCommonTest, VisibleContentRectInsideSurface) {
   LayerImpl* root = root_layer();
   LayerImpl* surface = AddChild<LayerImpl>(root);