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

meta charset and meta equiv="Content-Type" seem unable to switch away from UTF-8 #24898

Open
pshaughn opened this issue Nov 28, 2019 · 3 comments
Labels
A-content/parsers Related to parsing HTML and XML A-content/script Related to the script thread

Comments

@pshaughn
Copy link
Member

WPT test html/semantics/scripting-1/the-script-element/script-charset-02 tests out various things that the <meta charset=> tag should do, and none of them are happening. Charsets do exist and can be selected other ways, but the meta tag isn't doing it in this test.

@jdm jdm added A-content/parsers Related to parsing HTML and XML A-content/script Related to the script thread labels Nov 28, 2019
@pshaughn
Copy link
Member Author

pshaughn commented Dec 3, 2019

It looks like every WPT test in html/syntax that tries to get to a non-utf8 charset via a meta tag, either charset or http-equiv, fails by staying in utf8.

  • html/syntax/parsing-html-fragments/the-input-byte-stream-007.html
  • html/syntax/parsing-html-fragments/the-input-byte-stream-009.html
  • html/syntax/parsing-html-fragments/the-input-byte-stream-030.html
  • html/syntax/parsing/meta-inhead-insertion-mode.html
  • html/syntax/parsing/quotes-in-meta.html

@pshaughn pshaughn changed the title meta charset="windows-1250" seems to do nothing meta charset and meta equiv="Content-Type" seem unable to switch away from UTF-8 Dec 3, 2019
@pshaughn
Copy link
Member Author

pshaughn commented Dec 4, 2019

Since this needs to happen very early during parsing, and not via DOM, it is probably not dependent on whether http-equiv is commented out in the IDL; charset isn't even in the IDL and is supposed to work.

@jdm
Copy link
Member

jdm commented Jan 23, 2020

This is related to #6414.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-content/parsers Related to parsing HTML and XML A-content/script Related to the script thread
Projects
Development

No branches or pull requests

2 participants