Opened 15 years ago
Closed 15 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)
Change History (4)
Changed 15 years ago by
comment:1 Changed 15 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
This is fixed in the latest SVN
comment:2 Changed 15 years ago by
Resolution: | fixed |
---|---|
Status: | closed → 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 15 years ago by
Resolution: | → fixed |
---|---|
Status: | reopened → 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.
patched jQuery.js (started from version 1.2.1)