-
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
Remove Unicode serialization of an origin #2689
Conversation
And rename “ASCII serialization of an origin” to “serialization of an origin” though allow both to be used. Not all implementations implemented the Unicode variant and the ones that did, did so in a way that made it hard to use. Fixes #2568.
See whatwg/html#2568 for discussion. See whatwg/html#2689 for the corresponding change to the HTML Standard. Closes #297.
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.
Changes LGTM, although I will let you decide when landing this is good, with regard to the corresponding URL spec changes and tests.
This should land first with a pointer to all the tests and the corresponding URL Standard change. Then the URL Standard change can land a day later when the terms are indexed by Shepherd. I'm not in a rush since I don't think this will conflict much so I'll give folks another 24h or so. |
See whatwg/html#2568 for discussion. See whatwg/html#2689 for the corresponding change to the HTML Standard. Tests: web-platform-tests/wpt#5944. Closes #297.
Made obsolete by whatwg#2689.
And rename “ASCII serialization of an origin” to “serialization of an
origin” though allow both to be used.
Not all implementations implemented the Unicode variant and the ones
that did, did so in a way that made it hard to use.
Fixes #2568.