[WebPayments] Factor out sheet-specific logic in Controllers.

chrome/browser/ui/views/payments/payment_sheet_view_controller.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_UI_VIEWS_PAYMENTS_PAYMENT_SHEET_VIEW_CONTROLLER_H_
+#define CHROME_BROWSER_UI_VIEWS_PAYMENTS_PAYMENT_SHEET_VIEW_CONTROLLER_H_
+
+#include "chrome/browser/ui/views/payments/payment_request_sheet_controller.h"
+

[please use gerrit instead, 2016/12/15 19:11:15]

No need for a newline.

[anthonyvd, 2016/12/15 19:55:07]

Done.

+#include "ui/views/controls/button/button.h"
+
+namespace views {
+class View;

[please use gerrit instead, 2016/12/15 19:11:16]

No need to forward-declare or #include in the header the types that are used
only to override parent methods.

[anthonyvd, 2016/12/15 19:55:07]

Done.

+}
+
+namespace payments {
+
+// The PaymentRequestSheetController subtype for the Payment Sheet screen of the
+// Payment Request dialog.
+class PaymentSheetViewController : public PaymentRequestSheetController,
+                                   public views::ButtonListener {
+ public:
+  PaymentSheetViewController(PaymentRequestImpl* impl,

[please use gerrit instead, 2016/12/15 19:11:15]

Forward declare PaymentRequestImpl and PaymentRequestDialog. Add a comment about
ownership of impl and dialog parameters.

[anthonyvd, 2016/12/15 19:55:07]

Forward declare: done.
Added ownership comment to base class (which this ctor only forwards to).

+                             PaymentRequestDialog* dialog);
+  ~PaymentSheetViewController() override;
+
+  // PaymentRequestSheetController:
+  std::unique_ptr<views::View> CreateView() override;
+
+ private:
+  // views::ButtonListener:
+  void ButtonPressed(views::Button* sender, const ui::Event& event) override;
+
+  DISALLOW_COPY_AND_ASSIGN(PaymentSheetViewController);

[please use gerrit instead, 2016/12/15 19:11:16]

#include "base/macros.h"

[anthonyvd, 2016/12/15 19:55:07]

Done.

+};
+
+}  // namespace payments
+
+#endif  // CHROME_BROWSER_UI_VIEWS_PAYMENTS_PAYMENT_SHEET_VIEW_CONTROLLER_H_