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

Unified Diff: webkit/appcache/appcache_quota_client.cc

Issue 15925005: [Quota][Clean up] Drop non-informative StorageType parameter on GetOriginsForType callback (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 7 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: webkit/appcache/appcache_quota_client.cc
diff --git a/webkit/appcache/appcache_quota_client.cc b/webkit/appcache/appcache_quota_client.cc
index d17d49e24f533cf5dd990a138b48c586d07127e9..a629c996749bfabd9a2ec339c9285f2defefc9ca 100644
--- a/webkit/appcache/appcache_quota_client.cc
+++ b/webkit/appcache/appcache_quota_client.cc
@@ -106,7 +106,7 @@ void AppCacheQuotaClient::GetOriginsForHost(
const GetOriginsCallback& callback) {
DCHECK(!callback.is_null());
if (host.empty()) {
- callback.Run(std::set<GURL>(), type);
+ callback.Run(std::set<GURL>());
return;
}
GetOriginsHelper(type, host, callback);
@@ -162,7 +162,7 @@ void AppCacheQuotaClient::GetOriginsHelper(
DCHECK(!quota_manager_is_destroyed_);
if (!service_) {
- callback.Run(std::set<GURL>(), type);
+ callback.Run(std::set<GURL>());
return;
}
@@ -174,7 +174,7 @@ void AppCacheQuotaClient::GetOriginsHelper(
}
if (type != quota::kStorageTypeTemporary) {
- callback.Run(std::set<GURL>(), type);
+ callback.Run(std::set<GURL>());
return;
}
@@ -185,7 +185,7 @@ void AppCacheQuotaClient::GetOriginsHelper(
if (opt_host.empty() || iter->first.host() == opt_host)
origins.insert(iter->first);
}
- callback.Run(origins, type);
+ callback.Run(origins);
}
void AppCacheQuotaClient::ProcessPendingRequests() {
« no previous file with comments | « content/browser/indexed_db/indexed_db_quota_client_unittest.cc ('k') | webkit/appcache/appcache_quota_client_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698