Closed
Bug 735401
(LA-picpasso)
Opened 13 years ago
Closed 12 years ago
PicPasso add-on creates zombie compartments
Categories
(WebExtensions :: General, defect)
WebExtensions
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: kmag, Unassigned)
References
()
Details
(Whiteboard: [MemShrink:P3])
The PicPasso add-on creates zombie compartments after clicking its toolbar button. The following steps create a zombie compartment, but they may not be the only way to do so, and not all of the steps may be required:
1) Visit http://gmail.com/
2) Press the PicPasso toolbar button (at this point, security is broken for the page)
3) Close the tab
After minimizing memory multiple times, the following related compartments remain:
http://googleads.g.doubleclick.net/pagead/viewthroughconversion/1015888343/?random=1331667169853&cv=7&fst=1331667169853&num=1&fmt=2&value=0&label=aV-CCKGCwgIQ1_O05AM&bg=ffffff&hl=en&guid=ON&u_h=800&u_w=1280&u_ah=800&u_aw=1280&u_cd=24&u_his=1&u_tz=-240&u_java=true&u_nplug=12&u_nmime=199&url=http%3A//www.picpasso.com/register&ctc_id=CAIVAgAAAB0CAAAA&ct_cookie_present=false [pid=Main]
https://accounts.google.com/ServiceLogin?service=mail&passive=true&rm=false&continue=http://mail.google.com/mail/&scc=1<mpl=default<mplcache=2 [pid=Main]
Author CCed.
Comment 1•13 years ago
|
||
Jan: is there any update you can give us on fixing these memory leaks?
Comment 2•13 years ago
|
||
The developer could not reconstruct the memory leaks in Firefox 11, but is currently trying to solve it. I will let you know, if I hear more from him.
Updated•13 years ago
|
Component: General → Add-ons
Product: Core → Tech Evangelism
Comment 3•13 years ago
|
||
Jan: did the developer manage to solve the problem?
Comment 4•13 years ago
|
||
We cannot reconstruct the zombie compartment. Therefore we canot really solve it... Would it be OK to upload a new update, which can be tested again?
Comment 5•13 years ago
|
||
(In reply to Jan Schnitzler from comment #4)
> We cannot reconstruct the zombie compartment. Therefore we canot really
> solve it... Would it be OK to upload a new update, which can be tested again?
If you can't reproduce with your current version with the steps in Comment #0 then yes. We will check it again.
Comment 6•13 years ago
|
||
Jan, are you working on an update for us to test?
Comment 7•13 years ago
|
||
We are currently working on it and i hope we can submit it next week. Sorry for the delay...
Comment 8•13 years ago
|
||
Jan: I see your latest version was reviewed and the editor still found the leak - do you not see the leak with the steps to reproduce in Comment #0?
Comment 9•13 years ago
|
||
Hello all, I'm Igor, the principal developer. Jan is away, so I'll take over.
As Jan said, I followed the steps from Comment #0 but could not reproduce the memory leak.
1. When I start Firefox, I see only one PicPasso-related compartment: picPasso.sqlite.
2. After opening gmail.com, a number of PicPasso-related compartments appear, mainly inner windows with the uri about:blank (for PicPasso's own windows), but also some external.
3. However, when I close the Gmail tab, reopen about:memory and click 10 times on "Minimize memory usage", I see no PicPasso compartements, except the picPasso.sqlite. There are some inner windows, but, to my understanding, they belong to firebug.
Is there a possibility to post screenshots?
Comment 10•13 years ago
|
||
Igor, which version of Firefox are you using? If it's FF15, the fix in bug 695480 might be preventing the zombie compartment from manifesting.
Comment 11•13 years ago
|
||
(In reply to Nicholas Nethercote [:njn] from comment #10)
> Igor, which version of Firefox are you using? If it's FF15, the fix in bug
> 695480 might be preventing the zombie compartment from manifesting.
I'm using Firefox 12 and am referring to PicPasso 1.0.18, i.e. the new version.
Reporter | ||
Comment 12•12 years ago
|
||
I can no longer reproduce this issue.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•6 years ago
|
Component: Add-ons → General
Product: Tech Evangelism → WebExtensions
You need to log in
before you can comment on or make changes to this bug.
Description
•