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

Unified Diff: cc/resources/tiling_set_eviction_queue.h

Issue 1051473002: cc: Switch tiling set eviction queue to consider combined priority. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Clean up Created 5 years, 8 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/eviction_tile_priority_queue.cc ('k') | cc/resources/tiling_set_eviction_queue.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/resources/tiling_set_eviction_queue.h
diff --git a/cc/resources/tiling_set_eviction_queue.h b/cc/resources/tiling_set_eviction_queue.h
index b7027c4b5888914e3796d58fcb972a94ab5852b1..6662a7566d2b29ba6a1e2f2adffe62c2341a9c41 100644
--- a/cc/resources/tiling_set_eviction_queue.h
+++ b/cc/resources/tiling_set_eviction_queue.h
@@ -61,21 +61,13 @@ namespace cc {
// true (like it should be when there is a twin layer with a twin tiling set),
// eviction queue does not return shared which are out of order because their
// priority for tree priority is lowered or raised by a twin layer.
-// * If tree_priority is SAME_PRIORITY_FOR_BOTH_TREES, this happens for
-// a tile specific lower priority tree eviction queue (because priority for
-// tree priority is a merged priority).
-// * If tree priority is NEW_CONTENT_TAKES_PRIORITY, this happens for
-// an active tree eviction queue (because priority for tree priority is
-// the pending priority).
-// * If tree_priority is SMOOTHNESS_TAKES_PRIORITY, this happens for a pending
-// tree eviction queue (because priority for tree priority is the active
-// priority).
+// This happens for a tile specific lower priority tree eviction queue
+// (because eviction priority the combined priority).
// Those skipped shared out of order tiles are when returned only by the twin
// eviction queue.
class CC_EXPORT TilingSetEvictionQueue {
public:
TilingSetEvictionQueue(PictureLayerTilingSet* tiling_set,
- TreePriority tree_priority,
bool skip_shared_out_of_order_tiles);
~TilingSetEvictionQueue();
@@ -92,17 +84,12 @@ class CC_EXPORT TilingSetEvictionQueue {
PictureLayerTilingSet::TilingRange CurrentTilingRange() const;
size_t CurrentTilingIndex() const;
- bool IsSharedOutOfOrderTile(const Tile* tile) const;
- size_t TilingIndexWithRequiredForActivationTiles() const;
PictureLayerTilingSet* tiling_set_;
WhichTree tree_;
- TreePriority tree_priority_;
- bool skip_all_shared_tiles_;
bool skip_shared_out_of_order_tiles_;
bool processing_soon_border_rect_;
- bool processing_tiling_with_required_for_activation_tiles_;
- size_t tiling_index_with_required_for_activation_tiles_;
+ bool processing_required_for_activation_tiles_;
TilePriority::PriorityBin current_priority_bin_;
PictureLayerTiling* current_tiling_;
« no previous file with comments | « cc/resources/eviction_tile_priority_queue.cc ('k') | cc/resources/tiling_set_eviction_queue.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698