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

Unified Diff: cc/resources/picture_layer_tiling.h

Issue 428533008: cc: Remove vectors from tiling eviction tile iterator. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: update 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/resources/picture_layer_tiling.h
diff --git a/cc/resources/picture_layer_tiling.h b/cc/resources/picture_layer_tiling.h
index e0a368df236c962213582ecf6aa86b2ed27837e4..d23edd874dee089f2ac4b4189c9bec37e40e1ccc 100644
--- a/cc/resources/picture_layer_tiling.h
+++ b/cc/resources/picture_layer_tiling.h
@@ -101,24 +101,21 @@ class CC_EXPORT PictureLayerTiling {
public:
TilingEvictionTileIterator();
TilingEvictionTileIterator(PictureLayerTiling* tiling,
- TreePriority tree_priority);
+ TreePriority tree_priority,
+ TilePriority::PriorityBin type,
+ bool required_for_activation);
~TilingEvictionTileIterator();
operator bool() const;
const Tile* operator*() const;
Tile* operator*();
TilingEvictionTileIterator& operator++();
- TilePriority::PriorityBin get_type() {
- DCHECK(*this);
- const TilePriority& priority =
- (*tile_iterator_)->priority_for_tree_priority(tree_priority_);
- return priority.priority_bin;
- }
private:
PictureLayerTiling* tiling_;
TreePriority tree_priority_;
std::vector<Tile*>::iterator tile_iterator_;
+ std::vector<Tile*>* eviction_tiles_;
};
~PictureLayerTiling();
@@ -315,7 +312,10 @@ class CC_EXPORT PictureLayerTiling {
bool has_soon_border_rect_tiles_;
bool has_eventually_rect_tiles_;
- std::vector<Tile*> eviction_tiles_cache_;
+ std::vector<Tile*> eventually_eviction_tiles_;
+ std::vector<Tile*> soon_eviction_tiles_;
+ std::vector<Tile*> now_required_for_activation_eviction_tiles_;
+ std::vector<Tile*> now_not_required_for_activation_eviction_tiles_;
bool eviction_tiles_cache_valid_;
TreePriority eviction_cache_tree_priority_;

Powered by Google App Engine
This is Rietveld 408576698