Bug Tracker

Modify

Ticket #1918 (closed bug: fixed)

Opened 6 years ago

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

Change History

Changed 6 years ago by pieter_degraeuwe

patched jQuery.js (started from version 1.2.1)

comment:1 Changed 6 years ago by brandon

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

This is fixed in the latest SVN

comment:2 Changed 6 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 6 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.

Please follow the  bug reporting guidlines and use  jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.

View

Add a comment

Modify Ticket

Action
as closed
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.