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

Unified Diff: cc/tiles/tile_manager.cc

Issue 2407443002: cc: Properly detect tiles that are needed now. (Closed)
Patch Set: qfix: update Created 4 years, 2 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/tiles/raster_tile_priority_queue_all.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/tiles/tile_manager.cc
diff --git a/cc/tiles/tile_manager.cc b/cc/tiles/tile_manager.cc
index 56f6664c6df4d272c77c4adba09bfbcd2f944617..60d3519a6f87da9d61436eb92e03be6523554d12 100644
--- a/cc/tiles/tile_manager.cc
+++ b/cc/tiles/tile_manager.cc
@@ -673,7 +673,8 @@ TileManager::PrioritizedWorkToSchedule TileManager::AssignGpuMemoryToTiles() {
break;
}
- bool tile_is_needed_now = priority.priority_bin == TilePriority::NOW;
+ bool tile_is_needed_now =
+ tile->required_for_activation() || tile->required_for_draw();
if (!tile->is_solid_color_analysis_performed() &&
tile->use_picture_analysis() && kUseColorEstimator) {
// We analyze for solid color here, to decide to continue
@@ -696,8 +697,7 @@ TileManager::PrioritizedWorkToSchedule TileManager::AssignGpuMemoryToTiles() {
}
// Prepaint tiles that are far away are only processed for images.
- if (!tile->required_for_activation() && !tile->required_for_draw() &&
- prioritized_tile.is_process_for_images_only()) {
+ if (!tile_is_needed_now && prioritized_tile.is_process_for_images_only()) {
work_to_schedule.tiles_to_process_for_images.push_back(prioritized_tile);
continue;
}
« no previous file with comments | « cc/tiles/raster_tile_priority_queue_all.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698