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

Unified Diff: chrome/browser/browsing_data_remover.cc

Issue 8070001: Use base::Callback in Quota related code. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: pass as cref, and fix style. Created 9 years, 2 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: chrome/browser/browsing_data_remover.cc
diff --git a/chrome/browser/browsing_data_remover.cc b/chrome/browser/browsing_data_remover.cc
index 3a6f91d0739e84a07ae62b816fb93584013d6f6e..3d9df64da3a638f79d497dde7b064a1cdc6acd60 100644
--- a/chrome/browser/browsing_data_remover.cc
+++ b/chrome/browser/browsing_data_remover.cc
@@ -488,8 +488,9 @@ void BrowsingDataRemover::ClearQuotaManagedDataOnIOThread() {
// timeframe, and deal with the resulting set in
// OnGotPersistentQuotaManagedOrigins.
profile_->GetQuotaManager()->GetOriginsModifiedSince(
- quota::kStorageTypePersistent, delete_begin_, NewCallback(this,
- &BrowsingDataRemover::OnGotQuotaManagedOrigins));
+ quota::kStorageTypePersistent, delete_begin_,
+ base::Bind(&BrowsingDataRemover::OnGotQuotaManagedOrigins,
+ base::Unretained(this)));
} else {
// Otherwise, we don't need to deal with persistent storage.
--quota_managed_storage_types_to_delete_count_;
@@ -498,8 +499,9 @@ void BrowsingDataRemover::ClearQuotaManagedDataOnIOThread() {
// Do the same for temporary quota, regardless, passing the resulting set into
// OnGotTemporaryQuotaManagedOrigins.
profile_->GetQuotaManager()->GetOriginsModifiedSince(
- quota::kStorageTypeTemporary, delete_begin_, NewCallback(this,
- &BrowsingDataRemover::OnGotQuotaManagedOrigins));
+ quota::kStorageTypeTemporary, delete_begin_,
+ base::Bind(&BrowsingDataRemover::OnGotQuotaManagedOrigins,
+ base::Unretained(this)));
}
void BrowsingDataRemover::OnGotQuotaManagedOrigins(
@@ -512,9 +514,10 @@ void BrowsingDataRemover::OnGotQuotaManagedOrigins(
if (special_storage_policy_->IsStorageProtected(origin->GetOrigin()))
continue;
++quota_managed_origins_to_delete_count_;
- quota_manager_->DeleteOriginData(origin->GetOrigin(),
- type, NewCallback(this,
- &BrowsingDataRemover::OnQuotaManagedOriginDeletion));
+ quota_manager_->DeleteOriginData(
+ origin->GetOrigin(), type,
+ base::Bind(&BrowsingDataRemover::OnQuotaManagedOriginDeletion,
+ base::Unretained(this)));
}
--quota_managed_storage_types_to_delete_count_;

Powered by Google App Engine
This is Rietveld 408576698