Time-based removal of temporary file systems via BrowsingDataRemover

chrome/browser/browsing_data_remover.h

Index: chrome/browser/browsing_data_remover.h
diff --git a/chrome/browser/browsing_data_remover.h b/chrome/browser/browsing_data_remover.h
index 9f940246b817b744e8b423ebcf4937d18ee5c30d..6c6e9cf5a00a5231f2ad21e43f51f85b8cf878ba 100644
--- a/chrome/browser/browsing_data_remover.h
+++ b/chrome/browser/browsing_data_remover.h
@@ -16,6 +16,7 @@
 #include "content/browser/cancelable_request.h"
 #include "content/common/notification_observer.h"
 #include "content/common/notification_registrar.h"
+#include "webkit/quota/quota_types.h"
 
 class ExtensionSpecialStoragePolicy;
 class IOThread;
@@ -157,6 +158,11 @@ class BrowsingDataRemover : public NotificationObserver,
   // Invoked on the FILE thread to delete HTML5 file systems.
   void ClearFileSystemsOnFILEThread();
 
+  // Callback to respond to QuotaManager::GetOriginsModifiedSince.
+  void ProcessTemporaryQuotaModifiedInTimeframe(const std::set<GURL>&);
+
+  void DeleteOriginDataCallback(quota::QuotaStatusCode);
+
   // Callback when the appcache has been cleared. Invokes
   // NotifyAndDeleteIfDone.
   void OnClearedAppCache();
@@ -239,6 +245,9 @@ class BrowsingDataRemover : public NotificationObserver,
   bool waiting_for_clear_gears_data_;
   bool waiting_for_clear_file_systems_;

[michaeln, 2011/07/21 23:49:15]

I was thinking that use of the QuotaManager could completely replace the removal
logic for databases, appcaches, and filesystems (for both from-time and
for-all-time removal); so the flags and methods related to dealing with each of
those individually those could be removed. So you'd have some new methods
like...

void ClearQuotaManagedDataOnIOThread();
void OnGotQuotaManagedOrigins(const std::set<GURL>&);
void OnDeletedQuotaManagedOrigin(quota::QuotaStatusCode);

And a bunch of appcache/filesystem/database specific methods and members
removed.

 
+  // Tracking how many origins need to be deleted.
+  int temporary_quota_origins_to_delete_count_;
+
   ObserverList<Observer> observer_list_;
 
   // Used if we need to clear history.