1016038 - languagechange is emitted twice for each language change

Status: RESOLVED FIXED

(Firefox OS Graveyard :: General, defect)

Description

[Staś Małolepszy :stas]

I wanted to start using the languagechange event introduced in bug 889335 to handle language changes in the Gaia's l10n library (bug 1013929).

However, it looks like because of how we first reset the value of intl.language_code in http://mxr.mozilla.org/mozilla-central/source/b2g/chrome/content/settings.js#82 (bug 796079) in order to make sure the new value is prepended to the default value of the pref, the event is emitted twice, which doesn't make it very helpful.

This happens in Firefox OS only, because we use mozSettings' language.current there.

Comment 1

[Staś Małolepszy :stas]

Attachment: use getDefaultBranch instead of clearUserPref

Pike told me about getDefaultBranch() which works well here.  Vivien, can you take a look?

Comment 2

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

Comment on attachment 8428837 [details] [diff] [review]
use getDefaultBranch instead of clearUserPref

Review of attachment 8428837 [details] [diff] [review]:
-----------------------------------------------------------------

LGTM.

Comment 3

[Staś Małolepszy :stas]

Note that in the future we might want to change what value we store in language.current.  Right now it's a single locale code, but with bug 1005927, it will make sense to store the whole accept_languages string, which will make the task of updating intl.accept_languages easier.

Comment 4

[Staś Małolepszy :stas]

Comment on attachment 8428837 [details] [diff] [review]
use getDefaultBranch instead of clearUserPref

Try: https://tbpl.mozilla.org/?tree=Try&rev=fc5043fcda26

Comment 5

[Staś Małolepszy :stas]

I realize now that I attached the output of a diff command instead of 'hg export.' My bad.  Let me know if you'd like me to attach a new patch, with correct commit metadata.

Comment 6

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/integration/b2g-inbound/rev/450492b8afb8

I was able to get the commit information from your Try push, so that wasn't a huge deal. One request, though. Please do try to use commit messages that explain what your patch is doing rather than restating the problem it's solving. Thanks :)
https://developer.mozilla.org/en-US/docs/Developer_Guide/Committing_Rules_and_Responsibilities#Checkin_comment

Comment 7

[Staś Małolepszy :stas]

Thanks, Ryan!  I just copy&pasted the summary of this bug, but of course, it would have made sense to reword the commit messages as per the guidelines.  I'll keep that in mind!

Comment 8

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/450492b8afb8