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

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: Remove the LTI variable 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 657e72f9d52b88cddb8d6b6d70697fc2ccfc2f0d..ee132e6cf8fd9a53a43b748015dd40fa53784bd6 100644
--- a/cc/trees/layer_tree_host_impl.cc
+++ b/cc/trees/layer_tree_host_impl.cc
@@ -231,7 +231,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();
}
@@ -856,8 +856,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();
@@ -1238,7 +1237,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",
@@ -1447,7 +1446,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