Add functions to clean URLs from the activity log

chrome/browser/extensions/activity_log/counting_policy.h

Index: chrome/browser/extensions/activity_log/counting_policy.h
diff --git a/chrome/browser/extensions/activity_log/counting_policy.h b/chrome/browser/extensions/activity_log/counting_policy.h
index 3911bcad3ff47a8b398f61fa601188de1ae02c32..09a739f04a6392ca4880643dabaae128ab842fa9 100644
--- a/chrome/browser/extensions/activity_log/counting_policy.h
+++ b/chrome/browser/extensions/activity_log/counting_policy.h
@@ -36,6 +36,9 @@ class CountingPolicy : public ActivityLogDatabasePolicy {
     retention_time_ = delta;
   }
 
+  // Clean the URL data stored for this policy.
+  virtual void RemoveURLs(const std::vector<GURL>&);

[mvrable, 2013/08/26 20:38:50]

virtual void RemoveURLs(const std::vector<GURL>&) OVERRIDE;

[karenlees, 2013/08/26 22:58:36]

Done.

+
   // The main database table, and the name for a read-only view that
   // decompresses string values for easier parsing.
   static const char* kTableName;
@@ -66,6 +69,10 @@ class CountingPolicy : public ActivityLogDatabasePolicy {
       const std::string& extension_id,
       const int days_ago);
 
+  // The implementation of RemoveURLs; this must only run on the database
+  // thread.
+  virtual void DoRemoveURLs(const std::vector<GURL>& restrict_urls);

[mvrable, 2013/08/26 20:38:50]

Doesn't need to be virtual since the method is private.

[karenlees, 2013/08/26 22:58:36]

Done - but should I make it protected instead, incase this gets subclassed?

[mvrable, 2013/08/27 17:17:37]

Personal opinion: it doesn't need to be as we can always go back later and make
it protected virtual if we need to subclass.  But I don't have a strong feeling
on this.

+
   // Cleans old records from the activity log database.
   bool CleanOlderThan(sql::Connection* db, const base::Time& cutoff);