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

chrome/browser/ui/views/payments/order_summary_view_controller.cc

Index: chrome/browser/ui/views/payments/order_summary_view_controller.cc
diff --git a/chrome/browser/ui/views/payments/order_summary_view_controller.cc b/chrome/browser/ui/views/payments/order_summary_view_controller.cc
new file mode 100644
index 0000000000000000000000000000000000000000..2158461e8f26910c38bfef2807ec07aba08f61f6
--- /dev/null
+++ b/chrome/browser/ui/views/payments/order_summary_view_controller.cc
@@ -0,0 +1,58 @@
+// 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/ui/views/payments/order_summary_view_controller.h"
+
+#include "base/strings/utf_string_conversions.h"
+#include "chrome/browser/ui/views/payments/payment_request_dialog.h"
+#include "chrome/browser/ui/views/payments/payment_request_views_util.h"
+#include "chrome/grit/generated_resources.h"
+#include "ui/base/l10n/l10n_util.h"
+#include "ui/views/controls/button/md_text_button.h"
+#include "ui/views/layout/grid_layout.h"
+
+namespace {
+
+// The tag for the button that navigates back to the payment sheet.
+constexpr int kBackButtonTag = 0;
+
+}  // namespace
+
+namespace payments {
+
+OrderSummaryViewController::OrderSummaryViewController(
+    PaymentRequestImpl* impl, PaymentRequestDialog* dialog)
+  : PaymentRequestSheetController(impl, dialog) {}
+
+OrderSummaryViewController::~OrderSummaryViewController() {}
+
+std::unique_ptr<views::View> OrderSummaryViewController::CreateView() {
+  views::View* content_view = new views::View;

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

#include "ui/views/view.h"

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

Done.

+
+  views::GridLayout* layout = new views::GridLayout(content_view);
+  content_view->SetLayoutManager(layout);
+  views::ColumnSet* columns = layout->AddColumnSet(0);
+  columns->AddColumn(views::GridLayout::TRAILING, views::GridLayout::CENTER,
+                     0, views::GridLayout::USE_PREF, 0, 0);
+
+  layout->StartRow(0, 0);
+  views::LabelButton* back_button =

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

#include "ui/views/controls/button/label_button.h"

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

Done.

+      views::MdTextButton::CreateSecondaryUiBlueButton(
+          this, base::ASCIIToUTF16("Back"));

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

Don't forget to properly i18n this eventually.

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

Absolutely, this is going to be an arrow icon ultimately.

+  back_button->set_tag(kBackButtonTag);
+  layout->AddView(back_button);
+
+  return payments::CreatePaymentView(
+      l10n_util::GetStringUTF16(IDS_PAYMENT_REQUEST_ORDER_SUMMARY_TITLE),

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

Drive-by:

1) Place this string in components/autofill_strings.grdp

2) Let Android use it via formatter_data="android_java"

3) Remove IDS_PAYMENTS_ORDER_SUMMARY_LABEL from android_chrome_strings.grdp.

4) Use R.string.payment_request_order_summary_title instead of
R.string.payments_order_summary_label in PaymentRequestUI.java.

5) Optional: Use IDS_PAYMENT_REQUEST_ORDER_SUMMARY_TITLE instead of
IDS_IOS_PAYMENT_REQUEST_PAYMENT_ITEMS_TITLE.

It's OK to make these changes in a follow up patch. You don't need an Android
build setup, because the change would be mechanical.

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

Will do!

+      content_view);
+}
+
+void OrderSummaryViewController::ButtonPressed(
+    views::Button* sender, const ui::Event& event) {
+  DCHECK_EQ(kBackButtonTag, sender->tag());

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

#include "base/logging.h"

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

Done.

+
+  dialog()->GoBack();
+}
+
+}  // namespace payments