rsolomakhin requested changes on this pull request.
@romandev: One nit. Otherwise looks OK.
>
- e.source.postMessage(paymentAppResponse);
- window.close();
- }
+ navigator.serviceWorker.controller.postMessage(paymentAppResponse);
+ window.close();
`window.close()` will have no effect (at least in Chrome). Let's remove it. Chrome closes the payment handler window when the service worker responds to the `paymentrequest` event.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/w3c/payment-handler/pull/290#pullrequestreview-156974309