Cull autofill entries older than 60 days.

chrome/browser/webdata/autofill_table.h

Index: chrome/browser/webdata/autofill_table.h
diff --git a/chrome/browser/webdata/autofill_table.h b/chrome/browser/webdata/autofill_table.h
index 4430f17cba4472d8c404c74898a51a207ed339ff..2b77e1a9b7e51f1da843e00b6a4a033f19282d4b 100644
--- a/chrome/browser/webdata/autofill_table.h
+++ b/chrome/browser/webdata/autofill_table.h
@@ -154,17 +154,27 @@ class AutofillTable : public WebDatabaseTable {
                                       const base::Time& delete_end,
                                       std::vector<AutofillChange>* changes);
 
+  // Removes rows from autofill_dates if they were accessed strictly before
+  // |AutofillEntry::ExpirationTime()|. Removes the corresponding row from the
+  // autofill table. Also culls timestamps to only two. TODO(georgey): remove
+  // culling in future versions.
+  bool RemoveExpiredFormElements(std::vector<AutofillChange>* changes);
+
   // Removes from autofill_dates rows with given pair_id where date_created lies
-  // between delte_begin and delte_end.
+  // between |delete_begin| and |delete_end|.
   bool RemoveFormElementForTimeRange(int64 pair_id,
                                      const base::Time& delete_begin,
                                      const base::Time& delete_end,
                                      int* how_many);
 
   // Increments the count in the row corresponding to |pair_id| by
-  // |delta|.  Removes the row from the table and sets the
-  // |was_removed| out parameter to true if the count becomes 0.
-  bool AddToCountOfFormElement(int64 pair_id, int delta, bool* was_removed);
+  // |delta|.  Removes the row from the table and if the count becomes 0.

[Ilya Sherman, 2012/03/21 23:43:30]

It seems like this method should not be responsible for removing the row from
the table, ever -- it should just update the count.  Am I missing some reason
why that's not so?

[GeorgeY, 2012/03/23 21:53:21]

No. It is in NOTREACHED() land anyway - removed.

+  bool AddToCountOfFormElement(int64 pair_id, int delta);
+
+  // Counts how many timestamp data rows are in the |autofill_dates| table for
+  // a given |pair_id|. GetCountOfFormElement() on the other hand gives the
+  // |count| property for a given id.
+  int CountTimestampsData(int64 pair_id);
 
   // Gets the pair_id and count entries from name and value specified in
   // |element|.  Sets *pair_id and *count to 0 if there is no such row in
@@ -187,6 +197,9 @@ class AutofillTable : public WebDatabaseTable {
   // Adds a new row to the autofill_dates table.
   bool InsertPairIDAndDate(int64 pair_id, const base::Time& date_created);
 
+  // Deletes last access to the Autofill data from the autofill_dates table.
+  bool DeleteLastAccess(int64 pair_id);
+
   // Removes row from the autofill tables given |pair_id|.
   bool RemoveFormElementForID(int64 pair_id);