Create MetricsPrivateDelegate for metricsPrivate behavior

components/metrics/metrics_service_accessor.h

Index: components/metrics/metrics_service_accessor.h
diff --git a/components/metrics/metrics_service_accessor.h b/components/metrics/metrics_service_accessor.h
index eabf8a122391e27c6bf3eac08ed15c6ab94ee2ad..5d02555d5e2646e45974ec2036f44e4fb4085d57 100644
--- a/components/metrics/metrics_service_accessor.h
+++ b/components/metrics/metrics_service_accessor.h
@@ -29,8 +29,8 @@ class MetricsServiceAccessor {
   // Returns whether metrics reporting is enabled, using the value of the
   // kMetricsReportingEnabled pref in |pref_service| to determine whether user
   // has enabled reporting.
-  // NOTE: This method currently does not return the correct value on ChromeOS
-  // and Android due to http://crbug.com/362192 and http://crbug.com/532084. See
+  // NOTE: This method currently does not return the correct value on Android

[Rahul Chaturvedi, 2016/09/16 23:11:04]

Did you mean to change this comment?

[michaelpg, 2016/09/16 23:51:56]

yes, it's outdated -- it was fixed for Chrome OS in
https://codereview.chromium.org/1411863002. The referenced method,
ChromeMetricsServiceAccessor::IsMetricsAndCrashReportingEnabled(), only has
special cases for Android now (and in tests).

[Rahul Chaturvedi, 2016/09/19 18:11:09]

Acknowledged.

+  // due to http://crbug.com/362192 and http://crbug.com/532084. See
   // ChromeMetricsServiceAccessor::IsMetricsAndCrashReportingEnabled().
   static bool IsMetricsReportingEnabled(PrefService* pref_service);