Update provider on Reading List model update

components/ntp_snippets/reading_list/reading_list_suggestions_provider.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/ntp_snippets/reading_list/reading_list_suggestions_provider.h"
 
 #include <algorithm>
 #include <vector>
 
 #include "base/bind.h"
@@ skipping 109 matching lines @@
   FetchReadingListInternal();
 }
 
 void ReadingListSuggestionsProvider::ReadingListModelBeingDeleted(
     const ReadingListModel* model) {
   DCHECK(model == reading_list_model_);
   scoped_observer_.Remove(reading_list_model_);
   reading_list_model_ = nullptr;
 }
 
+void ReadingListSuggestionsProvider::ReadingListDidApplyChanges(
+    ReadingListModel* model) {
+  DCHECK(model == reading_list_model_);
+
+  FetchReadingListInternal();
+}
+
+void ReadingListSuggestionsProvider::ReadingListModelCompletedBatchUpdates(
+    const ReadingListModel* model) {
+  DCHECK(model == reading_list_model_);
+
+  FetchReadingListInternal();
+}
+
 void ReadingListSuggestionsProvider::FetchReadingListInternal() {
   if (!reading_list_model_)
     return;
 
+  if (reading_list_model_->IsPerformingBatchUpdates())

[Olivier, 2017/04/03 13:13:53]

super nit: group with the if above.

[Marc Treib, 2017/04/04 16:57:49]

nit: Braces please, also above

[gambard, 2017/04/05 08:08:55]

Done.

[gambard, 2017/04/05 08:08:55]

Done.

+    return;
+
   DCHECK(reading_list_model_->loaded());
   std::vector<const ReadingListEntry*> entries;
   for (const GURL& url : reading_list_model_->Keys()) {
     const ReadingListEntry* entry = reading_list_model_->GetEntryByURL(url);
     if (!entry->IsRead()) {
       entries.emplace_back(entry);
     }
   }
 
   if (entries.size() > kMaxEntries) {
@@ skipping 41 matching lines @@
 void ReadingListSuggestionsProvider::NotifyStatusChanged(
     CategoryStatus new_status) {
   if (category_status_ == new_status) {
     return;
   }
   category_status_ = new_status;
   observer()->OnCategoryStatusChanged(this, provided_category_, new_status);
 }
 
 }  // namespace ntp_snippets