Changeset 4292

Timestamp:

12/21/07 04:53:33 (9 months ago)

Author:

brandon.aaron

Message:

Fixed memory leak in IE with non-native event types

Files:

• trunk/jquery/src/event.js (modified) (3 diffs)

trunk/jquery/src/event.js

@@ -50 +50 @@
                     return val;
         
-                val = jQuery.event.handle.apply(elem, arguments);
+                val = jQuery.event.handle.apply(arguments.callee.elem, arguments);
         
                 return val;
             });
+        // Add elem as a property of the handle function
+        // This is to prevent a memory leak with non-native
+        // event in IE.
+        handle.elem = elem;
             
             // Handle multiple events seperated by a space
@@ -88 +92 @@
                 jQuery.event.global[type] = true;
             });
+        
+        // Nullify elem to prevent memory leaks in IE
+        elem = null;
     },
 
@@ -151 +158 @@
             for ( ret in events ) break;
             if ( !ret ) {
+                var handle = jQuery.data( elem, "handle" );
+                if ( handle ) handle.elem = null;
                 jQuery.removeData( elem, "events" );
                 jQuery.removeData( elem, "handle" );