1616280 - Investigate using shadow DOM to hide Remote L10n translations for from local translations

Status: RESOLVED FIXED

(Firefox :: Messaging System, defect, P2)

Description

[Andrei Oprea [:andreio]]

Why: we would be able to use HTML markup in our remote translations. For context any messaging surface that uses Remote L10n now cannot support HTML markup.

The global Localization instance does not expect another one (like the Remote L10n one) to exist and will overwrite our translations. We can hide content from it by using custom elements and then connecting the root to the Remote L10n localization. By default the mutation observer does not check the shadow DOM.

Comment 1

[Andrei Oprea [:andreio]]

Attachment: Bug 1616280 - Use shadow DOM to hide Remote L10n translations for from local translations

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/7afc6bac547c
Use shadow DOM to hide Remote L10n translations for from local translations r=k88hudson

Comment 3

[Cristina Coroiu [:ccoroiu]]

Backed out changeset 7afc6bac547c (bug 1616280) for browser-chrome failures at dom/l10n/DOMLocalization.cpp

Backout: https://hg.mozilla.org/integration/autoland/rev/6aa0ab986adc18ef357b57d48913531a9e874577

Failure push: https://treeherder.mozilla.org/#/jobs?repo=autoland&revision=7afc6bac547c9a5135c40c07cb05de6905caf034

Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=298119543&repo=autoland&lineNumber=3873

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/9e7bb5bc539b
Use shadow DOM to hide Remote L10n translations for from local translations r=k88hudson

Comment 5

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/9e7bb5bc539b

Comment 6

[BugBot [:suhaib / :marco/ :calixte]]

Since the status are different for nightly and release, what's the status for beta?
For more information, please visit auto_nag documentation.