1292486 - Get rid of nsIUnicodeNormalizer interface

Status: RESOLVED FIXED

(Core :: Internationalization, defect, P3)

Description

[Makoto Kato [:m_kato]]

After fix bug 1292476, we can remove this interface if ICU is turned on.

But some Addons uses this interface, so we might have to remove this on mobile only...

Comment 1

[Makoto Kato [:m_kato]]

Attachment: Bug 1292486 - Get rid of nsIUnicodeNormalizer interface.

m-c and c-c don't use nsIUnicodeNormalizer interface now.

Review commit: https://reviewboard.mozilla.org/r/191508/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/191508/

Comment 2

[Jonathan Kew [:jfkthame]]

Comment on attachment 8920508 [details]
Bug 1292486 - Get rid of nsIUnicodeNormalizer interface.

https://reviewboard.mozilla.org/r/191508/#review196710

Yes, it's time to get rid of this.

You should also delete the normalization_data.h file, only used by nsUnicodeNormalizer.cpp.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/dcf5bf004564
Get rid of nsIUnicodeNormalizer interface. r=jfkthame

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/5dd84312ab4c
Port bug 1292486: Get rid of nsIUnicodeNormalizer interface (remove unicharutil.xpt from package manifests). rs=bustage-fix

Comment 5

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

https://hg.mozilla.org/mozilla-central/rev/dcf5bf004564