[Offline Pages] Filtering expired pages and fix consistency check.

components/offline_pages/offline_page_model_impl.h

Index: components/offline_pages/offline_page_model_impl.h
diff --git a/components/offline_pages/offline_page_model_impl.h b/components/offline_pages/offline_page_model_impl.h
index 34dc791d037ef563ea8491da5d7d75bb110309e2..940830f956aa11ebdb149160e53a403740dc95f2 100644
--- a/components/offline_pages/offline_page_model_impl.h
+++ b/components/offline_pages/offline_page_model_impl.h
@@ -196,9 +196,10 @@ class OfflinePageModelImpl : public OfflinePageModel, public KeyedService {
   // Callbacks for checking if offline pages are missing archive files.
   void ScanForMissingArchiveFiles(
       const std::set<base::FilePath>& archive_paths);
-  void OnRemoveOfflinePagesMissingArchiveFileDone(
-      const std::vector<std::pair<int64_t, ClientId>>& offline_client_id_pairs,
-      DeletePageResult result);
+  void OnDeleteHeadlessArchivesDone(const std::vector<base::FilePath>& archives,

[fgorski, 2016/06/07 22:46:15]

Please explain what "headless" means here and why you chose it. 

[romax, 2016/06/08 01:31:26]

added some explaination in comment..I'm choosing headless since i was always
considering page item and archive files like pointers and objects... page items
in metadata store are pointers and archives are objects...or linked list? I
think there's a similar term in linked list.
But TBH I'd like some suggestions for names. However I'd like to have some
'terms' for things we have here within our group which we know what they are
while they're not too long to write.

+                                    bool success);
+  void OnExpireArchivelessFilesDone(const std::vector<int64_t>& offline_ids,
+                                    bool success);
 
   // Steps for clearing all.
   void OnRemoveAllFilesDoneForClearAll(const base::Closure& callback,