[Autofill] Update credit card type detection logic.

components/autofill/core/browser/validation.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/validation.h"
 
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/string_piece.h"
 #include "base/strings/string_util.h"
 #include "base/strings/utf_string_conversions.h"
@@ skipping 39 matching lines @@
   if (cc_year == current_year && cc_month < current_month)
     return false;
 
   return true;
 }
 
 bool IsValidCreditCardNumber(const base::string16& text) {
   base::string16 number = CreditCard::StripSeparators(text);
 
   // Credit card numbers are at most 19 digits in length [1]. 12 digits seems to
-  // be a fairly safe lower-bound [2].
+  // be a fairly safe lower-bound [2].  Specific card issuers have more rigidly
+  // defined sizes.
   // [1] http://www.merriampark.com/anatomycc.htm
   // [2] http://en.wikipedia.org/wiki/Bank_card_number
-  const size_t kMinCreditCardDigits = 12;
-  const size_t kMaxCreditCardDigits = 19;
-  if (number.size() < kMinCreditCardDigits ||
-      number.size() > kMaxCreditCardDigits)
+  const std::string type = CreditCard::GetCreditCardType(text);
+  if (type == kAmericanExpressCard && number.size() != 15)
+    return false;
+  if (type == kDinersCard && number.size() != 14)
+    return false;
+  if (type == kDiscoverCard && number.size() != 16)
+    return false;
+  if (type == kJCBCard && number.size() != 16)
+    return false;
+  if (type == kMasterCard && number.size() != 16)
+    return false;
+  if (type == kVisaCard && number.size() != 13 && number.size() != 16)
+    return false;
+  if (type == kGenericCard && (number.size() < 12 || number.size() > 19))
     return false;
 
   // Use the Luhn formula [3] to validate the number.
   // [3] http://en.wikipedia.org/wiki/Luhn_algorithm
   int sum = 0;
   bool odd = false;
   for (base::string16::reverse_iterator iter = number.rbegin();
        iter != number.rend();
        ++iter) {
     if (!IsAsciiDigit(*iter))
@@ skipping 110 matching lines @@
                                        number_string.begin() + 9),
                          &serial)
       || serial == 0) {
     return false;
   }
 
   return true;
 }
 
 }  // namespace autofill