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

Unified Diff: components/offline_pages/core/offline_page_model_impl.cc

Issue 2564163002: [Offline Pages] Remove load state from public OfflinePageModel API. (Closed)
Patch Set: Fix broken tests. Created 4 years 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/core/offline_page_model_impl.cc
diff --git a/components/offline_pages/core/offline_page_model_impl.cc b/components/offline_pages/core/offline_page_model_impl.cc
index 011dfbe2a1670cf224c0f9d3590e227b9ee7c57f..7466f1f145a505eccddf8711d17fc54f86ee8cf0 100644
--- a/components/offline_pages/core/offline_page_model_impl.cc
+++ b/components/offline_pages/core/offline_page_model_impl.cc
@@ -665,10 +665,6 @@ OfflinePageStorageManager* OfflinePageModelImpl::GetStorageManager() {
return storage_manager_.get();
}
-bool OfflinePageModelImpl::is_loaded() const {
- return is_loaded_;
-}
-
OfflineEventLogger* OfflinePageModelImpl::GetLogger() {
return &offline_event_logger_;
}
@@ -831,10 +827,6 @@ void OfflinePageModelImpl::FinalizeModelLoad() {
// All actions below are meant to be taken regardless of successful load of
// the store.
- // Inform observers the load is done.
- for (Observer& observer : observers_)
- observer.OfflinePageModelLoaded(this);
-
// Run all the delayed tasks.
for (const auto& delayed_task : delayed_tasks_)
delayed_task.Run();

Powered by Google App Engine
This is Rietveld 408576698