Support named months for Autofill implicit learning

chrome/browser/autofill/credit_card.cc

 // Copyright (c) 2011 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 "chrome/browser/autofill/credit_card.h"
 
 #include <stddef.h>
 #include <string>
 
 #include "base/basictypes.h"
 #include "base/logging.h"
 #include "base/string16.h"
 #include "base/string_number_conversions.h"
 #include "base/string_split.h"
 #include "base/string_util.h"
 #include "base/utf_string_conversions.h"
+#include "chrome/browser/autofill/autofill_country.h"
 #include "chrome/browser/autofill/autofill_regexes.h"
 #include "chrome/browser/autofill/autofill_type.h"
 #include "chrome/browser/autofill/field_types.h"
 #include "chrome/common/guid.h"
 #include "grit/generated_resources.h"
 #include "ui/base/l10n/l10n_util.h"
+#include "unicode/dtfmtsym.h"
+#include "unicode/uloc.h"
 
 namespace {
 
 const char16 kCreditCardObfuscationSymbol = '*';
 
 const AutofillFieldType kAutofillCreditCardTypes[] = {
   CREDIT_CARD_NAME,
   CREDIT_CARD_NUMBER,
   CREDIT_CARD_TYPE,
   CREDIT_CARD_EXP_MONTH,
@@ skipping 79 matching lines @@
     case 19:
       if (first_four_digits == 6334 || first_four_digits == 6767)
         return kSoloCard;
 
       break;
   }
 
   return kGenericCard;
 }
 
-bool ConvertDate(const string16& date, int* num) {
-  if (!date.empty()) {
-    bool converted = base::StringToInt(date, num);
-    DCHECK(converted);
-    if (!converted)
-      return false;
-  } else {
-    // Clear the value.
+bool ConvertDate(const string16& date, bool is_month, int* num) {

[dhollowa, 2011/06/07 16:33:00]

This feels like it is really two separate functions |ConvertMonth| and
|ConvertYear|.

[Ilya Sherman, 2011/06/08 00:10:12]

Done.

+  // If the |date| is empty, clear the value.
+  if (date.empty()) {
     *num = 0;
+    return true;
   }
 
-  return true;
+  // Try parsing the |date| as a number.
+  if (base::StringToInt(date, num))
+    return true;
+
+  // Try parsing the |date| as a named month, e.g. "January" or "Jan".
+  if (is_month) {
+    string16 lowercased_date = StringToLowerASCII(date);
+
+    UErrorCode status = U_ZERO_ERROR;
+    icu::Locale locale(AutofillCountry::ApplicationLocale().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.getMonths(num_months);
+    for (int32_t i = 0; i < num_months; ++i) {
+      const string16 month = string16(months[i].getBuffer(),
+                                      months[i].length());
+      if (lowercased_date == StringToLowerASCII(month)) {
+        *num = i + 1;  // Adjust from 0-indexed to 1-indexed.
+        return true;
+      }
+    }
+
+    months = date_format_symbols.getShortMonths(num_months);
+    for (int32_t i = 0; i < num_months; ++i) {
+      const string16 month = string16(months[i].getBuffer(),
+                                      months[i].length());
+      if (lowercased_date == StringToLowerASCII(month)) {
+        *num = i + 1;  // Adjust from 0-indexed to 1-indexed.
+        return true;
+      }
+    }
+  }
+
+

[dhollowa, 2011/06/07 16:33:00]

nit: extra space.

[Ilya Sherman, 2011/06/08 00:10:12]

Done.

+
+  NOTREACHED();
+  *num = 0;
+  return false;
 }
 
 }  // namespace
 
 CreditCard::CreditCard(const std::string& guid)
     : type_(kGenericCard),
       expiration_month_(0),
       expiration_year_(0),
       guid_(guid) {
 }
@@ skipping 331 matching lines @@
 
 string16 CreditCard::Expiration2DigitYearAsString() const {
   if (expiration_year_ == 0)
     return string16();
 
   return base::IntToString16(Expiration2DigitYear());
 }
 
 void CreditCard::SetExpirationMonthFromString(const string16& text) {
   int month;
-  if (!ConvertDate(text, &month))
+  if (!ConvertDate(text, true, &month))
     return;
 
   SetExpirationMonth(month);
 }
 
 void CreditCard::SetExpirationYearFromString(const string16& text) {
   int year;
-  if (!ConvertDate(text, &year))
+  if (!ConvertDate(text, false, &year))
     return;
 
   SetExpirationYear(year);
 }
 
 void CreditCard::SetNumber(const string16& number) {
   number_ = number;
   type_ = GetCreditCardType(StripSeparators(number_));
 }
 
@@ skipping 67 matching lines @@
 // send these strings to WK, which then asks WebKitClientImpl to load the image
 // data.
 const char* const kAmericanExpressCard = "americanExpressCC";
 const char* const kDinersCard = "dinersCC";
 const char* const kDiscoverCard = "discoverCC";
 const char* const kGenericCard = "genericCC";
 const char* const kJCBCard = "jcbCC";
 const char* const kMasterCard = "masterCardCC";
 const char* const kSoloCard = "soloCC";
 const char* const kVisaCard = "visaCC";