Mike Pinkerton (not reading bugmail) Reporter
	Description • 19 years ago

We're leaking a dom window every time you close the browser window. Not sure
when this regressed.

cc'ing jst in case it's more fallout from the split-window bug.

	Simon Fraser [no longer active] Assignee
	Comment 1 • 19 years ago

Firefox is leaking 2 dom windows for each main window that you create and then
close.

	Mike Pinkerton (not reading bugmail) Reporter
	Comment 2 • 19 years ago

well then!

	Mike Schroepfer
	Comment 3 • 19 years ago

JST - Split window related?

	Johnny Stenback (:jst)
	Comment 4 • 19 years ago

Hmm. Seems like Mac only, or anyone see this on non-Mac platforms?

	Johnny Stenback (:jst)
	Comment 5 • 19 years ago

Ok, I actually do see this on win32 and linux as well. This is a leak that crept
in *after* the splitwindow landing, when that change landed, we didn't leak a
single window object after starting firefox and opening another window, and then
closing firefox. 

	Johnny Stenback (:jst)
	Comment 6 • 19 years ago

I see us leaking one document object, the URL loaded into the leaked document is
"chrome://browser/content/feedview/feedview.xsl". There's no JS reference to
that document, so it seems like a leak through C++ code somewhere... digging
deeper...

	Caleb
	Comment 7 • 19 years ago

Might be related to bug 303043.

Also see bug 303043 comment 8.

	Simon Fraser [no longer active] Assignee
	Comment 8 • 19 years ago

Would we get this feedview stuff in Camino?

	Johnny Stenback (:jst)
	Comment 9 • 19 years ago

No idea...

	Johnny Stenback (:jst)
	Comment 10 • 19 years ago

The feedview leaks have been fixed on the trunk and branches, does this leak
still  exist in Camino?

	Simon Fraser [no longer active] Assignee
	Comment 11 • 19 years ago

I still see one nsGlobalWindow leaked per main window in Camino when loading a
simple page.

After heavy browsing sessions, I've seen 50-60 windows leaked.

	Johnny Stenback (:jst)
	Comment 12 • 19 years ago

Someone needs to enable GC_MARK_DEBUG to start with to see if we're leaking due
to JS references not being torn down or if it's a C++ leak somewhere. That'd be
where I'd start. Feel free to take this bug, I won't be able to get to a place
where I can debug this any time soon. I can help whoever takes on the task of
finding where this leak is tho...

	Mike Schroepfer
	Comment 13 • 19 years ago

Simon - is it possible for you to take a swag at this?

	Simon Fraser [no longer active] Assignee
	Comment 14 • 19 years ago

We're leaking webshells, which entrain dom windows.

I think it may be attributable to this change:

http://bonsai.mozilla.org/cvsview2.cgi?diff_mode=context&whitespace_mode=show&root=/cvsroot&subdir=mozilla/embedding/browser/webBrowser&command=DIFF_FRAMESET&file=nsWebBrowser.cpp&rev2=1.152&rev1=1.151

which bryner made.

	Simon Fraser [no longer active] Assignee
	Comment 15 • 19 years ago

It looks like we have a ref cycle between nsSecureBrowserUIImpl and docShell,
which I don't see being broken anywhere.

	Simon Fraser [no longer active] Assignee
	Comment 16 • 19 years ago

This patch breaks the ref cycle between the secureBrowserUI/docShell/domWindow.
I'm not sure if this is the best solution.

	Johnny Stenback (:jst)
	Comment 18 • 19 years ago

Seems like this had nothing to do with the split window changes. Updating
dependencies...

	Boris Zbarsky [:bzbarsky]
	Comment 19 • 19 years ago

Comment on attachment 193614 [details] [diff] [review]
Better patch

Thanks, Simon!

	Simon Fraser [no longer active] Assignee
	Comment 20 • 19 years ago

Comment on attachment 193614 [details] [diff] [review]
Better patch

This needs to be fixed on the branch too.

	Simon Fraser [no longer active] Assignee
	Comment 21 • 19 years ago

Checked into the trunk and branch.