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

Unified Diff: remoting/host/setup/host_starter.cc

Issue 23606019: Refactor the daemon controller so that the callbacks are called on the caller thread. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix the license Created 7 years, 3 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
« no previous file with comments | « remoting/host/setup/host_starter.h ('k') | remoting/host/setup/native_messaging_host.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/host/setup/host_starter.cc
diff --git a/remoting/host/setup/host_starter.cc b/remoting/host/setup/host_starter.cc
index 6758f58cb0f3f1138f539c3478e1950a5c24b396..1fd3c17a33016b5c165d0ae938f102fc71bdf894 100644
--- a/remoting/host/setup/host_starter.cc
+++ b/remoting/host/setup/host_starter.cc
@@ -20,10 +20,10 @@ namespace remoting {
HostStarter::HostStarter(
scoped_ptr<gaia::GaiaOAuthClient> oauth_client,
scoped_ptr<remoting::ServiceClient> service_client,
- scoped_ptr<remoting::DaemonController> daemon_controller)
+ scoped_refptr<remoting::DaemonController> daemon_controller)
: oauth_client_(oauth_client.Pass()),
service_client_(service_client.Pass()),
- daemon_controller_(daemon_controller.Pass()),
+ daemon_controller_(daemon_controller),
consent_to_data_collection_(false),
unregistering_host_(false),
weak_ptr_factory_(this),
@@ -42,11 +42,11 @@ scoped_ptr<HostStarter> HostStarter::Create(
scoped_ptr<remoting::ServiceClient> service_client(
new remoting::ServiceClient(
chromoting_hosts_url, url_request_context_getter));
- scoped_ptr<remoting::DaemonController> daemon_controller(
+ scoped_refptr<remoting::DaemonController> daemon_controller(
remoting::DaemonController::Create());
return scoped_ptr<HostStarter>(
new HostStarter(oauth_client.Pass(), service_client.Pass(),
- daemon_controller.Pass()));
+ daemon_controller));
}
void HostStarter::StartHost(
« no previous file with comments | « remoting/host/setup/host_starter.h ('k') | remoting/host/setup/native_messaging_host.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698