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

Unified Diff: components/autofill/core/browser/webdata/autofill_table.h

Issue 947693002: Add use_count and use_date to unmasked server cards as well. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 10 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: components/autofill/core/browser/webdata/autofill_table.h
diff --git a/components/autofill/core/browser/webdata/autofill_table.h b/components/autofill/core/browser/webdata/autofill_table.h
index a645e90e0b11dcdaf74f1c17b15aabc2ef1b6b79..542eb0273a70bb249b4b91e9e4ddbb45d28557f7 100644
--- a/components/autofill/core/browser/webdata/autofill_table.h
+++ b/components/autofill/core/browser/webdata/autofill_table.h
@@ -69,7 +69,7 @@ struct FormFieldData;
// country_code
// use_count The number of times this profile has been used to fill
// a form.
-// last_use The date this profile was last used to fill a form.
+// use_date The date this profile was last used to fill a form.
// date_modified The date on which this profile was last modified.
// Added in version 30.
// origin The domain of origin for this profile.
@@ -127,7 +127,7 @@ struct FormFieldData;
// Stores encrypted credit card number.
// use_count The number of times this card has been used to fill
// a form.
-// last_use The date this card was last used to fill a form.
+// use_date The date this card was last used to fill a form.
// date_modified The date on which this entry was last modified.
// Added in version 30.
// origin The domain of origin for this profile.
@@ -161,6 +161,9 @@ struct FormFieldData;
// masked_credit_cards table to get the rest of the data.
// card_number_encrypted
// Full card number, encrypted.
+// use_count The number of times this card has been used to fill
+// a form.
+// use_date The date this card was last used to fill a form.
brettw 2015/02/23 19:04:34 Can we call this (and the other one above) "last_u
Evan Stade 2015/02/23 22:57:22 I'd like to be consistent with the names used in t
//
// server_addresses This table contains Autofill address data synced from
// the wallet server. It's basically the same as the
@@ -311,6 +314,9 @@ class AutofillTable : public WebDatabaseTable {
const base::string16& full_number);
bool MaskServerCreditCard(const std::string& id);
+ // Updates the use count and last use date for an unmasked server card.
+ bool UpdateUnmaskedCardUsageStats(const CreditCard& credit_card);
+
// Removes rows from autofill_profiles and credit_cards if they were created
// on or after |delete_begin| and strictly before |delete_end|. Returns the
// list of deleted profile guids in |profile_guids|. Return value is true if
@@ -369,6 +375,7 @@ class AutofillTable : public WebDatabaseTable {
bool MigrateToVersion57AddFullNameField();
bool MigrateToVersion60AddServerCards();
bool MigrateToVersion61AddUsageStats();
+ bool MigrateToVersion62AddUsageStatsForUnmaskedCards();
// Max data length saved in the table;
static const size_t kMaxDataLength;

Powered by Google App Engine
This is Rietveld 408576698