Index: cc/trees/damage_tracker_unittest.cc |
diff --git a/cc/trees/damage_tracker_unittest.cc b/cc/trees/damage_tracker_unittest.cc |
index 9a00403a82154086934292fa45dfa81cec1c301c..6fd2929156bbe91ca08186fc36730b5c98fd903b 100644 |
--- a/cc/trees/damage_tracker_unittest.cc |
+++ b/cc/trees/damage_tracker_unittest.cc |
@@ -29,6 +29,7 @@ void ExecuteCalculateDrawProperties(LayerImpl* root, |
// Sanity check: The test itself should create the root layer's render |
// surface, so that the surface (and its damage tracker) can |
// persist across multiple calls to this function. |
+ ASSERT_TRUE(root->render_surface()); |
ASSERT_FALSE(render_surface_layer_list->size()); |
FakeLayerTreeHostImpl::RecursiveUpdateNumChildren(root); |
@@ -37,7 +38,6 @@ void ExecuteCalculateDrawProperties(LayerImpl* root, |
root, root->bounds(), render_surface_layer_list, |
root->layer_tree_impl()->current_render_surface_list_id()); |
LayerTreeHostCommon::CalculateDrawProperties(&inputs); |
- ASSERT_TRUE(root->render_surface()); |
} |
void ClearDamageForAllSurfaces(LayerImpl* layer) { |
@@ -94,7 +94,8 @@ class DamageTrackerTest : public testing::Test { |
root->SetPosition(gfx::PointF()); |
root->SetBounds(gfx::Size(500, 500)); |
root->SetDrawsContent(true); |
- root->SetForceRenderSurface(true); |
+ root->SetHasRenderSurface(true); |
+ root->render_surface()->SetContentRect(gfx::Rect(0, 0, 500, 500)); |
child->SetPosition(gfx::PointF(100.f, 100.f)); |
child->SetBounds(gfx::Size(30, 30)); |
@@ -123,7 +124,8 @@ class DamageTrackerTest : public testing::Test { |
root->SetPosition(gfx::PointF()); |
root->SetBounds(gfx::Size(500, 500)); |
root->SetDrawsContent(true); |
- root->SetForceRenderSurface(true); |
+ root->SetHasRenderSurface(true); |
+ root->render_surface()->SetContentRect(gfx::Rect(0, 0, 500, 500)); |
child1->SetPosition(gfx::PointF(100.f, 100.f)); |
child1->SetBounds(gfx::Size(30, 30)); |
@@ -131,7 +133,7 @@ class DamageTrackerTest : public testing::Test { |
// its own RenderSurface. This layer does not draw, but is intended to |
// create its own RenderSurface. |
child1->SetDrawsContent(false); |
- child1->SetForceRenderSurface(true); |
+ child1->SetHasRenderSurface(true); |
child2->SetPosition(gfx::PointF(11.f, 11.f)); |
child2->SetBounds(gfx::Size(18, 18)); |
@@ -542,7 +544,7 @@ TEST_F(DamageTrackerTest, VerifyDamageForImageFilter) { |
// Setting the filter will damage the whole surface. |
ClearDamageForAllSurfaces(root.get()); |
- child->SetForceRenderSurface(true); |
+ child->SetHasRenderSurface(true); |
child->SetFilters(filters); |
EmulateDrawingOneFrame(root.get()); |
root_damage_rect = |
@@ -935,7 +937,7 @@ TEST_F(DamageTrackerTest, VerifyDamageForAddingAndRemovingRenderSurfaces) { |
// CASE 1: If a descendant surface disappears, its entire old area becomes |
// exposed. |
ClearDamageForAllSurfaces(root.get()); |
- child1->SetForceRenderSurface(false); |
+ child1->SetHasRenderSurface(false); |
EmulateDrawingOneFrame(root.get()); |
// Sanity check that there is only one surface now. |
@@ -960,7 +962,7 @@ TEST_F(DamageTrackerTest, VerifyDamageForAddingAndRemovingRenderSurfaces) { |
// Then change the tree so that the render surface is added back. |
ClearDamageForAllSurfaces(root.get()); |
- child1->SetForceRenderSurface(true); |
+ child1->SetHasRenderSurface(true); |
EmulateDrawingOneFrame(root.get()); |
@@ -1063,7 +1065,7 @@ TEST_F(DamageTrackerTest, VerifyDamageForReplica) { |
reflection.Scale3d(-1.0, 1.0, 1.0); |
grand_child1_replica->SetTransform(reflection); |
grand_child1->SetReplicaLayer(std::move(grand_child1_replica)); |
- grand_child1->SetForceRenderSurface(true); |
+ grand_child1->SetHasRenderSurface(true); |
} |
EmulateDrawingOneFrame(root.get()); |
@@ -1114,7 +1116,7 @@ TEST_F(DamageTrackerTest, VerifyDamageForReplica) { |
// reflection to damage the target surface. |
ClearDamageForAllSurfaces(root.get()); |
grand_child1->SetReplicaLayer(nullptr); |
- grand_child1->SetForceRenderSurface(false); |
+ grand_child1->SetHasRenderSurface(false); |
EmulateDrawingOneFrame(root.get()); |
ASSERT_EQ(old_content_rect.width(), |
child1->render_surface()->content_rect().width()); |
@@ -1148,7 +1150,7 @@ TEST_F(DamageTrackerTest, VerifyDamageForMask) { |
mask_layer->SetPosition(child->position()); |
mask_layer->SetBounds(child->bounds()); |
child->SetMaskLayer(std::move(mask_layer)); |
- child->SetForceRenderSurface(true); |
+ child->SetHasRenderSurface(true); |
} |
LayerImpl* mask_layer = child->mask_layer(); |
@@ -1237,7 +1239,7 @@ TEST_F(DamageTrackerTest, VerifyDamageForReplicaMask) { |
reflection.Scale3d(-1.0, 1.0, 1.0); |
grand_child1_replica->SetTransform(reflection); |
grand_child1->SetReplicaLayer(std::move(grand_child1_replica)); |
- grand_child1->SetForceRenderSurface(true); |
+ grand_child1->SetHasRenderSurface(true); |
} |
LayerImpl* grand_child1_replica = grand_child1->replica_layer(); |
@@ -1314,7 +1316,7 @@ TEST_F(DamageTrackerTest, VerifyDamageForReplicaMaskWithTransformOrigin) { |
reflection.Scale3d(-1.0, 1.0, 1.0); |
grand_child1_replica->SetTransform(reflection); |
grand_child1->SetReplicaLayer(std::move(grand_child1_replica)); |
- grand_child1->SetForceRenderSurface(true); |
+ grand_child1->SetHasRenderSurface(true); |
} |
LayerImpl* grand_child1_replica = grand_child1->replica_layer(); |
@@ -1381,8 +1383,7 @@ TEST_F(DamageTrackerTest, VerifyDamageForEmptyLayerList) { |
// tracker does not crash when it receives an empty layer_list. |
scoped_ptr<LayerImpl> root = LayerImpl::Create(host_impl_.active_tree(), 1); |
- root->SetForceRenderSurface(true); |
- EmulateDrawingOneFrame(root.get()); |
+ root->SetHasRenderSurface(true); |
root->draw_properties().render_target = root.get(); |
ASSERT_EQ(root.get(), root->render_target()); |