Merging pull request 183 for #7793.
authorjeresig <jeresig@gmail.com>
Mon, 17 Jan 2011 20:45:07 +0000 (15:45 -0500)
committerjeresig <jeresig@gmail.com>
Mon, 17 Jan 2011 20:45:07 +0000 (15:45 -0500)
1  2 
src/event.js
test/unit/event.js

diff --combined src/event.js
@@@ -63,7 -63,7 +63,7 @@@ jQuery.event = 
                var eventKey = elem.nodeType ? "events" : "__events__",
                        events = elemData[ eventKey ],
                        eventHandle = elemData.handle;
 -                      
 +
                if ( typeof events === "function" ) {
                        // On plain objects events is a fn that holds the the data
                        // which prevents this data from being JSON serialized
                                        }
                                }
                        }
 -                      
 -                      if ( special.add ) { 
 -                              special.add.call( elem, handleObj ); 
 +
 +                      if ( special.add ) {
 +                              special.add.call( elem, handleObj );
  
                                if ( !handleObj.handler.guid ) {
                                        handleObj.handler.guid = handler.guid;
                if ( !elemData || !events ) {
                        return;
                }
 -              
 +
                if ( typeof events === "function" ) {
                        elemData = events;
                        events = events.events;
                                namespaces = type.split(".");
                                type = namespaces.shift();
  
 -                              namespace = new RegExp("(^|\\.)" + 
 +                              namespace = new RegExp("(^|\\.)" +
                                        jQuery.map( namespaces.slice(0).sort(), fcleanup ).join("\\.(?:.*\\.)?") + "(\\.|$)");
                        }
  
                                isClick = jQuery.nodeName( target, "a" ) && targetType === "click",
                                special = jQuery.event.special[ targetType ] || {};
  
 -                      if ( (!special._default || special._default.call( elem, event ) === false) && 
 +                      if ( (!special._default || special._default.call( elem, event ) === false) &&
                                !isClick && !(target && target.nodeName && jQuery.noData[target.nodeName.toLowerCase()]) ) {
  
                                try {
                                        event.handler = handleObj.handler;
                                        event.data = handleObj.data;
                                        event.handleObj = handleObj;
 -      
 +
                                        var ret = handleObj.handler.apply( this, args );
  
                                        if ( ret !== undefined ) {
                        add: function( handleObj ) {
                                jQuery.event.add( this,
                                        liveConvert( handleObj.origType, handleObj.selector ),
 -                                      jQuery.extend({}, handleObj, {handler: liveHandler, guid: handleObj.handler.guid}) ); 
 +                                      jQuery.extend({}, handleObj, {handler: liveHandler, guid: handleObj.handler.guid}) );
                        },
  
                        remove: function( handleObj ) {
@@@ -583,7 -583,7 +583,7 @@@ jQuery.removeEvent = document.removeEve
                if ( elem.removeEventListener ) {
                        elem.removeEventListener( type, handle, false );
                }
 -      } : 
 +      } :
        function( elem, type, handle ) {
                if ( elem.detachEvent ) {
                        elem.detachEvent( "on" + type, handle );
@@@ -600,12 -600,10 +600,12 @@@ jQuery.Event = function( src ) 
        if ( src && src.type ) {
                this.originalEvent = src;
                this.type = src.type;
 +
                // Events bubbling up the document may have been marked as prevented
                // by a handler lower down the tree; reflect the correct value.
-               this.isDefaultPrevented = (src.defaultPrevented || src.returnValue === false ||
+               this.isDefaultPrevented = (src.defaultPrevented || src.returnValue === false || 
                        src.getPreventDefault && src.getPreventDefault()) ? returnTrue : returnFalse;
 +
        // Event type
        } else {
                this.type = src;
@@@ -636,7 -634,7 +636,7 @@@ jQuery.Event.prototype = 
                if ( !e ) {
                        return;
                }
 -              
 +
                // if preventDefault exists run it on the original event
                if ( e.preventDefault ) {
                        e.preventDefault();
@@@ -732,7 -730,7 +732,7 @@@ if ( !jQuery.support.submitBubbles ) 
                                                return trigger( "submit", this, arguments );
                                        }
                                });
 -       
 +
                                jQuery.event.add(this, "keypress.specialSubmit", function( e ) {
                                        var elem = e.target,
                                                type = elem.type;
@@@ -794,7 -792,7 +794,7 @@@ if ( !jQuery.support.changeBubbles ) 
                if ( e.type !== "focusout" || elem.type !== "radio" ) {
                        jQuery.data( elem, "_change_data", val );
                }
 -              
 +
                if ( data === undefined || val === data ) {
                        return;
                }
  
        jQuery.event.special.change = {
                filters: {
 -                      focusout: testChange, 
 +                      focusout: testChange,
  
                        beforedeactivate: testChange,
  
@@@ -879,15 -877,15 +879,15 @@@ if ( document.addEventListener ) 
                                if ( focusCounts[fix]++ === 0 ) {
                                        document.addEventListener( orig, handler, true );
                                }
 -                      }, 
 -                      teardown: function() { 
 +                      },
 +                      teardown: function() {
                                if ( --focusCounts[fix] === 0 ) {
                                        document.removeEventListener( orig, handler, true );
                                }
                        }
                };
  
 -              function handler( e ) { 
 +              function handler( e ) {
                        e = jQuery.event.fix( e );
                        e.type = fix;
                        return jQuery.event.trigger( e, null, e.target );
@@@ -904,7 -902,7 +904,7 @@@ jQuery.each(["bind", "one"], function( 
                        }
                        return this;
                }
 -              
 +
                if ( jQuery.isFunction( data ) || data === false ) {
                        fn = data;
                        data = undefined;
@@@ -944,20 -942,20 +944,20 @@@ jQuery.fn.extend(
  
                return this;
        },
 -      
 +
        delegate: function( selector, types, data, fn ) {
                return this.live( types, data, fn, selector );
        },
 -      
 +
        undelegate: function( selector, types, fn ) {
                if ( arguments.length === 0 ) {
                                return this.unbind( "live" );
 -              
 +
                } else {
                        return this.die( types, null, fn, selector );
                }
        },
 -      
 +
        trigger: function( type, data ) {
                return this.each(function() {
                        jQuery.event.trigger( type, data, this );
@@@ -1014,12 -1012,12 +1014,12 @@@ jQuery.each(["live", "die"], function( 
                var type, i = 0, match, namespaces, preType,
                        selector = origSelector || this.selector,
                        context = origSelector ? this : jQuery( this.context );
 -              
 +
                if ( typeof types === "object" && !types.preventDefault ) {
                        for ( var key in types ) {
                                context[ name ]( key, data, types[key], selector );
                        }
 -                      
 +
                        return this;
                }
  
                                context.unbind( "live." + liveConvert( type, selector ), fn );
                        }
                }
 -              
 +
                return this;
        };
  });
@@@ -1085,7 -1083,7 +1085,7 @@@ function liveHandler( event ) 
        if ( event.liveFired === this || !events || !events.live || event.target.disabled || event.button && event.type === "click" ) {
                return;
        }
 -      
 +
        if ( event.namespace ) {
                namespace = new RegExp("(^|\\.)" + event.namespace.split(".").join("\\.(?:.*\\.)?") + "(\\.|$)");
        }
@@@ -1183,4 -1181,21 +1183,4 @@@ jQuery.each( ("blur focus focusin focus
        }
  });
  
 -// Prevent memory leaks in IE
 -// Window isn't included so as not to unbind existing unload events
 -// More info:
 -//  - http://isaacschlueter.com/2006/10/msie-memory-leaks/
 -if ( window.attachEvent && !window.addEventListener ) {
 -      jQuery(window).bind("unload", function() {
 -              for ( var id in jQuery.cache ) {
 -                      if ( jQuery.cache[ id ].handle ) {
 -                              // Try/Catch is to handle iframes being unloaded, see #4280
 -                              try {
 -                                      jQuery.event.remove( jQuery.cache[ id ].handle.elem );
 -                              } catch(e) {}
 -                      }
 -              }
 -      });
 -}
 -
  })( jQuery );
diff --combined test/unit/event.js
@@@ -4,20 -4,20 +4,20 @@@ test("null or undefined handler", funct
        expect(2);
    // Supports Fixes bug #7229
    try {
 -  
 +
      jQuery("#firstp").click(null);
 -  
 +
      ok(true, "Passing a null handler will not throw an exception");
  
 -  } catch (e) {}  
 +  } catch (e) {}
  
    try {
 -  
 +
      jQuery("#firstp").click(undefined);
 -  
 +
      ok(true, "Passing an undefined handler will not throw an exception");
  
 -  } catch (e) {}  
 +  } catch (e) {}
  });
  
  test("bind(), with data", function() {
@@@ -175,7 -175,7 +175,7 @@@ test("bind(), no data", function() 
  
  test("bind/one/unbind(Object)", function(){
        expect(6);
 -      
 +
        var clickCounter = 0, mouseoverCounter = 0;
        function handler(event) {
                if (event.type == "click")
                else if (event.type == "mouseover")
                        mouseoverCounter++;
        };
 -      
 +
        function handlerWithData(event) {
                if (event.type == "click")
                        clickCounter += event.data;
                else if (event.type == "mouseover")
                        mouseoverCounter += event.data;
        };
 -      
 +
        function trigger(){
                $elem.trigger("click").trigger("mouseover");
        }
 -      
 +
        var $elem = jQuery("#firstp")
                // Regular bind
                .bind({
                        click:handlerWithData,
                        mouseover:handlerWithData
                }, 2 );
 -      
 +
        trigger();
 -      
 +
        equals( clickCounter, 3, "bind(Object)" );
        equals( mouseoverCounter, 3, "bind(Object)" );
 -      
 +
        trigger();
        equals( clickCounter, 4, "bind(Object)" );
        equals( mouseoverCounter, 4, "bind(Object)" );
 -      
 +
        jQuery("#firstp").unbind({
                click:handler,
                mouseover:handler
  
  test("live/die(Object), delegate/undelegate(String, Object)", function() {
        expect(6);
 -      
 +
        var clickCounter = 0, mouseoverCounter = 0,
                $p = jQuery("#firstp"), $a = $p.find("a:first");
 -      
 +
        var events = {
                click: function( event ) {
                        clickCounter += ( event.data || 1 );
                        mouseoverCounter += ( event.data || 1 );
                }
        };
 -      
 +
        function trigger() {
                $a.trigger("click").trigger("mouseover");
        }
 -      
 +
        $a.live( events );
        $p.delegate( "a", events, 2 );
 -      
 +
        trigger();
        equals( clickCounter, 3, "live/delegate" );
        equals( mouseoverCounter, 3, "live/delegate" );
 -      
 +
        $p.undelegate( "a", events );
 -      
 +
        trigger();
        equals( clickCounter, 4, "undelegate" );
        equals( mouseoverCounter, 4, "undelegate" );
 -      
 +
        $a.die( events );
 -      
 +
        trigger();
        equals( clickCounter, 4, "die" );
        equals( mouseoverCounter, 4, "die" );
  
  test("live/delegate immediate propagation", function() {
        expect(2);
 -      
 +
        var $p = jQuery("#firstp"), $a = $p.find("a:first"), lastClick;
 -      
 +
        lastClick = "";
 -      $a.live( "click", function(e) { 
 -              lastClick = "click1"; 
 +      $a.live( "click", function(e) {
 +              lastClick = "click1";
                e.stopImmediatePropagation();
        });
        $a.live( "click", function(e) {
        $a.trigger( "click" );
        equals( lastClick, "click1", "live stopImmediatePropagation" );
        $a.die( "click" );
 -      
 +
        lastClick = "";
 -      $p.delegate( "a", "click", function(e) { 
 -              lastClick = "click1"; 
 +      $p.delegate( "a", "click", function(e) {
 +              lastClick = "click1";
                e.stopImmediatePropagation();
        });
        $p.delegate( "a", "click", function(e) {
        $p.undelegate( "click" );
  });
  
- test("bind/delegate bubbling, isDefaultPrevented (Bug #7793)", function() {
+ test("bind/delegate bubbling, isDefaultPrevented", function() {
        expect(2);
        var $anchor2 = jQuery( "#anchor2" ),
                $main = jQuery( "#main" ),
                fakeClick = function($jq) {
                        // Use a native click so we don't get jQuery simulated bubbling
                        if ( document.createEvent ) {
-                               var e = document.createEvent( "MouseEvents" );
-                               e.initEvent( "click", true, true );
+                               var e = document.createEvent( 'MouseEvents' );
+                               e.initEvent( "click", true, true ); 
                                $jq[0].dispatchEvent(e);
                        }
                        else if ( $jq[0].click ) {
                e.preventDefault();
        });
        $main.delegate("#foo", "click", function(e) {
-               equals( e.isDefaultPrevented(), true, "isDefaultPrevented true passed to bubbled event" );
+               var orig = e.originalEvent;
++
+               if ( typeof(orig.defaultPrevented) === "boolean" || typeof(orig.returnValue) === "boolean" || orig.getPreventDefault ) {
+                       equals( e.isDefaultPrevented(), true, "isDefaultPrevented true passed to bubbled event" );
 -              }
 -              else {
++
++              } else {
+                       // Opera < 11 doesn't implement any interface we can use, so give it a pass
+                       ok( true, "isDefaultPrevented not supported by this browser, test skipped" );
+               }
        });
        fakeClick( $anchor2 );
        $anchor2.unbind( "click" );
  test("bind(), iframes", function() {
        // events don't work with iframes, see #939 - this test fails in IE because of contentDocument
        var doc = jQuery("#loadediframe").contents();
 -      
 +
        jQuery("div", doc).bind("click", function() {
                ok( true, "Binding to element inside iframe" );
        }).click().unbind('click');
@@@ -395,7 -402,7 +403,7 @@@ test("bind(), namespaced events, clone
  
  test("bind(), multi-namespaced events", function() {
        expect(6);
 -      
 +
        var order = [
                "click.test.abc",
                "click.test.abc",
                "click.test",
                "custom.test2"
        ];
 -      
 +
        function check(name, msg){
                same(name, order.shift(), msg);
        }
        jQuery("#firstp").bind("click.test.abc",function(e){
                check("click.test.abc", "Namespaced click triggered");
        });
 -      
 +
        // Those would not trigger/unbind (#5303)
        jQuery("#firstp").trigger("click.a.test");
        jQuery("#firstp").unbind("click.a.test");
@@@ -488,7 -495,7 +496,7 @@@ test("bind(), make sure order is mainta
  
        elem.unbind("click");
  });
 - 
 +
  test("bind(), with different this object", function() {
        expect(4);
        var thisObject = { myThis: true },
                        equals( this, thisObject, "bind() with different this object and data" );
                        equals( event.data, data, "bind() with different this object and data" );
                };
 -      
 +
        jQuery("#firstp")
                .bind("click", jQuery.proxy(handler1, thisObject)).click().unbind("click", handler1)
                .bind("click", data, jQuery.proxy(handler2, thisObject)).click().unbind("click", handler2);
@@@ -566,29 -573,29 +574,29 @@@ test("bind()/trigger()/unbind() on plai
  
        // Make sure it doesn't complain when no events are found
        jQuery(obj).unbind("test");
 -      
 +
        equals( obj.__events__, undefined, "Make sure events object is removed" );
  });
  
  test("unbind(type)", function() {
        expect( 0 );
 -      
 +
        var $elem = jQuery("#firstp"),
                message;
  
        function error(){
                ok( false, message );
        }
 -      
 +
        message = "unbind passing function";
        $elem.bind('error1', error).unbind('error1',error).triggerHandler('error1');
 -      
 +
        message = "unbind all from event";
        $elem.bind('error1', error).unbind('error1').triggerHandler('error1');
 -      
 +
        message = "unbind all";
        $elem.bind('error1', error).unbind().triggerHandler('error1');
 -      
 +
        message = "unbind many with function";
        $elem.bind('error1 error2',error)
                 .unbind('error1 error2', error )
        $elem.bind('error1 error2',error)
                 .unbind('error1 error2')
                 .trigger('error1').triggerHandler('error2');
 -      
 +
        message = "unbind without a type or handler";
        $elem.bind("error1 error2.test",error)
                 .unbind()
  
  test("unbind(eventObject)", function() {
        expect(4);
 -      
 +
        var $elem = jQuery("#firstp"),
                num;
  
                $elem.trigger('foo').triggerHandler('bar');
                equals( num, expected, "Check the right handlers are triggered" );
        }
 -      
 +
        $elem
                // This handler shouldn't be unbound
                .bind('foo', function(){
                .bind('bar', function(){
                        num += 4;
                });
 -              
 +
        assert( 7 );
        assert( 5 );
 -      
 +
        $elem.unbind('bar');
        assert( 1 );
 -      
 -      $elem.unbind(); 
 +
 +      $elem.unbind();
        assert( 0 );
  });
  
@@@ -667,25 -674,25 +675,25 @@@ test("trigger() shortcuts", function() 
                ok( !close[0], "Context element does not exist, direct access to element must return undefined" );
                return false;
        }).click();
 -      
 +
        jQuery("#check1").click(function() {
                ok( true, "click event handler for checkbox gets fired twice, see #815" );
        }).click();
 -      
 +
        var counter = 0;
        jQuery('#firstp')[0].onclick = function(event) {
                counter++;
        };
        jQuery('#firstp').click();
        equals( counter, 1, "Check that click, triggers onclick event handler also" );
 -      
 +
        var clickCounter = 0;
        jQuery('#simon1')[0].onclick = function(event) {
                clickCounter++;
        };
        jQuery('#simon1').click();
        equals( clickCounter, 1, "Check that click, triggers onclick event handler on an a tag also" );
 -      
 +
        jQuery('<img />').load(function(){
                ok( true, "Trigger the load event, using the shortcut .load() (#2819)");
        }).load();
@@@ -763,7 -770,7 +771,7 @@@ test("trigger(type, [data], [fn])", fun
                pass = false;
        }
        ok( pass, "Trigger focus on hidden element" );
 -      
 +
        pass = true;
        try {
                jQuery('table:first').bind('test:test', function(){}).trigger('test:test');
@@@ -803,28 -810,28 +811,28 @@@ test("jQuery.Event.currentTarget", func
  
  test("trigger(eventObject, [data], [fn])", function() {
        expect(25);
 -      
 +
        var $parent = jQuery('<div id="par" />').hide().appendTo('body'),
                $child = jQuery('<p id="child">foo</p>').appendTo( $parent );
 -      
 -      var event = jQuery.Event("noNew");      
 +
 +      var event = jQuery.Event("noNew");
        ok( event != window, "Instantiate jQuery.Event without the 'new' keyword" );
        equals( event.type, "noNew", "Verify its type" );
 -      
 +
        equals( event.isDefaultPrevented(), false, "Verify isDefaultPrevented" );
        equals( event.isPropagationStopped(), false, "Verify isPropagationStopped" );
        equals( event.isImmediatePropagationStopped(), false, "Verify isImmediatePropagationStopped" );
 -      
 +
        event.preventDefault();
        equals( event.isDefaultPrevented(), true, "Verify isDefaultPrevented" );
        event.stopPropagation();
        equals( event.isPropagationStopped(), true, "Verify isPropagationStopped" );
 -      
 +
        event.isPropagationStopped = function(){ return false };
        event.stopImmediatePropagation();
        equals( event.isPropagationStopped(), true, "Verify isPropagationStopped" );
        equals( event.isImmediatePropagationStopped(), true, "Verify isPropagationStopped" );
 -      
 +
        $parent.bind('foo',function(e){
                // Tries bubbling
                equals( e.type, 'foo', 'Verify event type when passed passing an event object' );
                equals( e.currentTarget.id, 'par', 'Verify event.target when passed passing an event object' );
                equals( e.secret, 'boo!', 'Verify event object\'s custom attribute when passed passing an event object' );
        });
 -      
 +
        // test with an event object
        event = new jQuery.Event("foo");
        event.secret = 'boo!';
        $child.trigger(event);
 -      
 +
        // test with a literal object
        $child.trigger({type:'foo', secret:'boo!'});
 -      
 +
        $parent.unbind();
  
        function error(){
                ok( false, "This assertion shouldn't be reached");
        }
 -      
 +
        $parent.bind('foo', error );
 -      
 +
        $child.bind('foo',function(e, a, b, c ){
                equals( arguments.length, 4, "Check arguments length");
                equals( a, 1, "Check first custom argument");
                equals( b, 2, "Check second custom argument");
                equals( c, 3, "Check third custom argument");
 -              
 +
                equals( e.isDefaultPrevented(), false, "Verify isDefaultPrevented" );
                equals( e.isPropagationStopped(), false, "Verify isPropagationStopped" );
                equals( e.isImmediatePropagationStopped(), false, "Verify isImmediatePropagationStopped" );
 -              
 +
                // Skips both errors
                e.stopImmediatePropagation();
 -              
 +
                return "result";
        });
 -      
 +
        // We should add this back in when we want to test the order
        // in which event handlers are iterated.
        //$child.bind('foo', error );
 -      
 +
        event = new jQuery.Event("foo");
        $child.trigger( event, [1,2,3] ).unbind();
        equals( event.result, "result", "Check event.result attribute");
 -      
 +
        // Will error if it bubbles
        $child.triggerHandler('foo');
 -      
 +
        $child.unbind();
        $parent.unbind().remove();
  });
  
  test("jQuery.Event.currentTarget", function(){
        expect(1);
 -      
 +
        var counter = 0,
                $elem = jQuery('<button>a</button>').click(function(e){
                equals( e.currentTarget, this, "Check currentTarget on "+(counter++?"native":"fake") +" event" );
        });
 -      
 +
        // Fake event
        $elem.trigger('click');
 -      
 +
        // Cleanup
        $elem.unbind();
  });
  
  test("toggle(Function, Function, ...)", function() {
        expect(16);
 -      
 +
        var count = 0,
                fn1 = function(e) { count++; },
                fn2 = function(e) { count--; },
                });
                return false;
        }).click().click().click();
 -      
 +
        var turn = 0;
        var fns = [
                function(){
                        turn = 3;
                }
        ];
 -      
 +
        var $div = jQuery("<div>&nbsp;</div>").toggle( fns[0], fns[1], fns[2] );
        $div.click();
        equals( turn, 1, "Trying toggle with 3 functions, attempt 1 yields 1");
        equals( turn, 1, "Trying toggle with 3 functions, attempt 4 yields 1");
        $div.click();
        equals( turn, 2, "Trying toggle with 3 functions, attempt 5 yields 2");
 -      
 +
        $div.unbind('click',fns[0]);
        var data = jQuery.data( $div[0], 'events' );
        ok( !data, "Unbinding one function from toggle unbinds them all");
@@@ -1087,7 -1094,7 +1095,7 @@@ test(".live()/.die()", function() 
        // Test binding with different this object, event data, and trigger data
        jQuery("#foo").live("click", true, jQuery.proxy(function(e, data){
                equals( e.data, true, "live with with different this object, event data, and trigger data" );
 -              equals( this.foo, "bar", "live with with different this object, event data, and trigger data" ); 
 +              equals( this.foo, "bar", "live with with different this object, event data, and trigger data" );
                equals( data, true, "live with with different this object, event data, and trigger data")
        }, { foo: "bar" }));
        jQuery("#foo").trigger("click", true).die("click");
  
        // Cleanup
        jQuery("#nothiddendiv").die("foo", callback);
 -      
 +
        // Make sure we don't loose the target by DOM modifications
        // after the bubble already reached the liveHandler
        var livec = 0, elemDiv = jQuery("#nothiddendivchild").html('<span></span>').get(0);
 -      
 +
        jQuery("#nothiddendivchild").live("click", function(e){ jQuery("#nothiddendivchild").html(''); });
        jQuery("#nothiddendivchild").live("click", function(e){ if(e.target) {livec++;} });
 -      
 +
        jQuery("#nothiddendiv span").click();
        equals( jQuery("#nothiddendiv span").length, 0, "Verify that first handler occurred and modified the DOM." );
        equals( livec, 1, "Verify that second handler occurred even with nuked target." );
 -      
 +
        // Cleanup
        jQuery("#nothiddendivchild").die("click");
  
        // Verify that .live() ocurs and cancel buble in the same order as
        // we would expect .bind() and .click() without delegation
        var lived = 0, livee = 0;
 -      
 +
        // bind one pair in one order
        jQuery('span#liveSpan1 a').live('click', function(){ lived++; return false; });
        jQuery('span#liveSpan1').live('click', function(){ livee++; });
        jQuery('span#liveSpan2 a').click();
        equals( lived, 1, "Verify that only one first handler occurred." );
        equals( livee, 0, "Verify that second handler doesn't." );
 -      
 +
        // Cleanup
        jQuery("span#liveSpan1 a").die("click")
        jQuery("span#liveSpan1").die("click");
        jQuery("span#liveSpan2 a").die("click");
        jQuery("span#liveSpan2").die("click");
 -      
 +
        // Test this, target and currentTarget are correct
 -      jQuery('span#liveSpan1').live('click', function(e){ 
 +      jQuery('span#liveSpan1').live('click', function(e){
                equals( this.id, 'liveSpan1', 'Check the this within a live handler' );
                equals( e.currentTarget.id, 'liveSpan1', 'Check the event.currentTarget within a live handler' );
                equals( e.target.nodeName.toUpperCase(), 'A', 'Check the event.target within a live handler' );
        });
 -      
 +
        jQuery('span#liveSpan1 a').click();
 -      
 +
        jQuery('span#liveSpan1').die('click');
  
        // Work with deep selectors
@@@ -1333,18 -1340,18 +1341,18 @@@ test("live with change", function()
        expect(8);
  
        var selectChange = 0, checkboxChange = 0;
 -      
 +
        var select = jQuery("select[name='S1']")
        select.live("change", function() {
                selectChange++;
        });
 -      
 -      var checkbox = jQuery("#check2"), 
 +
 +      var checkbox = jQuery("#check2"),
                checkboxFunction = function(){
                        checkboxChange++;
                }
        checkbox.live("change", checkboxFunction);
 -      
 +
        // test click on select
  
        // second click that changed it
        select[0].selectedIndex = select[0].selectedIndex ? 0 : 1;
        select.trigger("change");
        equals( selectChange, 1, "Change on click." );
 -      
 +
        // test keys on select
        selectChange = 0;
        select[0].selectedIndex = select[0].selectedIndex ? 0 : 1;
        select.trigger("change");
        equals( selectChange, 1, "Change on keyup." );
 -      
 +
        // test click on checkbox
        checkbox.trigger("change");
        equals( checkboxChange, 1, "Change on checkbox." );
 -      
 +
        // test blur/focus on text
        var text = jQuery("#name"), textChange = 0, oldTextVal = text.val();
        text.live("change", function() {
  
        text.val(oldTextVal);
        text.die("change");
 -      
 +
        // test blur/focus on password
        var password = jQuery("#name"), passwordChange = 0, oldPasswordVal = password.val();
        password.live("change", function() {
  
        password.val(oldPasswordVal);
        password.die("change");
 -      
 +
        // make sure die works
 -      
 +
        // die all changes
        selectChange = 0;
        select.die("change");
        select[0].selectedIndex = select[0].selectedIndex ? 0 : 1;
        select.trigger("change");
        equals( selectChange, 0, "Die on keyup works." );
 -      
 +
        // die specific checkbox
        checkbox.die("change", checkboxFunction);
        checkbox.trigger("change");
  
  test("live with submit", function() {
        var count1 = 0, count2 = 0;
 -      
 +
        jQuery("#testForm").live("submit", function(ev) {
                count1++;
                ev.preventDefault();
        jQuery("#testForm input[name=sub1]").submit();
        equals( count1, 1, "Verify form submit." );
        equals( count2, 1, "Verify body submit." );
 -      
 +
        jQuery("#testForm").die("submit");
        jQuery("body").die("submit");
  });
@@@ -1603,7 -1610,7 +1611,7 @@@ test(".delegate()/.undelegate()", funct
        // Test binding with different this object, event data, and trigger data
        jQuery("#body").delegate("#foo", "click", true, jQuery.proxy(function(e, data){
                equals( e.data, true, "delegate with with different this object, event data, and trigger data" );
 -              equals( this.foo, "bar", "delegate with with different this object, event data, and trigger data" ); 
 +              equals( this.foo, "bar", "delegate with with different this object, event data, and trigger data" );
                equals( data, true, "delegate with with different this object, event data, and trigger data")
        }, { foo: "bar" }));
        jQuery("#foo").trigger("click", true);
  
        // Cleanup
        jQuery("#body").undelegate("#nothiddendiv", "foo", callback);
 -      
 +
        // Make sure we don't loose the target by DOM modifications
        // after the bubble already reached the liveHandler
        var livec = 0, elemDiv = jQuery("#nothiddendivchild").html('<span></span>').get(0);
 -      
 +
        jQuery("#body").delegate("#nothiddendivchild", "click", function(e){ jQuery("#nothiddendivchild").html(''); });
        jQuery("#body").delegate("#nothiddendivchild", "click", function(e){ if(e.target) {livec++;} });
 -      
 +
        jQuery("#nothiddendiv span").click();
        equals( jQuery("#nothiddendiv span").length, 0, "Verify that first handler occurred and modified the DOM." );
        equals( livec, 1, "Verify that second handler occurred even with nuked target." );
 -      
 +
        // Cleanup
        jQuery("#body").undelegate("#nothiddendivchild", "click");
  
        // Verify that .live() ocurs and cancel buble in the same order as
        // we would expect .bind() and .click() without delegation
        var lived = 0, livee = 0;
 -      
 +
        // bind one pair in one order
        jQuery("#body").delegate('span#liveSpan1 a', 'click', function(){ lived++; return false; });
        jQuery("#body").delegate('span#liveSpan1', 'click', function(){ livee++; });
        jQuery('span#liveSpan2 a').click();
        equals( lived, 1, "Verify that only one first handler occurred." );
        equals( livee, 0, "Verify that second handler doesn't." );
 -      
 +
        // Cleanup
        jQuery("#body").undelegate("click");
 -      
 +
        // Test this, target and currentTarget are correct
 -      jQuery("#body").delegate('span#liveSpan1', 'click', function(e){ 
 +      jQuery("#body").delegate('span#liveSpan1', 'click', function(e){
                equals( this.id, 'liveSpan1', 'Check the this within a delegate handler' );
                equals( e.currentTarget.id, 'liveSpan1', 'Check the event.currentTarget within a delegate handler' );
                equals( e.target.nodeName.toUpperCase(), 'A', 'Check the event.target within a delegate handler' );
        });
 -      
 +
        jQuery('span#liveSpan1 a').click();
 -      
 +
        jQuery("#body").undelegate('span#liveSpan1', 'click');
  
        // Work with deep selectors
@@@ -1789,18 -1796,18 +1797,18 @@@ test("delegate with change", function()
        expect(8);
  
        var selectChange = 0, checkboxChange = 0;
 -      
 +
        var select = jQuery("select[name='S1']");
        jQuery("#body").delegate("select[name='S1']", "change", function() {
                selectChange++;
        });
 -      
 -      var checkbox = jQuery("#check2"), 
 +
 +      var checkbox = jQuery("#check2"),
                checkboxFunction = function(){
                        checkboxChange++;
                }
        jQuery("#body").delegate("#check2", "change", checkboxFunction);
 -      
 +
        // test click on select
  
        // second click that changed it
        select[0].selectedIndex = select[0].selectedIndex ? 0 : 1;
        select.trigger("change");
        equals( selectChange, 1, "Change on click." );
 -      
 +
        // test keys on select
        selectChange = 0;
        select[0].selectedIndex = select[0].selectedIndex ? 0 : 1;
        select.trigger("change");
        equals( selectChange, 1, "Change on keyup." );
 -      
 +
        // test click on checkbox
        checkbox.trigger("change");
        equals( checkboxChange, 1, "Change on checkbox." );
 -      
 +
        // test blur/focus on text
        var text = jQuery("#name"), textChange = 0, oldTextVal = text.val();
        jQuery("#body").delegate("#name", "change", function() {
  
        text.val(oldTextVal);
        jQuery("#body").die("change");
 -      
 +
        // test blur/focus on password
        var password = jQuery("#name"), passwordChange = 0, oldPasswordVal = password.val();
        jQuery("#body").delegate("#name", "change", function() {
  
        password.val(oldPasswordVal);
        jQuery("#body").undelegate("#name", "change");
 -      
 +
        // make sure die works
 -      
 +
        // die all changes
        selectChange = 0;
        jQuery("#body").undelegate("select[name='S1']", "change");
        select[0].selectedIndex = select[0].selectedIndex ? 0 : 1;
        select.trigger("change");
        equals( selectChange, 0, "Die on keyup works." );
 -      
 +
        // die specific checkbox
        jQuery("#body").undelegate("#check2", "change", checkboxFunction);
        checkbox.trigger("change");
  
  test("delegate with submit", function() {
        var count1 = 0, count2 = 0;
 -      
 +
        jQuery("#body").delegate("#testForm", "submit", function(ev) {
                count1++;
                ev.preventDefault();
        jQuery("#testForm input[name=sub1]").submit();
        equals( count1, 1, "Verify form submit." );
        equals( count2, 1, "Verify body submit." );
 -      
 +
        jQuery("#body").undelegate();
        jQuery(document).undelegate();
  });