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..95017667832012a8c58ae270a52a2a146585e05a 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 true if the refresh actually triggered a fetch. Returns false if
+  // refreshing was skipped (because the refresh interval hasn't passed yet).

[tschumann, 2017/04/20 18:53:11]

nit: i'd simplify the last sentence. what about:
Returns false if no fetch was issued because the service has fresh data in it's
cache.
 

[fhorschig, 2017/04/24 13:52:34]

Sounds good!

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