[Offline pages] Adding expiration capability to OfflinePageModel

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 d5fe0fb782649a99befb40dc2d0806e089dc813c..4145c6c92b44666ecd42e56839f5dc2e405f01f4 100644
--- a/components/offline_pages/offline_page_model.h
+++ b/components/offline_pages/offline_page_model.h
@@ -215,6 +215,11 @@ class OfflinePageModel : public KeyedService,
   // observers.
   void CheckForExternalFileDeletion();
 
+  // Marks pages as expired and removes their respective files from the archive
+  // directory.
+  void ExpirePages(const std::vector<int64_t>& offline_ids,
+                   base::Time expiration_time);

[jianli, 2016/05/19 20:39:54]

nit: const &

[fgorski, 2016/05/20 17:54:23]

Done.

+
   // Returns the policy controller.
   ClientPolicyController* GetPolicyController();
 
@@ -329,6 +334,11 @@ class OfflinePageModel : public KeyedService,
   void DoDeletePagesByURLPredicate(const UrlPredicate& predicate,
                                    const DeletePageCallback& callback);
 
+  // Callback completing page expiration.
+  void OnExpirePageDone(int64_t offline_id,
+                        base::Time expiration_time,

[jianli, 2016/05/19 20:39:54]

nit: ditto

[fgorski, 2016/05/20 17:54:23]

Done.

+                        bool success);
+
   void RunWhenLoaded(const base::Closure& job);
 
   // Persistent store for offline page metadata.