OnDoodleConfigRevalidated

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 174 matching lines @@
 
   // Handle the case where the new config is already expired.
   bool expired = time_to_live <= base::TimeDelta();
   const base::Optional<DoodleConfig>& new_config =
       expired ? base::nullopt : doodle_config;
 
   // Determine the download outcome *before* updating the cached config.
   DownloadOutcome outcome =
       DetermineDownloadOutcome(cached_config_, new_config, state, expired);
 
-  // If the config changed, update our cache and notify observers.
   // Note that this checks both for existence changes as well as changes of the
   // configs themselves.
-  if (cached_config_ != new_config) {
+  if (cached_config_ == new_config) {
+    for (auto& observer : observers_) {
+      observer.OnDoodleConfigRevalidated();
+    }
+  } else {
+    // The config changed. Update our cache and the prefs, and notify observers.
+    cached_config_ = new_config;
+
     UpdateCachedConfig(time_to_live, new_config);
 
     for (auto& observer : observers_) {
       observer.OnDoodleConfigUpdated(cached_config_);
     }
   }
 
   // Even if the configs are identical, the time-to-live might have changed.
   // (Re-)schedule the cache expiry.
   if (cached_config_.has_value()) {
@@ skipping 24 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