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

Unified Diff: cc/layers/picture_layer_impl.cc

Issue 140513006: cc: Simplify picture layer tiling update tile priorities. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 11 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/layers/picture_layer_impl.cc
diff --git a/cc/layers/picture_layer_impl.cc b/cc/layers/picture_layer_impl.cc
index 0826186f4df64540bcb5ae86521a2d7f17c3b5bc..29b4bd3292c4517dc59371cd9038f16c5fafeb6a 100644
--- a/cc/layers/picture_layer_impl.cc
+++ b/cc/layers/picture_layer_impl.cc
@@ -351,15 +351,9 @@ void PictureLayerImpl::UpdateTilePriorities() {
layer_tree_impl()->settings().max_tiles_for_interest_area;
tilings_->UpdateTilePriorities(
tree,
- viewport_size,
viewport_in_content_space,
visible_content_rect(),
- last_bounds_,
- bounds(),
- last_content_scale_,
contents_scale_x(),
- last_screen_space_transform_,
- current_screen_space_transform,
current_frame_time_in_seconds,
max_tiles_for_interest_area);
@@ -726,7 +720,7 @@ void PictureLayerImpl::MarkVisibleResourcesAsRequired() const {
// This iteration is over the visible content rect which is potentially
// less conservative than projecting the viewport into the layer.
// Ignore tiles that are know to be outside the viewport.
- if (iter->priority(PENDING_TREE).distance_to_visible_in_pixels != 0)
+ if (iter->priority(PENDING_TREE).distance_to_visible != 0)
enne (OOO) 2014/01/31 23:33:36 I think this can go away as we only mark tiles in
vmpstr 2014/02/03 20:27:24 Good point. Done.
continue;
missing_region.Subtract(iter.geometry_rect());
@@ -795,7 +789,7 @@ bool PictureLayerImpl::MarkVisibleTilesAsRequired(
// This iteration is over the visible content rect which is potentially
// less conservative than projecting the viewport into the layer.
// Ignore tiles that are know to be outside the viewport.
- if (tile->priority(PENDING_TREE).distance_to_visible_in_pixels != 0)
+ if (tile->priority(PENDING_TREE).distance_to_visible != 0)
continue;
// If the missing region doesn't cover it, this tile is fully

Powered by Google App Engine
This is Rietveld 408576698