Bug Tracker

Opened 8 years ago

Closed 8 years ago

#1918 closed bug (fixed)

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 8 years ago.
patched jQuery.js (started from version 1.2.1)

Download all attachments as: .zip

Change History (4)

Changed 8 years ago by pieter_degraeuwe

patched jQuery.js (started from version 1.2.1)

comment:1 Changed 8 years ago by brandon

  • Resolution set to fixed
  • Status changed from new to closed

This is fixed in the latest SVN

comment:2 Changed 8 years ago by paaboo

  • Resolution fixed deleted
  • Status changed from closed to reopened

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

});

});

comment:3 Changed 8 years ago by brandon

  • Resolution set to fixed
  • Status changed from reopened to closed

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.

Note: See TracTickets for help on using tickets.