Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

"fetch a module script tree" is asynchronous #230

Closed
annevk opened this issue Jun 22, 2016 · 0 comments · Fixed by #375
Closed

"fetch a module script tree" is asynchronous #230

annevk opened this issue Jun 22, 2016 · 0 comments · Fixed by #375
Assignees

Comments

@annevk
Copy link
Member

annevk commented Jun 22, 2016

The specification assumes it isn't. @bzbarsky

bfgeek added a commit that referenced this issue Apr 6, 2017
Instead of relying on "in parallel" this change makes all the thread
hopping explicit via. message passing and queue tasks on the appropriate
event loop.

 - Fixes #372, changes arguments to fetch a module script graph.

 - Fixes #370, running scripts in parallel. Now the script explicitly are
   run within a task queued on the worklet global scope's event loop.

 - Fixes #318, actually runs the event loop.

 - Fixes #230, treats "fetch a module worker script graph" as
   asynchronous.

 - Probably fixes #225 ? I think I'm grabbing the correct state at each
   thread hop.
@tabatkins tabatkins added the ready label Apr 6, 2017
bfgeek added a commit that referenced this issue Apr 11, 2017
* [worklets] Multiple fixes to the {{Worklet/import()}} algorithm.

Instead of relying on "in parallel" this change makes all the thread
hopping explicit via. message passing and queue tasks on the appropriate
event loop.

 - Fixes #372, changes arguments to fetch a module script graph.

 - Fixes #370, running scripts in parallel. Now the script explicitly are
   run within a task queued on the worklet global scope's event loop.

 - Fixes #318, actually runs the event loop.

 - Fixes #230, treats "fetch a module worker script graph" as
   asynchronous.

 - Probably fixes #225 ? I think I'm grabbing the correct state at each
   thread hop.
@tabatkins tabatkins removed the ready label Apr 11, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants