[Offline pages] Adding capability to free up space used by Offline pages

components/offline_pages/offline_page_model.cc

Index: components/offline_pages/offline_page_model.cc
diff --git a/components/offline_pages/offline_page_model.cc b/components/offline_pages/offline_page_model.cc
index e89ae2add15ad730c64a7120bfad6fc60eb214cc..f58bceec427b57d37428701874c4b1a394f1754c 100644
--- a/components/offline_pages/offline_page_model.cc
+++ b/components/offline_pages/offline_page_model.cc
@@ -22,6 +22,8 @@ namespace offline_pages {
 
 namespace {
 
+const base::TimeDelta kPageCleanUpThreshold = base::TimeDelta::FromDays(30);

[jianli, 2015/08/21 23:39:21]

nit: add comment like: pages which are created 30 days earlier will be deleted
in order to free up space.

[fgorski, 2015/08/24 18:47:14]

Done.

+
 SavePageResult ToSavePageResult(ArchiverResult archiver_result) {
   SavePageResult result;
   switch (archiver_result) {
@@ -141,6 +143,17 @@ const std::vector<OfflinePageItem> OfflinePageModel::GetAllPages() const {
   return offline_pages;
 }
 
+const std::vector<OfflinePageItem> OfflinePageModel::GetPagesToCleanUp() const {

[jianli, 2015/08/21 23:39:21]

add test

[fgorski, 2015/08/24 18:47:14]

Done.

+  DCHECK(is_loaded_);
+  std::vector<OfflinePageItem> offline_pages;
+  base::Time now = base::Time::Now();
+  for (const auto& id_page_pair : offline_pages_) {
+    if (now - id_page_pair.second.creation_time > kPageCleanUpThreshold)
+      offline_pages.push_back(id_page_pair.second);
+  }
+  return offline_pages;
+}
+
 bool OfflinePageModel::GetPageByBookmarkId(
     int64 bookmark_id,
     OfflinePageItem* offline_page) const {