leak setting innerHTML to INPUTs

RESOLVED FIXED

Status

()

RESOLVED FIXED
14 years ago
10 years ago

People

(Reporter: ajschult784, Unassigned)

Tracking

({memory-leak, testcase})

Trunk
x86
Linux
memory-leak, testcase
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment)

(Reporter)

Description

14 years ago
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).
(Reporter)

Comment 1

14 years ago
Created attachment 176012 [details]
testcase

this leaks ~1MB/s with linux trunk 2005030101
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?
And what changed between those builds was that bug 268513 was backed out. 
Chances are, this is the same issue...
Depends on: 268513
(Reporter)

Comment 4

14 years ago
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.
Andrew, could you check in tomorrow's build, now that bug 268513 is refixed? 
This issue ought to be gone too...
(Reporter)

Comment 6

14 years ago
current CVS doesn't leak on the testcase.

FIXED by bug 268513
Status: NEW → RESOLVED
Last Resolved: 14 years ago
Resolution: --- → FIXED

Updated

10 years ago
Component: DOM: HTML → DOM: Core & HTML
QA Contact: ian → general
You need to log in before you can comment on or make changes to this bug.