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

Unified Diff: third_party/WebKit/LayoutTests/http/tests/payments/payment-app-manager.html

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: third_party/WebKit/LayoutTests/http/tests/payments/payment-app-manager.html
diff --git a/third_party/WebKit/LayoutTests/http/tests/payments/payment-app-manager.html b/third_party/WebKit/LayoutTests/http/tests/payments/payment-app-manager.html
index 05cbe8d127a6658dfd77944df688bccf95205969..5cb34aee4306e5600b83394b2712e965154b2533 100644
--- a/third_party/WebKit/LayoutTests/http/tests/payments/payment-app-manager.html
+++ b/third_party/WebKit/LayoutTests/http/tests/payments/payment-app-manager.html
@@ -17,7 +17,14 @@ promise_test(test => {
.then(state => {
assert_equals(state, 'activated');
return registration.paymentAppManager.setManifest({
- label: 'Payment App'
+ name: 'Payment App',
+ icon: 'payment-app-icon',
+ options: [{
+ name: 'Visa ****',
+ icon: 'payment-app-icon',
+ id: 'payment-app-id',
+ enabledMethods: ['visa']
+ }]
});
})
.then(result => {
@@ -40,7 +47,14 @@ promise_test(test => {
.then(state => {
assert_equals(state, 'installed');
return registration.paymentAppManager.setManifest({
- label: 'Payment App'
+ name: 'Payment App',
+ icon: 'payment-app-icon',
+ options: [{
+ name: 'Visa ****',
+ icon: 'payment-app-icon',
+ id: 'payment-app-id',
+ enabledMethods: ['visa']
+ }]
});
})
.then(result => {
@@ -67,7 +81,14 @@ promise_test(test => {
assert_equals(registration.waiting, null);
assert_equals(registration.active, null);
return registration.paymentAppManager.setManifest({
- label: 'Payment App'
+ name: 'Payment App',
+ icon: 'payment-app-icon',
+ options: [{
+ name: 'Visa ****',
+ icon: 'payment-app-icon',
+ id: 'payment-app-id',
+ enabledMethods: ['visa']
+ }]
});
})
.then(unreached_fulfillment(test))
@@ -82,10 +103,10 @@ promise_test(test => {
var script_url = 'resources/empty-worker.js';
var scope = 'resources/';
var manifest = {
- label: 'Payment App',
+ name: 'Payment App',
icon: 'payment-app-icon',
options: [{
- label: 'Visa ****',
+ name: 'Visa ****',
icon: 'payment-app-icon',
id: 'payment-app-id',
enabledMethods: ['visa2']
@@ -116,10 +137,10 @@ promise_test(test => {
var script_url = 'resources/empty-worker.js';
var scope = 'resources/';
var manifest = {
- label: 'Payment App',
+ name: 'Payment App',
icon: 'payment-app-icon',
options: [{
- label: 'Visa ****',
+ name: 'Visa ****',
icon: 'payment-app-icon',
id: 'payment-app-id',
enabledMethods: ['visa2']

Powered by Google App Engine
This is Rietveld 408576698