1402053 - Remove our obsolete implementation of Unicode normalization, now that we can use ICU on all platforms

Status: RESOLVED FIXED

(Core :: Internationalization, enhancement)

Description

[Jonathan Kew [:jfkthame]]

Now that we have ICU available on all platforms, we can completely remove our obsolete implementation of normalization.

Comment 1

[Jonathan Kew [:jfkthame]]

Attachment: Remove our obsolete implementation of Unicode normalization, now that we can use ICU on all platforms

Comment 2

[Jonathan Kew [:jfkthame]]

Comment on attachment 8910879 [details] [diff] [review]
Remove our obsolete implementation of Unicode normalization, now that we can use ICU on all platforms

Review of attachment 8910879 [details] [diff] [review]:
-----------------------------------------------------------------

Hmm, splinter isn't showing this as nicely as I'd have liked: what I actually did was to "hg rm nsUnicodeNormalizer.cpp" and then "hg mv nsUnicodeNormalizer_ICU.cpp nsUnicodeNormalizer.cpp", but it doesn't indicate the rename, it just looks like an edit.

Comment 3

[Makoto Kato [:m_kato]]

Comment on attachment 8910879 [details] [diff] [review]
Remove our obsolete implementation of Unicode normalization, now that we can use ICU on all platforms

Review of attachment 8910879 [details] [diff] [review]:
-----------------------------------------------------------------

We can remove nsIUnicodeNormalizer after resolving bug 1402048.  But since there is unused #include "nsIUnicodeNormalizer.h" in c-c, I will file a bug for c-c.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/513b9e8f28e6
Remove our obsolete implementation of Unicode normalization, now that we can use ICU on all platforms. r=m_kato

Comment 5

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/513b9e8f28e6