The user's consent to crash dumps reporting can now be set via the UI (Windows only). The checkbox …

remoting/host/plugin/daemon_controller.h

Index: remoting/host/plugin/daemon_controller.h
diff --git a/remoting/host/plugin/daemon_controller.h b/remoting/host/plugin/daemon_controller.h
index 04fde56b786391276c6fb571a56475a03cd7138b..5b1cca79db7b2d83a166f015398e40df50d8e7db 100644
--- a/remoting/host/plugin/daemon_controller.h
+++ b/remoting/host/plugin/daemon_controller.h
@@ -79,6 +79,9 @@ class DaemonController {
   // Callback type for GetVersion().
   typedef base::Callback<void (const std::string&)> GetVersionCallback;
 
+  // Callback type for GetUsageStatsConsent().
+  typedef base::Callback<void (bool, bool, bool)> GetUsageStatsConsentCallback;

[Jamie, 2012/06/21 18:47:52]

I think this typedef could do with documentation to explain what the three bools
represent.

[alexeypa (please no reviews), 2012/06/21 23:30:26]

Done.

+
   virtual ~DaemonController() {}
 
   // Return the "installed/running" state of the daemon process.
@@ -103,7 +106,8 @@ class DaemonController {
   // into SetConfig() and Start() once we have basic host setup flow
   // working.
   virtual void SetConfigAndStart(scoped_ptr<base::DictionaryValue> config,
-                                 const CompletionCallback& done_callback) = 0;
+                                 bool consent,
+                                 const CompletionCallback& done) = 0;
 
   // Updates current host configuration with the values specified in
   // |config|. Changes must take effect before the call completes.
@@ -129,6 +133,10 @@ class DaemonController {
   // major.minor.build.patch, if it is installed, or "" otherwise.
   virtual void GetVersion(const GetVersionCallback& done_callback) = 0;
 
+  // Get the user's consent to crash reporting.
+  virtual void GetUsageStatsConsent(
+      const GetUsageStatsConsentCallback& done) = 0;
+
   static scoped_ptr<DaemonController> Create();
 };