Add billing address as a mandatory field of Payments credit cards.

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 <stddef.h>
 
 #include "base/logging.h"
 #include "base/strings/string_number_conversions.h"
@@ skipping 123 matching lines @@
 
   if (card.number().empty())
     status |= CREDIT_CARD_NO_NUMBER;
 
   if (card.GetInfo(autofill::AutofillType(autofill::CREDIT_CARD_NAME_FULL),
                    app_locale)
           .empty()) {
     status |= CREDIT_CARD_NO_CARDHOLDER;
   }
 
+  if (card.billing_address_id().empty()) {

[sebsg, 2017/05/02 22:58:55]

This would be a bit more involved but we might want to check that the billing
address id is associated with a profile that we have on file. We can add it
later if you want (but let's create a bug in that case).

[MAD, 2017/05/03 16:15:19]

Done.

+    status |= CREDIT_CARD_NO_BILLING_ADDRESS;
+  }
+
   return status;
 }
 
 base::string16 GetCompletionMessageForCard(CreditCardCompletionStatus status) {
   switch (status) {
     case CREDIT_CARD_COMPLETE:
       return base::string16();
     case CREDIT_CARD_EXPIRED:
       return l10n_util::GetStringUTF16(
           IDS_PAYMENTS_VALIDATION_INVALID_CREDIT_CARD_EXPIRED);
     case CREDIT_CARD_NO_CARDHOLDER:
       return l10n_util::GetStringUTF16(IDS_PAYMENTS_NAME_ON_CARD_REQUIRED);
     case CREDIT_CARD_NO_NUMBER:
       return l10n_util::GetStringUTF16(
           IDS_PAYMENTS_CARD_NUMBER_INVALID_VALIDATION_MESSAGE);
+    case CREDIT_CARD_NO_BILLING_ADDRESS:
+      return l10n_util::GetStringUTF16(
+          IDS_PAYMENTS_CARD_BILLING_ADDRESS_REQUIRED);
     default:
       // Multiple things are missing
       return l10n_util::GetStringUTF16(IDS_PAYMENTS_MORE_INFORMATION_REQUIRED);
   }
 }
 
 bool IsValidEmailAddress(const base::string16& text) {
   // E-Mail pattern as defined by the WhatWG. (4.10.7.1.5 E-Mail state)
   const base::string16 kEmailPattern = base::ASCIIToUTF16(
       "^[a-zA-Z0-9.!#$%&'*+/=?^_`{|}~-]+@"
@@ skipping 196 matching lines @@
     return AMEX_CVC_LENGTH;
 
   return GENERAL_CVC_LENGTH;
 }
 
 bool IsUPIVirtualPaymentAddress(const base::string16& value) {
   return MatchesPattern(value, base::ASCIIToUTF16(kUPIVirtualPaymentAddressRe));
 }
 
 }  // namespace autofill