Re: [w3c/ServiceWorker] Add API for postMessage with PostMessageOptions (#1344)

annevk requested changes on this pull request.



> @@ -309,6 +309,7 @@ spec: webappsec-referrer-policy; urlPrefix: https://w3c.github.io/webappsec-refe
         readonly attribute USVString scriptURL;
         readonly attribute ServiceWorkerState state;
         void postMessage(any message, optional sequence<object> transfer = []);
+        void postMessage(any message, PostMessageOptions options);

This should be marked optional, the previous line should have its second argument marked as non-optional (and non-defaulting).

-- 
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/ServiceWorker/pull/1344#pullrequestreview-169209880