This is a little more complicated than I expected and might require a small refactor I was planning on anyway, to remove the dependency on scriptloader from moduleloader. This would allow the scriptloadrunnable to be merged with scriptloader and then the worker ref is only kept when we are over on the main thread, which would make sense.
Bug 1823391 Comment 2 Edit History
Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.
This is a little more complicated than I thought. The strong ref is released if the worker finishes; in this case, we are never finishing, and we are never cancelling. I can add code to try to shutdown, but it looks like we are still waiting, so that may not work. Are there other tests failing? It looks like there was a crash somewhere triggering this behavior thats since been removed.
This is a little more complicated than I thought. The strong ref is released if the worker finishes; in this case, we are never finishing, and we are never cancelling. I can add code to try to shutdown, but if we are still waiting for something that may not work. Are there other tests failing? It looks like there was a crash somewhere triggering this behavior thats since been removed.