Closed Bug 719782 Opened 12 years ago Closed 12 years ago

Addon Customize Your Web causes zombie compartment

Categories

(Core :: General, defect)

9 Branch
x86
Windows 7
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: u433711, Unassigned)

References

Details

(Whiteboard: [MemShrink:P3])

Attachments

(2 files)

Attached file google_cyw.xml
User Agent: Mozilla/5.0 (Windows NT 6.1; rv:9.0.1) Gecko/20100101 Firefox/9.0.1
Build ID: 20111220165912

Steps to reproduce:

What steps will reproduce the problem?
1. open firefox
2. browse to about:memory
3. review the memory compartments
4. open a new tab within firefox and browse to www.google.com
5. again review the memory compartments
6. notice the google compartment?
7. now close the tab on which you opened www.google.com
8. refresh the memorypage a couple of times, the google compartment should be closed
9. now add the script from the attachment in cyw
10. make sure the script is active
11. now again op a new tab with www.google.com
12. review the memory and notice the google compartment
13. now again close the tab with www.google.com
14. refresh the memory page several times.
15. on my machine the google compartment remains open resulting in a so-called zombie compartment

The attachment contains a script which can be loaded in Customize Your Web. Just opening tab, browsing to www.google.com and closing a tab is enough to trigger the creation of the zombie compartment.

I have used Firefox 9.01 and CYW 0.4.3.3Build201201122030 on Windows 7 US-English



Actual results:

CYW (Customize Your Web) can create shortcut actions on webpages. Creating a shortcut action in CYW results in a zombie compartment.


Expected results:

The compartment should be gone
Whiteboard: [MemShrink]
> The development of CYW is currently suspended.

Again, a great candidate for a warning message on the AMO page.
Whiteboard: [MemShrink] → [MemShrink:P3]
The add-on was updated in January, so I wouldn't call its development really suspended. I have contacted the developer.
Thank you for your investigations.
I've found the code part and changed it and uploaded a new version to amo.

Diffs of changed parts can be found here whereas the only interessing parts are in AbstractShortcutAction and ListviewAction
http://code.google.com/p/customizeyourweb/source/detail?r=105
Thanks for fixing, Rudolf.  The current version listed on AMO is 0.4.3.3Build201201122030, does that have the fix?  I notice that the add-on description starts with "Warning: This addon has a memory leak, thus increasing the memory consumption of Firefox over time!" -- thanks for your honesty, that's very considerate of your users.

If that is the updated version, if com.bugzilla@schutterhoef.nl can confirm that it fixes the problem that would be great.
I just reviewed and tested version 0.4.3.4Build201202102300. It closed the memory leak.
Status: UNCONFIRMED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Just as a formality, I have also tested the issue and have found it to be resolved. Thanks Rudolf for fixing.
You need to log in before you can comment on or make changes to this bug.