-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Relevant global object #3613
Comments
(My |
Per spec, no. https://heycam.github.io/webidl/#es-platform-objects explicitly says:
And yes, Firefox does change it on adopt.
Mmm. Good question; I hadn't thought about this before because in Firefox the two are always the same. We would probably need to audit all uses of relevant global on nodes and see what they're really after... |
I'm not sure how to approach this issue, as it appears to be all Firefox-specific concerns. |
Concern 2 is only a concern for non-Firefox browers, no? |
Can "Relevant global object", "node's global" and "node document's global" be clearly defined, here? What are the distinctions between the three terms? How to determine and verify each case? |
Is |
See https://html.spec.whatwg.org/multipage/webappapis.html#relevant "node's global" and "node document's global" are shorthand for "the relevant global object of the node" and "the relevant global object of the node's node document", in the https://dom.spec.whatwg.org/#concept-node-document sense.
I'm not sure what you're asking. For purposes of this issue, what probably needs to happen is identifying all APIs exposed on nodes that use the "relevant global" concept, testing each one with a node that has a different-global node document, and seeing what browsers do in practice in terms of which global they use. |
Not yet gathering the full scope of the inquiry. This is what tried so far
Trying to get |
The relevant global of everything in your testcase is the same. The testcase of interest that involves
But the bigger question is what should happen for all APIs on nodes that use the relevant global. |
I wonder about two things:
window.event
: Extra window.event tests web-platform-tests/wpt#10329.)cc @bzbarsky @domenic
The text was updated successfully, but these errors were encountered: