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

RESOLVED FIXED in Firefox 58

Status

()

enhancement
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: jfkthame, Assigned: jfkthame)

Tracking

unspecified
mozilla58
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox58 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

2 years ago
Now that we have ICU available on all platforms, we can completely remove our obsolete implementation of normalization.
(Assignee)

Comment 2

2 years ago
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 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.
Attachment #8910879 - Flags: review?(m_kato) → review+
Blocks: 1402193
Blocks: 1292486

Comment 4

2 years ago
Pushed by jkew@mozilla.com:
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

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/513b9e8f28e6
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla58
You need to log in before you can comment on or make changes to this bug.