[Webkit-unassigned] [Bug 238291] [WebAssembly][Modules] Support Wasm module import from a JS Worker module

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Mar 29 23:31:51 PDT 2022


https://bugs.webkit.org/show_bug.cgi?id=238291

--- Comment #8 from Yusuke Suzuki <ysuzuki at apple.com> ---
(In reply to Asumu Takikawa from comment #7)
> I've just refined this patch to solve the problem in IMO a better way.
> 
> Instead of modifying WorkerEventLoop.cpp at all, this adds a new parameter
> called `useTimeout` to WorkerRunLoop's `runInMode`. It defaults to false,
> but when it is true, the run loop will trigger timer-based timeouts even in
> a non-default run mode (such as for module loading).
> 
> Then in the use of the run loop for Worker module loading, we set the
> parameter to true and schedule a timer so that eventually the run loop will
> time out and run JS VM microtasks if needed.
> 
> This should also fix the test failure that came up in the previous patch
> with `LayoutTests/http/wpt/resource-timing/rt-initiatorType.worker.html`.

Hmmm, I think probably the original approach would be simpler because it requires 20ms delay to run timer-scheduled tasks. If we are loading content from memory-cache, then I think 20ms is slow.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20220330/74fa4572/attachment.htm>


More information about the webkit-unassigned mailing list