[WebPayments] Move views-specific code to c/b/ui/views/

chrome/browser/ui/views/payments/payment_request_dialog.h

 // 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.
 
-#ifndef CHROME_BROWSER_PAYMENTS_UI_PAYMENT_REQUEST_DIALOG_H_
-#define CHROME_BROWSER_PAYMENTS_UI_PAYMENT_REQUEST_DIALOG_H_
+#ifndef CHROME_BROWSER_UI_VIEWS_PAYMENTS_PAYMENT_REQUEST_DIALOG_H_
+#define CHROME_BROWSER_UI_VIEWS_PAYMENTS_PAYMENT_REQUEST_DIALOG_H_
 
-#include "components/payments/payment_request.mojom.h"
 #include "ui/views/controls/label.h"
 #include "ui/views/window/dialog_delegate.h"
 
 namespace payments {
 
+class PaymentRequestImpl;
+
 class PaymentRequestDialog : public views::DialogDelegateView {
  public:
-  explicit PaymentRequestDialog(
-      payments::mojom::PaymentRequestClientPtr client);
+  explicit PaymentRequestDialog(PaymentRequestImpl* impl);
   ~PaymentRequestDialog() override;
 
   // views::WidgetDelegate
   ui::ModalType GetModalType() const override;
 
   // views::View
   gfx::Size GetPreferredSize() const override;
 
   // views::DialogDelegate
   bool Cancel() override;
 
  private:
-  payments::mojom::PaymentRequestClientPtr client_;
+  PaymentRequestImpl* impl_;  // Not owned.

[please use gerrit instead, 2016/11/28 15:16:41]

Clearly describe the lifetime of PaymentRequestImpl and PaymentRequestDialog in
comments. Raw pointers with comments "not owned" are scary.

[anthonyvd, 2016/11/28 16:12:18]

Done.

Totally agree with you. Added a comment, let me know what you think. 

   std::unique_ptr<views::Label> label_;
 
   DISALLOW_COPY_AND_ASSIGN(PaymentRequestDialog);
 };
 
 }  // namespace payments
 
-#endif  // CHROME_BROWSER_PAYMENTS_UI_PAYMENT_REQUEST_DIALOG_H_
+#endif  // CHROME_BROWSER_UI_VIEWS_PAYMENTS_PAYMENT_REQUEST_DIALOG_H_