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

Unified Diff: components/offline_pages/offline_page_storage_manager.h

Issue 1968563003: [Offline pages] Extracting callbacks from the Offline Page Model (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 7 months 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_storage_manager.h
diff --git a/components/offline_pages/offline_page_storage_manager.h b/components/offline_pages/offline_page_storage_manager.h
index 73901de71c233cc7990dbcdea6178727e06f2d89..884ddd391a79675acd8a52a13ef4025994b3d574 100644
--- a/components/offline_pages/offline_page_storage_manager.h
+++ b/components/offline_pages/offline_page_storage_manager.h
@@ -36,7 +36,7 @@ class OfflinePageStorageManager {
// Callback used when calling ClearPagesIfNeeded.
// int: the number of deleted pages.
// DeletePageResult: result of deleting pages.
- typedef base::Callback<void(int, OfflinePageModel::DeletePageResult)>
+ typedef base::Callback<void(int, DeletePageResult)>
ClearPageCallback;
explicit OfflinePageStorageManager(OfflinePageModel* model);
@@ -52,17 +52,17 @@ class OfflinePageStorageManager {
// to |GetAllPages|.
void ClearExpiredPages(
const ClearPageCallback& callback,
- const OfflinePageModel::MultipleOfflinePageItemResult& pages);
+ const MultipleOfflinePageItemResult& pages);
// Gets offline IDs of all expired pages and return in |offline_ids|.
void GetExpiredPageIds(
- const OfflinePageModel::MultipleOfflinePageItemResult& pages,
+ const MultipleOfflinePageItemResult& pages,
std::vector<int64_t>& offline_ids);
// Callback when expired pages has been deleted.
void OnExpiredPagesDeleted(const ClearPageCallback& callback,
int pages_to_clear,
- OfflinePageModel::DeletePageResult result);
+ DeletePageResult result);
// Determine if manager should clear pages.
bool ShouldClearPages();

Powered by Google App Engine
This is Rietveld 408576698