Make it compatible with electrolysis

mkiol / GNotifier

Thunderbird add-on that replaces built-in notifications with the OS native notifications

https://addons.mozilla.org/thunderbird/addon/gnotifier/

GNU General Public License v3.0

164 stars 25 forks source link

Make it compatible with electrolysis #144

Closed moimael closed 7 years ago

moimael commented 7 years ago

Hi,

e10s start to be deployed to more and more people, and this addon is marked as not compatible with it. Could you check if this is easily feasible ? From what I see it might need some work to port it to WebExtensions.

https://compatibility-lookup.services.mozilla.com/ https://developer.mozilla.org/en-US/Add-ons/Working_with_multiprocess_Firefox

Thanks.

mkiol commented 7 years ago

Thank you for bringing this up. I didn't notice any issues related to multiprocess, so upcoming new version will be marked as "compatible". You right, GNotifier uses some low-level and XPCOM apis, but fortunately, it seems that it doesn't cause any incompatibilities with electrolysis.

[duplicate of #133]

mkiol commented 7 years ago

Latest release is marked as "compatible". If you find any problems related to electrolysis please reopen this issue.