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

[Ilya Sherman, 2016/09/23 22:39:59]

Is your CL changing something about the correctness of this method's return
value on ChromeOS?

[michaelpg, 2016/09/23 23:48:15]

No. The method seems to have already been corrected on Chrome OS in November,
making the comment incorrect:
  https://bugs.chromium.org/p/chromium/issues/detail?id=532084#c3

If that's correct, would you prefer me to make this comment change in a
separate CL?

[Ilya Sherman, 2016/09/27 01:33:13]

Yes please, and send the code review to gayane@, as she is the most familiar
with the details of this code currently =)

[michaelpg, 2016/09/27 16:45:16]

Looks like gayane@ beat me to it! https://codereview.chromium.org/2351873002

[Ilya Sherman, 2016/09/27 21:16:49]

Great!  Please make sure to remember to revert your changes to this file =)

[michaelpg, 2016/09/30 22:15:01]

Done.

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