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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: cc/layers/picture_layer_impl_unittest.cc

Issue 1418273002: cc: Move draw params from SetExternalDrawConstraints to OnDraw (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 5 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
« no previous file with comments | « cc/layers/picture_layer_impl.cc ('k') | cc/output/output_surface.h » ('j') | cc/trees/layer_tree_host_impl.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/layers/picture_layer_impl_unittest.cc
diff --git a/cc/layers/picture_layer_impl_unittest.cc b/cc/layers/picture_layer_impl_unittest.cc
index 1839690cfc53247fb021559f844bc2ea6b325e02..616d2928f41e54e0cf8ca8130f975d5eab7dfab6 100644
--- a/cc/layers/picture_layer_impl_unittest.cc
+++ b/cc/layers/picture_layer_impl_unittest.cc
@@ -490,17 +490,11 @@ TEST_F(PictureLayerImplTest, ExternalViewportRectForPrioritizingTiles) {
// Update tiles with viewport for tile priority as (0, 0, 100, 100) and the
// identify transform for tile priority.
- bool resourceless_software_draw = false;
- gfx::Rect viewport = gfx::Rect(layer_bounds),
- viewport_rect_for_tile_priority = gfx::Rect(0, 0, 100, 100);
+ gfx::Rect viewport_rect_for_tile_priority = gfx::Rect(0, 0, 100, 100);
gfx::Transform transform, transform_for_tile_priority;
- host_impl_.SetExternalDrawConstraints(transform,
- viewport,
- viewport,
- viewport_rect_for_tile_priority,
- transform_for_tile_priority,
- resourceless_software_draw);
+ host_impl_.SetExternalTilePriorityConstraints(viewport_rect_for_tile_priority,
+ transform_for_tile_priority);
bool update_lcd_text = false;
host_impl_.active_tree()->UpdateDrawProperties(update_lcd_text);
@@ -528,12 +522,8 @@ TEST_F(PictureLayerImplTest, ExternalViewportRectForPrioritizingTiles) {
viewport_rect_for_tile_priority = gfx::Rect(200, 200, 100, 100);
transform_for_tile_priority.Translate(100, 100);
transform_for_tile_priority.Rotate(45);
- host_impl_.SetExternalDrawConstraints(transform,
- viewport,
- viewport,
- viewport_rect_for_tile_priority,
- transform_for_tile_priority,
- resourceless_software_draw);
+ host_impl_.SetExternalTilePriorityConstraints(viewport_rect_for_tile_priority,
+ transform_for_tile_priority);
host_impl_.active_tree()->UpdateDrawProperties(update_lcd_text);
gfx::Transform screen_to_view(gfx::Transform::kSkipInitialization);
@@ -574,12 +564,7 @@ TEST_F(PictureLayerImplTest, InvalidViewportForPrioritizingTiles) {
bool resourceless_software_draw = false;
gfx::Rect viewport = gfx::Rect(layer_bounds);
gfx::Transform transform;
- host_impl_.SetExternalDrawConstraints(transform,
- viewport,
- viewport,
- viewport,
- transform,
- resourceless_software_draw);
+ host_impl_.SetExternalTilePriorityConstraints(viewport, transform);
active_layer_->draw_properties().visible_layer_rect = viewport;
active_layer_->draw_properties().screen_space_transform = transform;
active_layer_->UpdateTiles(resourceless_software_draw);
@@ -598,12 +583,7 @@ TEST_F(PictureLayerImplTest, InvalidViewportForPrioritizingTiles) {
transform.Translate(1.f, 1.f);
active_layer_->draw_properties().visible_layer_rect = viewport;
active_layer_->draw_properties().screen_space_transform = transform;
- host_impl_.SetExternalDrawConstraints(transform,
- viewport,
- viewport,
- viewport,
- transform,
- resourceless_software_draw);
+ host_impl_.SetExternalTilePriorityConstraints(viewport, transform);
active_layer_->UpdateTiles(resourceless_software_draw);
// Transform for tile priority is updated.
@@ -616,12 +596,7 @@ TEST_F(PictureLayerImplTest, InvalidViewportForPrioritizingTiles) {
// Keep expanded viewport but mark it valid. Should update tile viewport.
host_impl_.AdvanceToNextFrame(base::TimeDelta::FromMilliseconds(200));
resourceless_software_draw = false;
- host_impl_.SetExternalDrawConstraints(transform,
- viewport,
- viewport,
- viewport,
- transform,
- resourceless_software_draw);
+ host_impl_.SetExternalTilePriorityConstraints(viewport, transform);
active_layer_->UpdateTiles(resourceless_software_draw);
EXPECT_TRANSFORMATION_MATRIX_EQ(
@@ -639,14 +614,11 @@ TEST_F(PictureLayerImplTest, ViewportRectForTilePriorityIsCached) {
host_impl_.AdvanceToNextFrame(base::TimeDelta::FromMilliseconds(200));
- bool resourceless_software_draw = false;
- gfx::Rect viewport = gfx::Rect(layer_bounds);
gfx::Rect viewport_rect_for_tile_priority(0, 0, 100, 100);
- gfx::Transform transform, transform_for_tile_priority;
+ gfx::Transform transform_for_tile_priority;
- host_impl_.SetExternalDrawConstraints(
- transform, viewport, viewport, viewport_rect_for_tile_priority,
- transform_for_tile_priority, resourceless_software_draw);
+ host_impl_.SetExternalTilePriorityConstraints(viewport_rect_for_tile_priority,
+ transform_for_tile_priority);
bool update_lcd_text = false;
host_impl_.active_tree()->UpdateDrawProperties(update_lcd_text);
@@ -656,9 +628,8 @@ TEST_F(PictureLayerImplTest, ViewportRectForTilePriorityIsCached) {
host_impl_.AdvanceToNextFrame(base::TimeDelta::FromMilliseconds(200));
gfx::Rect another_viewport_rect_for_tile_priority(11, 11, 50, 50);
- host_impl_.SetExternalDrawConstraints(
- transform, viewport, viewport, another_viewport_rect_for_tile_priority,
- transform_for_tile_priority, resourceless_software_draw);
+ host_impl_.SetExternalTilePriorityConstraints(
+ another_viewport_rect_for_tile_priority, transform_for_tile_priority);
// Didn't call UpdateDrawProperties yet. The viewport rect for tile priority
// should remain to be the previously cached value.
@@ -1815,15 +1786,10 @@ TEST_F(NoLowResPictureLayerImplTest, MarkRequiredOffscreenTiles) {
gfx::Size layer_bounds(200, 200);
gfx::Transform transform;
- gfx::Transform transform_for_tile_priority;
bool resourceless_software_draw = false;
gfx::Rect viewport(0, 0, 100, 200);
- host_impl_.SetExternalDrawConstraints(transform,
- viewport,
- viewport,
- viewport,
- transform,
- resourceless_software_draw);
+ host_impl_.SetExternalTilePriorityConstraints(viewport, transform);
+ host_impl_.SetExternalViewportForTesting(viewport);
scoped_refptr<FakeDisplayListRasterSource> pending_raster_source =
FakeDisplayListRasterSource::CreateFilled(layer_bounds);
@@ -1877,12 +1843,8 @@ TEST_F(NoLowResPictureLayerImplTest,
gfx::Transform transform;
gfx::Transform transform_for_tile_priority;
bool resourceless_software_draw = false;
- host_impl_.SetExternalDrawConstraints(transform,
- viewport,
- viewport,
- external_viewport_for_tile_priority,
- transform_for_tile_priority,
- resourceless_software_draw);
+ host_impl_.SetExternalTilePriorityConstraints(
+ external_viewport_for_tile_priority, transform_for_tile_priority);
host_impl_.AdvanceToNextFrame(base::TimeDelta::FromMilliseconds(1));
bool update_lcd_text = false;
host_impl_.pending_tree()->UpdateDrawProperties(update_lcd_text);
@@ -3626,12 +3588,7 @@ TEST_F(NoLowResPictureLayerImplTest, InvalidViewportForPrioritizingTiles) {
bool resourceless_software_draw = false;
gfx::Rect viewport = gfx::Rect(layer_bounds);
gfx::Transform transform;
- host_impl_.SetExternalDrawConstraints(transform,
- viewport,
- viewport,
- viewport,
- transform,
- resourceless_software_draw);
+ host_impl_.SetExternalTilePriorityConstraints(viewport, transform);
active_layer_->draw_properties().visible_layer_rect = viewport;
active_layer_->draw_properties().screen_space_transform = transform;
active_layer_->UpdateTiles(resourceless_software_draw);
@@ -3650,12 +3607,7 @@ TEST_F(NoLowResPictureLayerImplTest, InvalidViewportForPrioritizingTiles) {
transform.Translate(1.f, 1.f);
active_layer_->draw_properties().visible_layer_rect = viewport;
active_layer_->draw_properties().screen_space_transform = transform;
- host_impl_.SetExternalDrawConstraints(transform,
- viewport,
- viewport,
- viewport,
- transform,
- resourceless_software_draw);
+ host_impl_.SetExternalTilePriorityConstraints(viewport, transform);
active_layer_->UpdateTiles(resourceless_software_draw);
// Transform for tile priority is updated.
@@ -3668,12 +3620,7 @@ TEST_F(NoLowResPictureLayerImplTest, InvalidViewportForPrioritizingTiles) {
// Keep expanded viewport but mark it valid. Should update tile viewport.
host_impl_.AdvanceToNextFrame(base::TimeDelta::FromMilliseconds(200));
resourceless_software_draw = false;
- host_impl_.SetExternalDrawConstraints(transform,
- viewport,
- viewport,
- viewport,
- transform,
- resourceless_software_draw);
+ host_impl_.SetExternalTilePriorityConstraints(viewport, transform);
active_layer_->UpdateTiles(resourceless_software_draw);
EXPECT_TRANSFORMATION_MATRIX_EQ(
@@ -4789,13 +4736,8 @@ TEST_F(PictureLayerImplTest, ScrollPastLiveTilesRectAndBack) {
ActivateTree();
EXPECT_TRUE(active_layer_->HighResTiling()->has_tiles());
- host_impl_.SetExternalDrawConstraints(
- gfx::Transform(), // transform
- gfx::Rect(), // clip
- gfx::Rect(), // viewport
- gfx::Rect(0, 5000, 100, 100), // viewport_rect_for_tile_priority
- gfx::Transform(), // transform_for_tile_priority
- false);
+ host_impl_.SetExternalTilePriorityConstraints(gfx::Rect(0, 5000, 100, 100),
+ gfx::Transform());
SetupPendingTreeWithFixedTileSize(pending_raster_source, tile_size,
gfx::Rect());
@@ -4806,13 +4748,8 @@ TEST_F(PictureLayerImplTest, ScrollPastLiveTilesRectAndBack) {
EXPECT_FALSE(active_layer_->HighResTiling()->has_tiles());
EXPECT_TRUE(active_layer_->HighResTiling()->live_tiles_rect().IsEmpty());
- host_impl_.SetExternalDrawConstraints(
- gfx::Transform(), // transform
- gfx::Rect(), // clip
- gfx::Rect(), // viewport
- gfx::Rect(0, 110, 100, 100), // viewport_rect_for_tile_priority
- gfx::Transform(), // transform_for_tile_priority
- false);
+ host_impl_.SetExternalTilePriorityConstraints(gfx::Rect(0, 110, 100, 100),
+ gfx::Transform());
SetupPendingTreeWithFixedTileSize(pending_raster_source, tile_size,
gfx::Rect());
« no previous file with comments | « cc/layers/picture_layer_impl.cc ('k') | cc/output/output_surface.h » ('j') | cc/trees/layer_tree_host_impl.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698