SubtreeShouldBeSkipped uses information from property trees

cc/trees/layer_tree_impl_unittest.cc

Index: cc/trees/layer_tree_impl_unittest.cc
diff --git a/cc/trees/layer_tree_impl_unittest.cc b/cc/trees/layer_tree_impl_unittest.cc
index 8cf4f0142556bfd3bc9480bce12313d13a9eb23f..31e18339823b4b29edd36829d7c81039cce2c36e 100644
--- a/cc/trees/layer_tree_impl_unittest.cc
+++ b/cc/trees/layer_tree_impl_unittest.cc
@@ -2356,10 +2356,21 @@ TEST_F(LayerTreeImplTest, SelectionBoundsWithLargeTransforms) {
   ViewportSelection output;
   host_impl().active_tree()->GetViewportSelection(&output);
 
-  // edge_bottom and edge_top aren't allowed to have NaNs, so the selection
-  // should be empty.
-  EXPECT_EQ(ViewportSelectionBound(), output.start);
-  EXPECT_EQ(ViewportSelectionBound(), output.end);
+  ViewportSelectionBound expected_start;
+  ViewportSelectionBound expected_end;
+
+  expected_start.type = SELECTION_BOUND_LEFT;
+  expected_start.edge_top = gfx::PointF(10, 10);
+  expected_start.edge_bottom = gfx::PointF(10, 20);
+  expected_start.visible = true;
+
+  expected_end.type = SELECTION_BOUND_RIGHT;
+  expected_end.edge_top = gfx::PointF(50, 10);
+  expected_end.edge_bottom = gfx::PointF(50, 30);
+  expected_end.visible = true;
+
+  EXPECT_EQ(expected_start, output.start);
+  EXPECT_EQ(expected_end, output.end);

[ajuma, 2016/03/24 15:11:59]

I think this might be another case where we have a layer that's winding up in
the RSLL but not getting draw properties computed.

 }
 
 TEST_F(LayerTreeImplTest, NumLayersTestOne) {