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

Investigate per-document event queues #6281

Closed
jdm opened this issue Jun 4, 2015 · 2 comments
Closed

Investigate per-document event queues #6281

jdm opened this issue Jun 4, 2015 · 2 comments

Comments

@jdm
Copy link
Member

@jdm jdm commented Jun 4, 2015

Currently we effectively have a per-origin event queue, and it's mixed in with the general script task events too. We should look into implementing a design like the one proposed for Gecko in https://bugzilla.mozilla.org/show_bug.cgi?id=715376, which yields benefits like tracking CPU usage per page (https://bugzilla.mozilla.org/show_bug.cgi?id=924582).

@jdm jdm added the A-content/script label Jun 4, 2015
@jdm
Copy link
Member Author

@jdm jdm commented Jul 20, 2015

Consider this for #5236. Advantages also include having to pass around far fewer PipelineId values and having the ScriptTask do much less routing. Additionally, any events queued after a document chooses to shut down can just be ignored, but they can also sit in the bfcache.

@jdm
Copy link
Member Author

@jdm jdm commented Apr 19, 2016

Gecko's proposed model is the one we're already using.

@jdm jdm closed this Apr 19, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant
You can’t perform that action at this time.