[Payments] Better default selections on iOS

ios/chrome/browser/payments/payment_request.mm

 // 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 "ios/chrome/browser/payments/payment_request.h"
 
+#include "base/containers/adapters.h"
 #include "base/strings/utf_string_conversions.h"
 #include "components/autofill/core/browser/autofill_data_util.h"
 #include "components/autofill/core/browser/autofill_profile.h"
 #include "components/autofill/core/browser/credit_card.h"
 #include "components/autofill/core/browser/personal_data_manager.h"
 #include "components/payments/core/currency_formatter.h"
 #include "components/payments/core/payment_request_data_util.h"
 #include "components/payments/core/payments_profile_comparator.h"
 #include "ios/chrome/browser/application_context.h"
 #include "ios/web/public/payments/payment_request.h"
 
 #if !defined(__has_feature) || !__has_feature(objc_arc)
 #error "This file requires ARC support."
 #endif
 
 PaymentRequest::PaymentRequest(
     const web::PaymentRequest& web_payment_request,
     autofill::PersonalDataManager* personal_data_manager)
     : web_payment_request_(web_payment_request),
       personal_data_manager_(personal_data_manager),
       selected_shipping_profile_(nullptr),
       selected_contact_profile_(nullptr),
       selected_credit_card_(nullptr),
-      selected_shipping_option_(nullptr) {
+      selected_shipping_option_(nullptr),
+      profile_comparator_(GetApplicationContext()->GetApplicationLocale(),
+                          *this) {
+  PopulateShippingOptionCache();
   PopulateProfileCache();
   PopulateCreditCardCache();
-  PopulateShippingOptionCache();
 }
 
 PaymentRequest::~PaymentRequest() {}
 
-void PaymentRequest::set_payment_details(const web::PaymentDetails& details) {
+void PaymentRequest::UpdatePaymentDetails(const web::PaymentDetails& details) {
   web_payment_request_.details = details;
   PopulateShippingOptionCache();
 }
 
 bool PaymentRequest::request_shipping() const {
   return web_payment_request_.options.request_shipping;
 }
 
 bool PaymentRequest::request_payer_name() const {
   return web_payment_request_.options.request_payer_name;
@@ skipping 35 matching lines @@
   return credit_cards_.front();
 }
 
 bool PaymentRequest::CanMakePayment() const {
   return !credit_cards_.empty();
 }
 
 void PaymentRequest::PopulateProfileCache() {
   const std::vector<autofill::AutofillProfile*>& profiles_to_suggest =
       personal_data_manager_->GetProfilesToSuggest();
+  // Return early if the user has no stored Autofill profiles.
+  if (profiles_to_suggest.empty())
+    return;
+
   profile_cache_.reserve(profiles_to_suggest.size());
   for (const auto* profile : profiles_to_suggest) {
     profile_cache_.push_back(*profile);
     shipping_profiles_.push_back(&profile_cache_.back());
-    contact_profiles_.push_back(&profile_cache_.back());
   }
 
-  payments::PaymentsProfileComparator comparator(
-      GetApplicationContext()->GetApplicationLocale(), *this);
+  // If the merchant provided a shipping option, select a suitable default
+  // shipping profile. We pick the profile that is most complete, going down
+  // the list in Frecency order.
+  // TODO(crbug.com/719652): Have a proper ordering of shipping addresses by
+  // completeness.
+  if (selected_shipping_option_) {
+    selected_shipping_profile_ = shipping_profiles_[0];
+    for (autofill::AutofillProfile* profile : shipping_profiles_) {
+      if (profile_comparator_.IsShippingComplete(profile)) {
+        selected_shipping_profile_ = profile;
+        break;
+      }
+    }
+  }
 
-  // TODO(crbug.com/602666): Implement deduplication and prioritization rules
-  // for shipping profiles.
+  // Contact profiles are deduped and ordered in completeness.
+  contact_profiles_ =
+      profile_comparator_.FilterProfilesForContact(shipping_profiles_);
 
-  contact_profiles_ = comparator.FilterProfilesForContact(contact_profiles_);
-
-  if (!shipping_profiles_.empty())
-    selected_shipping_profile_ = shipping_profiles_[0];
+  // If the highest-ranking contact profile is usable, select it. Otherwise,
+  // select none.
   if (!contact_profiles_.empty() &&
-      comparator.IsContactInfoComplete(contact_profiles_[0])) {
+      profile_comparator_.IsContactInfoComplete(contact_profiles_[0])) {
     selected_contact_profile_ = contact_profiles_[0];
   }
 }
 
 void PaymentRequest::PopulateCreditCardCache() {
   // TODO(crbug.com/709036): Validate method data.
   payments::data_util::ParseBasicCardSupportedNetworks(
       web_payment_request_.method_data, &supported_card_networks_,
       &basic_card_specified_networks_);
 
@@ skipping 14 matching lines @@
   }
 
   // TODO(crbug.com/602666): Implement prioritization rules for credit cards.
 
   if (!credit_cards_.empty())
     selected_credit_card_ = credit_cards_[0];
 }
 
 void PaymentRequest::PopulateShippingOptionCache() {
   shipping_options_.clear();
+  selected_shipping_option_ = nullptr;
+  if (web_payment_request_.details.shipping_options.empty())
+    return;
+
   shipping_options_.reserve(
       web_payment_request_.details.shipping_options.size());
   std::transform(std::begin(web_payment_request_.details.shipping_options),
                  std::end(web_payment_request_.details.shipping_options),
                  std::back_inserter(shipping_options_),
                  [](web::PaymentShippingOption& option) { return &option; });
 
-  selected_shipping_option_ = nullptr;
-  for (auto* shipping_option : shipping_options_) {
+  for (auto* shipping_option : base::Reversed(shipping_options_)) {
     if (shipping_option->selected) {
       // If more than one option has |selected| set, the last one in the
       // sequence should be treated as the selected item.
       selected_shipping_option_ = shipping_option;
+      break;
     }
   }
 }