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

Unified Diff: cc/trees/layer_tree_host_impl.cc

Issue 12259027: cc: Simplify the logic for deciding to update tile priorities. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Sper simple x2 Created 7 years, 8 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/trees/layer_tree_host_impl.cc
diff --git a/cc/trees/layer_tree_host_impl.cc b/cc/trees/layer_tree_host_impl.cc
index bfa05e39fa1d0f480db280aa7fe9a21f413508bc..f6d0f7f9c136e16ddc7bff9494bfc18557381bfc 100644
--- a/cc/trees/layer_tree_host_impl.cc
+++ b/cc/trees/layer_tree_host_impl.cc
@@ -229,7 +229,7 @@ void LayerTreeHostImpl::CommitComplete() {
// more lazily when needed prior to drawing.
if (settings_.impl_side_painting) {
pending_tree_->set_needs_update_draw_properties();
- pending_tree_->UpdateDrawProperties(LayerTreeImpl::UPDATE_PENDING_TREE);
+ pending_tree_->UpdateDrawProperties();
} else {
active_tree_->set_needs_update_draw_properties();
}
@@ -854,8 +854,7 @@ bool LayerTreeHostImpl::PrepareToDraw(FrameData* frame,
gfx::Rect device_viewport_damage_rect) {
TRACE_EVENT0("cc", "LayerTreeHostImpl::PrepareToDraw");
- active_tree_->UpdateDrawProperties(
- LayerTreeImpl::UPDATE_ACTIVE_TREE_FOR_DRAW);
+ active_tree_->UpdateDrawProperties();
frame->render_surface_layer_list = &active_tree_->RenderSurfaceLayerList();
frame->render_passes.clear();
@@ -1230,7 +1229,7 @@ bool LayerTreeHostImpl::ActivatePendingTreeIfNeeded() {
CHECK(tile_manager_);
- pending_tree_->UpdateDrawProperties(LayerTreeImpl::UPDATE_PENDING_TREE);
+ pending_tree_->UpdateDrawProperties();
TRACE_EVENT_ASYNC_STEP1("cc",
"PendingTree", pending_tree_.get(), "activate",
@@ -1439,7 +1438,7 @@ void LayerTreeHostImpl::DidChangeTopControlsPosition() {
}
bool LayerTreeHostImpl::EnsureRenderSurfaceLayerList() {
- active_tree_->UpdateDrawProperties(LayerTreeImpl::UPDATE_ACTIVE_TREE);
+ active_tree_->UpdateDrawProperties();
return !active_tree_->RenderSurfaceLayerList().empty();
}

Powered by Google App Engine
This is Rietveld 408576698