284396 - leak setting innerHTML to INPUTs

Status: RESOLVED FIXED

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

Description

[Andrew Schultz]

This is spun off from bug 173807.  At the URL, click "more" and then mousever
the various words in the new window.  I'll attach a testcase that leaks ~1MB/s.
 The testcase sets a DIV's innerHTML to be a bunch of INPUTs (repeatedly).

Comment 1

[Andrew Schultz]

Attachment: testcase

this leaks ~1MB/s with linux trunk 2005030101

Comment 2

[Boris Zbarsky [:bzbarsky]]

Interesting.  I get this leaking (or at least growing memory) with build
2005-02-28-06 but not with build 2005-02-27-06.

Andrew, do you see the same, or do you see the problem in both builds?

Comment 3

[Boris Zbarsky [:bzbarsky]]

And what changed between those builds was that bug 268513 was backed out. 
Chances are, this is the same issue...

Comment 4

[Andrew Schultz]

I crash on the testcase with build 2005-02-27-05 and 2005-02-24-05.  :(

I tried this with valgrind, but it saw no leaks, so things are just not being
freed in a timely manner.

Comment 5

[Boris Zbarsky [:bzbarsky]]

Andrew, could you check in tomorrow's build, now that bug 268513 is refixed? 
This issue ought to be gone too...

Comment 6

[Andrew Schultz]

current CVS doesn't leak on the testcase.

FIXED by bug 268513