Last Comment Bug 681206 - loadFrameScript has no unload counterpart to allow bootstrapped addons to properly clean up
: loadFrameScript has no unload counterpart to allow bootstrapped addons to pro...
Status: RESOLVED FIXED
: dev-doc-complete
Product: Core
Classification: Components
Component: IPC (show other bugs)
: unspecified
: All All
: -- normal (vote)
: ---
Assigned To: Olli Pettay [:smaug] (reviewing overload)
:
:
Mentors:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2011-08-23 01:56 PDT by Will Elwood (:Will)
Modified: 2011-11-08 06:05 PST (History)
4 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
patch (3.59 KB, patch)
2011-08-23 11:57 PDT, Olli Pettay [:smaug] (reviewing overload)
jst: review+
Details | Diff | Splinter Review

Description Will Elwood (:Will) 2011-08-23 01:56:08 PDT
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] 2011-08-23 07:16:28 PDT
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] (reviewing overload) 2011-08-23 11:57:15 PDT
Created attachment 555153 [details] [diff] [review]
patch
Comment 3 Olli Pettay [:smaug] (reviewing overload) 2011-08-24 03:48:17 PDT
http://hg.mozilla.org/mozilla-central/rev/e0acef471ab2
Comment 4 Eric Shepherd [:sheppy] 2011-09-30 12:30:49 PDT
Is this on Aurora (Fx9)?
Comment 5 Olli Pettay [:smaug] (reviewing overload) 2011-09-30 12:32:28 PDT
Yes

Note You need to log in before you can comment on or make changes to this bug.