Re: [w3c/payment-handler] Update Example6 and Example7 (#290)

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

Received on Wednesday, 19 September 2018 19:00:27 UTC