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 98 matching lines @@
     const DismissedSuggestionsCallback& callback) {
   // TODO(crbug.com/702241): Implement this method.
 }
 void ReadingListSuggestionsProvider::ClearDismissedSuggestionsForDebugging(
     Category category) {
   // TODO(crbug.com/702241): Implement this method.
 }
 
 void ReadingListSuggestionsProvider::ReadingListModelLoaded(
     const ReadingListModel* model) {
   DCHECK(model == reading_list_model_);

[Olivier, 2017/04/03 11:18:07]

if (model->IsPerformingBatchUpdates())
     return;

or add it in FetchReadingListInternal

[gambard, 2017/04/03 11:25:07]

Done.

   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_);
+
+  if (model->IsPerformingBatchUpdates())
+    return;
+
+  FetchReadingListInternal();
+}
+
+void ReadingListSuggestionsProvider::ReadingListModelCompletedBatchUpdates(
+    const ReadingListModel* model) {
+  DCHECK(model == reading_list_model_);
+
+  FetchReadingListInternal();
+}
+
 void ReadingListSuggestionsProvider::FetchReadingListInternal() {
   if (!reading_list_model_)
     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);
@@ skipping 45 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