Refactor the daemon controller so that the callbacks are called on the caller thread.

remoting/host/plugin/host_script_object.h

Index: remoting/host/plugin/host_script_object.h
diff --git a/remoting/host/plugin/host_script_object.h b/remoting/host/plugin/host_script_object.h
index e1120ec36731e701cc83005730e7c6930aace687..f12172e7a268c37d130f2db84b4894721f7c3577 100644
--- a/remoting/host/plugin/host_script_object.h
+++ b/remoting/host/plugin/host_script_object.h
@@ -255,10 +255,9 @@ class HostNPScriptObject {
       scoped_ptr<base::ListValue> paired_clients);
 
   // Callback handler for DaemonController::GetUsageStatsConsent().
-  void InvokeGetUsageStatsConsentCallback(const ScopedRefNPObject& callback,
-                                          bool supported,
-                                          bool allowed,
-                                          bool set_by_policy);
+  void InvokeGetUsageStatsConsentCallback(
+      const ScopedRefNPObject& callback,

[Wez, 2013/09/10 09:05:37]

nit: This, and any other methods that get bound to run off the plugin thread,
should be passed scoped_ptr<ScopedRefNPObject>&

(NPObject ref-counting isn't thread-safe)

[alexeypa (please no reviews), 2013/09/10 16:59:22]

Done.

+      const DaemonController::UsageStatsConsent& consent);
 
   //////////////////////////////////////////////////////////
   // Basic helper methods used for both It2Me and Me2me.
@@ -315,7 +314,7 @@ class HostNPScriptObject {
   // Me2Me host state.
 
   // Platform-specific installation & configuration implementation.
-  scoped_ptr<DaemonController> daemon_controller_;
+  scoped_refptr<DaemonController> daemon_controller_;
 
   // TODO(sergeyu): Replace this thread with
   // SequencedWorkerPool. Problem is that SequencedWorkerPool relies