Validate saved credit card number in interactive autocomplete

components/autofill/browser/credit_card.cc

Index: components/autofill/browser/credit_card.cc
diff --git a/components/autofill/browser/credit_card.cc b/components/autofill/browser/credit_card.cc
index 8a96fa9955f15ed0157f4e24d825c0f27b828e30..5aca667646ddec508d01b824c91641ff91966990 100644
--- a/components/autofill/browser/credit_card.cc
+++ b/components/autofill/browser/credit_card.cc
@@ -17,6 +17,7 @@
 #include "base/strings/string_split.h"
 #include "base/strings/string_util.h"
 #include "base/strings/utf_string_conversions.h"
+#include "base/time.h"
 #include "components/autofill/browser/autofill_field.h"
 #include "components/autofill/browser/autofill_regexes.h"
 #include "components/autofill/browser/autofill_type.h"
@@ -585,6 +586,13 @@ bool CreditCard::IsComplete() const {
       expiration_year_ != 0;
 }
 
+bool CreditCard::IsValid() const {
+  return IsComplete() && autofill::IsValidCreditCardExpirationDate(
+                             Expiration4DigitYearAsString(),

[Evan Stade, 2013/06/17 19:17:57]

nit: I would rather not make this depend on IsComplete.

I also don't think it makes sense to convert dates to a string just so they can
immediately be converted back to numerical values. So I'd recommend adding
another function to validation.cc

[please use gerrit instead, 2013/06/17 21:40:24]

Done.

+                             ExpirationMonthAsString(),
+                             base::Time::Now());
+}
+
 void CreditCard::GetSupportedTypes(FieldTypeSet* supported_types) const {
   supported_types->insert(CREDIT_CARD_NAME);
   supported_types->insert(CREDIT_CARD_NUMBER);