📰 The Status card reports disabled sync states

components/ntp_snippets/ntp_snippets_service.cc

Index: components/ntp_snippets/ntp_snippets_service.cc
diff --git a/components/ntp_snippets/ntp_snippets_service.cc b/components/ntp_snippets/ntp_snippets_service.cc
index 040c1c82c926f186caa88dfb4c7ff668ed2a6d8f..df529bb9686185623e26cccc3693fc2bae320d3e 100644
--- a/components/ntp_snippets/ntp_snippets_service.cc
+++ b/components/ntp_snippets/ntp_snippets_service.cc
@@ -658,13 +658,6 @@ void NTPSnippetsService::FinishInitialization() {
 
   FOR_EACH_OBSERVER(NTPSnippetsServiceObserver, observers_,
                     NTPSnippetsServiceLoaded());
-
-  // Start a fetch if we don't have any snippets yet, or a fetch was requested
-  // earlier.
-  if (ready() && (snippets_.empty() || fetch_after_load_)) {

[dgn, 2016/06/13 17:27:15]

redundant, if we transition to ready, we have the code to start a fetch in
EnterState(). Updated the code there to address empty snippets instead of this
bit.

-    fetch_after_load_ = false;
-    FetchSnippets();
-  }
 }
 
 NTPSnippetsService::State NTPSnippetsService::GetStateForDependenciesStatus() {
@@ -705,7 +698,7 @@ void NTPSnippetsService::EnterState(State state) {
 
       // If the service was previously disabled, we will need to start a fetch
       // because otherwise there won't be any.
-      bool fetch_snippets = state_ == State::DISABLED || fetch_after_load_;
+      bool fetch_snippets = snippets_.empty() || fetch_after_load_;
       DVLOG(1) << "Entering state: READY";
       state_ = State::READY;
       fetch_after_load_ = false;