[Webkit-unassigned] [Bug 28760] Make the canonical names (of TextEncoding) robust to changes in ICU's alias table

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=28760





--- Comment #3 from Jungshik Shin <jshin at chromium.org>  2009-09-02 12:47:35 PDT ---
(In reply to comment #2)
> Won't this change what JavaScript code sees as document.charset? If so, there's
> certain potential for negative web site compatibility effects - which is
> difficult to justify by ease of writing regression tests in my opinion.

In theory, all those JS codes and server-side codes behind them should do the
case-insensitive matching for charset names, but in practice, you're right that
there's a risk. I'll see what other browsers emit for document.charset (capital
or lowercase).

It seems that it's only GB18030 vs gb18030 that has changed in ICU 4.2. An
alternative to lowercasing all charset names is to special-case GB18030 vs
gb18030 in TextCodecICU.cpp probably enclosed with #ifdef (to make it ICU
version-specific).

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.