Re: [w3c/ServiceWorker] async waitUntil microtask issue (#1213)

I guess the setTimeout() case is already tested here:

https://github.com/w3c/web-platform-tests/blob/a16738c524a65946b0ff7df6be37ae076ff5e860/service-workers/service-worker/resources/extendable-event-async-waituntil.js#L17

So I guess we can just mark this test case as returning OK:

https://github.com/w3c/web-platform-tests/blob/a16738c524a65946b0ff7df6be37ae076ff5e860/service-workers/service-worker/resources/extendable-event-async-waituntil.js#L20

Maybe with a comment that its ok since the dispatching flag is set.

If we really wanted to we could add a 3rd test case that adds a setTimeout() before the async_microtask_waituntil to show that it should fail if the dispatch flag is not set.

-- 
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/issues/1213#issuecomment-338304564

Received on Friday, 20 October 2017 19:44:35 UTC