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

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

Issue 289373007: Rename MetricsServiceHelper to ChromeMetricsServiceAccessor. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Another rebas. Created 6 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_service.h ('k') | chrome/browser/metrics/metrics_service_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/metrics/metrics_service.cc
diff --git a/chrome/browser/metrics/metrics_service.cc b/chrome/browser/metrics/metrics_service.cc
index b6fca9f1eb9fea25efa4eab499137c5e9d5f297a..052efbdf8273c5da5547dde5ead10f8464cf2b29 100644
--- a/chrome/browser/metrics/metrics_service.cc
+++ b/chrome/browser/metrics/metrics_service.cc
@@ -1762,51 +1762,3 @@ bool MetricsService::IsPluginProcess(int process_type) {
process_type == content::PROCESS_TYPE_PPAPI_PLUGIN ||
process_type == content::PROCESS_TYPE_PPAPI_BROKER);
}
-
-// static
-bool MetricsServiceHelper::IsMetricsReportingEnabled() {
- bool result = false;
- const PrefService* local_state = g_browser_process->local_state();
- if (local_state) {
- const PrefService::Preference* uma_pref =
- local_state->FindPreference(prefs::kMetricsReportingEnabled);
- if (uma_pref) {
- bool success = uma_pref->GetValue()->GetAsBoolean(&result);
- DCHECK(success);
- }
- }
- return result;
-}
-
-bool MetricsServiceHelper::IsCrashReportingEnabled() {
-#if defined(GOOGLE_CHROME_BUILD)
-#if defined(OS_CHROMEOS)
- bool reporting_enabled = false;
- chromeos::CrosSettings::Get()->GetBoolean(chromeos::kStatsReportingPref,
- &reporting_enabled);
- return reporting_enabled;
-#elif defined(OS_ANDROID)
- // Android has its own settings for metrics / crash uploading.
- const PrefService* prefs = g_browser_process->local_state();
- return prefs->GetBoolean(prefs::kCrashReportingEnabled);
-#else
- return MetricsServiceHelper::IsMetricsReportingEnabled();
-#endif
-#else
- return false;
-#endif
-}
-
-void MetricsServiceHelper::AddMetricsServiceObserver(
- MetricsServiceObserver* observer) {
- MetricsService* metrics_service = g_browser_process->metrics_service();
- if (metrics_service)
- metrics_service->AddObserver(observer);
-}
-
-void MetricsServiceHelper::RemoveMetricsServiceObserver(
- MetricsServiceObserver* observer) {
- MetricsService* metrics_service = g_browser_process->metrics_service();
- if (metrics_service)
- metrics_service->RemoveObserver(observer);
-}
« no previous file with comments | « chrome/browser/metrics/metrics_service.h ('k') | chrome/browser/metrics/metrics_service_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698