[WebPayments] Implementing Profile filter and dedupe

components/payments/content/payment_request_state.cc

 // 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.
 
 #include "components/payments/content/payment_request_state.h"
 
 #include <set>
 
 #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/content/payment_request_spec.h"
 #include "components/payments/content/payment_response_helper.h"
 #include "components/payments/core/autofill_payment_instrument.h"
+#include "components/payments/core/profile_util.h"
 
 namespace payments {
 
 PaymentRequestState::PaymentRequestState(
     PaymentRequestSpec* spec,
     Delegate* delegate,
     const std::string& app_locale,
     autofill::PersonalDataManager* personal_data_manager)
     : is_ready_to_pay_(false),
       app_locale_(app_locale),
@@ skipping 122 matching lines @@
   std::vector<autofill::AutofillProfile*> profiles =
       personal_data_manager_->GetProfilesToSuggest();
 
   // PaymentRequest may outlive the Profiles returned by the Data Manager.
   // Thus, we store copies, and return a vector of pointers to these copies
   // whenever Profiles are requested. The same is true for credit cards.
   for (size_t i = 0; i < profiles.size(); i++) {
     profile_cache_.push_back(
         base::MakeUnique<autofill::AutofillProfile>(*profiles[i]));
 
-    // TODO(tmartino): Implement deduplication rules specific to shipping and
-    // contact profiles.
+    // TODO(tmartino): Implement deduplication rules specific to shipping
+    // profiles.
     shipping_profiles_.push_back(profile_cache_[i].get());
-    contact_profiles_.push_back(profile_cache_[i].get());
   }
 
+  std::vector<autofill::AutofillProfile*> raw_profiles_for_filtering(
+      profile_cache_.size());
+  std::transform(profile_cache_.begin(), profile_cache_.end(),
+                 raw_profiles_for_filtering.begin(),
+                 [](const std::unique_ptr<autofill::AutofillProfile>& p) {
+                   return p.get();
+                 });
+
+  contact_profiles_ = profile_util::FilterProfilesForContact(
+      raw_profiles_for_filtering, GetApplicationLocale(), spec_);
+
   // Create the list of available instruments.
   const std::vector<autofill::CreditCard*>& cards =
       personal_data_manager_->GetCreditCardsToSuggest();
   const std::set<std::string>& supported_card_networks =
       spec_->supported_card_networks_set();
   for (autofill::CreditCard* card : cards) {
     std::string basic_card_network =
         autofill::data_util::GetPaymentRequestData(card->type())
             .basic_card_payment_type;
     if (!supported_card_networks.count(basic_card_network))
@@ skipping 52 matching lines @@
   // instruments are listed/created in the flow.
   // TODO(crbug.com/702063): A masked card may not satisfy IsValid().
   return selected_instrument_ != nullptr && selected_instrument_->IsValid();
 }
 
 bool PaymentRequestState::ArePaymentOptionsSatisfied() {
   // TODO(mathp): Have a measure of shipping address completeness.
   if (spec_->request_shipping() && selected_shipping_profile_ == nullptr)
     return false;
 
-  // TODO(mathp): Make an encompassing class to validate contact info.
-  if (spec_->request_payer_name() &&
-      (selected_contact_profile_ == nullptr ||
-       selected_contact_profile_
-           ->GetInfo(autofill::AutofillType(autofill::NAME_FULL), app_locale_)
-           .empty())) {
-    return false;
-  }
-  if (spec_->request_payer_email() &&
-      (selected_contact_profile_ == nullptr ||
-       selected_contact_profile_
-           ->GetInfo(autofill::AutofillType(autofill::EMAIL_ADDRESS),
-                     app_locale_)
-           .empty())) {
-    return false;
-  }
-  if (spec_->request_payer_phone() &&
-      (selected_contact_profile_ == nullptr ||
-       selected_contact_profile_
-           ->GetInfo(autofill::AutofillType(autofill::PHONE_HOME_WHOLE_NUMBER),
-                     app_locale_)
-           .empty())) {
-    return false;
-  }
-
-  return true;
+  profile_util::PaymentsProfileComparator comparator(app_locale_, spec_);
+  return comparator.IsContactInfoComplete(selected_contact_profile_);
 }
 
 void PaymentRequestState::UpdateSelectedShippingOption() {
   selected_shipping_option_ = nullptr;
 
   // As per the spec, the selected shipping option should initially be the last
   // one in the array that has its selected field set to true.
   auto selected_shipping_option_it = std::find_if(
       spec_->details().shipping_options.rbegin(),
       spec_->details().shipping_options.rend(),
       [](const payments::mojom::PaymentShippingOptionPtr& element) {
         return element->selected;
       });
   if (selected_shipping_option_it != spec_->details().shipping_options.rend()) {
     selected_shipping_option_ = selected_shipping_option_it->get();
   }
 }
 
 }  // namespace payments