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

Use ASCII serialization for origins #1151

Merged
merged 1 commit into from
Jun 13, 2019
Merged

Use ASCII serialization for origins #1151

merged 1 commit into from
Jun 13, 2019

Conversation

jungkees
Copy link
Collaborator

@jungkees jungkees commented May 26, 2017

This patch makes it to use ASCII serialization for
(Extendable)MessageEvent.origin instead of Unicode serialization which
has been removed from HTML.

HTML issue: whatwg/html#2568.
HTML change: whatwg/html@c3b2511.

Fixes #1142.


Preview | Diff

@jungkees jungkees requested review from annevk and jakearchibald May 26, 2017 09:24
@jungkees
Copy link
Collaborator Author

jungkees commented May 26, 2017

TODO

  • Tests

@jungkees
Copy link
Collaborator Author

@annevk, so, we should test e.origin returns a punycode string for an origin containing a unicode IDN, right? If so, any good way to set the domain name in Unicode for WPT?

@annevk
Copy link
Member

annevk commented Jun 26, 2017

Yeah, see all the tests I wrote for the change in HTML. There's quite a few tricks to do this and I employed most I think.

@jungkees jungkees mentioned this pull request Sep 12, 2018
@jakearchibald
Copy link
Contributor

@jungkees if there are tests for MessageEvent#origin, can we reuse them for this?

This patch makes it to use ASCII serialization for
(Extendable)MessageEvent.origin instead of Unicode serialization which
has been removed from HTML.

HTML issue: whatwg/html#2568.
HTML change: whatwg/html@c3b2511.

Fixes #1142.
@jungkees
Copy link
Collaborator Author

Merging the PR. I opened an issue in WPT for adding tests.

@jungkees jungkees merged commit a47c6d8 into master Jun 13, 2019
@jungkees jungkees deleted the origin-serialization branch June 13, 2019 18:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use ASCII serialization for origins
3 participants