Handle some edge cases with binding events to the window object (which is also a...
authorJohn Resig <jeresig@gmail.com>
Mon, 11 Oct 2010 22:20:57 +0000 (18:20 -0400)
committerJohn Resig <jeresig@gmail.com>
Mon, 11 Oct 2010 22:20:57 +0000 (18:20 -0400)
src/event.js
test/unit/event.js

index 7a10a12..d949d0f 100644 (file)
@@ -282,7 +282,7 @@ jQuery.event = {
                        delete elemData.handle;
 
                        if ( typeof elemData === "function" ) {
-                               delete elem.events;
+                               jQuery.removeData( elem, "events" );
 
                        } else if ( jQuery.isEmptyObject( elemData ) ) {
                                jQuery.removeData( elem );
@@ -346,7 +346,7 @@ jQuery.event = {
                // Trigger the event, it is assumed that "handle" is a function
                var handle = elem.nodeType ?
                        jQuery.data( elem, "handle" ) :
-                       elem.events && elem.events.handle;
+                       (jQuery.data( elem, "events" ) || {}).handle;
 
                if ( handle ) {
                        handle.apply( elem, data );
index d9dcc97..b093e8e 100644 (file)
@@ -1830,11 +1830,15 @@ test("Non DOM element events", function() {
 });
 
 test("window resize", function() {
-       expect(1);
+       expect(2);
+
+       jQuery(window).unbind();
 
        jQuery(window).bind("resize", function(){
                ok( true, "Resize event fired." );
        }).resize().unbind("resize");
+
+       ok( !jQuery(window).data("events"), "Make sure all the events are gone." );
 });
 
 /*