New issue
Advanced search Search tips

Issue 648534 link

Starred by 6 users

Issue metadata

Status: Assigned
Owner:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Task



Sign in to add a comment

IMplement Fast(er) DOM Accessors

Project Member Reported by vogelheim@chromium.org, Sep 20 2016

Issue description

Change description:
Re-implement select DOM accessors for improved speed:

Use V8-assembled accessors, which allows us to re-implement several simple DOM accessors (firstChild, nextChild, previousSibling, nextSibling) so that V8 can compile the accessor code directly within V8, at least in the common case. This way, an access to these accessors from JavaScript can be executed entirely within the V8 runtime and does not require V8 to jump back into the C++ runtime and into the Blink bindings.



Changes to API surface:
n/a

(The implementation follows https://dom.spec.whatwg.org/#interface-node, as does the current implementation. This work should not be observable except for performance.)


Links:
- Spec: https://dom.spec.whatwg.org/#interface-node
- Not quite-up-to-date design doc: 
https://docs.google.com/document/d/1vUikYhhI6LlD2GYpijrPTm5f5MuouMJ0AQhFhFH2XGY/view
- Implementation tracking bug: https://crbug.com/508898


Support in other browsers:
(I didn't check, but I'm fairly certain they all implement DOM access.)


Other:
[Label OWP-Type-ChangeBehavior] is technically wrong, since this doesn't change behaviour. I picked it because I didn't find a label for performance-only change.
 

Comment 1 by owe...@chromium.org, Sep 12 2017

Labels: migrated-launch-owp Type-Task
This issue has been automatically relabelled type=task because type=launch-owp issues are now officially deprecated. The deprecation is because they were creating confusion about how to get launch approvals, which should be instead done via type=launch issues.

We recommend this issue be used for implementation tracking (for public visibility), but if you already have an issue for that, you may mark this as duplicate.

For more details see here: https://docs.google.com/document/d/1JA6RohjtZQc26bTrGoIE_bSXGXUDQz8vc6G0n_sZJ2o/edit

For any questions, please contact owencm, sshruthi, larforge

Sign in to add a comment