Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3554)

Unified Diff: cc/trees/occlusion_tracker_unittest.cc

Issue 1912893002: cc : Stop pushing properties not used by LayerImpl to LayerImpl. (2) (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 8 months 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/occlusion_tracker_unittest.cc
diff --git a/cc/trees/occlusion_tracker_unittest.cc b/cc/trees/occlusion_tracker_unittest.cc
index 35c29f7db03f25891dfbd4023a855138f33bb5c1..c322a462a7ee22cb5359c1c8ac4af1455028f88a 100644
--- a/cc/trees/occlusion_tracker_unittest.cc
+++ b/cc/trees/occlusion_tracker_unittest.cc
@@ -108,7 +108,7 @@ class OcclusionTrackerTest : public testing::Test {
host_->host_impl()->active_tree()->SetRootLayer(std::move(layer));
- layer_ptr->SetForceRenderSurface(true);
+ layer_ptr->test_properties()->force_render_surface = true;
SetRootLayerOnMainThread(layer_ptr);
return layer_ptr;
@@ -132,7 +132,7 @@ class OcclusionTrackerTest : public testing::Test {
const gfx::PointF& position,
const gfx::Size& bounds) {
LayerImpl* layer = CreateLayer(parent, transform, position, bounds);
- layer->SetForceRenderSurface(true);
+ layer->test_properties()->force_render_surface = true;
return layer;
}
@@ -194,7 +194,7 @@ class OcclusionTrackerTest : public testing::Test {
bool opaque) {
TestContentLayerImpl* layer =
CreateDrawingLayer(parent, transform, position, bounds, opaque);
- layer->SetForceRenderSurface(true);
+ layer->test_properties()->force_render_surface = true;
return layer;
}
@@ -541,7 +541,7 @@ class OcclusionTrackerTestScaledRenderSurface : public OcclusionTrackerTest {
layer1_matrix.Scale(2.0, 2.0);
TestContentLayerImpl* layer1 = this->CreateDrawingLayer(
parent, layer1_matrix, gfx::PointF(), gfx::Size(100, 100), true);
- layer1->SetForceRenderSurface(true);
+ layer1->test_properties()->force_render_surface = true;
gfx::Transform layer2_matrix;
layer2_matrix.Translate(25.0, 25.0);
@@ -922,17 +922,17 @@ class OcclusionTrackerTestFilters : public OcclusionTrackerTest {
parent, layer_transform, gfx::PointF(30.f, 30.f), gfx::Size(500, 500),
true);
- blur_layer->SetForceRenderSurface(true);
+ blur_layer->test_properties()->force_render_surface = true;
FilterOperations filters;
filters.Append(FilterOperation::CreateBlurFilter(10.f));
blur_layer->SetFilters(filters);
- opaque_layer->SetForceRenderSurface(true);
+ opaque_layer->test_properties()->force_render_surface = true;
filters.Clear();
filters.Append(FilterOperation::CreateGrayscaleFilter(0.5f));
opaque_layer->SetFilters(filters);
- opacity_layer->SetForceRenderSurface(true);
+ opacity_layer->test_properties()->force_render_surface = true;
filters.Clear();
filters.Append(FilterOperation::CreateOpacityFilter(0.5f));
opacity_layer->SetFilters(filters);
@@ -1484,7 +1484,7 @@ class OcclusionTrackerTestDontOccludePixelsNeededForBackgroundFilter
LayerImpl* occluding_layer = this->CreateDrawingLayer(
parent, this->identity_matrix, gfx::PointF(occlusion_rect.origin()),
occlusion_rect.size(), true);
- occluding_layer->SetForceRenderSurface(false);
+ occluding_layer->test_properties()->force_render_surface = false;
this->CalcDrawEtc(parent);
TestOcclusionTrackerWithClip occlusion(gfx::Rect(0, 0, 200, 200));
@@ -1568,8 +1568,8 @@ class OcclusionTrackerTestTwoBackgroundFiltersReduceOcclusionTwice
gfx::Size(50, 50), true);
// Filters make the layers own surfaces.
- filtered_surface1->SetForceRenderSurface(true);
- filtered_surface2->SetForceRenderSurface(true);
+ filtered_surface1->test_properties()->force_render_surface = true;
+ filtered_surface2->test_properties()->force_render_surface = true;
FilterOperations filters;
filters.Append(FilterOperation::CreateBlurFilter(1.f));
filtered_surface1->SetBackgroundFilters(filters);
@@ -1645,7 +1645,7 @@ class OcclusionTrackerTestDontReduceOcclusionBelowBackgroundFilter
gfx::Size());
// Filters make the layer own a surface.
- filtered_surface->SetForceRenderSurface(true);
+ filtered_surface->test_properties()->force_render_surface = true;
FilterOperations filters;
filters.Append(FilterOperation::CreateBlurFilter(3.f));
filtered_surface->SetBackgroundFilters(filters);
@@ -1712,7 +1712,7 @@ class OcclusionTrackerTestDontReduceOcclusionIfBackgroundFilterIsOccluded
gfx::Size(50, 50), true);
// Filters make the layer own a surface.
- filtered_surface->SetForceRenderSurface(true);
+ filtered_surface->test_properties()->force_render_surface = true;
FilterOperations filters;
filters.Append(FilterOperation::CreateBlurFilter(3.f));
filtered_surface->SetBackgroundFilters(filters);
@@ -1791,7 +1791,7 @@ class OcclusionTrackerTestReduceOcclusionWhenBkgdFilterIsPartiallyOccluded
gfx::Size(10, 10), true);
// Filters make the layer own a surface.
- filtered_surface->SetForceRenderSurface(true);
+ filtered_surface->test_properties()->force_render_surface = true;
FilterOperations filters;
filters.Append(FilterOperation::CreateBlurFilter(3.f));
filtered_surface->SetBackgroundFilters(filters);
@@ -1866,7 +1866,7 @@ class OcclusionTrackerTestBlendModeDoesNotOcclude
gfx::Size(20, 22), true);
// Blend mode makes the layer own a surface.
- blend_mode_layer->SetForceRenderSurface(true);
+ blend_mode_layer->test_properties()->force_render_surface = true;
blend_mode_layer->SetBlendMode(SkXfermode::kMultiply_Mode);
this->CalcDrawEtc(parent);

Powered by Google App Engine
This is Rietveld 408576698