diff options
author | brianderson <brianderson@chromium.org> | 2015-08-11 12:29:44 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-08-11 19:30:19 +0000 |
commit | 7d42dad6d95bf258c721f9d15f734bb9d0af8014 (patch) | |
tree | c6d3386934ffb79e4d1cb4231fa5e34fc78bc841 /cc | |
parent | e5f477c7d8494811b67f1c4dda4afb6f214cf074 (diff) | |
download | chromium_src-7d42dad6d95bf258c721f9d15f734bb9d0af8014.zip chromium_src-7d42dad6d95bf258c721f9d15f734bb9d0af8014.tar.gz chromium_src-7d42dad6d95bf258c721f9d15f734bb9d0af8014.tar.bz2 |
cc: Rebalance tree priorities in SMOOTHNESS_TAKES_PRIORITY
Instead of only rasterizing pending NOW tiles before active
EVENTUALLY tiles, rasterize ALL pending tiles before active
EVENTUALLY tiles.
After rasterizing all pending NOW tiles, we should activate
very soon and won't need the active EVENTUALLY tiles anyway.
BUG=515722
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1275703004
Cr-Commit-Position: refs/heads/master@{#342868}
Diffstat (limited to 'cc')
-rw-r--r-- | cc/tiles/raster_tile_priority_queue_all.cc | 13 |
1 files changed, 7 insertions, 6 deletions
diff --git a/cc/tiles/raster_tile_priority_queue_all.cc b/cc/tiles/raster_tile_priority_queue_all.cc index 8f7bdbf..ea3c14f 100644 --- a/cc/tiles/raster_tile_priority_queue_all.cc +++ b/cc/tiles/raster_tile_priority_queue_all.cc @@ -136,13 +136,14 @@ RasterTilePriorityQueueAll::GetNextQueues() const { switch (tree_priority_) { case SMOOTHNESS_TAKES_PRIORITY: { - // If we're down to eventually bin tiles on the active tree, process the - // pending tree to allow tiles required for activation to be initialized - // when memory policy only allows prepaint. - if (active_priority.priority_bin == TilePriority::EVENTUALLY && - pending_priority.priority_bin == TilePriority::NOW) { + // If we're down to eventually bin tiles on the active tree and there + // is a pending tree, process the entire pending tree to allow tiles + // required for activation to be initialized when memory policy only + // allows prepaint. The eventually bin tiles on the active tree are + // lowest priority since that work is likely to be thrown away when + // we activate. + if (active_priority.priority_bin == TilePriority::EVENTUALLY) return pending_queues_; - } return active_queues_; } case NEW_CONTENT_TAKES_PRIORITY: { |