Offline Pages: Replace Observer::OfflinePageModelChanged with OfflinePageAdded.

components/offline_pages/offline_page_model.h

Index: components/offline_pages/offline_page_model.h
diff --git a/components/offline_pages/offline_page_model.h b/components/offline_pages/offline_page_model.h
index eb058128a8374c0c27e110a244394c904aa7b6c2..9197db740a71992352a353481c86ad578cd503e8 100644
--- a/components/offline_pages/offline_page_model.h
+++ b/components/offline_pages/offline_page_model.h
@@ -54,9 +54,9 @@ class OfflinePageModel : public base::SupportsUserData {
     // Invoked when the model has finished loading.
     virtual void OfflinePageModelLoaded(OfflinePageModel* model) = 0;
 
-    // Invoked when the model is being updated, due to adding, removing or
-    // updating an offline page.
-    virtual void OfflinePageModelChanged(OfflinePageModel* model) = 0;
+    // Invoked when the model is being updated due to adding an offline page.
+    virtual void OfflinePageAdded(OfflinePageModel* model,

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

Might be not this patch, but please consider removing "*model" parameter. There
is no practical case for an observer to observe multiple page models, right?

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

Ack, will do later.

+                                  const OfflinePageItem& added_page) = 0;
 
     // Invoked when an offline copy related to |offline_id| was deleted.
     virtual void OfflinePageDeleted(int64_t offline_id,