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

Unified Diff: cc/resources/picture_layer_tiling.cc

Issue 202753002: cc: Add a tiling iterator that doesn't include borders (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: danakj review Created 6 years, 9 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/base/tiling_data_unittest.cc ('k') | cc/resources/picture_pile.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/resources/picture_layer_tiling.cc
diff --git a/cc/resources/picture_layer_tiling.cc b/cc/resources/picture_layer_tiling.cc
index 3604465c68409b8bc2d90b7b0a35c9c062f72a2b..356ac1b0246a7c60913ba1f8c7da54b7920296b8 100644
--- a/cc/resources/picture_layer_tiling.cc
+++ b/cc/resources/picture_layer_tiling.cc
@@ -111,7 +111,10 @@ void PictureLayerTiling::SetCanUseLCDText(bool can_use_lcd_text) {
void PictureLayerTiling::CreateMissingTilesInLiveTilesRect() {
const PictureLayerTiling* twin_tiling = client_->GetTwinTiling(this);
- for (TilingData::Iterator iter(&tiling_data_, live_tiles_rect_); iter;
+ bool include_borders = true;
+ for (TilingData::Iterator iter(
+ &tiling_data_, live_tiles_rect_, include_borders);
+ iter;
++iter) {
TileMapKey key = iter.index();
TileMap::iterator find = tiles_.find(key);
@@ -162,7 +165,11 @@ void PictureLayerTiling::Invalidate(const Region& layer_region) {
content_rect.Intersect(live_tiles_rect_);
if (content_rect.IsEmpty())
continue;
- for (TilingData::Iterator iter(&tiling_data_, content_rect); iter; ++iter) {
+ bool include_borders = true;
+ for (TilingData::Iterator iter(
+ &tiling_data_, content_rect, include_borders);
+ iter;
+ ++iter) {
TileMapKey key(iter.index());
TileMap::iterator find = tiles_.find(key);
if (find == tiles_.end())
@@ -435,7 +442,9 @@ void PictureLayerTiling::UpdateTilePriorities(
1.0f / (contents_scale_ * layer_contents_scale);
// Assign now priority to all visible tiles.
- for (TilingData::Iterator iter(&tiling_data_, visible_rect_in_content_space);
+ bool include_borders = true;
+ for (TilingData::Iterator iter(
+ &tiling_data_, visible_rect_in_content_space, include_borders);
iter;
++iter) {
TileMap::iterator find = tiles_.find(iter.index());
@@ -737,7 +746,9 @@ PictureLayerTiling::TilingRasterTileIterator::TilingRasterTileIterator(
eventually_rect_in_content_space_(tiling_->current_eventually_rect_),
tree_(tree),
current_tile_(NULL),
- visible_iterator_(&tiling->tiling_data_, visible_rect_in_content_space_),
+ visible_iterator_(&tiling->tiling_data_,
+ visible_rect_in_content_space_,
+ true /* include_borders */),
spiral_iterator_(&tiling->tiling_data_,
skewport_in_content_space_,
visible_rect_in_content_space_,
« no previous file with comments | « cc/base/tiling_data_unittest.cc ('k') | cc/resources/picture_pile.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698