Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(395)

Unified Diff: components/offline_pages/offline_page_model_impl.h

Issue 2497703002: [Offline pages] Resetting offline page metadata store to handle LOAD/INIT failures (Closed)
Patch Set: Fixing newly added test Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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 d5a4a7d954915dbfa544a3e5d86f08b9f2b0d5b1..467efe49af3535753a8d8983d529cd98c8a54cc4 100644
--- a/components/offline_pages/offline_page_model_impl.h
+++ b/components/offline_pages/offline_page_model_impl.h
@@ -148,9 +148,16 @@ class OfflinePageModelImpl : public OfflinePageModel, public KeyedService {
void CheckMetadataConsistency();
// Callback for loading pages from the offline page metadata store.
- void OnLoadDone(const base::TimeTicks& start_time,
- OfflinePageMetadataStore::LoadStatus load_status,
- const std::vector<OfflinePageItem>& offline_pages);
+ void OnStoreInitialized(const base::TimeTicks& start_time,
+ int reset_attempts_left,
+ bool success);
+ void OnStoreResetDone(const base::TimeTicks& start_time,
+ int reset_attempts_left,
+ bool success);
+ void OnInitialGetOfflinePagesDone(
+ const base::TimeTicks& start_time,
+ const std::vector<OfflinePageItem>& offline_pages);
+ void FinalizeModelLoad();
// Steps for saving a page offline.
void OnCreateArchiveDone(const SavePageParams& save_page_params,
@@ -239,7 +246,7 @@ class OfflinePageModelImpl : public OfflinePageModel, public KeyedService {
OfflinePageStorageManager::ClearStorageResult result);
// Post task to clear storage.
- void PostClearStorageIfNeededTask();
+ void PostClearStorageIfNeededTask(bool delayed);
// Check if |offline_page| should be removed on cache reset by user.
bool IsRemovedOnCacheReset(const OfflinePageItem& offline_page) const;
« no previous file with comments | « components/offline_pages/offline_page_metadata_store_sql.cc ('k') | components/offline_pages/offline_page_model_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698