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

Unified Diff: cc/resources/picture_layer_tiling_perftest.cc

Issue 775483002: cc: Remove max tiles and skewport constants from tiling client. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: 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/resources/picture_layer_tiling_perftest.cc
diff --git a/cc/resources/picture_layer_tiling_perftest.cc b/cc/resources/picture_layer_tiling_perftest.cc
index 3944e4698dc92fa6a4144d04083ff3402ad64f49..a8ce42b10c904de8eb8ba17dbfafb3565bb9518c 100644
--- a/cc/resources/picture_layer_tiling_perftest.cc
+++ b/cc/resources/picture_layer_tiling_perftest.cc
@@ -44,11 +44,14 @@ class PictureLayerTilingPerfTest : public testing::Test {
}
void SetUp() override {
+ LayerTreeSettings defaults;
picture_layer_tiling_client_.SetTileSize(gfx::Size(256, 256));
- picture_layer_tiling_client_.set_max_tiles_for_interest_area(250);
picture_layer_tiling_client_.set_tree(PENDING_TREE);
picture_layer_tiling_ = PictureLayerTiling::Create(
- 1, gfx::Size(256 * 50, 256 * 50), &picture_layer_tiling_client_);
+ 1, gfx::Size(256 * 50, 256 * 50), &picture_layer_tiling_client_,
+ defaults.max_tiles_for_interest_area,
+ defaults.skewport_target_time_in_seconds,
+ defaults.skewport_extrapolation_limit_in_content_pixels);
picture_layer_tiling_->CreateAllTilesForTesting();
}
@@ -126,8 +129,12 @@ class PictureLayerTilingPerfTest : public testing::Test {
void RunRasterIteratorConstructTest(const std::string& test_name,
const gfx::Rect& viewport) {
gfx::Size bounds(viewport.size());
- picture_layer_tiling_ =
- PictureLayerTiling::Create(1, bounds, &picture_layer_tiling_client_);
+ LayerTreeSettings defaults;
+ picture_layer_tiling_ = PictureLayerTiling::Create(
+ 1, bounds, &picture_layer_tiling_client_,
+ defaults.max_tiles_for_interest_area,
+ defaults.skewport_target_time_in_seconds,
+ defaults.skewport_extrapolation_limit_in_content_pixels);
picture_layer_tiling_client_.set_tree(ACTIVE_TREE);
picture_layer_tiling_->ComputeTilePriorityRects(viewport, 1.0f, 1.0,
Occlusion());
@@ -151,8 +158,12 @@ class PictureLayerTilingPerfTest : public testing::Test {
int num_tiles,
const gfx::Rect& viewport) {
gfx::Size bounds(10000, 10000);
- picture_layer_tiling_ =
- PictureLayerTiling::Create(1, bounds, &picture_layer_tiling_client_);
+ LayerTreeSettings defaults;
+ picture_layer_tiling_ = PictureLayerTiling::Create(
+ 1, bounds, &picture_layer_tiling_client_,
+ defaults.max_tiles_for_interest_area,
+ defaults.skewport_target_time_in_seconds,
+ defaults.skewport_extrapolation_limit_in_content_pixels);
picture_layer_tiling_client_.set_tree(ACTIVE_TREE);
picture_layer_tiling_->ComputeTilePriorityRects(viewport, 1.0f, 1.0,
Occlusion());

Powered by Google App Engine
This is Rietveld 408576698