Support action button to fetch more content suggestions

components/ntp_snippets/content_suggestions_service.cc

 // Copyright 2016 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/content_suggestions_service.h"
 
 #include <algorithm>
 #include <iterator>
 
 #include "base/bind.h"
@@ skipping 169 matching lines @@
 // Private methods
 
 void ContentSuggestionsService::OnNewSuggestions(
     ContentSuggestionsProvider* provider,
     Category category,
     std::vector<ContentSuggestion> new_suggestions) {
   if (RegisterCategoryIfRequired(provider, category)) {
     NotifyCategoryStatusChanged(category);
   }
 
+  if (!IsCategoryStatusAvailable(provider->GetCategoryStatus(category))) {

[Bernhard Bauer, 2016/08/12 09:03:09]

Nit: The if-statement above uses braces for a single-line body as well, but
other places in this file don't. It would be good to be consistent (in either
direction).

[Philipp Keck, 2016/08/12 09:16:42]

Removed them here and just above and just below.

+    return;
+  }
+
   for (const ContentSuggestion& suggestion :
        suggestions_by_category_[category]) {
     id_category_map_.erase(suggestion.id());
   }
 
   for (const ContentSuggestion& suggestion : new_suggestions) {
     id_category_map_.insert(std::make_pair(suggestion.id(), category));
   }
 
   suggestions_by_category_[category] = std::move(new_suggestions);
@@ skipping 48 matching lines @@
   return true;
 }
 
 void ContentSuggestionsService::NotifyCategoryStatusChanged(Category category) {
   FOR_EACH_OBSERVER(
       Observer, observers_,
       OnCategoryStatusChanged(category, GetCategoryStatus(category)));
 }
 
 }  // namespace ntp_snippets