Closed Bug 109344 Opened 23 years ago Closed 23 years ago

the tail of 20 windows

Categories

(Core :: XUL, defect)

x86
Linux
defect
Not set
normal

Tracking

()

RESOLVED DUPLICATE of bug 45353

People

(Reporter: blythe, Assigned: blythe)

References

Details

Leak detected using SpaceTrace after a run of 20 about:blank windows opened, and
then closed all at once.  19 leaks of same 12288 size.

An a similar run, 20 about:blank windows, opening one at a time from the parent,
closing the parent, and so on, only 2 of these sized objects were leaked (the
lifespan of the allocations indicated they may have been early on in the cycle).

All seemed to revolve around XUL workings in the callstack.

I'd like a crack at fixing this as a learning experience if not already known
about and a fix under progress.
Hyatt says waterson just rewrote the area of nsXULElement code that I have
narrowed what I believe to be the leak down to.

Will need to get waterson to suggest the fix to implement, as it is a hard case
for me to consider without knowing much of what is going on.
Status: NEW → ASSIGNED
Summary: the tail of 20 windows → the tail of 20 windows
Heh, I think you found bug 45353. We should probably just rid of the attribute
freelist stuff, as it's not really as important that attributes be allocated
quickly anymore.
Depends on: 45353
You are of course correct, 45353 turns out to be what I've found.

The more windows at one time open, the more free list chunks allocated.

I can take on attempting to remove the free list code.

Rest of my comments to be found in original 45353 bug.
nevermind, waterson already has the patch in 45353.

*** This bug has been marked as a duplicate of 45353 ***
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → DUPLICATE
Component: XP Toolkit/Widgets: XUL → XUL
QA Contact: jrgmorrison → xptoolkit.widgets
You need to log in before you can comment on or make changes to this bug.