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

Unified Diff: components/offline_pages/offline_page_storage_manager.cc

Issue 2015793002: [Offline Pages] Linking storage manager and model with UMAs. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@wisp
Patch Set: Adding more UMAs and hooking up model and storage manager. 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.cc
diff --git a/components/offline_pages/offline_page_storage_manager.cc b/components/offline_pages/offline_page_storage_manager.cc
index c12e8c0b1b8977fce2536e331c189feae3c758fc..0c46ff164a0ac842619a5da3fe34e19dd5121374 100644
--- a/components/offline_pages/offline_page_storage_manager.cc
+++ b/components/offline_pages/offline_page_storage_manager.cc
@@ -28,7 +28,7 @@ OfflinePageStorageManager::OfflinePageStorageManager(
OfflinePageStorageManager::~OfflinePageStorageManager() {}
void OfflinePageStorageManager::ClearPagesIfNeeded(
- const ClearPagesCallback& callback) {
+ const ClearStorageCallback& callback) {
if (IsInProgress())
return;
clear_time_ = clock_->Now();
@@ -43,7 +43,7 @@ void OfflinePageStorageManager::SetClockForTesting(
}
void OfflinePageStorageManager::OnGetStorageStatsDoneForClearingPages(
- const ClearPagesCallback& callback,
+ const ClearStorageCallback& callback,
const ArchiveManager::StorageStats& stats) {
DCHECK(IsInProgress());
ClearMode mode = ShouldClearPages(stats);
@@ -58,7 +58,7 @@ void OfflinePageStorageManager::OnGetStorageStatsDoneForClearingPages(
}
void OfflinePageStorageManager::OnGetAllPagesDoneForClearingPages(
- const ClearPagesCallback& callback,
+ const ClearStorageCallback& callback,
const ArchiveManager::StorageStats& stats,
const MultipleOfflinePageItemResult& pages) {
std::vector<int64_t> page_ids_to_expire;
@@ -72,7 +72,7 @@ void OfflinePageStorageManager::OnGetAllPagesDoneForClearingPages(
}
void OfflinePageStorageManager::OnPagesExpired(
- const ClearPagesCallback& callback,
+ const ClearStorageCallback& callback,
size_t pages_expired,
const std::vector<int64_t>& page_ids_to_remove,
bool expiration_succeeded) {
@@ -86,7 +86,7 @@ void OfflinePageStorageManager::OnPagesExpired(
}
void OfflinePageStorageManager::OnOutdatedPagesCleared(
- const ClearPagesCallback& callback,
+ const ClearStorageCallback& callback,
size_t pages_cleared,
bool expiration_succeeded,
DeletePageResult result) {

Powered by Google App Engine
This is Rietveld 408576698