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

WPT failure: DOMContentLoaded fires before a task that is queued from inside a defer script fires #25050

Open
pshaughn opened this issue Dec 3, 2019 · 3 comments
Labels
A-content/dom Interacting with the DOM from web content C-upstream The person attempting to resolve the issue requires further help

Comments

@pshaughn
Copy link
Member

pshaughn commented Dec 3, 2019

https://wpt.fyi/results/html/syntax/parsing/DOMContentLoaded-defer.html?label=experimental&label=master&aligned is solid red at this time; Servo is failing just like everyone else. This looks like it's involving full tasks rather than microtasks, so it's not redundant with #25016.

@jdm jdm added the A-content/dom Interacting with the DOM from web content label Dec 3, 2019
@jdm jdm added this to To do in web-platform-test failures via automation Dec 3, 2019
@pwnorbitals
Copy link

How could one help with this issue ? I'm not sure how to get started

@jdm
Copy link
Member

jdm commented Nov 19, 2020

The test in question is https://github.com/servo/servo/blob/master/tests/wpt/web-platform-tests/html/syntax/parsing/DOMContentLoaded-defer.html (which can be run with ./mach test-wpt tests/wpt/web-platform-tests/html/syntax/parsing/DOMContentLoaded-defer.html). We currently mark this test as an expected failure, but so does every other browser. This suggests that either the actual test does not match the specification, or that all browsers do not implement the specification and (presumably) the specification should be amended to match actual behaviour. We should really file an issue upstream in https://github.com/web-platform-tests/wpt about this.

@jdm jdm added the C-upstream The person attempting to resolve the issue requires further help label Nov 19, 2020
@pwnorbitals
Copy link

Thanks for the guidance.
An issue already exists over there : web-platform-tests/wpt#4267
It had no progress for 4 years, so I will ping the issue there to see if any progress can be made

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-content/dom Interacting with the DOM from web content C-upstream The person attempting to resolve the issue requires further help
Projects
Development

No branches or pull requests

3 participants