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

Unified Diff: content/browser/payments/payment_app_manager_unittest.cc

Issue 2562873002: [PaymentApp] label field was changed to name field in PaymentAppOption. (Closed)
Patch Set: rebase Created 4 years 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: content/browser/payments/payment_app_manager_unittest.cc
diff --git a/content/browser/payments/payment_app_manager_unittest.cc b/content/browser/payments/payment_app_manager_unittest.cc
index 78dd9facf5d61de7ca7050c9a3017cbac2433064..3ad55132cb4281a9d16ca4ab4566b1171ab95f2f 100644
--- a/content/browser/payments/payment_app_manager_unittest.cc
+++ b/content/browser/payments/payment_app_manager_unittest.cc
@@ -59,7 +59,7 @@ void GetManifestCallback(bool* called,
payments::mojom::PaymentAppManifestPtr CreatePaymentAppManifestForTest() {
payments::mojom::PaymentAppOptionPtr option =
payments::mojom::PaymentAppOption::New();
- option->label = "Visa ****";
+ option->name = "Visa ****";
option->id = "payment-app-id";
option->icon = std::string("payment-app-icon");
option->enabled_methods.push_back("visa");
@@ -67,7 +67,7 @@ payments::mojom::PaymentAppManifestPtr CreatePaymentAppManifestForTest() {
payments::mojom::PaymentAppManifestPtr manifest =
payments::mojom::PaymentAppManifest::New();
manifest->icon = std::string("payment-app-icon");
- manifest->label = "Payment App";
+ manifest->name = "Payment App";
manifest->options.push_back(std::move(option));
return manifest;
@@ -160,10 +160,10 @@ TEST_F(PaymentAppManagerTest, SetAndGetManifest) {
ASSERT_TRUE(called);
ASSERT_EQ(read_error, payments::mojom::PaymentAppManifestError::NONE);
EXPECT_EQ(read_manifest->icon, std::string("payment-app-icon"));
- EXPECT_EQ(read_manifest->label, "Payment App");
+ EXPECT_EQ(read_manifest->name, "Payment App");
ASSERT_EQ(read_manifest->options.size(), 1U);
EXPECT_EQ(read_manifest->options[0]->icon, std::string("payment-app-icon"));
- EXPECT_EQ(read_manifest->options[0]->label, "Visa ****");
+ EXPECT_EQ(read_manifest->options[0]->name, "Visa ****");
EXPECT_EQ(read_manifest->options[0]->id, "payment-app-id");
ASSERT_EQ(read_manifest->options[0]->enabled_methods.size(), 1U);
EXPECT_EQ(read_manifest->options[0]->enabled_methods[0], "visa");

Powered by Google App Engine
This is Rietveld 408576698