-
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
Make definition of "origin" less spread out, and more layered #2761
Comments
Yes, please! |
I think this plan is generally sound, but I'm not sure I'd move the core definition from HTML to URL. From what I remember the last time we looked at that it made more sense in HTML or at least it didn't make more sense in URL. |
I don't feel strongly on that, but it does feel like at least large parts of the origin concept apply in other environments, e.g. Node.js, where it's a bit weird to implement part of the HTML spec. |
That is, check the type of response directly, instead of synthesizing an opaque origin for opaque responses and then comparing that to the entry settings object's origin. This helps with #1431 by removing various uses of the entry concept, and closes #2761 by removing the origin concept for image and media elements entirely, since it is now unused.
That is, check the type of response directly, instead of synthesizing an opaque origin for opaque responses and then comparing that to the entry settings object's origin. Closes #2813. Helps with #1431 by removing various uses of the entry concept, and closes #2761 by removing the origin concept for image and media elements entirely, since it is now unused.
Spinning off from #2759 (comment)
It's confusing that "origin" as defined right now is seemingly spread out across DOM/HTML/URL/File API. Not to mention the obsoleted "Origin of a URI" spec.
My suggestion would be:
<dfn>
instead of everyone linking to a central origin definition. This makes it clear they are properties of a (whatever), and their type is origin.Curious what @annevk thinks.
The text was updated successfully, but these errors were encountered: