1034227 - [email] cookie cache DOM not localized on app restart

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::E-Mail, defect)

Description

[James Burke [:jrburke]]

Since the DOM generated by the cookie cache is in the page before l10n.js is loaded, the l10n MutationObserver hooks (the l10n that is on master/2.1) will not see it:

* Load email with one account configured. Assume English as the starting language.
* Change language to Accented English.
* Change the folder to something other than Inbox, like Sent, then go back to Inbox.
* Kill the email app.
* Change the language in Settings app back to English.
* Relaunch email app.

Notice that the Last Sync label and the search text box placeholder at the top of the list of messages is incorrect.

The folder names, like "Inbox" may still be incorrect, but that is being tracked in bug 855198. This is just for the basic UI localization that is not from the backend data.

The fix is in mail_app.js, when it realizes it has a cachedNode, pass it to mozL10n first before starting up the card machinery.

Comment 1

[James Burke [:jrburke]]

Attachment: GitHub pull request

In mail_app.js, if it knows a cachedNode will be used for startup, it passes it to mozL10n.translateFragment first. This fixes the root of the issue.

The change in tmpl.js for bug 1005760 is also removed because now the l10n.js MutationObservers translate the nodes on insertion, so it is just doing extra unneeded work to do this in tmpl.js. This can be confirmed by doing the following:

* Load email with one account configured, open Compose, then close it. Assume start language is English.
* Change the Language in Settings app to "Accented English".
* Go back to email and open the Compose card, it is still correctly translated to Accented English.

Asking asuth to review email changes, and :gandalf to confirm that mozL10n.translateFragment is the correct API to use on master for this change.

Since this changeset relies on a master-only l10n.js API and behavior, it should not be auto-backported to any other branch.

Comment 2

[Zibi Braniecki [:zbraniecki][:gandalf]]

Comment on attachment 8450460 [details] [review]
GitHub pull request

sweet, looks good!

Comment 3

[Andrew Sutherland [:asuth] (he/him)]

Comment on attachment 8450460 [details] [review]
GitHub pull request

Alas, the Hall and Oates song "Private Eyes" does not work quite as well with "Mutation Observers" (so many syllables!), so the reference I was going to use is no good.  But we can all pretend it did.

Comment 4

[Andrew Sutherland [:asuth] (he/him)]

(marking whiteboard so uplift is not considered per https://wiki.mozilla.org/Release_Management/B2G_Landing)

Comment 5

[James Burke [:jrburke]]

Merged in master:
https://github.com/mozilla-b2g/gaia/commit/3db3f62cdf3c3255a912d44c288672c9c36a68e9

from pull request:
https://github.com/mozilla-b2g/gaia/pull/21364

Comment 7

[Andrew Sutherland [:asuth] (he/him)]

Note that this is being addressed on older versions of FxOS via bug 1008859

Comment 8

[Sue]

Attachment: verify_video.MP4

This issue has been verified successfully on Flame v2.1
See attachment: verify_video.MP4
Reproducing rate: 0/5
Flame 2.1 versions:
Gaia-Rev        5655269098c7e82254e56933f1af05b4abe2a2f3
Gecko-Rev       https://hg.mozilla.org/releases/mozilla-b2g34_v2_1/rev/86608c9389b5
Build-ID        20141204001201
Version         34.0
Device-Name     flame
FW-Release      4.4.2
FW-Incremental  eng.cltbld.20141204.034958
FW-Date         Thu Dec  4 03:50:09 EST 2014
Bootloader      L1TC00011880