Offline Pages: Replace Observer::OfflinePageModelChanged with OfflinePageAdded.

components/ntp_snippets/offline_pages/offline_page_proxy.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/offline_pages/offline_page_proxy.h"
 
 #include "base/bind.h"
 
+using offline_pages::MultipleOfflinePageItemCallback;
 using offline_pages::MultipleOfflinePageItemResult;
-using offline_pages::MultipleOfflinePageItemCallback;
+using offline_pages::OfflinePageItem;
 using offline_pages::OfflinePageModel;
+using SavePageResult = offline_pages::SavePageResult;
 
 namespace ntp_snippets {
 
 OfflinePageProxy::OfflinePageProxy(OfflinePageModel* offline_page_model)
     : offline_page_model_(offline_page_model), weak_ptr_factory_(this) {
   offline_page_model_->AddObserver(this);
 }
 
 void OfflinePageProxy::GetAllPages(
     const MultipleOfflinePageItemCallback& callback) {
@@ skipping 12 matching lines @@
 // Private methods
 
 OfflinePageProxy::~OfflinePageProxy() {
   offline_page_model_->RemoveObserver(this);
 }
 
 void OfflinePageProxy::OfflinePageModelLoaded(OfflinePageModel* model) {
   DCHECK_EQ(offline_page_model_, model);
 }
 
-void OfflinePageProxy::OfflinePageModelChanged(OfflinePageModel* model) {
+void OfflinePageProxy::OfflinePageAdded(OfflinePageModel* model,

[fgorski, 2016/11/02 15:51:06]

Please ping the owners of this class, as they may no longer need to fetch
offline pages, as they are getting direct access to a newly added item.

[dewittj, 2016/11/02 16:34:37]

That's the plan

+                                        SavePageResult result,
+                                        const OfflinePageItem* item) {
   DCHECK_EQ(offline_page_model_, model);
   FetchOfflinePagesAndNotify();
 }
 
 void OfflinePageProxy::OfflinePageDeleted(
     int64_t offline_id,
     const offline_pages::ClientId& client_id) {
   for (Observer& observer : observers_)
     observer.OfflinePageDeleted(offline_id, client_id);
 }
 
 void OfflinePageProxy::FetchOfflinePagesAndNotify() {
   offline_page_model_->GetAllPages(base::Bind(
       &OfflinePageProxy::OnOfflinePagesLoaded, weak_ptr_factory_.GetWeakPtr()));
 }
 
 void OfflinePageProxy::OnOfflinePagesLoaded(
     const MultipleOfflinePageItemResult& result) {
   for (Observer& observer : observers_)
     observer.OfflinePageModelChanged(result);
 }
 
 }  // namespace ntp_snippets