[Payment Request] Makes sure only complete credit cards can be selected

ios/chrome/browser/ui/payments/payment_request_manager.mm

 // Copyright 2016 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.
 
 #import "ios/chrome/browser/ui/payments/payment_request_manager.h"
 
 #include "base/ios/block_types.h"
 #include "base/ios/ios_util.h"
 #import "base/mac/bind_objc_block.h"
 #include "base/mac/foundation_util.h"
@@ skipping 642 matching lines @@
 
   [self terminateRequestWithErrorMessage:kCancelMessage callback:callback];
 }
 
 - (void)paymentRequestCoordinator:(PaymentRequestCoordinator*)coordinator
     didCompletePaymentRequestWithCard:(const autofill::CreditCard&)card
                      verificationCode:(const base::string16&)verificationCode {
   _pendingPaymentResponse.creditCard = card;
   _pendingPaymentResponse.verificationCode = verificationCode;
 
-  // TODO(crbug.com/714768): Make sure the billing address is set and valid
-  // before getting here. Once the bug is addressed, there will be no need to
-  // copy the address, *billing_address_ptr can be used to get the basic card
-  // response.
-  if (!card.billing_address_id().empty()) {
-    autofill::AutofillProfile* billingAddressPtr =
-        autofill::PersonalDataManager::GetProfileFromProfilesByGUID(
-            card.billing_address_id(), _paymentRequest->billing_profiles());
-    if (billingAddressPtr) {
-      _pendingPaymentResponse.billingAddress = *billingAddressPtr;
-      _addressNormalizationManager->StartNormalizingAddress(
-          &_pendingPaymentResponse.billingAddress);
-    }
-  }
+  DCHECK(!card.billing_address_id().empty());
+  autofill::AutofillProfile* billingAddress =
+      autofill::PersonalDataManager::GetProfileFromProfilesByGUID(
+          card.billing_address_id(), _paymentRequest->billing_profiles());
+  DCHECK(billingAddress);
+  _pendingPaymentResponse.billingAddress = *billingAddress;
+  _addressNormalizationManager->StartNormalizingAddress(
+      &_pendingPaymentResponse.billingAddress);
 
   if (_paymentRequest->request_shipping()) {
     // TODO(crbug.com/602666): User should get here only if they have selected
     // a shipping address.
     DCHECK(_paymentRequest->selected_shipping_profile());
     _pendingPaymentResponse.shippingAddress =
         *_paymentRequest->selected_shipping_profile();
     _addressNormalizationManager->StartNormalizingAddress(
         &_pendingPaymentResponse.shippingAddress);
   }
@@ skipping 95 matching lines @@
 #pragma mark - CRWWebStateObserver methods
 
 - (void)webState:(web::WebState*)webState
     didCommitNavigationWithDetails:
         (const web::LoadCommittedDetails&)load_details {
   [self dismissUI];
   [self enableCurrentWebState];
 }
 
 @end