- From: Ben Kelly <notifications@github.com>
- Date: Fri, 20 Oct 2017 19:44:07 +0000 (UTC)
- To: w3c/ServiceWorker <ServiceWorker@noreply.github.com>
- Cc: Subscribed <subscribed@noreply.github.com>
Received on Friday, 20 October 2017 19:44:35 UTC
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