Cleanup: quota::HostQuotaCallback do not need to pass host and type as arguments

chrome/browser/browsing_data/browsing_data_quota_helper_impl.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/browsing_data/browsing_data_quota_helper_impl.h"
 
 #include <map>
 #include <set>
 
 #include "base/bind.h"
@@ skipping 31 matching lines @@
 void BrowsingDataQuotaHelperImpl::RevokeHostQuota(const std::string& host) {
   if (!io_thread_->BelongsToCurrentThread()) {
     io_thread_->PostTask(
         FROM_HERE,
         base::Bind(&BrowsingDataQuotaHelperImpl::RevokeHostQuota, this, host));
     return;
   }
 
   quota_manager_->SetPersistentHostQuota(
       host, 0,
       base::Bind(&BrowsingDataQuotaHelperImpl::DidRevokeHostQuota,

[kinuko, 2012/09/18 12:03:56]

DidRevokeHostQuota doesn't seem to check host or type. It looks like we can
remove the args from DidRevokeHostQuota and we do not need to bind them for it.

[calvinlo, 2012/09/19 03:15:38]

Done.

-                 weak_factory_.GetWeakPtr()));
+                 weak_factory_.GetWeakPtr(), host,
+                 quota::kStorageTypePersistent));
 }
 
 BrowsingDataQuotaHelperImpl::BrowsingDataQuotaHelperImpl(
     base::MessageLoopProxy* ui_thread,
     base::MessageLoopProxy* io_thread,
     quota::QuotaManager* quota_manager)
     : BrowsingDataQuotaHelper(io_thread),
       quota_manager_(quota_manager),
       is_fetching_(false),
       ui_thread_(ui_thread),
@@ skipping 54 matching lines @@
   pending_hosts_.erase(itr);
   GetHostUsage(host, type);
 }
 
 void BrowsingDataQuotaHelperImpl::GetHostUsage(const std::string& host,
                                                quota::StorageType type) {
   DCHECK(quota_manager_.get());
   quota_manager_->GetHostUsage(
       host, type,
       base::Bind(&BrowsingDataQuotaHelperImpl::GotHostUsage,
-                 weak_factory_.GetWeakPtr()));
+                 weak_factory_.GetWeakPtr(), host, type));
 }
 
 void BrowsingDataQuotaHelperImpl::GotHostUsage(const std::string& host,
                                                quota::StorageType type,
                                                int64 usage) {
   switch (type) {
     case quota::kStorageTypeTemporary:
       quota_info_[host].temporary_usage = usage;
       break;
     case quota::kStorageTypePersistent:
@@ skipping 28 matching lines @@
 
     info->host = itr->first;
     result.push_back(*info);
   }
 
   callback_.Run(result);
   callback_.Reset();
 }
 
 void BrowsingDataQuotaHelperImpl::DidRevokeHostQuota(
-    quota::QuotaStatusCode status_unused,
     const std::string& host_unused,
     quota::StorageType type_unused,
+    quota::QuotaStatusCode status_unused,
     int64 quota_unused) {
 }