Leak nsGlobalWindow hiding a focused iframe

RESOLVED FIXED

Status

()

defect
RESOLVED FIXED
9 years ago
5 months ago

People

(Reporter: jruderman, Assigned: smaug)

Tracking

(Blocks 2 bugs, {memory-leak, testcase})

Trunk
x86
macOS
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(blocking2.0 betaN+)

Details

Attachments

(2 attachments)

1. Load the testcase
2. Quit Firefox

Result: Leak nsGlobalWindow+

If I change focus before quitting, the leak goes away.  So this affects fuzzing more than it affects real life.  But I'd like to be able to fuzz for leaks ;)
blocking2.0: --- → ?
Peter, can you have a look?
Assignee: nobody → peterv
blocking2.0: ? → betaN+
I'm having a hard time reproducing. I managed to make it leak once after trying about 20 times.
I can reproduce this easily on Linux.
I could take a look.
Assignee: peterv → Olli.Pettay
Posted patch patchSplinter Review
When shutting down we should just release the comptrs the singleton owns.

In normal cases focused windows changes all the time, so this isn't a "real"
leak bug.
Attachment #497540 - Flags: review?(peterv)
Status: NEW → ASSIGNED
Comment on attachment 497540 [details] [diff] [review]
patch

I don't suppose we could clear these when the iframe is hidden?
Attachment #497540 - Flags: review?(peterv) → review+
That is another (open) bug and may be fixed as well. But I think there might
still be cases when we don't clear the variables before shutdown.
http://hg.mozilla.org/mozilla-central/rev/dd9e50b46d31
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Smaug, what's the bug number for the issue in comment 5 / comment 6?
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.