681206 - loadFrameScript has no unload counterpart to allow bootstrapped addons to properly clean up

Status: RESOLVED FIXED

(Core :: IPC, defect)

Description

[Will Elwood (:Will)]

When a bootstrapped addon is shutdown, any global frame scripts it has previously loaded with the allow delayed load flag will remain and continue to execute for new tabs.

Every time the addon is started without restarting the application, frame scripts multiply.

Hopefully a solution similar to bug 481603 can be found.

Comment 1

[Josh Matthews [:jdm]]

Olli, it does strike me as strange that there's no way to stop a content script from being loaded in the future, but I'm surprised that this hasn't been an issue until now.

Comment 2

[Olli Pettay [:smaug][[e-mail]]]

Attachment: patch

Comment 3

[Olli Pettay [:smaug][[e-mail]]]

http://hg.mozilla.org/mozilla-central/rev/e0acef471ab2

Comment 4

[Eric Shepherd [:sheppy]]

Is this on Aurora (Fx9)?

Comment 5

[Olli Pettay [:smaug][[e-mail]]]

Yes

Comment 6

[Eric Shepherd [:sheppy]]

Documentation updated:

https://developer.mozilla.org/en/XPCOM_Interface_Reference/nsIChromeFrameMessageManager
https://developer.mozilla.org/en/The_message_manager
https://developer.mozilla.org/en/Firefox/Updating_add-ons_for_Firefox_9#Bootstrapped_add-ons_can_remove_delayed-load_scripts

Also mentioned on Firefox 9 for developers.