Add use_count and use_date to unmasked server cards as well.

components/autofill/core/browser/webdata/autofill_table.h

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef COMPONENTS_AUTOFILL_CORE_BROWSER_WEBDATA_AUTOFILL_TABLE_H_
 #define COMPONENTS_AUTOFILL_CORE_BROWSER_WEBDATA_AUTOFILL_TABLE_H_
 
 #include <vector>
 
 #include "base/compiler_specific.h"
@@ skipping 51 matching lines @@
 //   city
 //   state
 //   zipcode
 //   sorting_code       Similar to the zipcode column, but used for businesses
 //                      or organizations that might not be geographically
 //                      contiguous.  The canonical example is CEDEX in France.
 //                      Added in version 54.
 //   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.
 //                      Added in version 50.
 //   language_code      The BCP 47 language code used to format the address for
 //                      display. For example, a JP address with "ja" language
 //                      code starts with the postal code, but a JP address with
 //                      "ja-latn" language code starts with the recipient name.
 //                      Added in version 56.
 //
@@ skipping 37 matching lines @@
 //
 //   guid               A guid string to uniquely identify the profile.
 //                      Added in version 31.
 //   name_on_card
 //   expiration_month
 //   expiration_year
 //   card_number_encrypted
 //                      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.
 //                      Added in version 50.
 //
 // masked_credit_cards
 //                      This table contains "masked" credit card information
 //                      about credit cards stored on the server. It consists
 //                      of a short description and an ID, but not full payment
 //                      information. Writing to this table is only done by sync.
@@ skipping 13 matching lines @@
 //   exp_year           Four-digit year: 2017
 //
 // unmasked_credit_cards
 //                      When a masked credit credit card is unmasked and the
 //                      full number is downloaded, it will be stored here.
 //
 //   id                 Server ID. This can be joined with the id in the
 //                      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_used_date"? Also, the comments
should mention what units this is in (see comment about units below)

[Evan Stade, 2015/02/23 22:57:22]

I'd like to be consistent with the names used in the other tables (the comments
in this file were wrong). I've updated the docs regarding the units and when the
fields were added.

 //
 // server_addresses     This table contains Autofill address data synced from
 //                      the wallet server. It's basically the same as the
 //                      autofill_profiles table but locally immutable.
 //
 //   id                 String assigned by the server to identify this address.
 //                      This is opaque to the client.
 //   company_name
 //   street_address     The combined lines of the street address.
 //   address_1          Also known as "administrative area". This is normally
@@ skipping 130 matching lines @@
   // present in the new list will be preserved (even if the input is MASKED).
   void SetServerCreditCards(const std::vector<CreditCard>& credit_cards);
 
   // Cards synced from the server may be "masked" (only last 4 digits
   // available) or "unmasked" (everything is available). These functions set
   // that state.
   bool UnmaskServerCreditCard(const std::string& id,
                               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
   // all rows were successfully removed.  Returns false on database error.  In
   // that case, the output vector state is undefined, and may be partially
   // filled.
   bool RemoveAutofillDataModifiedBetween(
       const base::Time& delete_begin,
       const base::Time& delete_end,
       std::vector<std::string>* profile_guids,
@@ skipping 38 matching lines @@
   bool MigrateToVersion34ProfilesBasedOnCountryCode();
   bool MigrateToVersion35GreatBritainCountryCodes();
   bool MigrateToVersion37MergeAndCullOlderProfiles();
   bool MigrateToVersion51AddOriginColumn();
   bool MigrateToVersion54AddI18nFieldsAndRemoveDeprecatedFields();
   bool MigrateToVersion55MergeAutofillDatesTable();
   bool MigrateToVersion56AddProfileLanguageCodeForFormatting();
   bool MigrateToVersion57AddFullNameField();
   bool MigrateToVersion60AddServerCards();
   bool MigrateToVersion61AddUsageStats();
+  bool MigrateToVersion62AddUsageStatsForUnmaskedCards();
 
   // Max data length saved in the table;
   static const size_t kMaxDataLength;
 
  private:
   FRIEND_TEST_ALL_PREFIXES(AutofillTableTest, Autofill);
   FRIEND_TEST_ALL_PREFIXES(AutofillTableTest, Autofill_AddChanges);
   FRIEND_TEST_ALL_PREFIXES(AutofillTableTest, Autofill_RemoveBetweenChanges);
   FRIEND_TEST_ALL_PREFIXES(AutofillTableTest, Autofill_UpdateDontReplace);
   FRIEND_TEST_ALL_PREFIXES(
@@ skipping 63 matching lines @@
   // created (on the UI thread), and cached here so that it can be used for
   // migrations (on the DB thread).
   std::string app_locale_;
 
   DISALLOW_COPY_AND_ASSIGN(AutofillTable);
 };
 
 }  // namespace autofill
 
 #endif  // COMPONENTS_AUTOFILL_CORE_BROWSER_WEBDATA_AUTOFILL_TABLE_H_