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

Unified Diff: remoting/host/plugin/daemon_controller.h

Issue 10537182: The user's consent to crash dumps reporting can now be set via the UI (Windows only). The checkbox … (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebased Created 8 years, 6 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/installer/chromoting.wxs ('k') | remoting/host/plugin/daemon_controller_linux.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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..bf6734501833328bf76f58580d071822d8081cb1 100644
--- a/remoting/host/plugin/daemon_controller.h
+++ b/remoting/host/plugin/daemon_controller.h
@@ -79,6 +79,15 @@ class DaemonController {
// Callback type for GetVersion().
typedef base::Callback<void (const std::string&)> GetVersionCallback;
+ // Callback type for GetUsageStatsConsent(). |supported| indicates whether
+ // crash dump reporting is supported by the host. |allowed| indicates if
+ // crash dump reporting is allowed by the user. |set_by_policy| carries
+ // information whether the crash dump reporting is controlled by policy.
+ typedef base::Callback<void (
+ bool supported,
+ bool allowed,
+ bool set_by_policy)> GetUsageStatsConsentCallback;
+
virtual ~DaemonController() {}
// Return the "installed/running" state of the daemon process.
@@ -103,7 +112,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 +139,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();
};
« no previous file with comments | « remoting/host/installer/chromoting.wxs ('k') | remoting/host/plugin/daemon_controller_linux.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698