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

Unified Diff: chrome/browser/ui/webui/settings/metrics_reporting_handler.cc

Issue 2233443002: MD Settings: implement metrics reporting checkbox on desktop (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@desktop-uma
Patch Set: remove 2 using Created 4 years, 4 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
Index: chrome/browser/ui/webui/settings/metrics_reporting_handler.cc
diff --git a/chrome/browser/ui/webui/settings/metrics_reporting_handler.cc b/chrome/browser/ui/webui/settings/metrics_reporting_handler.cc
new file mode 100644
index 0000000000000000000000000000000000000000..1121c85f2160aeb3a15f394a32667d3da8dd1740
--- /dev/null
+++ b/chrome/browser/ui/webui/settings/metrics_reporting_handler.cc
@@ -0,0 +1,107 @@
+// Copyright 2016 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.
+
+#if defined(GOOGLE_CHROME_BUILD) && !defined(OS_CHROMEOS)
+
+#include "chrome/browser/ui/webui/settings/metrics_reporting_handler.h"
+
+#include "base/bind.h"
+#include "base/bind_helpers.h"
+#include "base/values.h"
+#include "chrome/browser/browser_process.h"
+#include "chrome/browser/metrics/chrome_metrics_service_accessor.h"
+#include "chrome/browser/metrics/metrics_reporting_state.h"
+#include "components/metrics/metrics_pref_names.h"
+#include "components/prefs/pref_service.h"
+#include "content/public/browser/web_ui.h"
+#include "policy/policy_constants.h"
+
+namespace settings {
+
+MetricsReportingHandler::MetricsReportingHandler() {}
+MetricsReportingHandler::~MetricsReportingHandler() {}
+
+void MetricsReportingHandler::RegisterMessages() {
+ web_ui()->RegisterMessageCallback("getMetricsReporting",
+ base::Bind(&MetricsReportingHandler::HandleGetMetricsReporting,
+ base::Unretained(this)));
+ web_ui()->RegisterMessageCallback("setMetricsReportingEnabled",
+ base::Bind(&MetricsReportingHandler::HandleSetMetricsReportingEnabled,
+ base::Unretained(this)));
+}
+
+void MetricsReportingHandler::OnJavascriptAllowed() {
+ pref_member_.reset(new BooleanPrefMember);
+ pref_member_->Init(metrics::prefs::kMetricsReportingEnabled,
+ g_browser_process->local_state(),
+ base::Bind(&MetricsReportingHandler::OnPrefChanged,
+ base::Unretained(this)));
+
+ policy_registrar_.reset(new policy::PolicyChangeRegistrar(
+ g_browser_process->policy_service(),
+ policy::PolicyNamespace(policy::POLICY_DOMAIN_CHROME, std::string())));
+ policy_registrar_->Observe(policy::key::kMetricsReportingEnabled,
+ base::Bind(&MetricsReportingHandler::OnPolicyChanged,
+ base::Unretained(this)));
+}
+
+void MetricsReportingHandler::OnJavascriptDisallowed() {
+ pref_member_.reset();
+ policy_registrar_.reset();
+}
+
+void MetricsReportingHandler::HandleGetMetricsReporting(
+ const base::ListValue* args) {
+ AllowJavascript();
+ const base::Value* callback_id;
+ CHECK(args->Get(0, &callback_id));
+ ResolveJavascriptCallback(*callback_id, *CreateMetricsReportingDict());
+}
+
+std::unique_ptr<base::DictionaryValue>
+ MetricsReportingHandler::CreateMetricsReportingDict() {
+ std::unique_ptr<base::DictionaryValue> dict(new base::DictionaryValue);
+ dict->SetBoolean("enabled",
+ ChromeMetricsServiceAccessor::IsMetricsAndCrashReportingEnabled());
+ dict->SetBoolean("managed", IsMetricsReportingPolicyManaged());
+ return dict;
+}
+
+void MetricsReportingHandler::HandleSetMetricsReportingEnabled(
+ const base::ListValue* args) {
+ if (IsMetricsReportingPolicyManaged()) {
+ NOTREACHED();
+ // NOTE: ChangeMetricsReportingState() already checks whether metrics
+ // reporting is managed by policy. Also, the UI really shouldn't be able to
+ // send this message when managed. However, in this specific case, there's a
+ // sane, graceful fallback so we might as well do that.
+ SendMetricsReportingChange();
+ return;
+ }
+
+ bool enabled;
+ CHECK(args->GetBoolean(0, &enabled));
+ ChangeMetricsReportingState(enabled);
+}
+
+void MetricsReportingHandler::OnPolicyChanged(const base::Value* previous,
+ const base::Value* current) {
+ SendMetricsReportingChange();
+}
+
+void MetricsReportingHandler::OnPrefChanged(const std::string& pref_name) {
+ DCHECK_EQ(metrics::prefs::kMetricsReportingEnabled, pref_name);
+ SendMetricsReportingChange();
+}
+
+void MetricsReportingHandler::SendMetricsReportingChange() {
+ CallJavascriptFunction(
+ "cr.webUIListenerCallback",
+ base::StringValue("metrics-reporting-change"),
+ *CreateMetricsReportingDict());
+}
+
+} // namespace settings
+
+#endif // defined(GOOGLE_CHROME_BUILD) && !defined(OS_CHROMEOS)

Powered by Google App Engine
This is Rietveld 408576698