Closed Bug 1366827 Opened 3 years ago Closed 3 years ago

Allow webextension experiments on release


(WebExtensions :: Experiments, enhancement)

Not set


(firefox55 fixed)

Tracking Status
firefox55 --- fixed


(Reporter: aswan, Assigned: _6a68)



(1 file)

The main Test Pilot extension and individual Test Pilot experiments are in the process of moving away from the Add-on SDK.  The goal is to use webextension experiments, but of course they'll need those to be usable on release.

If we land this change in 55, then AMO-signed experiments could theoretically run on release in 55 and 56 but only admins can sign experiments with the AMO CA.  Beginning in 57, only experiments signed with "Mozilla Extensions" will be installable.
Jared, should I re-assign this one to you?
Flags: needinfo?(jhirsch)
Yeah! Let's do it
Flags: needinfo?(jhirsch)
Assignee: aswan → jhirsch
There are a few places where we just need to remove existing RELEASE_OR_BETA checks:

And then tests that we need to not skip on release/beta:
A couple of instances in toolkit/mozapps/extensions/test/xpcshell/test_webextension.js
The skip-if clauses in .ini files in toolkit/components/extensions/test/xpcshell
Comment on attachment 8873938 [details]
Bug 1366827 - Enable WebExtension experiments in Beta and Release.

r=me with the comments addressed

::: toolkit/mozapps/extensions/internal/XPIInstall.jsm:160
(Diff revision 1)
>    dictionary: 64,
>    experiment: 128,
>  };
> -if (!AppConstants.RELEASE_OR_BETA)
> -   TYPES.apiextension = 256;
> +TYPES.apiextension = 256;

you can now just put this in the declaration above

::: toolkit/mozapps/extensions/internal/XPIProvider.jsm:257
(Diff revision 1)
>    dictionary: 64,
>    experiment: 128,
>  };
> -if (!AppConstants.RELEASE_OR_BETA)
> -  TYPES.apiextension = 256;
> +TYPES.apiextension = 256;

ditto the previous comment
Attachment #8873938 - Flags: review?(aswan) → review+
Pushed by
Enable WebExtension experiments in Beta and Release. r=aswan
Keywords: checkin-needed
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
Product: Toolkit → WebExtensions
You need to log in before you can comment on or make changes to this bug.