QuotaManager::DeleteOriginData now allows deletion of specific QuotaClients

webkit/quota/quota_client.h

Index: webkit/quota/quota_client.h
diff --git a/webkit/quota/quota_client.h b/webkit/quota/quota_client.h
index 312e5d0d244f4bc6f122e72d2fd7214e07384a80..4a601509d89610250fd503672edddf4e4dd6cc5c 100644
--- a/webkit/quota/quota_client.h
+++ b/webkit/quota/quota_client.h
@@ -30,14 +30,17 @@ class QuotaClient {
   virtual ~QuotaClient() {}
 
   enum ID {
-    kUnknown,
-    kFileSystem,
-    kDatabase,
-    kAppcache,
-    kIndexedDatabase,
+    kUnknown = 1 << 0,
+    kFileSystem = 1 << 1,
+    kDatabase = 1 << 2,
+    kAppcache = 1 << 3,
+    kIndexedDatabase = 1 << 4,
     kMockStart,  // This needs to be the end of the enum.

[kinuko, 2011/09/26 09:43:48]

what should we do if we want to test the feature with the mock storage?

   };
 
+  static const int kAllClientsMask = kFileSystem | kDatabase | kAppcache |
+                                     kIndexedDatabase;

[kinuko, 2011/09/26 09:43:48]

-1?

+
   virtual ID id() const = 0;
 
   // Called when the quota manager is destroyed.
@@ -67,6 +70,10 @@ class QuotaClient {
                                 DeletionCallback* callback) = 0;
 };
 
+const int kQuotaClientAllIDsMask = QuotaClient::kFileSystem |
+    QuotaClient::kDatabase | QuotaClient::kAppcache |
+    QuotaClient::kIndexedDatabase;

[kinuko, 2011/09/26 09:43:48]

is this a dupe of line 41?

+
 // TODO(dmikurube): Replace it to std::vector for efficiency.
 typedef std::list<QuotaClient*> QuotaClientList;