[WebPayments] Implement state transitions in desktop WebPayments dialog.

chrome/browser/ui/views/payments/order_summary_state.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/ui/views/payments/order_summary_state.h"
+
+#include "base/observer_list.h"
+#include "base/strings/utf_string_conversions.h"
+#include "ui/views/controls/button/md_text_button.h"
+#include "ui/views/controls/label.h"
+#include "ui/views/layout/grid_layout.h"
+
+namespace payments {
+
+const int kBackButtonTag = 0;

[sky, 2016/12/08 03:58:16]

constexpr, also move into anonymous namespace and add description.

[anthonyvd, 2016/12/08 20:31:22]

Done.

+
+OrderSummaryState::OrderSummaryState() {}
+
+OrderSummaryState::~OrderSummaryState() {}
+
+views::View* OrderSummaryState::CreateView() {
+  views::View* view = new views::View();
+  view->SetSize(gfx::Size(300, 300));

[sky, 2016/12/08 03:58:17]

Why are you setting the size? The size should come from the layout manager, or
the parent.

[anthonyvd, 2016/12/08 20:31:22]

Done.

+
+  views::GridLayout* layout = new views::GridLayout(view);
+  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);
+  layout->AddView(new views::Label(base::ASCIIToUTF16("Order Summary")));

[sky, 2016/12/08 03:58:16]

Shouldn't you be looking these names up from resources?

[anthonyvd, 2016/12/08 20:31:22]

Those controls are there temporarily to have something in the view (since the
layout itself isn't implemented at all yet). I changed the label strings to pull
their value from resources as those are here to stay. As for the buttons, those
are going to be replaced (by icons in some cases and clickable sections in some
others) so I left the temporary strings in.

+
+  layout->StartRow(0, 0);
+  views::LabelButton* back_button =
+      views::MdTextButton::CreateSecondaryUiBlueButton(
+          this, base::ASCIIToUTF16("Back"));
+  back_button->set_tag(kBackButtonTag);
+  layout->AddView(back_button);
+
+  return view;
+}
+
+void OrderSummaryState::AddObserver(Observer* observer) {
+  observers_.AddObserver(observer);
+}
+
+void OrderSummaryState::RemoveObserver(Observer* observer) {
+  observers_.RemoveObserver(observer);
+}
+
+void OrderSummaryState::ButtonPressed(
+    views::Button* sender, const ui::Event& event) {
+  if (!interactable())
+    return;
+
+  if (sender->tag() == kBackButtonTag) {

[sky, 2016/12/08 03:58:17]

DCHECK_EQ as there is only one button.

[anthonyvd, 2016/12/08 20:31:22]

Done.

+    for (auto& observer : observers_) {
+      observer.OnBackClicked();
+    }
+  }
+}
+
+}  // namespace payments