Closed Bug 1414872 Opened 2 years ago Closed 2 years ago
Fix handling a source locale as a fallback locale for requested list
59 bytes, text/x-review-board-request
we have a minor issue with how we currently workaround the limitations of `general.useragent.locale`. If the user launches a repackaged build for "it", the LocaleService::GetRequestedLocales will return ["it", "en-US"] as expected. But if the user manually sets the locales via `LocaleService::SetRequestedLocales(["it", "en-US"]);` or `LocaleService::SetRequestedLocales(["it"]);` then GetRequestedLocales will return only ["it"]. This limits us a bit when negotiating locale fallback chains in L20n scenario, and I'd like to have the "en-US" added there, until we can start storing the fallback chain correctly.
Summary: Hardcode "en-US" as LocaleService:SourceLocale → Fix handling a source locale as a fallback locale for requested list
Jonathan - I originally planned to get all `en-US` in LocaleService and place them in a single LocaleService::mSourceLocale but decided against it to make the diff smaller and easier to review since it's close to merge day. In 59 I'll add a SourceLocale (bug 1346877) and move us to a new pref that handles locale list instead of a single locale. For 58, I'd just like to fix this behavior with the most minimal patch. I hope you're ok with that!
Assignee: nobody → gandalf
Status: NEW → ASSIGNED
Comment on attachment 8925581 [details] Bug 1414872 - Fix always adding source locale to requested locales fallback chain. https://reviewboard.mozilla.org/r/196706/#review201934 Ah, I see... GetRequestedLocales normally adds that when initially caching mRequestedLocales, but if we then go through RequestedLocalesChanged then the cached value gets updated there but bypasses the code to add the fallback. OK, this seems fine for now.
Attachment #8925581 - Flags: review?(jfkthame) → review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/f993b9f8f20c Fix always adding source locale to requested locales fallback chain. r=jfkthame
You need to log in before you can comment on or make changes to this bug.