#705 closed bug (fixed)
Leak in IE (__ie_init)
Reported by: | 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! :)
Change History (1)
comment:1 Changed 16 years ago by
Milestone: | 1.1 |
---|---|
Resolution: | → fixed |
Status: | new → closed |
Version: | 1.0 |
Note: See
TracTickets for help on using
tickets.
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.