-
Notifications
You must be signed in to change notification settings - Fork 6
Description
Hello,
I'm doing some changes to this extension to be able to run it on Firefox (which will support WebExtensions soon) Fork here https://github.com/nukeador/pwdhash
I've found some functions that are not going to be implemented in Firefox because in fact they are deprecated by Chrome and I guess it should be safe to fix them here too with the recommended ones.
extension.onRequest
is nowruntime.onMessage
extension.sendRequest
is nowruntime.sendMessage
With these changes I'm able to make it work on Firefox but just using the @@
key, using F2 I get errors about event.initTextEvent
not implemented (which is not).
Ignoring the changes to the manifest.json to make it load, these changes should be fine back for the Chrome extension too.
It would be great if the rest of the code could be cross-browser in the future when Firefox implements all WebExtension APIs.
Cheers