Ntp: show AllDismissedItem when all sections have been dismissed.

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 <set>
 #include <utility>
@@ skipping 132 matching lines @@
   DCHECK(removed) << "The dismissed suggestion " << suggestion_id
                   << " has already been removed. Providers must not call"
                   << " OnNewSuggestions in response to DismissSuggestion.";
 }
 
 void ContentSuggestionsService::DismissCategory(Category category) {
   auto providers_it = providers_by_category_.find(category);
   if (providers_it == providers_by_category_.end())
     return;
 
+  dismissed_providers_by_category_[providers_it->first] = providers_it->second;
   providers_by_category_.erase(providers_it);
   categories_.erase(
       std::find(categories_.begin(), categories_.end(), category));
 }
 
+void ContentSuggestionsService::RestoreDismissedCategories() {
+  for (const auto& category_provider_pair : dismissed_providers_by_category_)

[Marc Treib, 2016/10/11 07:35:58]

Braces if the body doesn't fit on one line.

[Michael van Ouwerkerk, 2016/10/11 12:04:16]

Done.

+    RegisterCategoryIfRequired(category_provider_pair.second,
+                               category_provider_pair.first);
+  dismissed_providers_by_category_.clear();

[Marc Treib, 2016/10/11 07:35:58]

Should RegisterCategoryIfRequired remove the entry from
dismissed_providers_by_category_? That would make sure we don't have a provider
in both providers_by_category_ and dismissed_providers_by_category_. Right now
that'll happen if the status changes after the category has been dismissed.

[Michael van Ouwerkerk, 2016/10/11 12:04:16]

Done.

+}
+
 void ContentSuggestionsService::AddObserver(Observer* observer) {
   observers_.AddObserver(observer);
 }
 
 void ContentSuggestionsService::RemoveObserver(Observer* observer) {
   observers_.RemoveObserver(observer);
 }
 
 void ContentSuggestionsService::RegisterProvider(
     std::unique_ptr<ContentSuggestionsProvider> provider) {
@@ skipping 157 matching lines @@
           if (left.IsKnownCategory(KnownCategories::BOOKMARKS))
             return false;
           if (right.IsKnownCategory(KnownCategories::BOOKMARKS))
             return true;
         }
         return category_factory_.CompareCategories(left, right);
       });
 }
 
 }  // namespace ntp_snippets