[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);
+  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_) {
+std::vector<autofill::AutofillProfile*> PaymentRequest::GetProfiles() {
+  if (profile_cache_.empty()) {
     autofill::PersonalDataManager* data_manager =
         delegate_->GetPersonalDataManager();
-    auto profiles = data_manager->GetProfiles();
-    if (!profiles.empty())
-      profile_ = base::MakeUnique<autofill::AutofillProfile>(*profiles[0]);
+    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.
+    for (autofill::AutofillProfile* profile : profiles) {
+      profile_cache_.push_back(
+          base::MakeUnique<autofill::AutofillProfile>(*profile));
+    }
   }
-  return profile_ ? profile_.get() : nullptr;
+
+  std::vector<autofill::AutofillProfile*> return_profiles(
+      profile_cache_.size());
+  std::transform(profile_cache_.begin(), profile_cache_.end(),
+                 return_profiles.begin(),
+                 [](const std::unique_ptr<autofill::AutofillProfile>& profile) {
+                   return profile.get();
+                 });
+
+  return return_profiles;
+}
+
+autofill::AutofillProfile* PaymentRequest::GetSelectedShippingProfile() {

[please use gerrit instead, 2017/01/20 19:32:53]

Here and below:

Simple getters and settings should use hacker_case() and are preferred to be
defined in the header file. Use const to indicate that getters don't change the
internal state of this PaymentRequest object.

autofill::AutofillProfile* selected_shipping_profile() const {
  return selected_shipping_profile_;
}

void set_selected_shipping_profile(autofill::AutofillProfile* profile) {
  selected_shipping_profile_ = profile;
}

[tmartino, 2017/01/25 00:08:41]

Done

+  return selected_shipping_profile_;
+}
+
+void PaymentRequest::SetShippingProfile(autofill::AutofillProfile* profile) {
+  selected_shipping_profile_ = profile;
+}
+
+autofill::AutofillProfile* PaymentRequest::GetSelectedContactProfile() {
+  return selected_contact_profile_;
+}
+
+void PaymentRequest::SetContactProfile(autofill::AutofillProfile* profile) {
+  selected_contact_profile_ = profile;
 }
 
 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::SetDefaultProfileSelections() {
+  auto profiles = GetProfiles();
+  if (!profiles.empty()) {
+    // TODO(tmartino): Examine the details to determine whether the shipping
+    // address should be prefilled, and what constraints apply to profiles
+    // for prefilling the contact info.
+    SetShippingProfile(profiles[0]);
+    SetContactProfile(profiles[0]);
+  }
+}
+
 }  // namespace payments