Closed Bug 1793572 Opened 2 years ago Closed 2 years ago

Migrate WebRTC strings from .properties to Fluent

Categories

(Core :: WebRTC: Audio/Video, task)

task

Tracking

()

RESOLVED FIXED
107 Branch
Tracking Status
firefox107 --- fixed

People

(Reporter: eemeli, Assigned: eemeli)

References

(Blocks 1 open bug)

Details

Attachments

(3 files)

Some of these strings currently use the legacy PluralForm API, which we'd like to drop.

Assignee: nobody → earo
Status: NEW → ASSIGNED

The list of items is now formatted using ListFormat rather than manually, which requires a new message webrtc-sharing-menuitem. Based on a review of the existing solutions [1], it should be acceptable to use the same pattern for all of them, even though this may effect a small change for the zh-TW locale.

[1] https://transvision.flod.org/string/?entity=browser/chrome/browser/browser.properties:getUserMedia.sharingMenuCamera&repo=gecko_strings

Depends on D158760

The code is slightly cleaned up to not construct identifiers by concatenating strings, and to avoid using the requestTypes array when determining the prompt contents.

A few messages are moved to and from browser.ftl to group them better, though all of the "WebRTC Pop-up notifications" messages could also be moved from there to webrtcIndicator.ftl, which is now also included by browser.xhtml.

The message webrtc-allow-share-audio-capture-unsafe-delegation is added, as it appears to have been missed previously -- it's referred to in code, but the message itself was not defined.

Depends on D158761

Pushed by earo@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/7f7572b89897
Convert webrtcIndicator.properties to Fluent. r=mconley,flod
https://hg.mozilla.org/integration/autoland/rev/774cecf1d9d8
Convert getUserMedia.sharingMenu strings from browser.properties to Fluent. r=mconley,flod
https://hg.mozilla.org/integration/autoland/rev/bf745ae89c66
Convert WebRTCParent.jsm from .properties to Fluent. r=mconley,flod
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 107 Branch
Regressions: 1805319
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: