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

Unified Diff: cc/resources/tile_manager_perftest.cc

Issue 22831041: cc: Removed low priority bin from tile management. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebased Created 7 years, 4 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/resources/tile_manager.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/resources/tile_manager_perftest.cc
diff --git a/cc/resources/tile_manager_perftest.cc b/cc/resources/tile_manager_perftest.cc
index 336fa57668570441868f10f5f8cbf13000b50997..1a6060128fc91c7da0e1b375d2f8775214a7224d 100644
--- a/cc/resources/tile_manager_perftest.cc
+++ b/cc/resources/tile_manager_perftest.cc
@@ -91,7 +91,8 @@ class TileManagerPerfTest : public testing::Test {
case EVENTUALLY_AND_ACTIVE_BIN:
case EVENTUALLY_BIN:
return TilePriorityForEventualBin();
- case NEVER_AND_ACTIVE_BIN:
+ case AT_LAST_BIN:
+ case AT_LAST_AND_ACTIVE_BIN:
case NEVER_BIN:
return TilePriority();
default:
@@ -110,7 +111,8 @@ class TileManagerPerfTest : public testing::Test {
case EVENTUALLY_AND_ACTIVE_BIN:
case EVENTUALLY_BIN:
return NEVER_BIN;
- case NEVER_AND_ACTIVE_BIN:
+ case AT_LAST_BIN:
+ case AT_LAST_AND_ACTIVE_BIN:
case NEVER_BIN:
return NOW_BIN;
default:
@@ -149,8 +151,8 @@ class TileManagerPerfTest : public testing::Test {
void RunManageTilesTest(const std::string test_name,
unsigned tile_count,
unsigned priority_change_percent) {
- DCHECK_GE(100u, tile_count);
- DCHECK_LE(100u, priority_change_percent);
+ DCHECK_GE(tile_count, 100u);
+ DCHECK_LE(priority_change_percent, 100u);
num_runs_ = 0;
TileBinVector tiles;
CreateTiles(tile_count, &tiles);
« no previous file with comments | « cc/resources/tile_manager.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698