[webkit-reviews] review granted: [Bug 22116] Revise manual charset alias in TextCodecICU : [Attachment 25723] updated patch (with more fleshed-out log entries)

[bugzilla-daemon at webkit.org]

Alexey Proskuryakov <ap at webkit.org> has granted Jungshik Shin
<jshin at chromium.org>'s request for review:
Bug 22116: Revise manual charset alias in TextCodecICU
https://bugs.webkit.org/show_bug.cgi?id=22116

Attachment 25723: updated patch (with more fleshed-out log entries)
https://bugs.webkit.org/attachment.cgi?id=25723&action=review

------- Additional Comments from Alexey Proskuryakov <ap at webkit.org>
> However, EUC-CN will not be picked up in the double-for-loop (going through
> IANA/MIME names and their aliases) because EUC-CN in convrtrs.txt is NOT
tagged
> with any standard.

According to online converter explorer, it has both MIME and IANA standard
names (GB2312 and csGB2312), so it should be picked. But I'm probably a little
confused: ChangeLog says "Maps 'EUC-CN' to 'GBK'", while I don't see any
changes to EUC-CN behavior in code, only an added test for it, which passes in
ToT.

+// FIX: Have to add tests for Euro and a few new characters added to
ISO-8859-x 

We use FIXME for such comments.

All substantial changes look good, and this patch significantly cleans up code
and tests, r=me!