Implement QM::GetOriginsModifiedSince for browser data deleter support

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 beca602214dfd4b7d4739d28515b9d67df6962ab..0d9fd1099031de64ab519a8468da585c95920b01 100644
--- a/webkit/quota/quota_manager.h
+++ b/webkit/quota/quota_manager.h
@@ -166,6 +166,8 @@ class QuotaManager : public QuotaTaskObserver,
   class OriginDeletionDatabaseTask;
   class TemporaryOriginsRegistrationTask;
   class OriginAccessRecordDatabaseTask;
+  class OriginModifyRecordDatabaseTask;
+  class OriginsModifiedSinceRetrieveTask;
 
   class UsageAndQuotaDispatcherTask;
   class UsageAndQuotaDispatcherTaskForTemporary;
@@ -173,16 +175,16 @@ class QuotaManager : public QuotaTaskObserver,
 
   class AvailableSpaceQueryTask;
   class DumpQuotaTableTask;
-  class DumpLastAccessTimeTableTask;
+  class DumpOriginInfoTableTask;
 
   typedef QuotaDatabase::QuotaTableEntry QuotaTableEntry;
-  typedef QuotaDatabase::LastAccessTimeTableEntry LastAccessTimeTableEntry;
+  typedef QuotaDatabase::OriginInfoTableEntry OriginInfoTableEntry;
   typedef std::vector<QuotaTableEntry> QuotaTableEntries;
-  typedef std::vector<LastAccessTimeTableEntry> LastAccessTimeTableEntries;
+  typedef std::vector<OriginInfoTableEntry> OriginInfoTableEntries;
 
   typedef Callback1<const QuotaTableEntries&>::Type DumpQuotaTableCallback;
-  typedef Callback1<const LastAccessTimeTableEntries&>::Type
-      DumpLastAccessTimeTableCallback;
+  typedef Callback1<const OriginInfoTableEntries&>::Type
+      DumpOriginInfoTableCallback;
 
   struct EvictionContext {
     EvictionContext()
@@ -215,6 +217,7 @@ class QuotaManager : public QuotaTaskObserver,
       UsageAndQuotaDispatcherTaskMap;
 
   friend struct QuotaManagerDeleter;
+  friend class MockStorageClient;
   friend class QuotaManagerProxy;
   friend class QuotaManagerTest;
   friend class QuotaTemporaryStorageEvictor;
@@ -236,15 +239,26 @@ class QuotaManager : public QuotaTaskObserver,
   // (Might return empty list if no origin is tracked by the tracker.)
   void GetCachedOrigins(StorageType type, std::set<GURL>* origins);
 
-  // This internal method is separately defined mainly for testing.
+  // These internal methods are separately defined mainly for testing.
   void NotifyStorageAccessedInternal(
       QuotaClient::ID client_id,
       const GURL& origin,
       StorageType type,
       base::Time accessed_time);
+  void NotifyStorageModifiedInternal(
+      QuotaClient::ID client_id,
+      const GURL& origin,
+      StorageType type,
+      int64 delta,
+      base::Time modified_time);
+
+  void GetOriginsModifiedSince(

[michaeln, 2011/06/16 19:28:53]

should this be in the public interface?

[kinuko, 2011/06/23 06:37:40]

Done.

+      StorageType type,
+      base::Time modified_since,
+      GetOriginsCallback* callback);
 
   void DumpQuotaTable(DumpQuotaTableCallback* callback);
-  void DumpLastAccessTimeTable(DumpLastAccessTimeTableCallback* callback);
+  void DumpOriginInfoTable(DumpOriginInfoTableCallback* callback);
 
   // Methods for eviction logic.
   void StartEviction();