Closed Bug 594009 Opened 14 years ago Closed 13 years ago

Fix tabcandy-central leaks

Categories

(Firefox Graveyard :: Panorama, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX
Future

People

(Reporter: ehsan.akhgari, Unassigned)

References

Details

(Keywords: memory-leak)

I have pushed tabcandy-central to the try server, and I noticed that it's producing leaks on every test which opens the browser.  I bisected the tabcandy-central specific changesets to find the culprit, and this changeset is at least one cause of the leaks:

http://hg.mozilla.org/users/edward.lee_engineering.uiuc.edu/tabcandy-central/rev/12c394eb7eca

As far as I can tell, the window objects stored in the arrays added in that changeset are never cleared.  Ian, maybe we should add this to the cleanup code you wrote before tabcandy landed on mozilla-central?
Version: unspecified → Trunk
Yes, it looks like it! Thank you for tracking it down. 

Since we're moth-balling tabcandy-central until after ff4, I'll keep this bug and have it block bug 578512.
Blocks: 578512
Target Milestone: --- → Future
(In reply to comment #1)
> Since we're moth-balling tabcandy-central until after ff4, I'll keep this bug
> and have it block bug 578512.

Makes sense.
We'll use tabcandy-central as an archive, only.
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → WONTFIX
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.