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

Unified Diff: cc/layers/picture_layer_impl_unittest.cc

Issue 2946223003: cc: Add UMA to measure the raster duration for pending tree activation. (Closed)
Patch Set: tests Created 3 years, 6 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 | « no previous file | cc/trees/layer_tree_host_impl.h » ('j') | no next file with comments »
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 0d7ac4d0e9b1fd88095ed86439b38cc0476577ff..7ca0ff85f7ac26ecd506e2be11c5ed675e614114 100644
--- a/cc/layers/picture_layer_impl_unittest.cc
+++ b/cc/layers/picture_layer_impl_unittest.cc
@@ -2339,6 +2339,7 @@ TEST_F(PictureLayerImplTest, SyncTilingAfterGpuRasterizationToggles) {
EXPECT_TRUE(pending_layer()->tilings()->FindTilingWithScaleKey(1.f));
EXPECT_EQ(0u, active_layer()->tilings()->num_tilings());
+ host_impl()->NotifyReadyToActivate();
ActivateTree();
EXPECT_TRUE(active_layer()->tilings()->FindTilingWithScaleKey(1.f));
@@ -2355,6 +2356,7 @@ TEST_F(PictureLayerImplTest, SyncTilingAfterGpuRasterizationToggles) {
EXPECT_EQ(2u, active_layer()->release_tile_resources_count());
EXPECT_EQ(2u, pending_layer()->release_resources_count());
EXPECT_EQ(2u, active_layer()->release_resources_count());
+ host_impl()->NotifyReadyToActivate();
host_impl()->SetHasGpuRasterizationTrigger(true);
host_impl()->SetContentIsSuitableForGpuRasterization(false);
@@ -4738,6 +4740,7 @@ TEST_F(TileSizeTest, TileSizes) {
host_impl()->CommitComplete();
EXPECT_EQ(host_impl()->gpu_rasterization_status(),
GpuRasterizationStatus::OFF_VIEWPORT);
+ host_impl()->NotifyReadyToActivate();
// Default tile-size for large layers.
result = layer->CalculateTileSize(gfx::Size(10000, 10000));
@@ -4761,6 +4764,7 @@ TEST_F(TileSizeTest, TileSizes) {
EXPECT_EQ(host_impl()->gpu_rasterization_status(),
GpuRasterizationStatus::ON);
host_impl()->SetViewportSize(gfx::Size(2000, 2000));
+ host_impl()->NotifyReadyToActivate();
layer->set_gpu_raster_max_texture_size(host_impl()->device_viewport_size());
result = layer->CalculateTileSize(gfx::Size(10000, 10000));
« no previous file with comments | « no previous file | cc/trees/layer_tree_host_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698