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

Unified Diff: cc/resources/tiling_set_eviction_queue.h

Issue 736753002: cc: Implement geometry-based tile eviction (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Clean up Created 6 years 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/tiling_set_eviction_queue.h
diff --git a/cc/resources/tiling_set_eviction_queue.h b/cc/resources/tiling_set_eviction_queue.h
index f88ce3bd710b0c9c88d1426261741f807c15146d..432c8e9a035a8cb9e9cb1495ae45b463bda385ed 100644
--- a/cc/resources/tiling_set_eviction_queue.h
+++ b/cc/resources/tiling_set_eviction_queue.h
@@ -5,6 +5,8 @@
#ifndef CC_RESOURCES_TILING_SET_EVICTION_QUEUE_H_
#define CC_RESOURCES_TILING_SET_EVICTION_QUEUE_H_
+#include <vector>
+
#include "cc/base/cc_export.h"
#include "cc/resources/picture_layer_tiling_set.h"
@@ -14,7 +16,8 @@ class CC_EXPORT TilingSetEvictionQueue {
public:
TilingSetEvictionQueue();
TilingSetEvictionQueue(PictureLayerTilingSet* tiling_set,
- TreePriority tree_priority);
+ TreePriority tree_priority,
+ bool skip_shared_out_of_order_tiles);
~TilingSetEvictionQueue();
Tile* Top();
@@ -23,26 +26,37 @@ class CC_EXPORT TilingSetEvictionQueue {
bool IsEmpty() const;
private:
- bool AdvanceToNextCategory();
bool AdvanceToNextEvictionTile();
+ bool AdvanceToNextPriorityBin();
bool AdvanceToNextTilingRangeType();
bool AdvanceToNextValidTiling();
+ bool AdvanceToSkewport();
PictureLayerTilingSet::TilingRange CurrentTilingRange() const;
size_t CurrentTilingIndex() const;
+ bool IsSharedOutOfOrderTile(const Tile* tile) const;
+
+ void SetCurrentRequiredForActivationTilingIndex(size_t index);
PictureLayerTilingSet* tiling_set_;
TreePriority tree_priority_;
+ bool skip_shared_out_of_order_tiles_;
+ bool processing_required_for_activation_tiling_;
vmpstr 2014/12/09 02:10:59 I'm not sure what this means.. tilings aren't typi
USE eero AT chromium.org 2014/12/09 18:44:15 Yes. But it is also true if we are processing the
vmpstr 2014/12/09 19:47:55 I see. I think the second name is better, since it
USE eero AT chromium.org 2014/12/10 12:35:45 Done.
+ bool processing_soon_border_rect_;
- PictureLayerTiling::EvictionCategory current_category_;
+ TilePriority::PriorityBin current_priority_bin_;
+ PictureLayerTiling* current_tiling_;
size_t current_tiling_index_;
+ size_t current_required_for_activation_tiling_index_;
PictureLayerTilingSet::TilingRangeType current_tiling_range_type_;
Tile* current_eviction_tile_;
- const std::vector<Tile*>* eviction_tiles_;
- size_t next_eviction_tile_index_;
+ TilingData::ReverseSpiralDifferenceIterator spiral_iterator_;
+ TilingData::Iterator visible_iterator_;
+ std::vector<Tile*> unoccluded_now_tiles_;
+ size_t next_unoccluded_now_tile_index_;
};
} // namespace cc
-#endif // CC_RESOURCES_TILING_SET_RASTER_QUEUE_H_
+#endif // CC_RESOURCES_TILING_SET_EVICTION_QUEUE_H_

Powered by Google App Engine
This is Rietveld 408576698