[webkit-reviews] review requested: [Bug 12526] Safari ignores encoding description "charset=Shift_JIS" in invalid html : [Attachment 18110] proposed fix

[bugzilla-daemon at webkit.org]

Alexey Proskuryakov <ap at webkit.org> has asked  for review:
Bug 12526: Safari ignores encoding description "charset=Shift_JIS" in invalid
html
http://bugs.webkit.org/show_bug.cgi?id=12526

Attachment 18110: proposed fix
http://bugs.webkit.org/attachment.cgi?id=18110&action=edit

------- Additional Comments from Alexey Proskuryakov <ap at webkit.org>
This doesn't fix all the examples we have (per comment 3, there are sites that
would require an unacceptably large cut-off), but it fixes quite a few.

I still really dislike the idea that a browser would restart parsing if it sees
a charset declaration anywhere in the document. Hopefully, this brings us close
enough to real world compatibility and we won't have to implement that.