Side navigation
#705 closed bug (fixed)
Opened December 22, 2006 05:04PM UTC
Closed December 23, 2006 05:46PM UTC
Last modified June 20, 2007 02:47AM UTC
Leak in IE (__ie_init)
| Reported by: | adam@scorpiatech.com | Owned by: | |
|---|---|---|---|
| Priority: | major | Milestone: | |
| Component: | event | Version: | |
| Keywords: | leak, ie, document.ready | Cc: | |
| Blocked by: | Blocking: |
Description
Found a leak bug, this time in IE using the document.write/__ie_init hack to use $(document).ready(). Drip reports that this element is leaked using jQuery 1.0.4 (I've got IE7 running on XP SP2):
<SCRIPT id=__ie_init src="//:" defer __drip_hooked="true"></SCRIPT>
This simple patch fixes it without affecting $(document).ready():
@@ -1387,6 +1387,7 @@ new function(){
};
// Clear from memory
+ script.parentNode.removeChild(script);
script = null;
// If Safari is used
I can't think of another way to prevent the leak...but feel free to fix this however you see fit! :)
Attachments (0)
Change History (1)
Changed December 23, 2006 05:46PM UTC by comment:1
| milestone: | 1.1 |
|---|---|
| resolution: | → fixed |
| status: | new → closed |
| version: | 1.0 |
The script element is already removed in the "onreadystatechange" function. It does this be referring to 'this' instead of 'script' (avoiding the leak). I think this may be sufficient.