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

Unified Diff: cc/test/fake_tile_manager.cc

Issue 366113002: cc: Do not cleanup tiles with raster tasks. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: use scopedvector for released tiles. Created 6 years, 5 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
Index: cc/test/fake_tile_manager.cc
diff --git a/cc/test/fake_tile_manager.cc b/cc/test/fake_tile_manager.cc
index 1fc2e43e7f861a902be572e7745f1807cd44528d..b051389bfacd820fe974eccd8dbc1fb67d453347 100644
--- a/cc/test/fake_tile_manager.cc
+++ b/cc/test/fake_tile_manager.cc
@@ -99,7 +99,13 @@ void FakeTileManager::DidFinishRunningTasksForTesting() {
}
void FakeTileManager::Release(Tile* tile) {
+ // TODO(sohanjg): Find which tree the tile exists, and reset
+ // priority for it.
+ tile->SetPriority(ACTIVE_TREE, TilePriority());
+ tile->SetPriority(PENDING_TREE, TilePriority());
reveman 2014/07/28 02:24:35 please remove these and figure out how to reset pr
sohanjg 2014/07/28 10:10:19 As mentioned above, once we implement ::GetTree in
TileManager::Release(tile);
+
+ FreeResourcesForReleasedTiles();
CleanUpReleasedTiles();
}
« cc/test/fake_picture_layer_tiling_client.cc ('K') | « cc/test/fake_picture_layer_tiling_client.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698