[Autofill] Credit Card Autofill Last Used Date Experiment

components/autofill/core/browser/credit_card.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/credit_card.h"
 
 #include <stddef.h>
 #include <stdint.h>
 
 #include <algorithm>
@@ skipping 659 matching lines @@
   supported_types->insert(CREDIT_CARD_NUMBER);
   supported_types->insert(CREDIT_CARD_TYPE);
   supported_types->insert(CREDIT_CARD_EXP_MONTH);
   supported_types->insert(CREDIT_CARD_EXP_2_DIGIT_YEAR);
   supported_types->insert(CREDIT_CARD_EXP_4_DIGIT_YEAR);
   supported_types->insert(CREDIT_CARD_EXP_DATE_2_DIGIT_YEAR);
   supported_types->insert(CREDIT_CARD_EXP_DATE_4_DIGIT_YEAR);
 }
 
 base::string16 CreditCard::ExpirationMonthAsString() const {
-  if (expiration_month_ == 0)
-    return base::string16();
-
-  base::string16 month = base::IntToString16(expiration_month_);
-  if (expiration_month_ >= 10)
-    return month;
-
-  base::string16 zero = ASCIIToUTF16("0");
-  zero.append(month);
-  return zero;
+  return expiration_month_ == 0
+      ? base::string16()
+      : PadTo2Digit(expiration_month_);
 }
 
 base::string16 CreditCard::TypeForFill() const {
   return ::autofill::TypeForFill(type_);
 }
 
 base::string16 CreditCard::Expiration4DigitYearAsString() const {
   if (expiration_year_ == 0)
     return base::string16();
 
@@ skipping 76 matching lines @@
 
 void CreditCard::SetNumber(const base::string16& number) {
   number_ = number;
 
   // Set the type based on the card number, but only for full numbers, not
   // when we have masked cards from the server (last 4 digits).
   if (record_type_ != MASKED_SERVER_CARD)
     type_ = GetCreditCardType(StripSeparators(number_));
 }
 
+base::string16 CreditCard::LastUsedDateAsString(

[Mathieu, 2017/01/18 14:42:15]

Could we rename this GetLastUsedDataForDisplay?

[jiahuiguo, 2017/01/18 22:18:12]

Done.

+    bool show_time_detail,
+    bool show_expiration_date,
+    const std::string& app_locale) const {
+  base::Time::Exploded exploded;
+  use_date().LocalExplode(&exploded);
+  base::string16 month = GetShortMonthLabel(exploded.month, app_locale);
+  DCHECK(use_count() > 0);
+  // use_count() is initialized as 1 when the card is just added

[Jared Saul, 2017/01/18 18:43:44]

nit: Here and in the other comment on line 795, add period at end

[jiahuiguo, 2017/01/18 22:18:13]

Done.

+  if (use_count() == 1)

[Mathieu, 2017/01/18 14:42:15]

 put curlies {}

[jiahuiguo, 2017/01/18 22:18:12]

Done.

+    return l10n_util::GetStringFUTF16(
+        show_expiration_date
+            ? IDS_AUTOFILL_CREDIT_CARD_ADDED_TO_CHROME_DATE_LABEL_AND_EXP

[Mathieu, 2017/01/18 14:42:15]

let's not mention Chrome in the label name.
"IDS_AUTOFILL_CREDIT_CARD_ADDED__DATE_LABEL_AND_EXP"

[Jared Saul, 2017/01/18 18:43:44]

(But with s/ADDED__DATE/ADDED_DATE)

[jiahuiguo, 2017/01/18 22:18:12]

Done.

[jiahuiguo, 2017/01/18 22:18:12]

Done.

+            : IDS_AUTOFILL_CREDIT_CARD_ADDED_TO_CHROME_DATE_LABEL,
+        month,
+        PadTo2Digit(exploded.day_of_month));
+  // use_count() > 1 when the card has been used in autofill
+  if (show_time_detail) {
+    return l10n_util::GetStringFUTF16(
+        show_expiration_date
+            ? IDS_AUTOFILL_CREDIT_CARD_LAST_USED_DATE_LABEL_AND_EXP_WITH_DETAIL
+            : IDS_AUTOFILL_CREDIT_CARD_LAST_USED_DATE_LABEL_WITH_DETAIL,
+        month,
+        PadTo2Digit(exploded.day_of_month),
+        PadTo2Digit(exploded.hour),
+        PadTo2Digit(exploded.minute));
+  } else {
+    return l10n_util::GetStringFUTF16(
+        show_expiration_date
+            ? IDS_AUTOFILL_CREDIT_CARD_LAST_USED_DATE_LABEL_AND_EXP_NO_DETAIL
+            : IDS_AUTOFILL_CREDIT_CARD_LAST_USED_DATE_LABEL_NO_DETAIL,
+        month,
+        PadTo2Digit(exploded.day_of_month));
+  }
+}
+
+base::string16 CreditCard::PadTo2Digit(int32_t date_info) const {
+  DCHECK(date_info > 0);
+  base::string16 dateIn2Digit = base::IntToString16(date_info);

[Mathieu, 2017/01/18 14:42:15]

nit: C++ style is no camel-case

[jiahuiguo, 2017/01/18 22:18:13]

Done.

+  if (date_info >= 10)
+    return dateIn2Digit;
+
+  base::string16 padded_date = ASCIIToUTF16("0");
+  padded_date.append(dateIn2Digit);
+  return padded_date;
+}
+
+base::string16 CreditCard::GetShortMonthLabel(
+    int one_base_month,
+    const std::string& app_locale) const {
+  UErrorCode status = U_ZERO_ERROR;
+  icu::Locale locale(app_locale.c_str());
+  icu::DateFormatSymbols date_format_symbols(locale, status);
+  DCHECK(status == U_ZERO_ERROR || status == U_USING_FALLBACK_WARNING ||
+         status == U_USING_DEFAULT_WARNING);
+
+  int32_t num_months;
+  const icu::UnicodeString* months =
+      date_format_symbols.getShortMonths(num_months);
+  base::string16 short_month(
+      months[one_base_month - 1].getBuffer(),
+      static_cast<size_t>(months[one_base_month - 1].length()));
+  return short_month;
+}
+
 void CreditCard::RecordAndLogUse() {
   UMA_HISTOGRAM_COUNTS_1000("Autofill.DaysSinceLastUse.CreditCard",
                             (base::Time::Now() - use_date()).InDays());
   RecordUse();
 }
 
 // static
 bool CreditCard::ConvertMonth(const base::string16& month,
                               const std::string& app_locale,
                               int* num) {
@@ skipping 76 matching lines @@
 const char kDinersCard[] = "dinersCC";
 const char kDiscoverCard[] = "discoverCC";
 const char kGenericCard[] = "genericCC";
 const char kJCBCard[] = "jcbCC";
 const char kMasterCard[] = "masterCardCC";
 const char kMirCard[] = "mirCC";
 const char kUnionPay[] = "unionPayCC";
 const char kVisaCard[] = "visaCC";
 
 }  // namespace autofill