Log some information about the state of the homepage on Android.

chrome/browser/prefs/pref_metrics_service.h

Index: chrome/browser/prefs/pref_metrics_service.h
diff --git a/chrome/browser/prefs/pref_metrics_service.h b/chrome/browser/prefs/pref_metrics_service.h
index 7627e48b4d2f1f8418afe2f7868fd46a7d6860b4..cc4d98dae9a05cc82165244ae23a425a4ed8f7be 100644
--- a/chrome/browser/prefs/pref_metrics_service.h
+++ b/chrome/browser/prefs/pref_metrics_service.h
@@ -25,6 +25,18 @@ class PrefMetricsService : public KeyedService {
   explicit PrefMetricsService(Profile* profile);
   ~PrefMetricsService() override;
 
+  // Records metrics about the state of the homepage on launch.  On all
+  // platforms except Android this will be triggered automatically as part of
+  // initializing the PrefMetricsService.
+  //
+  // On Android, determining whether the homepage is enabled requires waiting
+  // for a response from a third party provider installed on the device.  Thus,
+  // this functionality is exposed to allow it to be called once all the
+  // necessary information is available.

[gab, 2016/05/25 14:03:24]

This comment should state what this method does, not how it's called.

The caller that omits OS_ANDROID is a good place to document how it's done
differently on Android (ref. other comment about changing if-def semantics).

[Mark P, 2016/05/25 17:01:53]

I agree with gab@ on this one; you can move everything but the first sentence of
this comment into the implementation.

[Ted C, 2016/05/25 18:13:07]

I personally think the comment about this being triggered during initialization
on all ports except android is relevant in the header.  Then readers will
understand quickly whether this is something they should be worrying about (w/o
having to look into the code).  I agree reasoning why android is different is
not important here and should be/is already in the code.

But if you still disagree strongly then I'll remove it as I don't want that to
be the thing that holds this up.

[gab, 2016/05/25 18:23:40]

If you want to mention where it's triggered, that mention should be on the
constructor of PrefMetricsService (where the trigger is).

Also, if you're going to state that it's not triggered on Android, keeping the
comment of what else is done on Android alongside makes sense too I'd say?

[Ted C, 2016/05/25 18:28:31]

I can't say I agree, but I'll just remove it entirely.

+  static void RecordHomePageLaunchMetrics(bool show_home_button,
+                                          bool homepage_is_ntp,
+                                          const GURL& homepage_url);
+
   class Factory : public BrowserContextKeyedServiceFactory {
    public:
     static Factory* GetInstance();