Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(4197)

Unified Diff: chrome/browser/ui/views/payments/credit_card_editor_view_controller_browsertest.cc

Issue 2742813004: [Payments] Refactor into PaymentRequestState and Spec (Closed)
Patch Set: don't stop rebasin' Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/views/payments/credit_card_editor_view_controller_browsertest.cc
diff --git a/chrome/browser/ui/views/payments/credit_card_editor_view_controller_browsertest.cc b/chrome/browser/ui/views/payments/credit_card_editor_view_controller_browsertest.cc
index 6d4bb54a9a07b48cb5046ab23d33a24d1f5b32bb..c6323bce1c103076b096830cece6d2bdda7f5e86 100644
--- a/chrome/browser/ui/views/payments/credit_card_editor_view_controller_browsertest.cc
+++ b/chrome/browser/ui/views/payments/credit_card_editor_view_controller_browsertest.cc
@@ -14,6 +14,7 @@
#include "components/autofill/core/browser/personal_data_manager.h"
#include "components/autofill/core/browser/test_autofill_clock.h"
#include "components/payments/content/payment_request.h"
+#include "components/payments/content/payment_request_spec.h"
#include "components/strings/grit/components_strings.h"
#include "content/public/test/browser_test_utils.h"
#include "testing/gtest/include/gtest/gtest.h"
@@ -324,7 +325,7 @@ IN_PROC_BROWSER_TEST_F(PaymentRequestCreditCardBasicCardTest,
GetPaymentRequests(GetActiveWebContents());
EXPECT_EQ(1u, requests.size());
std::vector<std::string> supported_card_networks =
- requests[0]->supported_card_networks();
+ requests[0]->spec()->supported_card_networks();
EXPECT_EQ(2u, supported_card_networks.size());
// The networks appear in the order in which they were specified by the
// merchant.
@@ -342,7 +343,7 @@ IN_PROC_BROWSER_TEST_F(PaymentRequestCreditCardBasicCardTest,
GetPaymentRequests(GetActiveWebContents());
EXPECT_EQ(1u, requests.size());
std::vector<std::string> supported_card_networks =
- requests[0]->supported_card_networks();
+ requests[0]->spec()->supported_card_networks();
// The default ordering is alphabetical.
EXPECT_EQ(8u, supported_card_networks.size());
EXPECT_EQ("amex", supported_card_networks[0]);
@@ -370,7 +371,7 @@ IN_PROC_BROWSER_TEST_F(PaymentRequestCreditCardBasicCardTest,
GetPaymentRequests(GetActiveWebContents());
EXPECT_EQ(1u, requests.size());
std::vector<std::string> supported_card_networks =
- requests[0]->supported_card_networks();
+ requests[0]->spec()->supported_card_networks();
// 'mastercard' is first because it was explicitely specified first. The rest
// is alphabetical.
EXPECT_EQ(8u, supported_card_networks.size());
@@ -397,7 +398,7 @@ IN_PROC_BROWSER_TEST_F(PaymentRequestCreditCardBasicCardTest,
GetPaymentRequests(GetActiveWebContents());
EXPECT_EQ(1u, requests.size());
std::vector<std::string> supported_card_networks =
- requests[0]->supported_card_networks();
+ requests[0]->spec()->supported_card_networks();
// 'visa' is first because it was explicitely specified first. The rest
// is alphabetical.
EXPECT_EQ(8u, supported_card_networks.size());
@@ -429,7 +430,7 @@ IN_PROC_BROWSER_TEST_F(PaymentRequestCreditCardBasicCardTest,
GetPaymentRequests(GetActiveWebContents());
EXPECT_EQ(1u, requests.size());
std::vector<std::string> supported_card_networks =
- requests[0]->supported_card_networks();
+ requests[0]->spec()->supported_card_networks();
EXPECT_EQ(3u, supported_card_networks.size());
EXPECT_EQ("mastercard", supported_card_networks[0]);
EXPECT_EQ("visa", supported_card_networks[1]);

Powered by Google App Engine
This is Rietveld 408576698