Bug Tracker

Ticket #1918 (closed bug: fixed)

Opened 7 years ago

Last modified 7 years ago

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:
Blocking: Blocked by:

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

jquery.js Download (78.7 KB) - added by pieter_degraeuwe 7 years ago.
patched jQuery.js (started from version 1.2.1)

Change History

Changed 7 years ago by pieter_degraeuwe

patched jQuery.js (started from version 1.2.1)

comment:1 Changed 7 years ago by brandon

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

This is fixed in the latest SVN

comment:2 Changed 7 years ago by paaboo

  • Status changed from closed to reopened
  • Resolution fixed deleted

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 7 years ago by brandon

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

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.