Offline Pages: Replace Observer::OfflinePageModelChanged with OfflinePageAdded.

components/offline_pages/downloads/download_ui_adapter.cc

Index: components/offline_pages/downloads/download_ui_adapter.cc
diff --git a/components/offline_pages/downloads/download_ui_adapter.cc b/components/offline_pages/downloads/download_ui_adapter.cc
index bb398cb8f025d372546d0d669b1137e76a555acd..0f0a0564bd38e67c2190d96314e11f84abd1448c 100644
--- a/components/offline_pages/downloads/download_ui_adapter.cc
+++ b/components/offline_pages/downloads/download_ui_adapter.cc
@@ -80,7 +80,8 @@ void DownloadUIAdapter::OfflinePageModelLoaded(OfflinePageModel* model) {
   // This signal is not used here.
 }
 
-void DownloadUIAdapter::OfflinePageModelChanged(OfflinePageModel* model) {
+void DownloadUIAdapter::OfflinePageAdded(OfflinePageModel* model,
+                                         const OfflinePageItem& added_page) {
   DCHECK(model == model_);
   model_->GetAllPages(

[Dmitry Titov, 2016/11/15 20:02:30]

Now the impl of this should change to do a direct add instead of re-get all.
Also the OnOfflinePagesChanged should change (likely by being removed?)

[dewittj, 2016/11/15 23:54:19]

I had that in here before (see patchset #2), but in the interest of compact,
focused patches I decided to put it in an upcoming CL.

       base::Bind(&DownloadUIAdapter::OnOfflinePagesChanged,