Record NTP.LogoShownTime for timely refreshs only

components/doodle/doodle_service.cc

 // Copyright 2017 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "components/doodle/doodle_service.h"
 
 #include <utility>
 
 #include "base/bind.h"
 #include "base/memory/ptr_util.h"
@@ skipping 62 matching lines @@
 void DoodleService::Shutdown() {}
 
 void DoodleService::AddObserver(Observer* observer) {
   observers_.AddObserver(observer);
 }
 
 void DoodleService::RemoveObserver(Observer* observer) {
   observers_.RemoveObserver(observer);
 }
 
-void DoodleService::Refresh() {
+bool DoodleService::Refresh() {
   base::TimeTicks now_ticks = tick_clock_->NowTicks();
   // Check if we have passed the minimum refresh interval.
   base::TimeDelta time_since_fetch = now_ticks - last_successful_fetch_;
   if (time_since_fetch < min_refresh_interval_) {
     RecordDownloadMetrics(OUTCOME_REFRESH_INTERVAL_NOT_PASSED,
                           base::TimeDelta());
-    return;
+    return false;
   }
   fetcher_->FetchDoodle(base::BindOnce(&DoodleService::DoodleFetched,
                                        base::Unretained(this), now_ticks));
+  return true;
 }
 
 // static
 bool DoodleService::DownloadOutcomeIsSuccess(DownloadOutcome outcome) {
   switch (outcome) {
     case OUTCOME_NEW_DOODLE:
     case OUTCOME_REVALIDATED_DOODLE:
     case OUTCOME_CHANGED_DOODLE:
     case OUTCOME_NO_DOODLE:
       return true;
@@ skipping 129 matching lines @@
     pref_service_->ClearPref(prefs::kCachedConfigExpiry);
   }
 }
 
 void DoodleService::DoodleExpired() {
   DCHECK(cached_config_.has_value());
   HandleNewConfig(DoodleState::NO_DOODLE, base::TimeDelta(), base::nullopt);
 }
 
 }  // namespace doodle