[WebPayments] Add the Payment Method section in the Payment Sheet

chrome/browser/payments/chrome_payment_request_delegate.h

 // 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.
 
 #ifndef CHROME_BROWSER_PAYMENTS_CHROME_PAYMENT_REQUEST_DELEGATE_H_
 #define CHROME_BROWSER_PAYMENTS_CHROME_PAYMENT_REQUEST_DELEGATE_H_
 
 #include "base/macros.h"
 #include "components/payments/payment_request_delegate.h"
 
 namespace payments {
 
 class PaymentRequest;
 
 class ChromePaymentRequestDelegate : public PaymentRequestDelegate {
  public:
   ChromePaymentRequestDelegate() {}

[Mathieu, 2017/01/12 21:27:10]

Let's create the ChromePaymentRequestDelegate with a WebContents* that it keeps
as member, and can reuse in GetPersonalDataManager.

I don't think the PaymentRequestDelegate interface needs to be specific to
WebContents (i.e. someone could provide a FooBarPaymentRequestDelegate that
didn't require WebContents to get a PersonalDataManager)

[anthonyvd, 2017/01/13 16:36:06]

Done.

   ~ChromePaymentRequestDelegate() override {}
 
   void ShowPaymentRequestDialog(PaymentRequest* request) override;
+  autofill::PersonalDataManager* GetDataManagerFromWebContents(

[Mathieu, 2017/01/12 21:27:10]

nit: would call GetPersonalDataManager()

[anthonyvd, 2017/01/13 16:09:18]

Done.

+      content::WebContents* web_contents) override;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(ChromePaymentRequestDelegate);
 };
 
 }  // namespace payments
 
 #endif  // CHROME_BROWSER_PAYMENTS_CHROME_PAYMENT_REQUEST_DELEGATE_H_