1719748 - Unify SharedIntlData in SpiderMonkey

Status: RESOLVED FIXED

(Core :: Internationalization, task, P3)

Description

[Greg Tatum [:gregtatum] (REO Fx132)]

https://searchfox.org/mozilla-central/source/js/src/builtin/intl/SharedIntlData.cpp

There is a call to get the calendars, and there are some enumerations of locale information. This could be split into two bugs probably if someone wanted to do some smaller work.

Comment 1

[Dan Minor [:dminor]]

We shouldn't start on this until we've unified more stuff. Blocking this on LanguageTag for now, we can revisit once that is complete.

Comment 2

[André Bargull [:anba]]

Attachment: Bug 1719748 - Part 1: Add an enumeration over the list of available locales. r=#platform-i18n-reviewers!

Comment 3

[André Bargull [:anba]]

Attachment: Bug 1719748 - Part 2: Add Collator::GetFirstCase(). r=#platform-i18n-reviewers!

Depends on D126487

Comment 4

[André Bargull [:anba]]

Attachment: Bug 1719748 - Part 3: Replace ICU calls when determining the list of upper-case first locales. r=#platform-i18n-reviewers!

Depends on D126488

Comment 5

[André Bargull [:anba]]

Attachment: Bug 1719748 - Part 4: Add TimeZone::GetAvailableTimeZones(). r=#platform-i18n-reviewers!

Depends on D126489

Comment 6

[André Bargull [:anba]]

Attachment: Bug 1719748 - Part 5: Replace ICU calls when determining the list of available time zones. r=#platform-i18n-reviewers!

Depends on D126490

Comment 7

[André Bargull [:anba]]

Attachment: Bug 1719748 - Part 6: Add an enumeration over the list of available locales. r=#platform-i18n-reviewers!

Depends on D126491

Comment 8

[André Bargull [:anba]]

Attachment: Bug 1719748 - Part 7: Replace remaining ICU calls in SharedIntlData. r=#platform-i18n-reviewers!

Depends on D126492

Comment 9

[André Bargull [:anba]]

Attachment: Bug 1719748 - Part 8: Add mozilla::intl::Locale::GetDefaultLocale(). r=#platform-i18n-reviewers!

Add GetDefaultLocale() to replace the two calls to uloc_getDefault().

Depends on D126493

Comment 10

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/c086451993cf
Part 1: Add an enumeration over the list of available locales. r=platform-i18n-reviewers,gregtatum
https://hg.mozilla.org/integration/autoland/rev/afe700a3d101
Part 2: Add Collator::GetFirstCase(). r=platform-i18n-reviewers,gregtatum
https://hg.mozilla.org/integration/autoland/rev/14276449fbef
Part 3: Replace ICU calls when determining the list of upper-case first locales. r=platform-i18n-reviewers,gregtatum
https://hg.mozilla.org/integration/autoland/rev/007974a9be0c
Part 4: Add TimeZone::GetAvailableTimeZones(). r=platform-i18n-reviewers,gregtatum
https://hg.mozilla.org/integration/autoland/rev/98c5261756e1
Part 5: Replace ICU calls when determining the list of available time zones. r=platform-i18n-reviewers,gregtatum
https://hg.mozilla.org/integration/autoland/rev/6b6c1a7f22a6
Part 6: Add an enumeration over the list of available locales. r=platform-i18n-reviewers,gregtatum
https://hg.mozilla.org/integration/autoland/rev/c9f7e931dda0
Part 7: Replace remaining ICU calls in SharedIntlData. r=platform-i18n-reviewers,gregtatum
https://hg.mozilla.org/integration/autoland/rev/73554dd49d46
Part 8: Add mozilla::intl::Locale::GetDefaultLocale(). r=platform-i18n-reviewers,gregtatum

Comment 11

[Cristian Tuns]

https://hg.mozilla.org/mozilla-central/rev/c086451993cf
https://hg.mozilla.org/mozilla-central/rev/afe700a3d101
https://hg.mozilla.org/mozilla-central/rev/14276449fbef
https://hg.mozilla.org/mozilla-central/rev/007974a9be0c
https://hg.mozilla.org/mozilla-central/rev/98c5261756e1
https://hg.mozilla.org/mozilla-central/rev/6b6c1a7f22a6
https://hg.mozilla.org/mozilla-central/rev/c9f7e931dda0
https://hg.mozilla.org/mozilla-central/rev/73554dd49d46