Page 1 of 1

[Fixed] image context menu cause zombie compartment

Posted: Fri Dec 02, 2011 3:09 pm
by dindog
just update to new 2.0b3268, and notice when I right click an image, it will hold a zombie compartment in about:memory.

Here is how I reproduce, no filter subscription needed:
1. open about:memory
2. open http://www.google.com
3. right-click on "google" the image, the context menu pop-up
4. close google tab, observe about:memory
5. the http://www.google.com compartment will stay there forever... until
6. you trigger another pop-up:
  • a) it's an image context menu, then this tab's compartment will take the zombie role..
    b) right click in blank area, the google zombie will be GC in a few second
So it seem the context menu of image hold a global object prevent GC. I am not sure whether it only the image context menu, but you'll easy fix other, if any, to locate the reason of this issue.

Re: image context menu cause zombie compartment

Posted: Mon Dec 05, 2011 2:51 pm
by Wladimir Palant
Please click "Minimize memory usage" in about:memory - does the compartment go away then? If it does then it isn't a zombie compartment.

For reference: I tried to reproduce in a Firefox 11 nightly, without success. I guess you are using Firefox 8?

Re: image context menu cause zombie compartment

Posted: Mon Dec 05, 2011 6:01 pm
by dindog
Nightly and dev version of ABP

I have this consecutive reproduce... just try that in 12-5 nightly and ABP 2.0b.3287. Win7 sp1 32bit

"Minimize memory usage" has no effect.

The key-point is you have to right click on a image, then close the page.

Re: image context menu cause zombie compartment

Posted: Tue Dec 06, 2011 9:58 am
by Wladimir Palant
No problem here with pretty much the same configuration. Are you sure that this isn't some other extension interfering? Adblock Plus definitely doesn't hold any references unless you actually click "Block image" (in which case the dialog opening holds a reference to the image and prevents the compartment from being released).

Re: image context menu cause zombie compartment

Posted: Tue Dec 06, 2011 10:22 am
by dindog
reproduce with fresh new profile, only ADP is installed.

And I also reproduce it in virtual machines, one is WinXP sp3 32bit, the other is Linux Fedora

Re: image context menu cause zombie compartment

Posted: Tue Dec 06, 2011 10:53 am
by Wladimir Palant
Don't know why I couldn't reproduce it at first, now I realized what's going on. Fixed: https://hg.adblockplus.org/adblockplus/rev/34b2b91586ff (will be in the next development build, in 25 minutes). Not a new issue however, this always worked like this (at least since 1.0).