Consolidates accessing and setting the UMA pref to be within metrics code

chrome/browser/metrics/metrics_reporting_state.h

Index: chrome/browser/metrics/metrics_reporting_state.h
diff --git a/chrome/browser/metrics/metrics_reporting_state.h b/chrome/browser/metrics/metrics_reporting_state.h
index c333d84e52b87cfbe39697c3be71c829659838a2..20a6d84f694160e6e1a8cd61fe183b38093f50fe 100644
--- a/chrome/browser/metrics/metrics_reporting_state.h
+++ b/chrome/browser/metrics/metrics_reporting_state.h
@@ -5,12 +5,19 @@
 #ifndef CHROME_BROWSER_METRICS_METRICS_REPORTING_STATE_H_
 #define CHROME_BROWSER_METRICS_METRICS_REPORTING_STATE_H_
 
-// Tries to set crash stats upload consent to |enabled|. Regardless of success,
-// if crash stats uploading is enabled, starts the MetricsService; otherwise
-// stops it.
-// Returns whether crash stats uploading is enabled.
-// TODO(tfarina): This method is pretty confusing. Clean this up and rename it
-// to something that makes more sense.
-bool ResolveMetricsReportingEnabled(bool enabled);
+#include "base/callback.h"
+#include "components/metrics/metrics_service.h"
+
+
+// Calls |SetGoogleUpdateSettings| function in FILE thread with a callback

[Alexei Svitkine (slow), 2014/09/04 13:58:03]

This is describing an implementation detail. Instead, have the comment describe
the purpose of this function.

[gayane -on leave until 09-2017, 2014/09/04 18:35:47]

Done.

+// |SetMetricsReporting| function which does the remaining necessary changes.
+void InitiateMetricsReportingChange(
+    base::Callback<void(bool)> callback_fn, bool enabled);

[Alexei Svitkine (slow), 2014/09/04 13:58:03]

Nit: Make the callback the last param.

[gayane -on leave until 09-2017, 2014/09/04 18:35:47]

Done.

+
+// Returns weather MetricsReporting is checked
+bool IsMetricsReportingEnabled();
+
+// Returns weather MetricsReporting can be modified by user
+bool IsMetricsReportingUserChangable();
 
 #endif  // CHROME_BROWSER_METRICS_METRICS_REPORTING_STATE_H_