QuotaManager::DeleteOriginData now allows deletion of specific QuotaClients

webkit/quota/quota_manager.h

Index: webkit/quota/quota_manager.h
diff --git a/webkit/quota/quota_manager.h b/webkit/quota/quota_manager.h
index 70254c09870768754ad22c10f79d54b42f4d4d81..5bf71c46fe6918c977efb2b6a85b8f9b5ad58c16 100644
--- a/webkit/quota/quota_manager.h
+++ b/webkit/quota/quota_manager.h
@@ -1,4 +1,4 @@
-// Copyright (c) 2011 The Chromium Authors. All rights reserved.
+// 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.
 
@@ -148,12 +148,20 @@ class QuotaManager : public QuotaTaskObserver,
     return origins_in_use_.find(origin) != origins_in_use_.end();
   }
 
-  // Called by UI.
+  // DeleteOriginData and DeleteHostData (surprisingly enough) delete data of a
+  // particular StorageType associated with either a specific origin or set of
+  // origins. Each method additionally requires a quota_client_mask which
+  // specifies the types of QuotaClients to delete from the origin. This is
+  // specified as a bitmask built from QuotaClient::IDs.

[kinuko, 2012/01/22 18:35:31]

It might be helpful to note that the caller specify the wildcard mask
(kAllClientsMask) to delete all.

[Mike West, 2012/01/22 20:54:38]

Good call. Done, both here and in MockQuotaManager.

+  //
+  // Both methods must be called on the UI thread.
   virtual void DeleteOriginData(const GURL& origin,
                                 StorageType type,
+                                int quota_client_mask,
                                 const StatusCallback& callback);
   void DeleteHostData(const std::string& host,
                       StorageType type,
+                      int quota_client_mask,
                       const StatusCallback& callback);
 
   // Called by UI and internal modules.