1647858 - Enable dom.manifest.enabled for all channels

Status: RESOLVED FIXED

(GeckoView :: General, task)

Description

[:Harald Kirschner :digitarald]

We want to ship a Web App Manifest inspection tool for DevTools.

• GeckoView supports manifests and most devs will optimize for GeckoView by opening Desktop Firefox and RDM
• We have feedback from developers that miss PWA inspection in Firefox.

dom.manifest.enabled is only enabled in Nightly and GeckoView. We'd want it enabled for all channels.

Marcos and I had a chat and agreed that we are both not aware of immediate risks. Developers should not depend on feature detection for manifest support, as most of its behavior is implicitly handled by browsers using the information from a static JSON file.

Comment 1

[:Harald Kirschner :digitarald]

Marcos, does this reflect your understanding as well? If so, we'd want to enable the App panel for 79 – do you think we can get this change in (could also be uplifted)?

Comment 2

[Marcos Caceres [:marcosc]]

(In reply to :Harald Kirschner :digitarald from comment #1)

Marcos, does this reflect your understanding as well? If so, we'd want to enable the App panel for 79 – do you think we can get this change in (could also be uplifted)?

yes, that sounds great! Anything I can do to help?

Comment 3

[Marcos Caceres [:marcosc]]

Attachment: Enable dom.manifest.enabled for all channels

Comment 4

[Marcos Caceres [:marcosc]]

try:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=98e86c22e79b00b29affd382406ee4ebde9d6791

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/27fec7a6d8ce
Enable dom.manifest.enabled for all channels r=Honza

Comment 6

[Cristian Brindusan [:cbrindusan]]

https://hg.mozilla.org/mozilla-central/rev/27fec7a6d8ce