Unify intl/locale/MozLocale and intl/components/Locale
Categories
(Core :: Internationalization, task, P3)
Tracking
()
Tracking | Status | |
---|---|---|
firefox96 | --- | fixed |
People
(Reporter: dminor, Assigned: dminor)
References
Details
(Whiteboard: [i18n-unification])
Attachments
(9 files)
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
Details | Review |
Bug 1719746 moved the LanguageTag implementation from SpiderMonkey to mozilla::intl::Locale. Unfortunately, we also have a mozilla:intl::Locale class in intl/Locale/MozLocale.h. This lead to crashes due to the wrong destructor being called!
To workaround this, I renamed the MozLocale version to mozilla::intl::Locale in Bug 1719746. This bug tracks unifying the two implementations.
Assignee | ||
Updated•4 years ago
|
Assignee | ||
Comment 1•4 years ago
|
||
Assignee | ||
Comment 2•4 years ago
|
||
Depends on D129205
Assignee | ||
Comment 3•4 years ago
|
||
Depends on D129206
Assignee | ||
Comment 4•4 years ago
|
||
Depends on D129207
Assignee | ||
Comment 5•4 years ago
|
||
Depends on D129208
Assignee | ||
Comment 6•4 years ago
|
||
Depends on D129209
Assignee | ||
Comment 7•4 years ago
|
||
Depends on D129210
Assignee | ||
Comment 8•4 years ago
|
||
Depends on D129211
Assignee | ||
Comment 9•4 years ago
|
||
Depends on D129212
Comment 10•4 years ago
|
||
Comment 11•4 years ago
|
||
Backed out 9 changesets (Bug 1736017) for causing bustages in nsLineBreaker.cpp
Log: https://treeherder.mozilla.org/logviewer?job_id=356254984&repo=autoland&lineNumber=11416
Backout: https://hg.mozilla.org/integration/autoland/rev/4b9b149dc90ecd2f6d786a3f80c6b684b669ac77
Assignee | ||
Comment 12•4 years ago
|
||
Oops, I forgot to try building after I rebased.
Assignee | ||
Comment 13•4 years ago
|
||
I think we need to figure out how we want to handle _
in the LocaleParser in Bug 1738340 before we can land the patches here.
Assignee | ||
Comment 14•3 years ago
|
||
After some local testing and try runs [1] [2] with an assertion added, I don't think we need to solve Bug 1738340 before landing this. The only location in tree where I'm certain we're using _
as a separator is not affected by the patch stack here.
[1] https://treeherder.mozilla.org/jobs?repo=try&revision=76e4d3a021133837594671f112b85be1bea70868
[2] https://treeherder.mozilla.org/jobs?repo=try&revision=1e13e700e4e7604a0b6eff6ab43ff22390b6335c
Comment 15•3 years ago
|
||
Comment 16•3 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/8cafb816ae31
https://hg.mozilla.org/mozilla-central/rev/98e29bc1f4fb
https://hg.mozilla.org/mozilla-central/rev/31dcf5dbf0b2
https://hg.mozilla.org/mozilla-central/rev/e16018b57faf
https://hg.mozilla.org/mozilla-central/rev/6f6bc4549520
https://hg.mozilla.org/mozilla-central/rev/1055cd0ae3a0
https://hg.mozilla.org/mozilla-central/rev/99bdf30baacd
https://hg.mozilla.org/mozilla-central/rev/fdf26f350832
https://hg.mozilla.org/mozilla-central/rev/5d984460e274
Description
•