Skip to main content

Bug Tracker

Side navigation

#1918 closed bug (fixed)

Opened November 13, 2007 09:34AM UTC

Closed December 14, 2007 03:37PM UTC

memory leak onReady() in IE + patch

Reported by: pieter_degraeuwe Owned by:
Priority: major Milestone: 1.2.2
Component: core Version: 1.2.1
Keywords: Cc:
Blocked by: Blocking:
Description

Guys,

Just adding the following causes a memoryleak in IE when you navigate a lot.

$().ready(function() {

});

I detected it using the tool Drip.exe; I found out that DOM elements with the id __ie_init were causing the leak (rings a bell?)

anyway, I did patch jquery.js (attached) by checking in the bindReady() for the existence of an __ie_init element.

If it already exists, I don't write it anymore.

I hope you can include this patch in the next release.

Attachments (1)
  • jquery.js (78.7 KB) - added by pieter_degraeuwe November 13, 2007 09:35AM UTC.

    patched jQuery.js (started from version 1.2.1)

Change History (3)

Changed November 14, 2007 04:38AM UTC by brandon comment:1

resolution: → fixed
status: newclosed

This is fixed in the latest SVN

Changed December 13, 2007 04:22PM UTC by paaboo comment:2

resolution: fixed
status: closedreopened

eee....the patched jquery.js also have the problom,

please try this:

----------------------------

$(document).ready(function(){

$("div.ppp").each(function(){

$(this).html("ddddd"); or $(this).text("xxxxx"); this line will leak

$(this).click(function(){alert(1);});//but this line have not memery leak

});

});

Changed December 14, 2007 03:37PM UTC by brandon comment:3

resolution: → fixed
status: reopenedclosed

This is resolved in the latest SVN. However, it is possible that text might still leak. If it does please create a new ticket for the text leak.