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

components/payments/core/autofill_payment_instrument.cc

 // Copyright 2017 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/payments/core/autofill_payment_instrument.h"
 
 #include <memory>
 
 #include "base/json/json_writer.h"
 #include "base/strings/utf_string_conversions.h"
@@ skipping 37 matching lines @@
 void AutofillPaymentInstrument::InvokePaymentApp(
     PaymentInstrument::Delegate* delegate) {
   DCHECK(delegate);
   // There can be only one FullCardRequest going on at a time. If |delegate_| is
   // not null, there's already an active request, which shouldn't happen.
   // |delegate_| is reset to nullptr when the request succeeds or fails.
   DCHECK(!delegate_);
   delegate_ = delegate;
 
   // Get the billing address.
-  // TODO(crbug.com/709776): Make sure the billing address is valid before
-  // getting here.
   if (!credit_card_.billing_address_id().empty()) {
     autofill::AutofillProfile* billing_address =
         autofill::PersonalDataManager::GetProfileFromProfilesByGUID(
             credit_card_.billing_address_id(), billing_profiles_);
     if (billing_address)
       billing_address_ = *billing_address;
   }
 
   is_waiting_for_billing_address_normalization_ = true;
   is_waiting_for_card_unmask_ = true;
 
   // Start the normalization of the billing address.
   // Use the country code from the profile if it is set, otherwise infer it
   // from the |app_locale_|.
   std::string country_code = base::UTF16ToUTF8(
       billing_address_.GetRawInfo(autofill::ADDRESS_HOME_COUNTRY));
   if (!autofill::data_util::IsValidCountryCode(country_code)) {
     country_code = autofill::AutofillCountry::CountryCodeForLocale(app_locale_);
   }
   payment_request_delegate_->GetAddressNormalizer()->StartAddressNormalization(
       billing_address_, country_code, /*timeout_seconds=*/5, this);
 
   payment_request_delegate_->DoFullCardRequest(credit_card_,
                                                weak_ptr_factory_.GetWeakPtr());
 }
 
 bool AutofillPaymentInstrument::IsCompleteForPayment() {
-  return autofill::GetCompletionStatusForCard(credit_card_, app_locale_) ==
+  return autofill::GetCompletionStatusForCard(credit_card_, app_locale_,
+                                              billing_profiles_) ==
          autofill::CREDIT_CARD_COMPLETE;
 }
 
 base::string16 AutofillPaymentInstrument::GetMissingInfoLabel() {
   return autofill::GetCompletionMessageForCard(
-      autofill::GetCompletionStatusForCard(credit_card_, app_locale_));
+      autofill::GetCompletionStatusForCard(credit_card_, app_locale_,
+                                           billing_profiles_));
 }
 
 bool AutofillPaymentInstrument::IsValidForCanMakePayment() {
   autofill::CreditCardCompletionStatus status =
-      autofill::GetCompletionStatusForCard(credit_card_, app_locale_);
+      autofill::GetCompletionStatusForCard(credit_card_, app_locale_,
+                                           billing_profiles_);
   // Card has to have a cardholder name and number for the purposes of
   // CanMakePayment. An expired card is still valid at this stage.
   return !(status & autofill::CREDIT_CARD_NO_CARDHOLDER ||
            status & autofill::CREDIT_CARD_NO_NUMBER);
 }
 
 void AutofillPaymentInstrument::OnFullCardRequestSucceeded(
     const autofill::CreditCard& card,
     const base::string16& cvc) {
   DCHECK(delegate_);
@@ skipping 41 matching lines @@
           .ToDictionaryValue();
   std::string stringified_details;
   base::JSONWriter::Write(*response_value, &stringified_details);
   delegate_->OnInstrumentDetailsReady(method_name(), stringified_details);
 
   delegate_ = nullptr;
   cvc_ = base::UTF8ToUTF16("");
 }
 
 }  // namespace payments