Implement storage manager for offline pages so that it can purge expired pages automatically. https://docs.google.com/a/google.com/document/d/16_7d4a4OpwV8o16CZxUsBt767P7FqpB0k6mJXRznfBo/edit?usp=sharing
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/ff3a7d889cd3326bc5f080cfd7d7663c2abdd604 commit ff3a7d889cd3326bc5f080cfd7d7663c2abdd604 Author: romax <romax@chromium.org> Date: Mon May 09 22:50:00 2016 [Offline Pages] Implement OfflinePageStorageManager. OfflinePageStorageManager should be responsible for making offline page model to clear expired offline pages automatically. BUG= 609357 Review-Url: https://codereview.chromium.org/1947323002 Cr-Commit-Position: refs/heads/master@{#392455} [modify] https://crrev.com/ff3a7d889cd3326bc5f080cfd7d7663c2abdd604/components/offline_pages.gypi [modify] https://crrev.com/ff3a7d889cd3326bc5f080cfd7d7663c2abdd604/components/offline_pages/BUILD.gn [modify] https://crrev.com/ff3a7d889cd3326bc5f080cfd7d7663c2abdd604/components/offline_pages/offline_page_model.cc [modify] https://crrev.com/ff3a7d889cd3326bc5f080cfd7d7663c2abdd604/components/offline_pages/offline_page_model.h [modify] https://crrev.com/ff3a7d889cd3326bc5f080cfd7d7663c2abdd604/components/offline_pages/offline_page_model_unittest.cc [add] https://crrev.com/ff3a7d889cd3326bc5f080cfd7d7663c2abdd604/components/offline_pages/offline_page_storage_manager.cc [add] https://crrev.com/ff3a7d889cd3326bc5f080cfd7d7663c2abdd604/components/offline_pages/offline_page_storage_manager.h [add] https://crrev.com/ff3a7d889cd3326bc5f080cfd7d7663c2abdd604/components/offline_pages/offline_page_storage_manager_unittest.cc
Reopening, having more related CLs.
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/44fdf4d230beaab374b3cb6d442e0d2fbeb14366 commit 44fdf4d230beaab374b3cb6d442e0d2fbeb14366 Author: romax <romax@chromium.org> Date: Thu May 12 02:31:19 2016 [Offline Pages] Introducing StorageManagerClient interface. BUG= 609357 Review-Url: https://codereview.chromium.org/1965633002 Cr-Commit-Position: refs/heads/master@{#393153} [modify] https://crrev.com/44fdf4d230beaab374b3cb6d442e0d2fbeb14366/components/offline_pages/offline_page_model.cc [modify] https://crrev.com/44fdf4d230beaab374b3cb6d442e0d2fbeb14366/components/offline_pages/offline_page_model.h [modify] https://crrev.com/44fdf4d230beaab374b3cb6d442e0d2fbeb14366/components/offline_pages/offline_page_storage_manager.cc [modify] https://crrev.com/44fdf4d230beaab374b3cb6d442e0d2fbeb14366/components/offline_pages/offline_page_storage_manager.h [modify] https://crrev.com/44fdf4d230beaab374b3cb6d442e0d2fbeb14366/components/offline_pages/offline_page_storage_manager_unittest.cc [modify] https://crrev.com/44fdf4d230beaab374b3cb6d442e0d2fbeb14366/components/offline_pages/offline_page_types.h
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/13aa507d5e0488ebbbe5e4f8afd0942e5435689b commit 13aa507d5e0488ebbbe5e4f8afd0942e5435689b Author: romax <romax@chromium.org> Date: Mon May 23 19:17:20 2016 [Offline Pages] Updated clearing logic in storage manager. Updated clearing logic in storage manager, adapted to new changes from archive manager. Also removed the storage manager related test in model_unittest, while adding some tests in storage_manager_unittest. BUG= 609357 Review-Url: https://codereview.chromium.org/1986673002 Cr-Commit-Position: refs/heads/master@{#395384} [modify] https://crrev.com/13aa507d5e0488ebbbe5e4f8afd0942e5435689b/components/offline_pages/archive_manager.cc [modify] https://crrev.com/13aa507d5e0488ebbbe5e4f8afd0942e5435689b/components/offline_pages/archive_manager.h [modify] https://crrev.com/13aa507d5e0488ebbbe5e4f8afd0942e5435689b/components/offline_pages/offline_page_model.cc [modify] https://crrev.com/13aa507d5e0488ebbbe5e4f8afd0942e5435689b/components/offline_pages/offline_page_storage_manager.cc [modify] https://crrev.com/13aa507d5e0488ebbbe5e4f8afd0942e5435689b/components/offline_pages/offline_page_storage_manager.h [modify] https://crrev.com/13aa507d5e0488ebbbe5e4f8afd0942e5435689b/components/offline_pages/offline_page_storage_manager_unittest.cc
Comment 1 by romax@chromium.org
, May 5 2016