[Offline pages] Updating the UpdateCallback in OPMStoreSQL

components/offline_pages/offline_page_metadata_store.h

Index: components/offline_pages/offline_page_metadata_store.h
diff --git a/components/offline_pages/offline_page_metadata_store.h b/components/offline_pages/offline_page_metadata_store.h
index 63a35e464bdcd448da68500251c49e7db293a62a..33a17aaba6c209d8630638625200bbcef7bb9c7d 100644
--- a/components/offline_pages/offline_page_metadata_store.h
+++ b/components/offline_pages/offline_page_metadata_store.h
@@ -7,6 +7,7 @@
 
 #include <stdint.h>
 
+#include <map>
 #include <vector>
 
 #include "base/callback.h"
@@ -57,7 +58,9 @@ class OfflinePageMetadataStore {
   typedef base::Callback<void(LoadStatus, const std::vector<OfflinePageItem>&)>
       LoadCallback;
   typedef base::Callback<void(ItemActionStatus)> AddCallback;
-  typedef base::Callback<void(bool)> UpdateCallback;
+  typedef base::Callback<void(const std::map<int64_t, ItemActionStatus>&,

[dougarnett, 2016/09/15 19:57:24]

perhaps in the future, if we never end up using SUCCESS status code,
ItemActionStatus could become FailureStatus or similar and then this typedef
would be a bit more self documenting.

[dougarnett, 2016/09/15 19:57:24]

nit - seems like successful collection should come first and then failure
collections (but realize this may be easier iteration to keep same order).

[fgorski, 2016/09/19 23:24:29]

Done.(Callback redesigned in patch 1)

[fgorski, 2016/09/19 23:24:29]

Done. (Callback redesigned in patch 1)

+                              const std::vector<OfflinePageItem>&)>
+      UpdateCallback;
   typedef base::Callback<void(bool)> ResetCallback;
 
   OfflinePageMetadataStore();