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

Unified Diff: remoting/host/remoting_me2me_host.cc

Issue 11412305: Use AutoThread in ChromotingHostContext & NPAPI plugin. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix ChromotingHostContext startup-check and thread checks in getters. Created 8 years 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: remoting/host/remoting_me2me_host.cc
diff --git a/remoting/host/remoting_me2me_host.cc b/remoting/host/remoting_me2me_host.cc
index 88f7bae39a0a174e441792b4ddc42c64a5bc7c52..6307b4443bf50b5de2944d0b61afae5db54c5672 100644
--- a/remoting/host/remoting_me2me_host.cc
+++ b/remoting/host/remoting_me2me_host.cc
@@ -1087,13 +1087,11 @@ int main(int argc, char** argv) {
// Create the main message loop and start helper threads.
MessageLoop message_loop(MessageLoop::TYPE_UI);
- base::Closure quit_message_loop = base::Bind(&QuitMessageLoop, &message_loop);
- scoped_ptr<remoting::ChromotingHostContext> context(
- new remoting::ChromotingHostContext(
+ scoped_ptr<remoting::ChromotingHostContext> context =
+ remoting::ChromotingHostContext::Create(
new remoting::AutoThreadTaskRunner(message_loop.message_loop_proxy(),
- quit_message_loop)));
-
- if (!context->Start())
+ MessageLoop::QuitClosure()));
+ if (context)
alexeypa (please no reviews) 2012/12/03 20:34:23 if (!context)
Wez 2012/12/03 22:38:46 Done.
return remoting::kInitializationFailed;
// Create & start the HostProcess using these threads.

Powered by Google App Engine
This is Rietveld 408576698