-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Make Add-on compatible with e10s (multiprocess) #10
Comments
Yes would be nice -- and easy, as only the add-on manifest needs to be changed to add:
Note that the current situation means people installing the add-on cannot benefit from e10s (except on Developer and Nightly channels). |
Not easy anymore. Firefox 57 (August for Developer Edition, November for Stable) will remove the ability to run XUL extensions, so this needs to be ported to WebExtensions. To get you started, here are the links you'll probably need:
...and given that Firefox is a bit more conservative than Chrome in what it allows that API to override, you need to test it out A.S.A.P. in case it's necessary to go on Bugzilla and get a fix discussed, implemented, and committed before Firefox 57 moves from nightly to aurora channel in August. |
Hi, There is a new addon called "Disable Ctrl-Q and Cmd-Q" that supports webextensions: The problem is that it does not work as expected in Linux, because of the bug https://bugzilla.mozilla.org/show_bug.cgi?id=1325692. So, for now I'm going to stick to the current addon and hope that the bug will be fixed until Firefox 57 be released... |
Friendly update, looks like https://bugzilla.mozilla.org/show_bug.cgi?id=1325692 got a "WONT FIX" which puts linux users of this extension in a particularly interesting position for FF57 |
According to the compatibility reporter and https://arewee10syet.com/, this addon is not compatible with e10s (aka Firefox Multiprocess). The add-on is not at all interacting with tab contents, so it seems like it can't be too hard to establish said compatibility. That would be great, because it would mean that once e10s is rolled out for more people, this add-on won't prevent anyone from using it :)
The text was updated successfully, but these errors were encountered: