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

Unified Diff: cc/trees/thread_proxy.cc

Issue 1201573002: cc: Remove BeginFrameSourcesConstructor. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 6 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/thread_proxy.cc
diff --git a/cc/trees/thread_proxy.cc b/cc/trees/thread_proxy.cc
index 94e8d89b7118cdbb4deec6686726189c51e08bba..bca8a8c6a89381b663fb6f3d650f8e9a3b1e4e02 100644
--- a/cc/trees/thread_proxy.cc
+++ b/cc/trees/thread_proxy.cc
@@ -1203,11 +1203,8 @@ void ThreadProxy::InitializeImplOnImplThread(CompletionEvent* completion) {
SchedulerSettings scheduler_settings(
layer_tree_host()->settings().ToSchedulerSettings());
impl().scheduler = Scheduler::Create(
- this,
- scheduler_settings,
- impl().layer_tree_host_id,
- ImplThreadTaskRunner(),
- impl().external_begin_frame_source.Pass());
+ this, scheduler_settings, impl().layer_tree_host_id,
+ ImplThreadTaskRunner(), impl().external_begin_frame_source.get());
impl().scheduler->SetVisible(impl().layer_tree_host_impl->visible());
impl_thread_weak_ptr_ = impl().weak_factory.GetWeakPtr();
completion->Signal();
@@ -1267,6 +1264,7 @@ void ThreadProxy::LayerTreeHostClosedOnImplThread(CompletionEvent* completion) {
impl().layer_tree_host_impl->resource_provider());
impl().current_resource_update_controller = nullptr;
impl().scheduler = nullptr;
+ impl().external_begin_frame_source = nullptr;
impl().layer_tree_host_impl = nullptr;
impl().weak_factory.InvalidateWeakPtrs();
// We need to explicitly shutdown the notifier to destroy any weakptrs it is
« cc/surfaces/display_scheduler_unittest.cc ('K') | « cc/trees/single_thread_proxy.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698