RESOLVED FIXED 14683
Incorrect charset at http://star.vnet.cn
https://bugs.webkit.org/show_bug.cgi?id=14683
Summary Incorrect charset at http://star.vnet.cn
Jungshik Shin
Reported 2007-07-19 22:46:53 PDT
The lefthand side of the above page is decoded as the default charset instead of gb2312. The righthand side is correctly decoded as gb2312. The lefthand side is loaded into an iframe in the page. The document loaded into the iframe at http://star.vnet.cn/Comments/three.asp?ChannelID=116&ClassID=170 does not specify charset anywhere (html meta or http). On the other hand, its parent document has meta charset declaration. In this scenario, usually inheriting charset from its parent helps.
Attachments
Alexey Proskuryakov
Comment 1 2007-07-20 00:03:05 PDT
We do not inherit the charset from parent frame to make Google Images (and Google Cache) work better, see bug 6118. This also fixes sites that use User-Agent sniffing to choose an encoding (e.g. the main frame's server may send x-mac-cyrillic to Safari, while a subframe may rely on the browser default being windows-1251, as that's the Windows Cyrillic encoding). We also match Firefox here. So, although it's a regression from shipping Safari/WebKit for this site, I think it's an evangelism issue.
Alexey Proskuryakov
Comment 2 2007-07-20 03:17:08 PDT
One way to fix this site without breaking the cases I mentioned above would be to inherit charsets specified in META, but not in an actual HTTP header. But it's not clear whether such a change would fix more sites than it would break.
Darin Adler
Comment 3 2007-07-20 10:17:15 PDT
Matching other browsers is more important than matching older versions of Safari, so I think Alexey's right and we want the behavior we have currently.
Alexey Proskuryakov
Comment 4 2007-07-21 13:11:59 PDT
Moving to evangelism component.
Robert Blaut
Comment 5 2008-10-12 23:21:57 PDT
The URL is dead.
Alexey Proskuryakov
Comment 6 2008-10-14 03:55:50 PDT
Closing now. Note that we have actually changed the behavior to inherit the charset as long as security considerations allow (other browsers did change in the meanwhile, too).
Note You need to log in before you can comment on or make changes to this bug.