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

Unified Diff: chrome/browser/ui/webui/quota_internals_proxy.cc

Issue 8070001: Use base::Callback in Quota related code. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase 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/ui/webui/quota_internals_proxy.cc
diff --git a/chrome/browser/ui/webui/quota_internals_proxy.cc b/chrome/browser/ui/webui/quota_internals_proxy.cc
index 322c0566645dc8377ac78631a645613c9dac1bbb..e217d376f5daf7dc343d8383a298a6311496fb38 100644
--- a/chrome/browser/ui/webui/quota_internals_proxy.cc
+++ b/chrome/browser/ui/webui/quota_internals_proxy.cc
@@ -16,7 +16,7 @@ namespace quota_internals {
QuotaInternalsProxy::QuotaInternalsProxy(QuotaInternalsHandler* handler)
: handler_(handler),
- callback_factory_(ALLOW_THIS_IN_INITIALIZER_LIST(this)) {
+ weak_factory_(ALLOW_THIS_IN_INITIALIZER_LIST(this)) {
}
QuotaInternalsProxy::~QuotaInternalsProxy() {}
@@ -55,30 +55,30 @@ void QuotaInternalsProxy::RequestInfo(
quota_manager_ = quota_manager;
quota_manager_->GetAvailableSpace(
- callback_factory_.NewCallback(
- &QuotaInternalsProxy::DidGetAvailableSpace));
+ base::Bind(&QuotaInternalsProxy::DidGetAvailableSpace,
+ weak_factory_.GetWeakPtr()));
quota_manager_->GetTemporaryGlobalQuota(
- callback_factory_.NewCallback(
- &QuotaInternalsProxy::DidGetGlobalQuota));
+ base::Bind(&QuotaInternalsProxy::DidGetGlobalQuota,
+ weak_factory_.GetWeakPtr()));
quota_manager_->GetGlobalUsage(
quota::kStorageTypeTemporary,
- callback_factory_.NewCallback(
- &QuotaInternalsProxy::DidGetGlobalUsage));
+ base::Bind(&QuotaInternalsProxy::DidGetGlobalUsage,
+ weak_factory_.GetWeakPtr()));
quota_manager_->GetGlobalUsage(
quota::kStorageTypePersistent,
- callback_factory_.NewCallback(
- &QuotaInternalsProxy::DidGetGlobalUsage));
+ base::Bind(&QuotaInternalsProxy::DidGetGlobalUsage,
+ weak_factory_.GetWeakPtr()));
quota_manager_->DumpQuotaTable(
- callback_factory_.NewCallback(
- &QuotaInternalsProxy::DidDumpQuotaTable));
+ base::Bind(&QuotaInternalsProxy::DidDumpQuotaTable,
+ weak_factory_.GetWeakPtr()));
quota_manager_->DumpOriginInfoTable(
- callback_factory_.NewCallback(
- &QuotaInternalsProxy::DidDumpOriginInfoTable));
+ base::Bind(&QuotaInternalsProxy::DidDumpOriginInfoTable,
+ weak_factory_.GetWeakPtr()));
std::map<std::string, std::string> stats;
quota_manager_->GetStatistics(&stats);
@@ -180,8 +180,8 @@ void QuotaInternalsProxy::GetHostUsage(const std::string& host,
DCHECK(quota_manager_);
quota_manager_->GetHostUsage(
host, type,
- callback_factory_.NewCallback(
- &QuotaInternalsProxy::DidGetHostUsage));
+ base::Bind(&QuotaInternalsProxy::DidGetHostUsage,
+ weak_factory_.GetWeakPtr()));
}
void QuotaInternalsProxy::RequestPerOriginInfo(quota::StorageType type) {
« no previous file with comments | « chrome/browser/ui/webui/quota_internals_proxy.h ('k') | content/browser/file_system/file_system_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698