cc/trees/layer_tree_host_impl_unittest.cc - Issue 465853004: Moving RenderSurface creation outside of CalcDrawProps

Unified Diff: cc/trees/layer_tree_host_impl_unittest.cc

Issue 465853004: Moving RenderSurface creation outside of CalcDrawProps (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master

Patch Set: Fixed formatting Created 6 years ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

View side-by-side diff with in-line comments

Download patch

Index: cc/trees/layer_tree_host_impl_unittest.cc

diff --git a/cc/trees/layer_tree_host_impl_unittest.cc b/cc/trees/layer_tree_host_impl_unittest.cc

index 8d0ee39ac6faf30a491828f0d8c2287ab3219f42..2b54b0c789b8a18cc1a2b645f48e9ca943977090 100644

--- a/cc/trees/layer_tree_host_impl_unittest.cc

+++ b/cc/trees/layer_tree_host_impl_unittest.cc

@@ -50,6 +50,7 @@

#include "cc/test/fake_video_frame_provider.h"

#include "cc/test/geometry_test_utils.h"

#include "cc/test/layer_test_common.h"

+#include "cc/test/layer_tree_test.h"

#include "cc/test/render_pass_test_common.h"

#include "cc/test/test_gpu_memory_buffer_manager.h"

#include "cc/test/test_shared_bitmap_manager.h"

@@ -180,6 +181,7 @@ class LayerTreeHostImplTest : public testing::Test,

root->SetContentBounds(gfx::Size(10, 10));

root->SetDrawsContent(true);

root->draw_properties().visible_content_rect = gfx::Rect(0, 0, 10, 10);

+ root->SetHasRenderSurface(true);

host_impl_->active_tree()->SetRootLayer(root.Pass());

}

@@ -226,6 +228,7 @@ class LayerTreeHostImplTest : public testing::Test,

root->SetBounds(content_size);

root->SetContentBounds(content_size);

root->SetPosition(gfx::PointF());

+ root->SetHasRenderSurface(true);

scoped_ptr<LayerImpl> scroll =

LayerImpl::Create(layer_tree_impl, kInnerViewportScrollLayerId);

@@ -567,20 +570,6 @@ TEST_F(LayerTreeHostImplTest, ReplaceTreeWhileScrolling) {

ExpectContains(*scroll_info, scroll_layer->id(), scroll_delta);

}

-TEST_F(LayerTreeHostImplTest, ClearRootRenderSurfaceAndScroll) {

- SetupScrollAndContentsLayers(gfx::Size(100, 100));

- host_impl_->SetViewportSize(gfx::Size(50, 50));

- DrawFrame();

- // We should be able to scroll even if the root layer loses its render surface

- // after the most recent render.

- host_impl_->active_tree()->root_layer()->ClearRenderSurface();

- host_impl_->active_tree()->set_needs_update_draw_properties();

- EXPECT_EQ(InputHandler::ScrollStarted,

- host_impl_->ScrollBegin(gfx::Point(), InputHandler::Wheel));

TEST_F(LayerTreeHostImplTest, WheelEventHandlers) {

SetupScrollAndContentsLayers(gfx::Size(100, 100));

host_impl_->SetViewportSize(gfx::Size(50, 50));

@@ -896,20 +885,6 @@ TEST_F(LayerTreeHostImplTest, ScrollWithUserUnscrollableLayers) {

EXPECT_VECTOR_EQ(gfx::Vector2dF(10, 20), overflow->TotalScrollOffset());

}

-TEST_F(LayerTreeHostImplTest,

- ClearRootRenderSurfaceAndHitTestTouchHandlerRegion) {

- SetupScrollAndContentsLayers(gfx::Size(100, 100));

- host_impl_->SetViewportSize(gfx::Size(50, 50));

- DrawFrame();

- // We should be able to hit test for touch event handlers even if the root

- // layer loses its render surface after the most recent render.

- host_impl_->active_tree()->root_layer()->ClearRenderSurface();

- host_impl_->active_tree()->set_needs_update_draw_properties();

- EXPECT_EQ(host_impl_->HaveTouchEventHandlersAt(gfx::Point()), false);

TEST_F(LayerTreeHostImplTest, ImplPinchZoom) {

LayerImpl* scroll_layer = SetupScrollAndContentsLayers(gfx::Size(100, 100));

host_impl_->SetViewportSize(gfx::Size(50, 50));

@@ -1449,6 +1424,7 @@ class LayerTreeHostImplOverridePhysicalTime : public LayerTreeHostImpl {

scroll->AddChild(contents.Pass()); \

root->AddChild(scroll.Pass()); \

+ root->SetHasRenderSurface(true); \

scrollbar->SetScrollLayerAndClipLayerByIds(2, 1); \

root->AddChild(scrollbar.Pass()); \

@@ -1593,6 +1569,7 @@ void LayerTreeHostImplTest::SetupMouseMoveAtWithDeviceScale(

scoped_ptr<LayerImpl> root =

LayerImpl::Create(host_impl_->active_tree(), 1);

root->SetBounds(viewport_size);

+ root->SetHasRenderSurface(true);

scoped_ptr<LayerImpl> scroll =

LayerImpl::Create(host_impl_->active_tree(), 2);

@@ -1798,6 +1775,7 @@ TEST_F(LayerTreeHostImplTest, WillDrawReturningFalseDoesNotCall) {

host_impl_->active_tree()->root_layer());

root->AddChild(DidDrawCheckLayer::Create(host_impl_->active_tree(), 2));

+ root->SetHasRenderSurface(true);

DidDrawCheckLayer* layer =

static_cast<DidDrawCheckLayer*>(root->children()[0]);

@@ -1838,7 +1816,7 @@ TEST_F(LayerTreeHostImplTest, DidDrawNotCalledOnHiddenLayer) {

DidDrawCheckLayer* root = static_cast<DidDrawCheckLayer*>(

host_impl_->active_tree()->root_layer());

root->SetMasksToBounds(true);

+ root->SetHasRenderSurface(true);

root->AddChild(DidDrawCheckLayer::Create(host_impl_->active_tree(), 2));

DidDrawCheckLayer* layer =

static_cast<DidDrawCheckLayer*>(root->children()[0]);

@@ -1891,6 +1869,7 @@ TEST_F(LayerTreeHostImplTest, WillDrawNotCalledOnOccludedLayer) {

static_cast<DidDrawCheckLayer*>(root->children()[0]);

root->AddChild(DidDrawCheckLayer::Create(host_impl_->active_tree(), 3));

+ root->SetHasRenderSurface(true);

DidDrawCheckLayer* top_layer =

static_cast<DidDrawCheckLayer*>(root->children()[1]);

// This layer covers the occluded_layer above. Make this layer large so it can

@@ -1923,6 +1902,7 @@ TEST_F(LayerTreeHostImplTest, DidDrawCalledOnAllLayers) {

static_cast<DidDrawCheckLayer*>(host_impl_->active_tree()->root_layer());

root->AddChild(DidDrawCheckLayer::Create(host_impl_->active_tree(), 2));

+ root->SetHasRenderSurface(true);

DidDrawCheckLayer* layer1 =

static_cast<DidDrawCheckLayer*>(root->children()[0]);

@@ -1930,7 +1910,7 @@ TEST_F(LayerTreeHostImplTest, DidDrawCalledOnAllLayers) {

DidDrawCheckLayer* layer2 =

static_cast<DidDrawCheckLayer*>(layer1->children()[0]);

- layer1->SetOpacity(0.3f);

+ layer1->SetHasRenderSurface(true);

layer1->SetShouldFlattenTransform(true);

EXPECT_FALSE(root->did_draw_called());

@@ -2002,7 +1982,7 @@ TEST_F(LayerTreeHostImplTest, PrepareToDrawSucceedsOnDefault) {

DidDrawCheckLayer::Create(host_impl_->active_tree(), 1));

DidDrawCheckLayer* root =

static_cast<DidDrawCheckLayer*>(host_impl_->active_tree()->root_layer());

+ root->SetHasRenderSurface(true);

bool tile_missing = false;

bool had_incomplete_tile = false;

bool is_animating = false;

@@ -2026,6 +2006,7 @@ TEST_F(LayerTreeHostImplTest, PrepareToDrawSucceedsWithAnimatedLayer) {

DidDrawCheckLayer::Create(host_impl_->active_tree(), 1));

DidDrawCheckLayer* root =

static_cast<DidDrawCheckLayer*>(host_impl_->active_tree()->root_layer());

+ root->SetHasRenderSurface(true);

bool tile_missing = false;

bool had_incomplete_tile = false;

bool is_animating = true;

@@ -2049,6 +2030,7 @@ TEST_F(LayerTreeHostImplTest, PrepareToDrawSucceedsWithMissingTiles) {

DidDrawCheckLayer::Create(host_impl_->active_tree(), 3));

DidDrawCheckLayer* root =

static_cast<DidDrawCheckLayer*>(host_impl_->active_tree()->root_layer());

+ root->SetHasRenderSurface(true);

LayerTreeHostImpl::FrameData frame;

EXPECT_EQ(DRAW_SUCCESS, host_impl_->PrepareToDraw(&frame));

@@ -2077,6 +2059,7 @@ TEST_F(LayerTreeHostImplTest, PrepareToDrawSucceedsWithIncompleteTile) {

DidDrawCheckLayer::Create(host_impl_->active_tree(), 3));

DidDrawCheckLayer* root =

static_cast<DidDrawCheckLayer*>(host_impl_->active_tree()->root_layer());

+ root->SetHasRenderSurface(true);

LayerTreeHostImpl::FrameData frame;

EXPECT_EQ(DRAW_SUCCESS, host_impl_->PrepareToDraw(&frame));

@@ -2106,6 +2089,7 @@ TEST_F(LayerTreeHostImplTest,

DidDrawCheckLayer::Create(host_impl_->active_tree(), 5));

DidDrawCheckLayer* root =

static_cast<DidDrawCheckLayer*>(host_impl_->active_tree()->root_layer());

+ root->SetHasRenderSurface(true);

LayerTreeHostImpl::FrameData frame;

EXPECT_EQ(DRAW_SUCCESS, host_impl_->PrepareToDraw(&frame));

@@ -2136,6 +2120,7 @@ TEST_F(LayerTreeHostImplTest,

DidDrawCheckLayer::Create(host_impl_->active_tree(), 5));

DidDrawCheckLayer* root =

static_cast<DidDrawCheckLayer*>(host_impl_->active_tree()->root_layer());

+ root->SetHasRenderSurface(true);

LayerTreeHostImpl::FrameData frame;

EXPECT_EQ(DRAW_SUCCESS, host_impl_->PrepareToDraw(&frame));

@@ -2164,6 +2149,7 @@ TEST_F(LayerTreeHostImplTest, PrepareToDrawSucceedsWhenHighResRequired) {

DidDrawCheckLayer::Create(host_impl_->active_tree(), 7));

DidDrawCheckLayer* root =

static_cast<DidDrawCheckLayer*>(host_impl_->active_tree()->root_layer());

+ root->SetHasRenderSurface(true);

LayerTreeHostImpl::FrameData frame;

EXPECT_EQ(DRAW_SUCCESS, host_impl_->PrepareToDraw(&frame));

@@ -2194,6 +2180,7 @@ TEST_F(LayerTreeHostImplTest,

DidDrawCheckLayer::Create(host_impl_->active_tree(), 7));

DidDrawCheckLayer* root =

static_cast<DidDrawCheckLayer*>(host_impl_->active_tree()->root_layer());

+ root->SetHasRenderSurface(true);

LayerTreeHostImpl::FrameData frame;

EXPECT_EQ(DRAW_SUCCESS, host_impl_->PrepareToDraw(&frame));

@@ -2225,6 +2212,7 @@ TEST_F(LayerTreeHostImplTest,

DidDrawCheckLayer::Create(host_impl_->active_tree(), 7));

DidDrawCheckLayer* root =

static_cast<DidDrawCheckLayer*>(host_impl_->active_tree()->root_layer());

+ root->SetHasRenderSurface(true);

LayerTreeHostImpl::FrameData frame;

EXPECT_EQ(DRAW_SUCCESS, host_impl_->PrepareToDraw(&frame));

@@ -2253,6 +2241,7 @@ TEST_F(LayerTreeHostImplTest,

TEST_F(LayerTreeHostImplTest, ScrollRootIgnored) {

scoped_ptr<LayerImpl> root = LayerImpl::Create(host_impl_->active_tree(), 1);

root->SetScrollClipLayer(Layer::INVALID_ID);

+ root->SetHasRenderSurface(true);

host_impl_->active_tree()->SetRootLayer(root.Pass());

DrawFrame();

@@ -2294,7 +2283,9 @@ class LayerTreeHostImplTopControlsTest : public LayerTreeHostImplTest {

root->SetIsContainerForFixedPositionLayers(true);

int inner_viewport_scroll_layer_id = root->id();

int page_scale_layer_id = root_clip->id();

+ root_clip->SetHasRenderSurface(true);

root_clip->AddChild(root.Pass());

+ root_clip->SetHasRenderSurface(true);

host_impl_->active_tree()->SetRootLayer(root_clip.Pass());

host_impl_->active_tree()->SetViewportLayersFromIds(

Layer::INVALID_ID, page_scale_layer_id, inner_viewport_scroll_layer_id,

@@ -2358,7 +2349,7 @@ class LayerTreeHostImplTopControlsTest : public LayerTreeHostImplTest {

root->SetPosition(gfx::PointF());

root->SetDrawsContent(false);

root->SetIsContainerForFixedPositionLayers(true);

+ root_clip->SetHasRenderSurface(true);

outer_clip->SetBounds(outer_viewport_size);

outer_scroll->SetScrollClipLayer(outer_clip->id());

outer_scroll->SetBounds(scroll_layer_size);

@@ -2884,6 +2875,7 @@ TEST_F(LayerTreeHostImplTest, ScrollNonCompositedRoot) {

scroll_layer->AddChild(content_layer.Pass());

scroll_clip_layer->AddChild(scroll_layer.Pass());

+ scroll_clip_layer->SetHasRenderSurface(true);

host_impl_->active_tree()->SetRootLayer(scroll_clip_layer.Pass());

host_impl_->SetViewportSize(surface_size);

DrawFrame();

@@ -2904,6 +2896,7 @@ TEST_F(LayerTreeHostImplTest, ScrollChildCallsCommitAndRedraw) {

root->SetBounds(surface_size);

root->SetContentBounds(contents_size);

root->AddChild(CreateScrollableLayer(2, contents_size, root.get()));

+ root->SetHasRenderSurface(true);

host_impl_->active_tree()->SetRootLayer(root.Pass());

host_impl_->SetViewportSize(surface_size);

DrawFrame();

@@ -2921,6 +2914,7 @@ TEST_F(LayerTreeHostImplTest, ScrollMissesChild) {

gfx::Size surface_size(10, 10);

scoped_ptr<LayerImpl> root = LayerImpl::Create(host_impl_->active_tree(), 1);

root->AddChild(CreateScrollableLayer(2, surface_size, root.get()));

+ root->SetHasRenderSurface(true);

host_impl_->active_tree()->SetRootLayer(root.Pass());

host_impl_->SetViewportSize(surface_size);

DrawFrame();

@@ -2937,6 +2931,7 @@ TEST_F(LayerTreeHostImplTest, ScrollMissesChild) {

TEST_F(LayerTreeHostImplTest, ScrollMissesBackfacingChild) {

gfx::Size surface_size(10, 10);

scoped_ptr<LayerImpl> root = LayerImpl::Create(host_impl_->active_tree(), 1);

+ root->SetHasRenderSurface(true);

scoped_ptr<LayerImpl> child =

CreateScrollableLayer(2, surface_size, root.get());

host_impl_->SetViewportSize(surface_size);

@@ -2974,6 +2969,7 @@ TEST_F(LayerTreeHostImplTest, ScrollBlockedByContentLayer) {

CreateScrollableLayer(2, surface_size, clip_layer.get());

scroll_layer->AddChild(content_layer.Pass());

clip_layer->AddChild(scroll_layer.Pass());

+ clip_layer->SetHasRenderSurface(true);

host_impl_->active_tree()->SetRootLayer(clip_layer.Pass());

host_impl_->SetViewportSize(surface_size);

@@ -2999,6 +2995,7 @@ TEST_F(LayerTreeHostImplTest, ScrollRootAndChangePageScaleOnMainThread) {

root_scrolling->SetIsContainerForFixedPositionLayers(true);

root_clip->AddChild(root_scrolling.Pass());

root->AddChild(root_clip.Pass());

+ root->SetHasRenderSurface(true);

host_impl_->active_tree()->SetRootLayer(root.Pass());

// The behaviour in this test assumes the page scale is applied at a layer

// above the clip layer.

@@ -3050,6 +3047,7 @@ TEST_F(LayerTreeHostImplTest, ScrollRootAndChangePageScaleOnImplThread) {

root_scrolling->SetIsContainerForFixedPositionLayers(true);

root_clip->AddChild(root_scrolling.Pass());

root->AddChild(root_clip.Pass());

+ root->SetHasRenderSurface(true);

host_impl_->active_tree()->SetRootLayer(root.Pass());

// The behaviour in this test assumes the page scale is applied at a layer

// above the clip layer.

@@ -3155,6 +3153,7 @@ TEST_F(LayerTreeHostImplTest, ScrollChildAndChangePageScaleOnMainThread) {

gfx::Size surface_size(30, 30);

scoped_ptr<LayerImpl> root = LayerImpl::Create(host_impl_->active_tree(), 1);

root->SetBounds(gfx::Size(5, 5));

+ root->SetHasRenderSurface(true);

scoped_ptr<LayerImpl> root_scrolling =

LayerImpl::Create(host_impl_->active_tree(), 2);

root_scrolling->SetBounds(surface_size);

@@ -3211,7 +3210,7 @@ TEST_F(LayerTreeHostImplTest, ScrollChildBeyondLimit) {

gfx::Size content_size(20, 20);

scoped_ptr<LayerImpl> root = LayerImpl::Create(host_impl_->active_tree(), 1);

root->SetBounds(surface_size);

+ root->SetHasRenderSurface(true);

scoped_ptr<LayerImpl> grand_child =

CreateScrollableLayer(3, content_size, root.get());

@@ -3256,6 +3255,7 @@ TEST_F(LayerTreeHostImplTest, ScrollWithoutBubbling) {

gfx::Size surface_size(20, 20);

gfx::Size viewport_size(10, 10);

scoped_ptr<LayerImpl> root = LayerImpl::Create(host_impl_->active_tree(), 1);

+ root->SetHasRenderSurface(true);

scoped_ptr<LayerImpl> root_scrolling =

CreateScrollableLayer(2, surface_size, root.get());

root_scrolling->SetIsContainerForFixedPositionLayers(true);

@@ -3365,6 +3365,7 @@ TEST_F(LayerTreeHostImplTest, ScrollEventBubbling) {

gfx::Size content_size(20, 20);

scoped_ptr<LayerImpl> root_clip =

LayerImpl::Create(host_impl_->active_tree(), 3);

+ root_clip->SetHasRenderSurface(true);

scoped_ptr<LayerImpl> root =

CreateScrollableLayer(1, content_size, root_clip.get());

// Make 'root' the clip layer for child: since they have the same sizes the

@@ -3408,6 +3409,7 @@ TEST_F(LayerTreeHostImplTest, ScrollBeforeRedraw) {

scoped_ptr<LayerImpl> root_scroll =

CreateScrollableLayer(2, surface_size, root_clip.get());

root_scroll->SetIsContainerForFixedPositionLayers(true);

+ root_clip->SetHasRenderSurface(true);

root_clip->AddChild(root_scroll.Pass());

host_impl_->active_tree()->SetRootLayer(root_clip.Pass());

host_impl_->active_tree()->SetViewportLayersFromIds(Layer::INVALID_ID, 1, 2,

@@ -3425,6 +3427,7 @@ TEST_F(LayerTreeHostImplTest, ScrollBeforeRedraw) {

CreateScrollableLayer(4, surface_size, root_clip2.get());

root_scroll2->SetIsContainerForFixedPositionLayers(true);

root_clip2->AddChild(root_scroll2.Pass());

+ root_clip2->SetHasRenderSurface(true);

host_impl_->active_tree()->SetRootLayer(root_clip2.Pass());

host_impl_->active_tree()->SetViewportLayersFromIds(Layer::INVALID_ID, 3, 4,

Layer::INVALID_ID);

@@ -3939,6 +3942,8 @@ TEST_F(LayerTreeHostImplTest, OverscrollChildWithoutBubbling) {

gfx::Size surface_size(10, 10);

scoped_ptr<LayerImpl> root_clip =

LayerImpl::Create(host_impl_->active_tree(), 4);

+ root_clip->SetHasRenderSurface(true);

scoped_ptr<LayerImpl> root =

CreateScrollableLayer(1, surface_size, root_clip.get());

@@ -4003,6 +4008,8 @@ TEST_F(LayerTreeHostImplTest, OverscrollChildEventBubbling) {

gfx::Size content_size(20, 20);

scoped_ptr<LayerImpl> root_clip =

LayerImpl::Create(host_impl_->active_tree(), 3);

+ root_clip->SetHasRenderSurface(true);

scoped_ptr<LayerImpl> root =

CreateScrollableLayer(1, content_size, root_clip.get());

root->SetIsContainerForFixedPositionLayers(true);

@@ -4059,6 +4066,8 @@ TEST_F(LayerTreeHostImplTest, NoOverscrollOnFractionalDeviceScale) {

float device_scale_factor = 1.5f;

scoped_ptr<LayerImpl> root_clip =

LayerImpl::Create(host_impl_->active_tree(), 3);

+ root_clip->SetHasRenderSurface(true);

scoped_ptr<LayerImpl> root =

CreateScrollableLayer(1, content_size, root_clip.get());

root->SetIsContainerForFixedPositionLayers(true);

@@ -4098,6 +4107,8 @@ TEST_F(LayerTreeHostImplTest, NoOverscrollWhenNotAtEdge) {

gfx::Size content_size(200, 200);

scoped_ptr<LayerImpl> root_clip =

LayerImpl::Create(host_impl_->active_tree(), 3);

+ root_clip->SetHasRenderSurface(true);

scoped_ptr<LayerImpl> root =

CreateScrollableLayer(1, content_size, root_clip.get());

root->SetIsContainerForFixedPositionLayers(true);

@@ -4243,6 +4254,7 @@ TEST_F(LayerTreeHostImplTest, BlendingOffWhenDrawingOpaqueLayers) {

root->SetBounds(gfx::Size(10, 10));

root->SetContentBounds(root->bounds());

root->SetDrawsContent(false);

+ root->SetHasRenderSurface(true);

host_impl_->active_tree()->SetRootLayer(root.Pass());

}

LayerImpl* root = host_impl_->active_tree()->root_layer();

@@ -4352,6 +4364,7 @@ TEST_F(LayerTreeHostImplTest, BlendingOffWhenDrawingOpaqueLayers) {

// carries the inherited opacity).

layer1->SetContentsOpaque(true);

layer1->SetOpacity(0.5f);

+ layer1->SetHasRenderSurface(true);

layer1->SetExpectation(false, true);

layer1->SetUpdateRect(gfx::Rect(layer1->content_bounds()));

layer2->SetExpectation(false, false);

@@ -4363,6 +4376,7 @@ TEST_F(LayerTreeHostImplTest, BlendingOffWhenDrawingOpaqueLayers) {

EXPECT_TRUE(layer1->quads_appended());

EXPECT_TRUE(layer2->quads_appended());

host_impl_->DidDrawAllLayers(frame);

+ layer1->SetHasRenderSurface(false);

// Draw again, but with child non-opaque, to make sure

// layer1 not culled.

@@ -4479,6 +4493,7 @@ class LayerTreeHostImplViewportCoveredTest : public LayerTreeHostImplTest {

host_impl_->active_tree()->set_background_color(SK_ColorGRAY);

host_impl_->active_tree()->SetRootLayer(

LayerImpl::Create(host_impl_->active_tree(), 1));

+ host_impl_->active_tree()->root_layer()->SetHasRenderSurface(true);

host_impl_->active_tree()->root_layer()->AddChild(

BlendStateCheckLayer::Create(host_impl_->active_tree(),

@@ -4769,6 +4784,7 @@ TEST_F(LayerTreeHostImplTest, ReshapeNotCalledUntilDraw) {

root->SetBounds(gfx::Size(10, 10));

root->SetContentBounds(gfx::Size(10, 10));

root->SetDrawsContent(true);

+ root->SetHasRenderSurface(true);

host_impl_->active_tree()->SetRootLayer(root.Pass());

EXPECT_FALSE(provider->TestContext3d()->reshape_called());

provider->TestContext3d()->clear_reshape_called();

@@ -4837,6 +4853,7 @@ TEST_F(LayerTreeHostImplTest, PartialSwapReceivesDamageRect) {

scoped_ptr<LayerImpl> root =

FakeDrawableLayerImpl::Create(layer_tree_host_impl->active_tree(), 1);

+ root->SetHasRenderSurface(true);

scoped_ptr<LayerImpl> child =

FakeDrawableLayerImpl::Create(layer_tree_host_impl->active_tree(), 2);

child->SetPosition(gfx::PointF(12.f, 13.f));

@@ -4903,7 +4920,7 @@ TEST_F(LayerTreeHostImplTest, RootLayerDoesntCreateExtraSurface) {

root->SetBounds(gfx::Size(10, 10));

root->SetContentBounds(gfx::Size(10, 10));

root->SetDrawsContent(true);

- root->SetForceRenderSurface(true);

+ root->SetHasRenderSurface(true);

root->AddChild(child.Pass());

host_impl_->active_tree()->SetRootLayer(root.Pass());

@@ -5152,7 +5169,7 @@ static scoped_ptr<LayerTreeHostImpl> SetupLayersForOpacity(

gfx::Rect child_rect(10, 10, 50, 50);

gfx::Rect grand_child_rect(5, 5, 150, 150);

- root->CreateRenderSurface();

+ root->SetHasRenderSurface(true);

root->SetPosition(root_rect.origin());

root->SetBounds(root_rect.size());

root->SetContentBounds(root->bounds());

@@ -5166,7 +5183,7 @@ static scoped_ptr<LayerTreeHostImpl> SetupLayersForOpacity(

child->SetContentBounds(child->bounds());

child->draw_properties().visible_content_rect = child_rect;

child->SetDrawsContent(false);

- child->SetForceRenderSurface(true);

+ child->SetHasRenderSurface(true);

grand_child->SetPosition(grand_child_rect.origin());

grand_child->SetBounds(grand_child_rect.size());

@@ -5246,6 +5263,7 @@ TEST_F(LayerTreeHostImplTest, LayersFreeTextures) {

scoped_ptr<LayerImpl> root_layer =

LayerImpl::Create(host_impl_->active_tree(), 1);

root_layer->SetBounds(gfx::Size(10, 10));

+ root_layer->SetHasRenderSurface(true);

scoped_refptr<VideoFrame> softwareFrame =

media::VideoFrame::CreateColorFrame(

@@ -5420,6 +5438,7 @@ TEST_F(LayerTreeHostImplTestWithDelegatingRenderer, FrameIncludesDamageRect) {

root->SetBounds(gfx::Size(10, 10));

root->SetContentBounds(gfx::Size(10, 10));

root->SetDrawsContent(true);

+ root->SetHasRenderSurface(true);

// Child layer is in the bottom right corner.

scoped_ptr<SolidColorLayerImpl> child =

@@ -5486,6 +5505,7 @@ TEST_F(LayerTreeHostImplTest, MaskLayerWithScaling) {

scoped_ptr<LayerImpl> scoped_root =

LayerImpl::Create(host_impl_->active_tree(), 1);

LayerImpl* root = scoped_root.get();

+ root->SetHasRenderSurface(true);

host_impl_->active_tree()->SetRootLayer(scoped_root.Pass());

scoped_ptr<LayerImpl> scoped_scaling_layer =

@@ -5501,6 +5521,7 @@ TEST_F(LayerTreeHostImplTest, MaskLayerWithScaling) {

scoped_ptr<FakeMaskLayerImpl> scoped_mask_layer =

FakeMaskLayerImpl::Create(host_impl_->active_tree(), 4);

FakeMaskLayerImpl* mask_layer = scoped_mask_layer.get();

+ content_layer->SetHasRenderSurface(true);

content_layer->SetMaskLayer(scoped_mask_layer.Pass());

gfx::Size root_size(100, 100);

@@ -5626,6 +5647,8 @@ TEST_F(LayerTreeHostImplTest, MaskLayerWithDifferentBounds) {

scoped_ptr<LayerImpl> scoped_root =

LayerImpl::Create(host_impl_->active_tree(), 1);

LayerImpl* root = scoped_root.get();

+ root->SetHasRenderSurface(true);

host_impl_->active_tree()->SetRootLayer(scoped_root.Pass());

scoped_ptr<LayerImpl> scoped_content_layer =

@@ -5637,6 +5660,7 @@ TEST_F(LayerTreeHostImplTest, MaskLayerWithDifferentBounds) {

FakeMaskLayerImpl::Create(host_impl_->active_tree(), 4);

FakeMaskLayerImpl* mask_layer = scoped_mask_layer.get();

content_layer->SetMaskLayer(scoped_mask_layer.Pass());

+ content_layer->SetHasRenderSurface(true);

gfx::Size root_size(100, 100);

root->SetBounds(root_size);

@@ -5780,6 +5804,8 @@ TEST_F(LayerTreeHostImplTest, ReflectionMaskLayerWithDifferentBounds) {

scoped_ptr<LayerImpl> scoped_root =

LayerImpl::Create(host_impl_->active_tree(), 1);

LayerImpl* root = scoped_root.get();

+ root->SetHasRenderSurface(true);

host_impl_->active_tree()->SetRootLayer(scoped_root.Pass());

scoped_ptr<LayerImpl> scoped_content_layer =

@@ -5791,11 +5817,13 @@ TEST_F(LayerTreeHostImplTest, ReflectionMaskLayerWithDifferentBounds) {

LayerImpl::Create(host_impl_->active_tree(), 2);

LayerImpl* replica_layer = scoped_replica_layer.get();

content_layer->SetReplicaLayer(scoped_replica_layer.Pass());

+ content_layer->SetHasRenderSurface(true);

scoped_ptr<FakeMaskLayerImpl> scoped_mask_layer =

FakeMaskLayerImpl::Create(host_impl_->active_tree(), 4);

FakeMaskLayerImpl* mask_layer = scoped_mask_layer.get();

replica_layer->SetMaskLayer(scoped_mask_layer.Pass());

+ replica_layer->SetHasRenderSurface(true);

gfx::Size root_size(100, 100);

root->SetBounds(root_size);

@@ -5935,6 +5963,7 @@ TEST_F(LayerTreeHostImplTest, ReflectionMaskLayerForSurfaceWithUnclippedChild) {

scoped_ptr<LayerImpl> scoped_root =

LayerImpl::Create(host_impl_->active_tree(), 1);

LayerImpl* root = scoped_root.get();

+ root->SetHasRenderSurface(true);

host_impl_->active_tree()->SetRootLayer(scoped_root.Pass());

scoped_ptr<LayerImpl> scoped_content_layer =

@@ -5951,11 +5980,13 @@ TEST_F(LayerTreeHostImplTest, ReflectionMaskLayerForSurfaceWithUnclippedChild) {

LayerImpl::Create(host_impl_->active_tree(), 4);

LayerImpl* replica_layer = scoped_replica_layer.get();

content_layer->SetReplicaLayer(scoped_replica_layer.Pass());

+ content_layer->SetHasRenderSurface(true);

scoped_ptr<FakeMaskLayerImpl> scoped_mask_layer =

FakeMaskLayerImpl::Create(host_impl_->active_tree(), 5);

FakeMaskLayerImpl* mask_layer = scoped_mask_layer.get();

replica_layer->SetMaskLayer(scoped_mask_layer.Pass());

+ replica_layer->SetHasRenderSurface(true);

gfx::Size root_size(100, 100);

root->SetBounds(root_size);

@@ -6059,6 +6090,8 @@ TEST_F(LayerTreeHostImplTest, MaskLayerForSurfaceWithClippedLayer) {

scoped_ptr<LayerImpl> scoped_root =

LayerImpl::Create(host_impl_->active_tree(), 1);

LayerImpl* root = scoped_root.get();

+ root->SetHasRenderSurface(true);

host_impl_->active_tree()->SetRootLayer(scoped_root.Pass());

scoped_ptr<LayerImpl> scoped_clipping_layer =

@@ -6080,6 +6113,7 @@ TEST_F(LayerTreeHostImplTest, MaskLayerForSurfaceWithClippedLayer) {

FakeMaskLayerImpl::Create(host_impl_->active_tree(), 6);

FakeMaskLayerImpl* mask_layer = scoped_mask_layer.get();

content_layer->SetMaskLayer(scoped_mask_layer.Pass());

+ content_layer->SetHasRenderSurface(true);

gfx::Size root_size(100, 100);

root->SetBounds(root_size);

@@ -6165,6 +6199,7 @@ TEST_F(LayerTreeHostImplTest, FarAwayQuadsDontNeedAA) {

scoped_ptr<LayerImpl> scoped_root =

LayerImpl::Create(host_impl_->pending_tree(), 1);

LayerImpl* root = scoped_root.get();

+ root->SetHasRenderSurface(true);

host_impl_->pending_tree()->SetRootLayer(scoped_root.Pass());

@@ -6666,6 +6701,8 @@ TEST_F(LayerTreeHostImplTest, TouchFlingShouldNotBubble) {

gfx::Size content_size(20, 20);

scoped_ptr<LayerImpl> root_clip =

LayerImpl::Create(host_impl_->active_tree(), 3);

+ root_clip->SetHasRenderSurface(true);

scoped_ptr<LayerImpl> root =

CreateScrollableLayer(1, content_size, root_clip.get());

root->SetIsContainerForFixedPositionLayers(true);

@@ -6710,6 +6747,7 @@ TEST_F(LayerTreeHostImplTest, TouchFlingShouldLockToFirstScrolledLayer) {

// the scroll doesn't bubble up to the parent layer.

gfx::Size surface_size(10, 10);

scoped_ptr<LayerImpl> root = LayerImpl::Create(host_impl_->active_tree(), 1);

+ root->SetHasRenderSurface(true);

scoped_ptr<LayerImpl> root_scrolling =

CreateScrollableLayer(2, surface_size, root.get());

@@ -6780,6 +6818,7 @@ TEST_F(LayerTreeHostImplTest, WheelFlingShouldBubble) {

gfx::Size content_size(20, 20);

scoped_ptr<LayerImpl> root_clip =

LayerImpl::Create(host_impl_->active_tree(), 3);

+ root_clip->SetHasRenderSurface(true);

scoped_ptr<LayerImpl> root_scroll =

CreateScrollableLayer(1, content_size, root_clip.get());

int root_scroll_id = root_scroll->id();

@@ -6983,6 +7022,7 @@ TEST_F(LayerTreeHostImplTest, LatencyInfoPassedToCompositorFrameMetadata) {

root->SetBounds(gfx::Size(10, 10));

root->SetContentBounds(gfx::Size(10, 10));

root->SetDrawsContent(true);

+ root->SetHasRenderSurface(true);

host_impl_->active_tree()->SetRootLayer(root.Pass());

@@ -7023,6 +7063,7 @@ TEST_F(LayerTreeHostImplTest, SelectionBoundsPassedToCompositorFrameMetadata) {

root->SetBounds(gfx::Size(10, 10));

root->SetContentBounds(gfx::Size(10, 10));

root->SetDrawsContent(true);

+ root->SetHasRenderSurface(true);

host_impl_->active_tree()->SetRootLayer(root.Pass());

@@ -7520,6 +7561,7 @@ class LayerTreeHostImplVirtualViewportTest : public LayerTreeHostImplTest {

page_scale->AddChild(inner_scroll.Pass());

inner_clip->AddChild(page_scale.Pass());

+ inner_clip->SetHasRenderSurface(true);

layer_tree_impl->SetRootLayer(inner_clip.Pass());

layer_tree_impl->SetViewportLayersFromIds(

Layer::INVALID_ID, kPageScaleLayerId, kInnerViewportScrollLayerId,

« cc/trees/layer_tree_host_common.cc ('K') | « cc/trees/layer_tree_host_common_unittest.cc ('k') | cc/trees/layer_tree_host_unittest.cc » ('j') | no next file with comments »