Last Comment Bug 747167 - HttpFox extension leaks many content compartments while running
: HttpFox extension leaks many content compartments while running
Product: Tech Evangelism
Classification: Other
Component: Add-ons (show other bugs)
: unspecified
: All All
-- normal (vote)
: ---
Assigned To: Nobody; OK to take it and work on it
Depends on:
Blocks: LeakyAddons
  Show dependency treegraph
Reported: 2012-04-19 13:57 PDT by Wladimir Palant
Modified: 2012-06-07 14:57 PDT (History)
7 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---

Patch that fixes some memory leaks (472 bytes, text/plain)
2012-05-16 14:15 PDT, asgerklasker
no flags Details

Description User image Wladimir Palant 2012-04-19 13:57:22 PDT
Just noticed that my Firefox has grown to 1 GB even though I restarted it yesterday. Turned out that I left HttpFox running and forgot about it. As a result I got tons of leaked content compartments.

Steps to reproduce:

* Install HttpFox 0.8.10 from
* After browser restart click the HttpFox icon in the toolbar, click "Start" in the HttpFox bottombar and close it.
* Open in a new tab and close the tab again.
* Check about:compartments

Expected results:

about:compartments doesn't show as an active compartment.

Actual results: shows up under "User Compartments" and "Ghost Windows". Opening HttpFox bottombar and clearing the entries allows the compartments to be released. Not sure why some sites leak while others don't.

Unfortunately, it is very easy to forget that HttpFox is running. A while ago I argued that it should stop collecting data when its bottombar is closed ( but nothing happened. But even if it keeps collecting data, there is no reason why it should leak compartments.
Comment 1 User image Nicholas Nethercote [:njn] 2012-04-19 17:23:43 PDT
HttpFox has 107,000 users according to AMO.
Comment 2 User image Jorge Villalobos [:jorgev] 2012-04-20 07:50:43 PDT
The developer has been contacted through AMO.
Comment 3 User image asgerklasker 2012-05-16 14:15:31 PDT
Created attachment 624532 [details]
Patch that fixes some memory leaks

There are two rather trivial bugs that prevent the important cleanup code from running. I've attached a patch that fixes them. I can't find the author's email address, so I'm posting it here.
Comment 4 User image Andrew Williamson [:eviljeff] 2012-05-23 08:35:57 PDT
Developer chased via info request.  I've mentioned the patches contributed and warned that we will downgrade if they don't make an effort to address the leak.
Comment 5 User image pappkamerad 2012-05-27 14:30:35 PDT
thanks for the patch. new version 0.8.11 was just uploaded and is waiting to be reviewed.

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