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
html: Add a test for _charset_. #10623
Conversation
HTML spec change: whatwg/html#3645 URL spec change: whatwg/url#382
<body> | ||
|
||
<iframe name="frame1"></iframe> | ||
<form accept-charset="windows-1252" target="frame1" action="/common/blank.html"> |
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.
Maybe make this value "iso-8859-1" so we can see that the bit of mapping that to "windows-1252" is implemented as well.
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.
Done.
<head> | ||
<meta charset="utf-8"> | ||
<link rel="help" href="https://html.spec.whatwg.org/multipage/form-control-infrastructure.html#constructing-the-form-data-set"> | ||
<link rel="help" href="https://fetch.spec.whatwg.org/#ref-for-utf-8"> |
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.
https://xhr.spec.whatwg.org/#dom-formdata is the reference you want here I think, but I think it's better to just leave out <link rel=help>
as they often go out-of-date.
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.
I changed this to https://fetch.spec.whatwg.org/#concept-bodyinit-extract, and added https://xhr.spec.whatwg.org/#dom-formdata too.
I want to mention Fetch spec here because it explains _charset_
value for new FormData(form)
is "UTF-8"
.
This makes the algorithms match Chrome, Firefox, and Safari. Change to the URL Standard: whatwg/url#382. Tests: web-platform-tests/wpt#10623. Fixes #3560.
This makes the algorithms match Chrome, Firefox, and Safari. Change to the URL Standard: whatwg/url#382. Tests: web-platform-tests/wpt#10623. Fixes whatwg#3560.
HTML spec change: whatwg/html#3645
URL spec change: whatwg/url#382