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

Unified Diff: cc/resources/picture_layer_tiling.h

Issue 411183002: cc: Optimize picture layer tiling raster tile iterator. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: 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
« no previous file with comments | « no previous file | cc/resources/picture_layer_tiling.cc » ('j') | cc/resources/picture_layer_tiling.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/resources/picture_layer_tiling.h
diff --git a/cc/resources/picture_layer_tiling.h b/cc/resources/picture_layer_tiling.h
index 37eea09cd95c506e6cce1fb95fd2cea229814c99..9246fd7c4c906846be5c3da316350f0bf9f03f74 100644
--- a/cc/resources/picture_layer_tiling.h
+++ b/cc/resources/picture_layer_tiling.h
@@ -61,16 +61,6 @@ class CC_EXPORT PictureLayerTiling {
TilingRasterTileIterator& operator++();
- gfx::Rect TileBounds() const {
- DCHECK(*this);
- if (type_ == TilePriority::NOW) {
- return tiling_->tiling_data_.TileBounds(visible_iterator_.index_x(),
- visible_iterator_.index_y());
- }
- return tiling_->tiling_data_.TileBounds(spiral_iterator_.index_x(),
- spiral_iterator_.index_y());
- }
-
private:
void AdvancePhase();
bool TileNeedsRaster(Tile* tile) const {
@@ -81,10 +71,6 @@ class CC_EXPORT PictureLayerTiling {
PictureLayerTiling* tiling_;
TilePriority::PriorityBin type_;
- gfx::Rect visible_rect_in_content_space_;
- gfx::Rect skewport_in_content_space_;
- gfx::Rect eventually_rect_in_content_space_;
- gfx::Rect soon_border_rect_in_content_space_;
WhichTree tree_;
Tile* current_tile_;
@@ -306,6 +292,11 @@ class CC_EXPORT PictureLayerTiling {
gfx::Rect current_soon_border_rect_;
gfx::Rect current_eventually_rect_;
+ bool has_visible_rect_tiles_;
+ bool has_skewport_rect_tiles_;
+ bool has_soon_border_rect_tiles_;
+ bool has_eventually_rect_tiles_;
+
std::vector<Tile*> eviction_tiles_cache_;
bool eviction_tiles_cache_valid_;
TreePriority eviction_cache_tree_priority_;
« no previous file with comments | « no previous file | cc/resources/picture_layer_tiling.cc » ('j') | cc/resources/picture_layer_tiling.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698