[Autofill] Credit Card Autofill Last Used Date Experiment

components/autofill/core/browser/personal_data_manager.cc

 // 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.
 
 #include "components/autofill/core/browser/personal_data_manager.h"
 
 #include <stddef.h>
 
 #include <algorithm>
 #include <list>
 #include <map>
 #include <string>
 #include <utility>
 
 #include "base/i18n/case_conversion.h"
+#include "base/i18n/time_formatting.h"
 #include "base/i18n/timezone.h"
 #include "base/memory/ptr_util.h"
 #include "base/profiler/scoped_tracker.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/string_util.h"
 #include "base/strings/utf_string_conversions.h"
 #include "build/build_config.h"
 #include "components/autofill/core/browser/address_i18n.h"
 #include "components/autofill/core/browser/autofill-inl.h"
 #include "components/autofill/core/browser/autofill_country.h"
@@ skipping 11 matching lines @@
 #include "components/autofill/core/common/autofill_pref_names.h"
 #include "components/autofill/core/common/autofill_switches.h"
 #include "components/autofill/core/common/autofill_util.h"
 #include "components/prefs/pref_service.h"
 #include "components/signin/core/browser/account_tracker_service.h"
 #include "components/signin/core/browser/signin_manager.h"
 #include "components/signin/core/common/signin_pref_names.h"
 #include "components/sync/driver/sync_service.h"
 #include "components/variations/variations_associated_data.h"
 #include "components/version_info/version_info.h"
+#include "grit/components_strings.h"
 #include "third_party/libaddressinput/src/cpp/include/libaddressinput/address_data.h"
 #include "third_party/libaddressinput/src/cpp/include/libaddressinput/address_formatter.h"
+#include "ui/base/l10n/l10n_util.h"
 
 namespace autofill {
 namespace {
 
 using ::i18n::addressinput::AddressField;
 using ::i18n::addressinput::GetStreetAddressLinesAsSingleLine;
 using ::i18n::addressinput::STREET_ADDRESS;
 
 template<typename T>
 class FormGroupMatchesByGUIDFunctor {
@@ skipping 182 matching lines @@
       suggestion_canon.length() >= field_contents_canon.length() &&
       GetTextSelectionStart(suggestion_canon, field_contents_canon, false) !=
           base::string16::npos) {
     *is_prefix_matched = false;
     return true;
   }
 
   return false;
 }
 
+// Returns the date when the credit card was last used in autofill. Shows
+// expiration date if |show_expiration_date| is true, and shows time detail
+// if |show_time_detail| is true.
+base::string16 GetLastUsedDateForDisplay(const CreditCard* credit_card,

[Mathieu, 2017/01/27 15:01:44]

I would put this function in AutofillDataModel (which is the base class of
AutofillProfile *and* CreditCard).

+                                         bool show_expiration_date,
+                                         bool show_time_detail,
+                                         const std::string& app_locale) {
+  DCHECK(credit_card->use_count() > 0);
+  // use_count() is initialized as 1 when the card is just added.
+  if (credit_card->use_count() == 1) {
+    return show_expiration_date
+               ? l10n_util::GetStringFUTF16(
+                     IDS_AUTOFILL_CREDIT_CARD_EXP_AND_ADDED_DATE,
+                     credit_card->GetInfo(
+                         AutofillType(CREDIT_CARD_EXP_DATE_2_DIGIT_YEAR),
+                         app_locale),
+                     base::TimeFormatWithPattern(credit_card->use_date(),
+                                                 "MMMdd"))
+               : l10n_util::GetStringFUTF16(
+                     IDS_AUTOFILL_CREDIT_CARD_ADDED_DATE,
+                     base::TimeFormatWithPattern(credit_card->use_date(),
+                                                 "MMMdd"));
+  }
+  // use_count() > 1 when the card has been used in autofill.
+
+  // If the card was last used in autofill more than a year ago,
+  // display "last used > 1 year ago" without showing date detail.
+  if ((base::Time::Now() - credit_card->use_date()).InDays() > 365) {
+    return show_expiration_date
+               ? l10n_util::GetStringFUTF16(
+                     IDS_AUTOFILL_CREDIT_CARD_EXP_AND_LAST_USED_YEAR_AGO,
+                     credit_card->GetInfo(
+                         AutofillType(CREDIT_CARD_EXP_DATE_2_DIGIT_YEAR),
+                         app_locale))
+               : l10n_util::GetStringUTF16(
+                     IDS_AUTOFILL_CREDIT_CARD_LAST_USED_YEAR_AGO);
+  }
+
+  // If the card was last used in autofill within a year, show date information.
+  if (show_time_detail) {
+    return show_expiration_date
+               ? l10n_util::GetStringFUTF16(
+                     IDS_AUTOFILL_CREDIT_CARD_EXP_AND_LAST_USED_DATE_DETAIL,
+                     credit_card->GetInfo(
+                         AutofillType(CREDIT_CARD_EXP_DATE_2_DIGIT_YEAR),
+                         app_locale),
+                     base::TimeFormatWithPattern(credit_card->use_date(),
+                                                 "MMMddjmm"))

[Mathieu, 2017/01/27 15:01:44]

Since we are ever only using two patterns (with time detail and without), we
should make them constants.

[jiahuiguo, 2017/02/03 07:11:17]

Done.

+               : l10n_util::GetStringFUTF16(
+                     IDS_AUTOFILL_CREDIT_CARD_LAST_USED_DATE_DETAIL,
+                     base::TimeFormatWithPattern(credit_card->use_date(),
+                                                 "MMMddjmm"));
+  } else {
+    return show_expiration_date
+               ? l10n_util::GetStringFUTF16(
+                     IDS_AUTOFILL_CREDIT_CARD_EXP_AND_LAST_USED_DATE,
+                     credit_card->GetInfo(
+                         AutofillType(CREDIT_CARD_EXP_DATE_2_DIGIT_YEAR),
+                         app_locale),
+                     base::TimeFormatWithPattern(credit_card->use_date(),
+                                                 "MMMdd"))
+               : l10n_util::GetStringFUTF16(
+                     IDS_AUTOFILL_CREDIT_CARD_LAST_USED_DATE,
+                     base::TimeFormatWithPattern(credit_card->use_date(),
+                                                 "MMMdd"));
+  }
+}
+
 }  // namespace
 
 const char kFrecencyFieldTrialName[] = "AutofillProfileOrderByFrecency";
 const char kFrecencyFieldTrialLimitParam[] = "limit";
 
 PersonalDataManager::PersonalDataManager(const std::string& app_locale)
     : database_(NULL),
       is_data_loaded_(false),
       pending_profiles_query_(0),
       pending_server_profiles_query_(0),
@@ skipping 1350 matching lines @@
       suggestion->backend_id = credit_card->guid();
       suggestion->match = prefix_matched_suggestion
                               ? Suggestion::PREFIX_MATCH
                               : Suggestion::SUBSTRING_MATCH;
 
       // If the value is the card number, the label is the expiration date.
       // Otherwise the label is the card number, or if that is empty the
       // cardholder name. The label should never repeat the value.
       if (type.GetStorableType() == CREDIT_CARD_NUMBER) {
         suggestion->value = credit_card->TypeAndLastFourDigits();
-        suggestion->label = credit_card->GetInfo(
-            AutofillType(CREDIT_CARD_EXP_DATE_2_DIGIT_YEAR), app_locale_);
+        if (IsAutofillCreditCardLastUsedDateDisplayExperimentEnabled() &&
+            !IsKeyboardAccessoryEnabled()) {
+          suggestion->label = GetLastUsedDateForDisplay(

[Mathieu, 2017/01/27 15:01:44]

using the suggestion above, you will be able to do
credit_card->GetLastUsedDateForDisplay(...);

[jiahuiguo, 2017/02/03 07:11:17]

Done.

+              credit_card, ShowExpirationDateInAutofillCreditCardLastUsedDate(),
+              ShowTimeDetailInAutofillCreditCardLastUsedDate(), app_locale_);
+        } else {
+          suggestion->label = credit_card->GetInfo(
+              AutofillType(CREDIT_CARD_EXP_DATE_2_DIGIT_YEAR), app_locale_);
+        }
         if (IsAutofillCreditCardPopupLayoutExperimentEnabled())
           ModifyAutofillCreditCardSuggestion(suggestion);
       } else if (credit_card->number().empty()) {
         if (type.GetStorableType() != CREDIT_CARD_NAME_FULL) {
           suggestion->label = credit_card->GetInfo(
               AutofillType(CREDIT_CARD_NAME_FULL), app_locale_);
         }
       } else {
 #if defined(OS_ANDROID)
         // Since Android places the label on its own row, there's more
@@ skipping 213 matching lines @@
     }
 
     // If the card was modified, apply the changes to the database.
     if (was_modified) {
       database_->UpdateCreditCard(*credit_card);
     }
   }
 }
 
 }  // namespace autofill