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)
Change History (3)
Changed November 14, 2007 04:38AM UTC by comment:1
| resolution: | → fixed |
|---|---|
| status: | new → closed |
Changed December 13, 2007 04:22PM UTC by comment:2
| 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
});
});
Changed December 14, 2007 03:37PM UTC by comment:3
| 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.
This is fixed in the latest SVN