Closed Bug 1524327 Opened 5 years ago Closed 4 years ago



(WebExtensions :: General, enhancement, P5)



(firefox74 fixed)

Tracking Status
firefox74 --- fixed


(Reporter: mixedpuppy, Assigned: mixedpuppy)



(Whiteboard: devrel-note)


(1 file)

At a minimum, we could rename that to ALLOW_EXTENSION_EXPERIMENTS or something, but we have both isPrivileged and experimentsAllowed looking at appconstant and/or a pref. It's not clear why that is being done when looking at the code.

Minimum approved!

Priority: -- → P5

Legacy extensions are no longer loaded, so we can drop the build config for it. We
still need flags for handling experimental APIs since what we require differs between builds
and distributions.

Assignee: nobody → mixedpuppy

Geoff, do we need any of this for Thunderbird or are we good with the way we use WebExtensions now?

Flags: needinfo?(geoff)

I think this is fine but I need to look more closely. Leaving the NI? until I can do that.

This is a problem though:

  get addonExperimentsAllowed() {
    return this.NIGHTLY_BUILD || this.MOZ_DEV_EDITION;

Geoff, I'll land this patch in the next day or two. In addition to the above, any build that has MOZ_REQUIRE_SIGNING set to false will be able to set the preference. Based on what is in mozilla-central, this patch essentially does not change what builds can set the expiremental preference.

Right. That's fine. I'd done nothing here because of a lack of Bugzilla action … but of course everything's happened on Phabricator and I forgot to subscribe to that. :-/

Flags: needinfo?(geoff)

This hasn't gone so well though:

xpcshell.ini:toolkit/mozapps/extensions/test/xpcshell/test_upgrade.js | xpcshell return code: 0
xpcshell.ini:toolkit/mozapps/extensions/test/xpcshell/test_upgrade.js | test_1 - [test_1 : 111] Extension 1 is active - false == true
xpcshell.ini:toolkit/mozapps/extensions/test/xpcshell/test_onPropertyChanged_appDisabled.js | xpcshell return code: 0
xpcshell.ini:toolkit/mozapps/extensions/test/xpcshell/test_onPropertyChanged_appDisabled.js | run_test - [run_test : 29] false == true
xpcshell.ini:toolkit/mozapps/extensions/test/xpcshell/test_strictcompatibility.js | xpcshell return code: 0
xpcshell.ini:toolkit/mozapps/extensions/test/xpcshell/test_strictcompatibility.js | test_1 - [test_1 : 113] Expected value of addon.appDisabled - true deepEqual false

Flags: needinfo?(mixedpuppy)
See Also: → 1608552

Geoff, the latest patch addresses those failures. I'll have to deal with them in more detail in the followup bug 1608552.

Flags: needinfo?(mixedpuppy)

We should announce this far and wide.

Flags: needinfo?(philipp)
Whiteboard: devrel-note
Pushed by
remove MOZ_ALLOW_LEGACY_EXTENSIONS and extensions.legacy.enabled r=zombie,aswan
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla74

There's one reference of MOZ_ALLOW_LEGACY_EXTENSIONS left in toolkit/mozapps/extensions/test/xpcshell/test_temporary.js

Pushed by
remove Thunderbird reference to MOZ_ALLOW_LEGACY_EXTENSIONS define (which was removed in bug 1524327). r=bustage-fix


If this ticket requires manual testing please provide the steps to reproduce/test. Otherwise, the "qe verify -" flag should be set.
Thank you

Flags: needinfo?(mixedpuppy)
Flags: needinfo?(mixedpuppy) → qe-verify-
Pushed by
Remove SeaMonkey reference to MOZ_ALLOW_LEGACY_EXTENSIONS define (which was removed in bug 1524327). r=bustage-fix (DONTBUILD)

Part of this fix adds a line to firefox.js setting pref("extensions.experiments.enabled", false);

Is that intentionally part of this bug? Experiments are not discussed explicitly as part of it anywhere that I see, and it has had the effect of disabling experiments where they were previously enabled.

Regressions: 1628188
See Also: → 1662704
Flags: needinfo?(philipp)
You need to log in before you can comment on or make changes to this bug.