Revert "[WebPayments] Clicking selected row goes back"

This reverts commit 90a73c37.

Reason for revert: <INSERT REASONING HERE>

Original change's description:
> [WebPayments] Clicking selected row goes back
> 
> Changing behavior when clicking on the currently selected item in an
> ItemList. Was a no-op; now goes back to the main sheet.
> 
> Bug: 748552
> Change-Id: I43f716145c059b6d95d5f01526b1224528fb0ad3
> Reviewed-on: https://chromium-review.googlesource.com/585370
> Reviewed-by: Anthony Vallee-Dubois <anthonyvd@chromium.org>
> Commit-Queue: Tommy Martino <tmartino@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#491751}

TBR=anthonyvd@chromium.org,tmartino@chromium.org

Change-Id: I8a5d7de3da2f0e7f47bff11e9c6980e31590c38d
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 748552
Reviewed-on: https://chromium-review.googlesource.com/600847Reviewed-by: Luna Lu <loonybear@chromium.org>
Commit-Queue: Luna Lu <loonybear@chromium.org>
Cr-Commit-Position: refs/heads/master@{#491790}

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

@@ -186,8 +186,7 @@ PaymentMethodViewController::PaymentMethodViewController(
     PaymentRequestSpec* spec,
     PaymentRequestState* state,
     PaymentRequestDialogView* dialog)
-    : PaymentRequestSheetController(spec, state, dialog),
-      payment_method_list_(dialog) {
+    : PaymentRequestSheetController(spec, state, dialog) {
   const std::vector<std::unique_ptr<PaymentInstrument>>& available_instruments =
       state->available_instruments();
   for (const auto& instrument : available_instruments) {

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

@@ -100,12 +100,16 @@ IN_PROC_BROWSER_TEST_F(PaymentMethodViewControllerTest,
   OpenPaymentMethodScreen();
   list_view = dialog_view()->GetViewByID(
       static_cast<int>(DialogViewID::PAYMENT_METHOD_SHEET_LIST_VIEW));
-
-  ResetEventObserver(DialogEvent::BACK_NAVIGATION);
-  // Clicking on the second card again should not modify any state, and should
-  // return to the main payment sheet.
+  // Clicking on the second card again should not modify any state.
   ClickOnDialogViewAndWait(list_view->child_at(1));
 
+  checkmark_view = list_view->child_at(0)->GetViewByID(
+      static_cast<int>(DialogViewID::CHECKMARK_VIEW));
+  checkmark_view2 = list_view->child_at(1)->GetViewByID(
+      static_cast<int>(DialogViewID::CHECKMARK_VIEW));
+  EXPECT_FALSE(checkmark_view->visible());
+  EXPECT_TRUE(checkmark_view2->visible());
+
   EXPECT_EQ(request->state()->available_instruments().back().get(),
             request->state()->selected_instrument());
 }

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

@@ -4,9 +4,6 @@
 
 #include "chrome/browser/ui/views/payments/payment_request_item_list.h"
 
-#include <utility>
-
-#include "chrome/browser/ui/views/payments/payment_request_dialog_view.h"
 #include "chrome/browser/ui/views/payments/payment_request_dialog_view_ids.h"
 #include "chrome/browser/ui/views/payments/payment_request_views_util.h"
 #include "components/payments/content/payment_request_state.h"
@@ -155,10 +152,6 @@ void PaymentRequestItemList::Item::ButtonPressed(views::Button* sender,
                                                  const ui::Event& event) {
   if (sender->id() == static_cast<int>(DialogViewID::EDIT_ITEM_BUTTON)) {
     EditButtonPressed();
-  } else if (selected_) {
-    // |dialog()| may be null in tests
-    if (list_->dialog())
-      list_->dialog()->GoBack();
   } else if (CanBeSelected()) {
     list()->SelectItem(this);
   } else {
@@ -177,8 +170,7 @@ void PaymentRequestItemList::Item::UpdateAccessibleName() {
   SetAccessibleName(accessible_content);
 }
 
-PaymentRequestItemList::PaymentRequestItemList(PaymentRequestDialogView* dialog)
-    : selected_item_(nullptr), dialog_(dialog) {}
+PaymentRequestItemList::PaymentRequestItemList() : selected_item_(nullptr) {}
 
 PaymentRequestItemList::~PaymentRequestItemList() {}
 

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

@@ -19,7 +19,6 @@ class View;
 
 namespace payments {
 
-class PaymentRequestDialogView;
 class PaymentRequestSpec;
 class PaymentRequestState;
 
@@ -124,7 +123,7 @@ class PaymentRequestItemList {
     DISALLOW_COPY_AND_ASSIGN(Item);
   };
 
-  explicit PaymentRequestItemList(PaymentRequestDialogView* dialog);
+  PaymentRequestItemList();
   virtual ~PaymentRequestItemList();
 
   // Adds an item to this list. |item->list()| should return this object.
@@ -141,8 +140,6 @@ class PaymentRequestItemList {
   // Deselects the currently selected item and selects |item| instead.
   void SelectItem(Item* item);
 
-  PaymentRequestDialogView* dialog() { return dialog_; }
-
  private:
   // Unselects the currently selected item. This is private so that the list can
   // use it when selecting a new item while avoiding consumers of this class
@@ -151,7 +148,6 @@ class PaymentRequestItemList {
 
   std::vector<std::unique_ptr<Item>> items_;
   Item* selected_item_;
-  PaymentRequestDialogView* dialog_;
 
   DISALLOW_COPY_AND_ASSIGN(PaymentRequestItemList);
 };

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

@@ -59,7 +59,7 @@ class TestListItem : public PaymentRequestItemList::Item {
 }  // namespace
 
 TEST(PaymentRequestItemListTest, TestAddItem) {
-  PaymentRequestItemList list(nullptr);
+  PaymentRequestItemList list;
 
   std::unique_ptr<views::View> list_view = list.CreateListView();
   EXPECT_FALSE(list_view->has_children());
@@ -90,7 +90,7 @@ TEST(PaymentRequestItemListTest, TestAddItem) {
 }
 
 TEST(PaymentRequestItemListTest, TestSelectItemResultsInSingleItemSelected) {
-  PaymentRequestItemList list(nullptr);
+  PaymentRequestItemList list;
 
   std::vector<std::unique_ptr<TestListItem>> items;
   items.push_back(base::MakeUnique<TestListItem>(&list, false));

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

@@ -368,7 +368,7 @@ ProfileListViewController::ProfileListViewController(
     PaymentRequestSpec* spec,
     PaymentRequestState* state,
     PaymentRequestDialogView* dialog)
-    : PaymentRequestSheetController(spec, state, dialog), list_(dialog) {}
+    : PaymentRequestSheetController(spec, state, dialog) {}
 
 ProfileListViewController::~ProfileListViewController() {}
 

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

@@ -4,8 +4,6 @@
 
 #include "chrome/browser/ui/views/payments/shipping_option_view_controller.h"
 
-#include <memory>
-
 #include "chrome/browser/ui/views/payments/payment_request_dialog_view.h"
 #include "chrome/browser/ui/views/payments/payment_request_views_util.h"
 #include "components/payments/content/payment_request_spec.h"
@@ -84,8 +82,7 @@ ShippingOptionViewController::ShippingOptionViewController(
     PaymentRequestSpec* spec,
     PaymentRequestState* state,
     PaymentRequestDialogView* dialog)
-    : PaymentRequestSheetController(spec, state, dialog),
-      shipping_option_list_(dialog) {
+    : PaymentRequestSheetController(spec, state, dialog) {
   spec->AddObserver(this);
   for (const auto& option : spec->GetShippingOptions()) {
     shipping_option_list_.AddItem(base::MakeUnique<ShippingOptionItem>(