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

Unified Diff: cc/trees/layer_tree_host_common_unittest.cc

Issue 2408243002: cc : Move screen space scale factor to root transform node (Closed)
Patch Set: comments Created 4 years, 2 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
« no previous file with comments | « cc/trees/layer_tree_host_common.cc ('k') | cc/trees/layer_tree_impl.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 050c97edf95ba3365e0e70069807d55773d72b7a..f9aa093080b79188d2f86a6fa21f28a9ab05f6b2 100644
--- a/cc/trees/layer_tree_host_common_unittest.cc
+++ b/cc/trees/layer_tree_host_common_unittest.cc
@@ -1292,7 +1292,8 @@ TEST_F(LayerTreeHostCommonTest, TransformAboveRootLayer) {
inputs.property_trees->needs_rebuild = true;
LayerTreeHostCommon::CalculateDrawPropertiesForTesting(&inputs);
gfx::Transform device_scaled_translate = translate;
- device_scaled_translate.Scale(device_scale_factor, device_scale_factor);
+ device_scaled_translate.matrix().postScale(device_scale_factor,
+ device_scale_factor, 1.f);
EXPECT_TRANSFORMATION_MATRIX_EQ(
device_scaled_translate,
root->draw_properties().target_space_transform);
@@ -1315,7 +1316,8 @@ TEST_F(LayerTreeHostCommonTest, TransformAboveRootLayer) {
inputs.property_trees->needs_rebuild = true;
LayerTreeHostCommon::CalculateDrawPropertiesForTesting(&inputs);
gfx::Transform page_scaled_translate = translate;
- page_scaled_translate.Scale(page_scale_factor, page_scale_factor);
+ page_scaled_translate.matrix().postScale(page_scale_factor,
+ page_scale_factor, 1.f);
EXPECT_TRANSFORMATION_MATRIX_EQ(
page_scaled_translate, root->draw_properties().target_space_transform);
EXPECT_TRANSFORMATION_MATRIX_EQ(
@@ -9650,6 +9652,39 @@ TEST_F(LayerTreeHostCommonTest, LayerWithInputHandlerAndZeroOpacity) {
test_layer->ScreenSpaceTransform());
}
+TEST_F(LayerTreeHostCommonTest, ClipParentDrawsIntoScaledRootSurface) {
+ LayerImpl* root = root_layer_for_testing();
+ LayerImpl* clip_layer = AddChild<LayerImpl>(root);
+ LayerImpl* clip_parent = AddChild<LayerImpl>(clip_layer);
+ LayerImpl* unclipped_desc_surface = AddChild<LayerImpl>(clip_parent);
+ LayerImpl* clip_child = AddChild<LayerImpl>(unclipped_desc_surface);
+
+ root->SetBounds(gfx::Size(100, 100));
+ clip_layer->SetBounds(gfx::Size(20, 20));
+ clip_layer->SetMasksToBounds(true);
+ clip_parent->SetBounds(gfx::Size(50, 50));
+ unclipped_desc_surface->SetBounds(gfx::Size(100, 100));
+ unclipped_desc_surface->SetDrawsContent(true);
+ unclipped_desc_surface->test_properties()->force_render_surface = true;
+ clip_child->SetBounds(gfx::Size(100, 100));
+ clip_child->SetDrawsContent(true);
+
+ clip_child->test_properties()->clip_parent = clip_parent;
+ clip_parent->test_properties()->clip_children =
+ base::MakeUnique<std::set<LayerImpl*>>();
+ clip_parent->test_properties()->clip_children->insert(clip_child);
+
+ float device_scale_factor = 1.f;
+ ExecuteCalculateDrawProperties(root, device_scale_factor);
+ EXPECT_EQ(gfx::Rect(20, 20), clip_child->clip_rect());
+ EXPECT_EQ(gfx::Rect(20, 20), clip_child->visible_layer_rect());
+
+ device_scale_factor = 2.f;
+ ExecuteCalculateDrawProperties(root, device_scale_factor);
+ EXPECT_EQ(gfx::Rect(40, 40), clip_child->clip_rect());
+ EXPECT_EQ(gfx::Rect(20, 20), clip_child->visible_layer_rect());
+}
+
TEST_F(LayerTreeHostCommonTest, ClipChildVisibleRect) {
LayerImpl* root = root_layer_for_testing();
LayerImpl* clip_parent = AddChildToRoot<LayerImpl>();
« no previous file with comments | « cc/trees/layer_tree_host_common.cc ('k') | cc/trees/layer_tree_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698