1867400 - Remove devtools.inspector.compatibility.enabled pref

Status: RESOLVED FIXED

(DevTools :: Inspector: Compatibility, task)

Description

[Daniel Holbert [:dholbert]]

We've got devtools.inspector.compatibility.enabled included in our bucket of "Experimental Features":
https://searchfox.org/mozilla-central/rev/12ea2c521cdd071a6d25b0894f31f8f23b18b76a/toolkit/components/featuregates/Features.toml#34-43

but it's been default-enabled on all channels since bug 1750619.

It seems like we should:
(a) remove it from the "Experimental features" listing
(b) maybe remove the pref altogether?

(Removing it from the experimental features listing would be useful, since it reduces noise in the about:preferences#experimental UI a bit, for a feature that's not-really-experimental-anymore. It also reduces noise in crash-report UI, where this shows up in the protected data ExperimentalFeatures data field, despite not being a useful differentiator since it's on by default for everyone.)

ni=daisuke since I think he

Comment 1

[Daniel Holbert [:dholbert]]

(In reply to Daniel Holbert [:dholbert] from comment #0)

ni=daisuke since I think he

er, sorry, submitted midsentence -- was going to say:

ni=daisuke to confirm that this is fine & maybe take this if interested, since I think he was one of the main folks working on this feature, based on the top portion of bug 1584464.

Comment 2

[Daisuke Akatsuka (:daisuke)]

Hi Daniel! Thank you very much for letting me know this.
Indeed, it seems that we don't need it anymore. I will fix it.

Comment 3

[Daisuke Akatsuka (:daisuke)]

Attachment: Bug 1867400: Remove devtools.inspector.compatibility.enabled pref

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/cdf72549c950
Remove devtools.inspector.compatibility.enabled pref r=settings-reviewers,devtools-reviewers,nchevobbe

Comment 5

[Narcis Beleuzu [:NarcisB]]

https://hg.mozilla.org/mozilla-central/rev/cdf72549c950