Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(8)

Unified Diff: chrome/browser/webdata/autofill_table.h

Issue 9585020: Cull autofill entries older than 60 days. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Addressed the comments Created 8 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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..e20811dfb297c783557536c503f1a3403113aedd 100644
--- a/chrome/browser/webdata/autofill_table.h
+++ b/chrome/browser/webdata/autofill_table.h
@@ -154,8 +154,15 @@ class AutofillTable : public WebDatabaseTable {
const base::Time& delete_end,
std::vector<AutofillChange>* changes);
+ // Removes rows from autofill_dates if they were accessed strictly before
+ // |delete_end|. Removes corresponding row from autofill table.
+ // Also culls timestamps to only two. TODO(georgey): remove culling in future
+ // versions.
+ bool RemoveExpiredFormElements(const base::Time& delete_end,
Ilya Sherman 2012/03/19 21:12:51 nit: I find the fact that we're passing |delete_en
GeorgeY 2012/03/20 20:47:58 sure
+ 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,
@@ -187,6 +194,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);

Powered by Google App Engine
This is Rietveld 408576698