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

Unified Diff: cc/resources/picture_layer_tiling_set.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/resources/picture_layer_tiling_set.cc
diff --git a/cc/resources/picture_layer_tiling_set.cc b/cc/resources/picture_layer_tiling_set.cc
index c8ac64822f7b0f42c49f55e2f31febb6d0eec3f2..862ecc4101634f49f99b2b11ff30b2a42ee056ea 100644
--- a/cc/resources/picture_layer_tiling_set.cc
+++ b/cc/resources/picture_layer_tiling_set.cc
@@ -303,36 +303,24 @@ PictureLayerTilingSet::CoverageIterator::operator bool() const {
void PictureLayerTilingSet::UpdateTilePriorities(
WhichTree tree,
- const gfx::Size& device_viewport,
const gfx::Rect& viewport_in_content_space,
const gfx::Rect& visible_content_rect,
- const gfx::Size& last_layer_bounds,
- const gfx::Size& current_layer_bounds,
- float last_layer_contents_scale,
- float current_layer_contents_scale,
- const gfx::Transform& last_screen_transform,
- const gfx::Transform& current_screen_transform,
+ float layer_contents_scale,
double current_frame_time_in_seconds,
size_t max_tiles_for_interest_area) {
gfx::Rect viewport_in_layer_space = gfx::ScaleToEnclosingRect(
viewport_in_content_space,
- 1.f / current_layer_contents_scale);
+ 1.f / layer_contents_scale);
gfx::Rect visible_layer_rect = gfx::ScaleToEnclosingRect(
visible_content_rect,
- 1.f / current_layer_contents_scale);
+ 1.f / layer_contents_scale);
for (size_t i = 0; i < tilings_.size(); ++i) {
tilings_[i]->UpdateTilePriorities(
tree,
- device_viewport,
viewport_in_layer_space,
visible_layer_rect,
- last_layer_bounds,
- current_layer_bounds,
- last_layer_contents_scale,
- current_layer_contents_scale,
- last_screen_transform,
- current_screen_transform,
+ layer_contents_scale,
current_frame_time_in_seconds,
max_tiles_for_interest_area);
}

Powered by Google App Engine
This is Rietveld 408576698