Bug 1792882 - test_interfaces.js should not check pref values. r=smaug

Differential Revision: https://phabricator.services.mozilla.com/D158359
This commit is contained in:
Peter Van der Beken 2022-09-30 06:39:09 +00:00
parent ef29c5398a
commit 5a37b5b0bc

View File

@ -3,6 +3,13 @@
// This is a list of all interfaces that are exposed to every webpage.
// Please only add things to this list with great care and proper review
// from the associated module peers.
//
// The test is supposed to check whether our actual exposure behavior
// matches what we expect, with the latter expressed in terms of outside
// observables like type of build (nightly, release), platform, secure
// context, etc. Testing based on prefs is thus the wrong check, as this
// means we'd also have to test whether the pref value matches what we
// expect in terms of outside observables.
// This file lists global interfaces we want exposed and verifies they
// are what we intend. Each entry in the arrays below can either be a
@ -661,9 +668,7 @@ let interfaceNamesInGlobalScope = [
{
name: "InstallTrigger",
insecureContext: true,
disabled: !SpecialPowers.Services.prefs.getBoolPref(
"extensions.InstallTrigger.enabled"
),
disabled: isEarlyBetaOrEarlier,
},
// IMPORTANT: Do not change this list without review from a DOM peer!
{ name: "IntersectionObserver", insecureContext: true },