-
Notifications
You must be signed in to change notification settings - Fork 23
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
Update Firefox extension to WebExtension. #47
Comments
Any news on this? |
So far no. I might have to suck it up and port it in a couple days before users start noticing that BPM is broken. I probably will do a shit job, but 🤷♂️. |
Update: This is going to be harder than it should be. However, I have a hutch changing the namespace might do. On the bright side, I think I fixed the options page. |
I think I'm going to owe you something nice if you pull this off. |
I think I pulled it off, and it was due to Just changing a few lines made it port nearly flawlessly (see below for why), on top of changing the manifest (which could be easily dictated in the Makefile. For instance, this line (cr-background.js)... chrome.browser.onMessage.addListener(function(message, sender, sendResponse) { ...had to be changed to... chrome.runtime.onMessage.addListener(function(message, sender, sendResponse) { Everything in the below tests work in both Chrome Canary and Firefox Nightly: Tested:
|
Our saviour Shugabuga. Thank you. |
Glad I was able to help out! Also, if anyone needs to use this build immediately (i.e. the latest version of BPM no longer runs in Firefox), I have compiled and listed it here. |
I wrote an alternative patch over at #49, which I think would be nicer long-term. |
Tomorrow legacy extensions are going to be disabled by default in Firefox Nightly 57 and in a month in Beta and in a month after that in Stable.
https://mail.mozilla.org/pipermail/dev-addons/2017-August/003059.html
The text was updated successfully, but these errors were encountered: