[Quota][Clean up] Drop non-informative StorageType parameter on GetOriginsForType callback

The caller of GetOriginsForType should know the StorageType,
so this parameter will not add any information to the callback.
Historically this was useful when we'd used old callbacks.

BUG=220029
TEST=should not break existing tests

Review URL: https://chromiumcodereview.appspot.com/15925005

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@202805 0039d316-1c4b-4281-b951-d872f2087c98

2 files changed, 7 insertions, 13 deletions

diff --git a/webkit/browser/database/database_quota_client.cc b/webkit/browser/database/database_quota_client.cc
index a1ea780..5902d1a 100644
--- a/webkit/browser/database/database_quota_client.cc
+++ b/webkit/browser/database/database_quota_client.cc
@@ -66,9 +66,8 @@ void GetOriginsForHostOnDBThread(
 
 void DidGetOrigins(
     const QuotaClient::GetOriginsCallback& callback,
-    std::set<GURL>* origins_ptr,
-    quota::StorageType type) {
-  callback.Run(*origins_ptr, type);
+    std::set<GURL>* origins_ptr) {
+  callback.Run(*origins_ptr);
 }
 
 void DidDeleteOriginData(
@@ -151,7 +150,7 @@ void DatabaseQuotaClient::GetOriginsForType(
 
   // All databases are in the temp namespace for now.
   if (type != quota::kStorageTypeTemporary) {
-    callback.Run(std::set<GURL>(), type);
+    callback.Run(std::set<GURL>());
     return;
   }
 
@@ -163,8 +162,7 @@ void DatabaseQuotaClient::GetOriginsForType(
                  base::Unretained(origins_ptr)),
       base::Bind(&DidGetOrigins,
                  callback,
-                 base::Owned(origins_ptr),
-                 type));
+                 base::Owned(origins_ptr)));
 }
 
 void DatabaseQuotaClient::GetOriginsForHost(
@@ -176,7 +174,7 @@ void DatabaseQuotaClient::GetOriginsForHost(
 
   // All databases are in the temp namespace for now.
   if (type != quota::kStorageTypeTemporary) {
-    callback.Run(std::set<GURL>(), type);
+    callback.Run(std::set<GURL>());
     return;
   }
 
@@ -189,8 +187,7 @@ void DatabaseQuotaClient::GetOriginsForHost(
                  host),
       base::Bind(&DidGetOrigins,
                  callback,
-                 base::Owned(origins_ptr),
-                 type));
+                 base::Owned(origins_ptr)));
 }
 
 void DatabaseQuotaClient::DeleteOriginData(
diff --git a/webkit/browser/database/database_quota_client_unittest.cc b/webkit/browser/database/database_quota_client_unittest.cc
index 8e508ac..abb7f21 100644
--- a/webkit/browser/database/database_quota_client_unittest.cc
+++ b/webkit/browser/database/database_quota_client_unittest.cc
@@ -192,10 +192,8 @@ class DatabaseQuotaClientTest : public testing::Test {
     usage_ = usage;
   }
 
-  void OnGetOriginsComplete(const std::set<GURL>& origins,
-      quota::StorageType type) {
+  void OnGetOriginsComplete(const std::set<GURL>& origins) {
     origins_ = origins;
-    type_ = type;
   }
 
   void OnDeleteOriginDataComplete(quota::QuotaStatusCode status) {
@@ -205,7 +203,6 @@ class DatabaseQuotaClientTest : public testing::Test {
   base::MessageLoop message_loop_;
   int64 usage_;
   std::set<GURL> origins_;
-  quota::StorageType type_;
   quota::QuotaStatusCode delete_status_;
   scoped_refptr<MockDatabaseTracker> mock_tracker_;
   base::WeakPtrFactory<DatabaseQuotaClientTest> weak_factory_;