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

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

Issue 2602213002: [PaymentApp] Fix a crash when enabledMethods called. (Closed)
Patch Set: [Payment] fix a crash when enabledMethods called. Created 3 years, 12 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
« no previous file with comments | « no previous file | third_party/WebKit/Source/modules/payments/PaymentAppManager.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 5cb34aee4306e5600b83394b2712e965154b2533..dba3cf6a8ecaf650fa1ed366164040e63356914d 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
@@ -190,4 +190,30 @@ promise_test(test => {
});
}, 'If there is no manifest data, then throws AbortError.');
+promise_test(test => {
+ var registration;
+ var script_url = 'resources/empty-worker.js';
+ var scope = 'resources/';
+
+ return service_worker_unregister_and_register(test, script_url, scope)
+ .then(r => {
+ registration = r;
+ return wait_for_state(test, registration.installing, 'activated');
+ })
+ .then(state => {
+ assert_equals(state, 'activated');
+ return registration.paymentAppManager.setManifest({
+ name: 'Payment App',
+ icon: 'payment-app-icon',
+ options: [{
+ name: 'Visa ****',
+ icon: 'payment-app-icon',
+ id: 'payment-app-id'
+ }]
+ });
+ })
+ .catch(unreached_rejection(test));
+ }, 'enabled_methods of PaymentAppOption is not required member. '
+ + 'so this test should be pass.');
+
</script>
« no previous file with comments | « no previous file | third_party/WebKit/Source/modules/payments/PaymentAppManager.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698