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
Add some more tests of NodeList iterability #4663
Conversation
Chrome (unstable channel)Testing web-platform-tests at revision a077848 All results/dom/nodes/NodeList-Iterable.html
|
Firefox (nightly channel)Testing web-platform-tests at revision a077848 All results/dom/nodes/NodeList-Iterable.html
|
dom/nodes/NodeList-Iterable.html
Outdated
} | ||
} | ||
|
||
assert_equals(total, 5); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we also confirm the order in which elements appear?
e.g. we can add class/id/title on each b
to identify each b's.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
7f2cb00
to
eafc8ef
Compare
No idea what the stability bot's problem is this time... |
eafc8ef
to
976f063
Compare
976f063
to
e1e3373
Compare
OK, bot problems were my bad messing up submodules. Good for a new review. |
r=me. |
Adapted from https://github.com/tmpvar/jsdom/blob/8b82e0c39bf856b072130e99cc60ca06ed75f7a7/test/living-dom/node-list.js.