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

Unified Diff: trunk/src/cc/trees/layer_tree_impl.cc

Issue 96073002: Revert 237848 "Use LatencyInfoSwapPromise to track LatencyInfo t..." (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 7 years, 1 month 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: trunk/src/cc/trees/layer_tree_impl.cc
===================================================================
--- trunk/src/cc/trees/layer_tree_impl.cc (revision 237855)
+++ trunk/src/cc/trees/layer_tree_impl.cc (working copy)
@@ -120,6 +120,9 @@
next_activation_forces_redraw_ = false;
}
+ target_tree->SetLatencyInfo(latency_info_);
+ latency_info_.Clear();
+
target_tree->PassSwapPromises(&swap_promise_list_);
target_tree->SetPageScaleFactorAndLimits(
@@ -461,6 +464,8 @@
pending_tree->SetCurrentlyScrollingLayer(
LayerTreeHostCommon::FindLayerInSubtree(pending_tree->root_layer(),
currently_scrolling_layer_ ? currently_scrolling_layer_->id() : 0));
+ pending_tree->SetLatencyInfo(latency_info_);
+ latency_info_.Clear();
}
static void DidBecomeActiveRecursive(LayerImpl* layer) {
@@ -694,6 +699,18 @@
scrollable_viewport_size - original_viewport_size);
}
+void LayerTreeImpl::SetLatencyInfo(const ui::LatencyInfo& latency_info) {
+ latency_info_.MergeWith(latency_info);
+}
+
+const ui::LatencyInfo& LayerTreeImpl::GetLatencyInfo() {
+ return latency_info_;
+}
+
+void LayerTreeImpl::ClearLatencyInfo() {
+ latency_info_.Clear();
+}
+
void LayerTreeImpl::QueueSwapPromise(scoped_ptr<SwapPromise> swap_promise) {
DCHECK(swap_promise);
if (swap_promise_list_.size() > kMaxQueuedSwapPromiseNumber)
@@ -708,9 +725,9 @@
new_swap_promise->clear();
}
-void LayerTreeImpl::FinishSwapPromises(CompositorFrameMetadata* metadata) {
+void LayerTreeImpl::FinishSwapPromises() {
for (size_t i = 0; i < swap_promise_list_.size(); i++)
- swap_promise_list_[i]->DidSwap(metadata);
+ swap_promise_list_[i]->DidSwap();
swap_promise_list_.clear();
}
« no previous file with comments | « trunk/src/cc/trees/layer_tree_impl.h ('k') | trunk/src/content/browser/renderer_host/render_widget_host_view_android.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698