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

Unified Diff: chrome/browser/metrics/metrics_reporting_state.cc

Issue 2014463003: Modifications for recording whether UMA/Crash default state for iOS (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 7 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 | « chrome/browser/metrics/metrics_reporting_state.h ('k') | chrome/common/pref_names.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/metrics/metrics_reporting_state.cc
diff --git a/chrome/browser/metrics/metrics_reporting_state.cc b/chrome/browser/metrics/metrics_reporting_state.cc
index 30a9dd588de445c6632869268f6c299e97e97cea..9e74f158f3c558422940129dbb06ffa5a7aaa118 100644
--- a/chrome/browser/metrics/metrics_reporting_state.cc
+++ b/chrome/browser/metrics/metrics_reporting_state.cc
@@ -124,11 +124,6 @@ void InitiateMetricsReportingChange(
base::Bind(&SetMetricsReporting, enabled, callback_fn));
}
-void RegisterMetricsReportingStatePrefs(PrefRegistrySimple* registry) {
- registry->RegisterIntegerPref(prefs::kMetricsDefaultOptIn,
- metrics::MetricsServiceClient::DEFAULT_UNKNOWN);
-}
-
bool IsMetricsReportingPolicyManaged() {
const PrefService* pref_service = g_browser_process->local_state();
const PrefService::Preference* pref =
@@ -136,20 +131,6 @@ bool IsMetricsReportingPolicyManaged() {
return pref && pref->IsManaged();
}
-void RecordMetricsReportingDefaultOptIn(PrefService* local_state, bool opt_in) {
- DCHECK(GetMetricsReportingDefaultOptIn(local_state) ==
- metrics::MetricsServiceClient::DEFAULT_UNKNOWN);
- local_state->SetInteger(prefs::kMetricsDefaultOptIn,
- opt_in ? metrics::MetricsServiceClient::OPT_IN
- : metrics::MetricsServiceClient::OPT_OUT);
-}
-
-metrics::MetricsServiceClient::EnableMetricsDefault
-GetMetricsReportingDefaultOptIn(PrefService* local_state) {
- return static_cast<metrics::MetricsServiceClient::EnableMetricsDefault>(
- local_state->GetInteger(prefs::kMetricsDefaultOptIn));
-}
-
// TODO(gayane): Add unittest which will check that observer on device settings
// will trigger this function and kMetricsReportinEnabled as well as metrics
// service state will be updated accordingly.
« no previous file with comments | « chrome/browser/metrics/metrics_reporting_state.h ('k') | chrome/common/pref_names.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698