343982 - Leak DOM windows and document loading CNN

Status: VERIFIED FIXED

(Core :: DOM: Core & HTML, defect)

Description

[Andrew Schultz]

If I load www.cnn.com with dbaron's leak logging, I leak 2 DOM windows and a document.  The part that triggers the leak it is:

var s = new Object;
window.a = s;
s.d = document;

Comment 1

[Andrew Schultz]

Attachment: testcase

Leaked inner window 95b9d00 (outer 9668448) at address 95b9d00.
 ... with URI "file:///home/andrew/mozbugs/343982/test.html".
Leaked outer window 9668448 at address 9668448.
Leaked document at address 922d998.
 ... with URI "file:///home/andrew/mozbugs/343982/test.html".

Comment 2

[Adam Guthrie]

I tracked down the regression range for this to between 2006-06-12 04:00 and 2006-06-13 04:00. 

Checkins: http://bonsai.mozilla.org/cvsquery.cgi?treeid=default&module=PhoenixTinderbox&branch=HEAD&branchtype=match&dir=&file=&filetype=match&who=&whotype=match&sortby=Date&hours=2&date=explicit&mindate=2006-06-12+03&maxdate=2006-06-13+05&cvsroot=%2Fcvsroot

The landing of the DOM agnostic branch and bug 176182 (still not sure what this was about) seem like likely candidates for regressions.

Comment 3

[Adam Guthrie]

dbaron's patch in bug 352911 fixes this.

Comment 4

[Adam Guthrie]

Fixed by bug 352911. It would be great if you could verify this, Andrew.

Comment 5

[Andrew Schultz]

no leak on the testcase or http://www.cnn.com/ with a current CVS build