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

remoting/host/setup/daemon_controller_delegate_win.h

Index: remoting/host/setup/daemon_controller_delegate_win.h
diff --git a/remoting/host/setup/daemon_controller_delegate_win.h b/remoting/host/setup/daemon_controller_delegate_win.h
new file mode 100644
index 0000000000000000000000000000000000000000..ded84787b137b679711a287d08fe609c817b2d10
--- /dev/null
+++ b/remoting/host/setup/daemon_controller_delegate_win.h
@@ -0,0 +1,81 @@
+// Copyright 2013 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef REMOTING_HOST_SETUP_DAEMON_CONTROLLER_DELEGATE_WIN_H_
+#define REMOTING_HOST_SETUP_DAEMON_CONTROLLER_DELEGATE_WIN_H_
+
+#include "base/memory/scoped_ptr.h"
+#include "base/timer/timer.h"
+#include "base/win/scoped_comptr.h"
+// chromoting_lib.h contains MIDL-generated declarations.
+#include "remoting/host/chromoting_lib.h"
+#include "remoting/host/setup/daemon_controller.h"
+
+namespace remoting {
+
+class DaemonInstallerWin;
+
+class DaemonControllerDelegateWin : public DaemonController::Delegate {
+ public:
+  DaemonControllerDelegateWin();
+  virtual ~DaemonControllerDelegateWin();
+
+  // DaemonController::Delegate interface.
+  virtual DaemonController::State GetState() OVERRIDE;
+  virtual scoped_ptr<base::DictionaryValue> GetConfig() OVERRIDE;
+  virtual void SetConfigAndStart(
+      scoped_ptr<base::DictionaryValue> config,
+      bool consent,
+      const DaemonController::CompletionCallback& done) OVERRIDE;
+  virtual void UpdateConfig(
+      scoped_ptr<base::DictionaryValue> config,
+      const DaemonController::CompletionCallback& done) OVERRIDE;
+  virtual void Stop(const DaemonController::CompletionCallback& done) OVERRIDE;
+  virtual void SetWindow(void* window_handle) OVERRIDE;
+  virtual std::string GetVersion() OVERRIDE;
+  virtual DaemonController::UsageStatsConsent GetUsageStatsConsent() OVERRIDE;
+
+ private:
+  // Activates an unprivileged instance of the daemon controller and caches it.
+  HRESULT ActivateController();
+
+  // Activates an instance of the daemon controller and caches it. If COM
+  // Elevation is supported (Vista+) the activated instance is elevated,
+  // otherwise it is activated under credentials of the caller.
+  HRESULT ActivateElevatedController();
+
+  // Releases the cached instance of the controller.
+  void ReleaseController();
+
+  // Procedes with the daemon configuration if the installation succeeded,
+  // otherwise reports the error.
+  void OnInstallationComplete(
+      scoped_ptr<base::DictionaryValue> config,
+      bool consent,
+      const DaemonController::CompletionCallback& done,
+      HRESULT hr);
+
+  // |control_| and |control2_| hold references to an instance of the daemon
+  // controller to prevent a UAC prompt on every operation.
+  base::win::ScopedComPtr<IDaemonControl> control_;
+  base::win::ScopedComPtr<IDaemonControl2> control2_;
+
+  // True if |control_| holds a reference to an elevated instance of the daemon
+  // controller.
+  bool control_is_elevated_;
+
+  // This timer is used to release |control_| after a timeout.
+  scoped_ptr<base::OneShotTimer<DaemonControllerDelegateWin> > release_timer_;
+
+  // Handle of the plugin window.
+  HWND window_handle_;
+
+  scoped_ptr<DaemonInstallerWin> installer_;
+
+  DISALLOW_COPY_AND_ASSIGN(DaemonControllerDelegateWin);
+};
+
+}  // namespace remoting
+
+#endif  // REMOTING_HOST_SETUP_DAEMON_CONTROLLER_DELEGATE_WIN_H_