[Fixed] image context menu cause zombie compartment

Posting here is no longer possible, please use the corresponding product forum.
Locked
dindog
Posts: 14
Joined: Sun Dec 12, 2010 7:07 pm

[Fixed] image context menu cause zombie compartment

Post 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.
Wladimir Palant

Re: image context menu cause zombie compartment

Post 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?
dindog
Posts: 14
Joined: Sun Dec 12, 2010 7:07 pm

Re: image context menu cause zombie compartment

Post 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.
Wladimir Palant

Re: image context menu cause zombie compartment

Post 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).
dindog
Posts: 14
Joined: Sun Dec 12, 2010 7:07 pm

Re: image context menu cause zombie compartment

Post 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
Wladimir Palant

Re: image context menu cause zombie compartment

Post 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).
Locked