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

Unified Diff: cc/tiles/tile_manager.h

Issue 2612413003: Clean up tile deletion (Closed)
Patch Set: fix_perftest Created 3 years, 11 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/tiles/tile_manager.h
diff --git a/cc/tiles/tile_manager.h b/cc/tiles/tile_manager.h
index 7a47febe84c08d20255dcca96ca1e46723c2fbf5..abecb651960a2f7593dbbdb4f58a4b1ce4878942 100644
--- a/cc/tiles/tile_manager.h
+++ b/cc/tiles/tile_manager.h
@@ -178,11 +178,6 @@ class CC_EXPORT TileManager {
raster_buffer_provider_ = raster_buffer_provider;
}
- void FreeResourcesAndCleanUpReleasedTilesForTesting() {
- FreeResourcesForReleasedTiles();
- CleanUpReleasedTiles();
- }
-
std::vector<Tile*> AllTilesForTesting() const {
std::vector<Tile*> tiles;
for (auto& tile_pair : tiles_)
@@ -207,14 +202,11 @@ class CC_EXPORT TileManager {
}
void OnRasterTaskCompleted(std::unique_ptr<RasterBuffer> raster_buffer,
- Tile* tile,
+ Tile::Id tile_id,
Resource* resource,
bool was_canceled);
protected:
- void FreeResourcesForReleasedTiles();
- void CleanUpReleasedTiles();
-
friend class Tile;
// Virtual for testing.
virtual void Release(Tile* tile);
@@ -326,8 +318,6 @@ class CC_EXPORT TileManager {
RasterTaskCompletionStats flush_stats_;
- std::vector<Tile*> released_tiles_;
-
TaskGraph graph_;
UniqueNotifier more_tiles_need_prepare_check_notifier_;
« no previous file with comments | « cc/test/fake_tile_manager.cc ('k') | cc/tiles/tile_manager.cc » ('j') | cc/tiles/tile_manager.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698