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

Unified Diff: webkit/quota/quota_manager_unittest.cc

Issue 10948006: Cleanup: quota::HostQuotaCallback do not need to pass host and type as arguments (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixes for Kinuko San Review #1 Created 8 years, 3 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: webkit/quota/quota_manager_unittest.cc
diff --git a/webkit/quota/quota_manager_unittest.cc b/webkit/quota/quota_manager_unittest.cc
index 43e4f852244ed5b3d23f9f9a8fcb819406a21110..0ba95a3a1cdf16faa39abd34d3890fbb347cdad4 100644
--- a/webkit/quota/quota_manager_unittest.cc
+++ b/webkit/quota/quota_manager_unittest.cc
@@ -128,7 +128,7 @@ class QuotaManagerTest : public testing::Test {
quota_manager_->GetPersistentHostQuota(
host,
base::Bind(&QuotaManagerTest::DidGetHostQuota,
- weak_factory_.GetWeakPtr()));
+ weak_factory_.GetWeakPtr(), host, type_));
}
void SetPersistentHostQuota(const std::string& host, int64 new_quota) {
@@ -139,7 +139,7 @@ class QuotaManagerTest : public testing::Test {
quota_manager_->SetPersistentHostQuota(
host, new_quota,
base::Bind(&QuotaManagerTest::DidGetHostQuota,
- weak_factory_.GetWeakPtr()));
+ weak_factory_.GetWeakPtr(), host, type_));
}
void GetGlobalUsage(StorageType type) {
@@ -308,13 +308,13 @@ class QuotaManagerTest : public testing::Test {
available_space_ = available_space;
}
- void DidGetHostQuota(QuotaStatusCode status,
- const std::string& host,
+ void DidGetHostQuota(const std::string& host,
StorageType type,
+ QuotaStatusCode status,
int64 quota) {
- quota_status_ = status;
host_ = host;
type_ = type;
+ quota_status_ = status;
quota_ = quota;
}

Powered by Google App Engine
This is Rietveld 408576698