[NTP Snippets] Don't notify about new suggestion when in a not-available state

components/ntp_snippets/remote/remote_suggestions_provider.cc

 // Copyright 2015 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/ntp_snippets/remote/remote_suggestions_provider.h"
 
 #include <algorithm>
 #include <iterator>
 #include <utility>
 
@@ skipping 467 matching lines @@
     return;
   }
   CategoryContent* content = &content_it->second;
   if (content->snippets.empty())
     return;
 
   database_->DeleteSnippets(GetSnippetIDVector(content->snippets));
   database_->DeleteImages(GetSnippetIDVector(content->snippets));
   content->snippets.clear();
 
-  NotifyNewSuggestions(category, *content);
+  if (IsCategoryStatusAvailable(content->status))
+    NotifyNewSuggestions(category, *content);
 }
 
 void RemoteSuggestionsProvider::GetDismissedSuggestionsForDebugging(
     Category category,
     const DismissedSuggestionsCallback& callback) {
   auto content_it = category_contents_.find(category);
   DCHECK(content_it != category_contents_.end());
   callback.Run(
       ConvertToContentSuggestions(category, content_it->second.dismissed));
 }
@@ skipping 535 matching lines @@
   // Note: Initializing the status service will run the callback right away with
   // the current state.
   snippets_status_service_->Init(
       base::Bind(&RemoteSuggestionsProvider::OnSnippetsStatusChanged,
                  base::Unretained(this)));
 
   // Always notify here even if we got nothing from the database, because we
   // don't know how long the fetch will take or if it will even complete.
   for (const auto& item : category_contents_) {
     Category category = item.first;
-    NotifyNewSuggestions(category, item.second);
+    const CategoryContent& content = item.second;
+    // Note: We might be in a non-available status here, e.g. DISABLED due to
+    // enterprise policy.
+    if (IsCategoryStatusAvailable(content.status))
+      NotifyNewSuggestions(category, content);
   }
 }
 
 void RemoteSuggestionsProvider::OnSnippetsStatusChanged(
     SnippetsStatus old_snippets_status,
     SnippetsStatus new_snippets_status) {
   switch (new_snippets_status) {
     case SnippetsStatus::ENABLED_AND_SIGNED_IN:
       if (old_snippets_status == SnippetsStatus::ENABLED_AND_SIGNED_OUT) {
         DCHECK(state_ == State::READY);
@@ skipping 225 matching lines @@
 RemoteSuggestionsProvider::CategoryContent::CategoryContent(CategoryContent&&) =
     default;
 
 RemoteSuggestionsProvider::CategoryContent::~CategoryContent() = default;
 
 RemoteSuggestionsProvider::CategoryContent&
 RemoteSuggestionsProvider::CategoryContent::operator=(CategoryContent&&) =
     default;
 
 }  // namespace ntp_snippets