[Web Payments] Implement item selection in lists.

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

 // Copyright 2017 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 "base/strings/utf_string_conversions.h"
 #include "chrome/browser/ui/views/payments/payment_request_browsertest_base.h"
 #include "chrome/browser/ui/views/payments/payment_request_dialog_view_ids.h"
 #include "components/autofill/core/browser/autofill_test_utils.h"
 #include "components/autofill/core/browser/personal_data_manager.h"
 #include "components/payments/content/payment_request.h"
@@ skipping 73 matching lines @@
   EXPECT_EQ(request->state()->available_instruments().front().get(),
             request->state()->selected_instrument());
   views::View* checkmark_view = list_view->child_at(0)->GetViewByID(
       static_cast<int>(DialogViewID::CHECKMARK_VIEW));
   EXPECT_TRUE(checkmark_view->visible());
 
   views::View* checkmark_view2 = list_view->child_at(1)->GetViewByID(
       static_cast<int>(DialogViewID::CHECKMARK_VIEW));
   EXPECT_FALSE(checkmark_view2->visible());
 
+  ResetEventObserver(DialogEvent::BACK_NAVIGATION);
   // Simulate selecting the second card.
   ClickOnDialogViewAndWait(list_view->child_at(1));
 
   EXPECT_EQ(request->state()->available_instruments().back().get(),
             request->state()->selected_instrument());
+
+  OpenPaymentMethodScreen();
+  list_view = dialog_view()->GetViewByID(
+      static_cast<int>(DialogViewID::PAYMENT_METHOD_SHEET_LIST_VIEW));
+  // Clicking on the second card again should not modify any state.
+  ClickOnDialogViewAndWait(list_view->child_at(1));

[Mathieu, 2017/03/21 19:27:36]

Do we not expect it to go back to the payment method screen? In a way, you're
"confirming" your choice...

[anthonyvd, 2017/03/21 20:29:14]

Good question, we can ask Bruno!

+
+  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());
 
-  // Clicking on the second card again should not modify any state.
-  ClickOnDialogViewAndWait(list_view->child_at(1));
-
   EXPECT_EQ(request->state()->available_instruments().back().get(),
             request->state()->selected_instrument());
-  EXPECT_FALSE(checkmark_view->visible());
-  EXPECT_TRUE(checkmark_view2->visible());
 }
 
 }  // namespace payments