|Submitter||Diff||Changes||Open Issues||Last Updated|
|Error loading review requests:|
See bug 1369399, comment 10. This bug has two causes. One, that I'm going to fix here (there's another case of RELEASE_OR_BETA in firefox.js) and another that might require a followup bug. That is that XPIProvider.e10sBlocksEnabling returns *true* for webextensions (including the link cleaner). Felipe, is that intended?
Actually, my question in comment 0 is probably because I was stepping through the code in Nightly and DevEdition is on Aurora, neither of which has an ADDON_ROLLOUT_POLICY. This is working as intended – we should just skip the addon checks for DevEdition all together.
Comment on attachment 8879352 [details] Bug 1374492 - Don't let extensions disable e10s in DevEdition. https://reviewboard.mozilla.org/r/150644/#review156366
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/5898d2436f40 Don't let extensions disable e10s in DevEdition. r=Felipe
So I looked at the patches and manually set `extensions.e10sBlocksEnabling` and `extensions.e10sMultiBlocksEnabling` to false so it has a `status: modified` attribute and that solved the restarting WebExtension issue for me. Thanks Blake! Hope this gets uplifted to DevEdition beta soon.
Please request Beta approval on this when you get a chance.
Comment on attachment 8879352 [details] Bug 1374492 - Don't let extensions disable e10s in DevEdition. Approval Request Comment [Feature/Bug causing the regression]: Project dawn [User impact if declined]: Webextensions (among others) will disable e10s on DevEdition. [Is this code covered by automated tests?]: No :( [Has the fix been verified in Nightly?]: Yes. [Needs manual test from QE? If yes, steps to reproduce]: No. [List of other uplifts needed for the feature/fix]: None. [Is the change risky?]: No. [Why is the change risky/not risky?]: It's restoring previous behavior. [String changes made/needed]: n/a
Comment on attachment 8879352 [details] Bug 1374492 - Don't let extensions disable e10s in DevEdition. enable e10s with webextensions in DevEdition builds, beta55+