[WebPayments] Start populating the Payment Sheet.

chrome/browser/payments/payment_request_impl.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 "chrome/browser/payments/payment_request_impl.h"
 
 #include <map>
 
 #include "base/lazy_instance.h"
 #include "chrome/browser/ui/browser_dialogs.h"
@@ skipping 44 matching lines @@
 
 PaymentRequestImpl::~PaymentRequestImpl() {}
 
 void PaymentRequestImpl::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);
   client_ = std::move(client);
+  details_ = std::move(details);

[please use gerrit instead, 2017/01/04 15:55:27]

Run |details| through validatePaymentDetails(details, &error_message) first:

https://cs.chromium.org/chromium/src/components/payments/payment_details_vali...

This data comes from the renderer process, which the browser process should not
trust. If the |details| are invalid, disconnect from the client_ and binding_,
also destroy PaymentRequestImpl.

[anthonyvd, 2017/01/04 19:08:55]

Done.

 }
 
 void PaymentRequestImpl::Show() {
   chrome::ShowPaymentRequestDialog(this);
 }
 
 void PaymentRequestImpl::Cancel() {
   client_->OnError(payments::mojom::PaymentErrorReason::USER_CANCEL);
 }
 
 void PaymentRequestImpl::OnError() {
   binding_.Close();
   // TODO(krb): Close the dialog here, but avoid double-free
   payment_request_factory.Get().UnassignPaymentRequest(web_contents_);
 }
 
 }  // namespace payments
 
 void CreatePaymentRequestHandler(
     content::WebContents* web_contents,
     mojo::InterfaceRequest<payments::mojom::PaymentRequest> request) {
   payment_request_factory.Get().AssignPaymentRequest(web_contents,
                                                      std::move(request));
 }