Skip to main content

Bug Tracker

Side navigation

#3553 closed enhancement ()

Opened October 29, 2008 11:56PM UTC

Closed November 11, 2010 11:09PM UTC

Last modified July 10, 2012 02:47PM UTC

use of removeChild in IE leaking memory

Reported by: crobinson Owned by: flesler
Priority: minor Milestone: 1.4
Component: core Version: 1.2.6
Keywords: removeChild memory leak Cc: crobinson
Blocked by: Blocking:
Description

attached is a zip file with a small test you can run after patching and 2 screen shots, on of the sIEve results of reqular jQuery vs a modified version I made to test the theory. In the screen shots you can see that the memory reclaimed in the Delta's its on average higher in the modified version.

Take note that the tests will run 250 iterations before it will stop.

This is only an issue in IE.

for a write up in more detail visit: http://www.outsidethediv.com/2008/10/removechild-vs-the-garbage-bin/

Attachments (3)
Change History (6)

Changed October 30, 2008 02:43AM UTC by flesler comment:1

need: ReviewTest Case
owner: → flesler
status: newassigned

Ok, haven't used sIEve more than once... shouldn't the #leaks column have a number that is diff than 0 if it'd have leaks ?

I see that the regular takes less time.

Also... while doing some googling on the issue, I found this:

http://trac.dojotoolkit.org/ticket/1727#comment:15

Can you try that with your demo and see if it solves the so called leak ?

Changed October 30, 2008 02:43AM UTC by flesler comment:2

cc: → crobinson

Changed July 15, 2009 08:31AM UTC by yehuda comment:3

milestone: 1.31.3.3

Changed October 14, 2010 03:14AM UTC by snover comment:4

status: assignedpending

This ticket has been marked as missing a test case. In an effort to reduce the number of outstanding tickets in the bug tracker, it will be closed automatically in 30 days. In order to prevent this from happening, please provide a working test case. If a test case has already been provided and our records are wrong, please respond to the ticket so that it can be fixed. Thank you!

Changed November 11, 2010 11:09PM UTC by trac-o-bot comment:5

status: pendingclosed

Automatically closed due to 14 days of inactivity.

Changed July 10, 2012 02:47PM UTC by anonymous comment:6

Replying to [comment:5 trac-o-bot]:

Automatically closed due to 14 days of inactivity.

There is a test case attached to this. This should be reopened