1511019 - localize "This Firefox" runtime name

Status: VERIFIED FIXED

(DevTools :: about:debugging, enhancement, P1)

Description

[Ola Gasidlo [:ogasidlo] [:ola] OOO - please ping :ladybenko:]

"This Firefox" localized in the sidebar,but it seems we are hard coding it as a `runtime` name (what gets displayed as part of the heading in the runtime page).

So we want to localize it here too.

Comment 1

[Julian Descottes [:jdescottes]]

At the moment the hardcoded "This Firefox" name in the runtimes state is not displayed anywhere. The runtime header for This Firefox only shows the name of app and its version (eg "Firefox Nightly 66 (66.0a1)").

In theory, we could do nothing since the string is not displayed, or make the name string optional in the runtime type and skip it for This Firefox. But I think for consistency it would be better to keep the string and localize it properly. The only issue with this is that the This Firefox runtime is created in the default state right at the beginning, before the localization module has been initialized.

We can either workaround this issue by loading the create-store.js module using a lazyRequire (in practice we need to make sure we don't try to create the state before the l10n module is available). Or we just remove This Firefox from the default state and we let aboutdebugging.js populate the state with This Firefox.

Comment 2

[Julian Descottes [:jdescottes]]

Attachment: Bug 1511019 - Localize runtime name This Firefox in aboutdebugging state;r=ola,ladybenko

This approach forces to have a new action here, but the alternative was to create
the store after initializing l10n.js, which breaks all the tests because our current helper
waitForRequestsSuccess is quite fragile.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/0eafc7a46433
Localize runtime name This Firefox in aboutdebugging state;r=Ola,ladybenko

Comment 4

[Arthur Iakab [arthur_iakab]]

https://hg.mozilla.org/mozilla-central/rev/0eafc7a46433

Comment 5

[Liviu Seplecan]

I've verified on Nightly 68.0a1, Firefox Developer Edition 67.0, release 66.0.1 - on all versions the "This Firefox" will appear at the sidebar - top left side.