Record NTP.LogoShownTime for timely refreshs only

components/doodle/doodle_service.h

Index: components/doodle/doodle_service.h
diff --git a/components/doodle/doodle_service.h b/components/doodle/doodle_service.h
index e4a9792a99149bad9e08cf054aad14232881ae96..705dbfafb63d20e4974713b8e6a856147d3da5b5 100644
--- a/components/doodle/doodle_service.h
+++ b/components/doodle/doodle_service.h
@@ -59,7 +59,9 @@ class DoodleService : public KeyedService {
   // Requests an asynchronous refresh of the DoodleConfig from the network.
   // After the update completes, the observers will be notified only if the
   // config changed.
-  void Refresh();
+  // Returns false if no fetch was issued because the service has fresh data in

[Marc Treib, 2017/04/26 10:17:16]

nit: No newline. Also, maybe just "Returns whether a network request was
actually sent."?

[fhorschig, 2017/04/27 10:31:44]

Hm? There is no unnecessary newline, or am I missing something?
"Returns true when a network request was actually sent."?
I want to be explicit about when it's true.

[Marc Treib, 2017/04/27 11:15:50]

I meant that there's a line break in the comment that seems unnecessary to me.
Not a big deal.
s/when/if/, otherwise okay. (I wanted to cover both cases, but this is fine.)

[fhorschig, 2017/04/27 11:49:20]

Done.

+  // it's cache.
+  bool Refresh();
 
  private:
   // Note: Keep in sync with the corresponding enum in histograms.xml. Never