[WebPayments] Adding something resembling Shipping Address list

components/payments/payment_request.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/payment_request.h"
 
 #include "base/memory/ptr_util.h"
 #include "components/autofill/core/browser/personal_data_manager.h"
 #include "components/payments/payment_details_validation.h"
 #include "components/payments/payment_request_delegate.h"
 #include "components/payments/payment_request_web_contents_manager.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/web_contents.h"
 
 namespace payments {
 
 PaymentRequest::PaymentRequest(
     content::WebContents* web_contents,
     std::unique_ptr<PaymentRequestDelegate> delegate,
     PaymentRequestWebContentsManager* manager,
     mojo::InterfaceRequest<payments::mojom::PaymentRequest> request)
     : web_contents_(web_contents),
       delegate_(std::move(delegate)),
       manager_(manager),
-      binding_(this, std::move(request)) {
+      binding_(this, std::move(request)),
+      selected_shipping_profile_(nullptr),
+      selected_contact_profile_(nullptr) {
   binding_.set_connection_error_handler(
       base::Bind(&PaymentRequest::OnError, base::Unretained(this)));
 }
 
 PaymentRequest::~PaymentRequest() {}
 
 void PaymentRequest::Init(
     payments::mojom::PaymentRequestClientPtr client,
     std::vector<payments::mojom::PaymentMethodDataPtr> methodData,
     payments::mojom::PaymentDetailsPtr details,
     payments::mojom::PaymentOptionsPtr options) {
   DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
   std::string error;
   if (!payments::validatePaymentDetails(details, &error)) {
     LOG(ERROR) << error;
     OnError();
     client_.reset();
     return;
   }
   client_ = std::move(client);
   details_ = std::move(details);
+  PopulateProfileCache();
+  SetDefaultProfileSelections();
 }
 
 void PaymentRequest::Show() {
   if (!client_.is_bound() || !binding_.is_bound()) {
     OnError();
     return;
   }
   delegate_->ShowPaymentRequestDialog(this);
 }
 
@@ skipping 11 matching lines @@
     const base::Optional<std::string> currency_system,
     const std::string& locale_name) {
   if (!currency_formatter_) {
     currency_formatter_.reset(
         new CurrencyFormatter(currency_code, currency_system, locale_name));
   }
 
   return currency_formatter_.get();
 }
 
-autofill::AutofillProfile* PaymentRequest::GetCurrentlySelectedProfile() {
-  // TODO(tmartino): Implement more sophisticated algorithm for populating
-  // this when it starts empty.
-  if (!profile_) {
-    autofill::PersonalDataManager* data_manager =
-        delegate_->GetPersonalDataManager();
-    auto profiles = data_manager->GetProfiles();
-    if (!profiles.empty())
-      profile_ = base::MakeUnique<autofill::AutofillProfile>(*profiles[0]);
-  }
-  return profile_ ? profile_.get() : nullptr;
+const std::vector<autofill::AutofillProfile*>&
+PaymentRequest::shipping_profiles() {
+  return shipping_profiles_;
+}
+
+const std::vector<autofill::AutofillProfile*>&
+PaymentRequest::contact_profiles() {
+  return contact_profiles_;
 }
 
 autofill::CreditCard* PaymentRequest::GetCurrentlySelectedCreditCard() {
   // TODO(anthonyvd): Change this code to prioritize server cards and implement
   // a way to modify this function's return value.
   autofill::PersonalDataManager* data_manager =
       delegate_->GetPersonalDataManager();
 
   const std::vector<autofill::CreditCard*> cards =
       data_manager->GetCreditCardsToSuggest();
 
   auto first_complete_card = std::find_if(
       cards.begin(),
       cards.end(),
       [] (autofill::CreditCard* card) {
         return card->IsValid();
   });
 
   return first_complete_card == cards.end() ? nullptr : *first_complete_card;
 }
 
+void PaymentRequest::PopulateProfileCache() {
+  autofill::PersonalDataManager* data_manager =
+      delegate_->GetPersonalDataManager();
+  std::vector<autofill::AutofillProfile*> profiles =
+      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.
+  for (size_t i = 0; i < profiles.size(); i++) {
+    profile_cache_.push_back(
+        base::MakeUnique<autofill::AutofillProfile>(*profiles[0]));

[please use gerrit instead, 2017/01/25 15:07:47]

profiles[i]

[tmartino, 2017/01/25 18:48:10]

Good catch, thanks.

+
+    // TODO(tmartino): Implement deduplication rules specific to shipping and
+    // contact profiles.
+    shipping_profiles_.push_back(profile_cache_[i].get());
+    contact_profiles_.push_back(profile_cache_[i].get());
+  }
+}
+
+void PaymentRequest::SetDefaultProfileSelections() {
+  if (!shipping_profiles().empty())
+    set_selected_shipping_profile(shipping_profiles()[0]);
+
+  if (!contact_profiles().empty())
+    set_selected_contact_profile(contact_profiles()[0]);
+}
+
 }  // namespace payments