Event: remove outdated originalEvent hack

markelog · markelog · commit 841e7fa7e797 · 2015-05-20T19:00:56.000+03:00
Ref jquerygh-2300 Ref 2506c5d
diff --git a/src/event.js b/src/event.js
@@ -602,9 +602,6 @@ jQuery.event = {
 	},
 
 	simulate: function( type, elem, event, bubble ) {
-		// Piggyback on a donor event to simulate a different one.
-		// Fake originalEvent to avoid donor's stopPropagation, but if the
-		// simulated event prevents default then we do the same on the donor.
 		var e = jQuery.extend(
 			new jQuery.Event(),
 			event,
@@ -614,10 +611,6 @@ jQuery.event = {
 			}
 		);
 
-		// This prevents stopPropagation(), stopImmediatePropagation(), and preventDefault() from
-		// preventing default on the donor event.
-		delete e.originalEvent;
-
 		if ( bubble ) {
 			jQuery.event.trigger( e, null, elem );
 		} else {
diff --git a/test/unit/event.js b/test/unit/event.js
@@ -2701,37 +2701,111 @@ test( "preventDefault() on focusin does not throw exception", function( assert )
 			.focus();
 } );
 
-test( "jQuery.event.simulate() event has no originalEvent", function( assert ) {
-	expect( 1 );
+test( "Donor event interference", function( assert ) {
+	assert.expect( 10 );
 
-	var done = assert.async(),
-		input = jQuery( "<input>" )
-		.on( "click", function( event ) {
-			assert.strictEqual( "originalEvent" in event, false,
-				"originalEvent not present on simulated event" );
-			done();
-		} );
-
-	jQuery.event.simulate( "click", input[ 0 ], new jQuery.Event(), true );
-} );
+	var html = "<div id='donor-outer'>" +
+		"<form id='donor-form'>" +
+			"<input id='donor-input' type='radio' />" +
+		"</form>" +
+	"</div>";
 
-test( "Donor event interference", function( assert ) {
-	assert.expect( 4 );
+	jQuery( "#qunit-fixture" ).append( html );
 
-	jQuery( "#donor-outer" ).on( "click", function() {
+	jQuery( "#donor-outer" ).on( "click", function( event ) {
 		assert.ok( true, "click bubbled to outer div" );
+		assert.equal( typeof event.originalEvent, "object", "make sure originalEvent exist" );
+		assert.equal( event.type, "click", "make sure event type is correct" );
 	} );
 	jQuery( "#donor-input" ).on( "click", function( event ) {
 		assert.ok( true, "got a click event from the input" );
 		assert.ok( !event.isPropagationStopped(), "propagation says it's not stopped" );
+		assert.equal( event.type, "click", "make sure event type is correct" );
+		assert.equal( typeof event.originalEvent, "object", "make sure originalEvent exist" );
 	} );
 	jQuery( "#donor-input" ).on( "change", function( event ) {
+		assert.equal( typeof event.originalEvent, "object", "make sure originalEvent exist" );
+		assert.equal( event.type, "change", "make sure event type is correct" );
 		assert.ok( true, "got a change event from the input" );
 		event.stopPropagation();
 	} );
 	jQuery( "#donor-input" )[0].click();
 } );
 
+test( "originalEvent property for IE8", function( assert ) {
+	if ( !(/msie 8\.0/i.test( window.navigator.userAgent )) ) {
+		assert.expect( 1 );
+		assert.ok( true, "Assertions should run only in IE" );
+		return;
+	}
+
+	assert.expect( 12 );
+
+	var html = "<div id='donor-outer'>" +
+		"<form id='donor-form'>" +
+			"<input id='donor-input' type='radio' />" +
+		"</form>" +
+	"</div>";
+
+	jQuery( "#qunit-fixture" ).append( html );
+
+	jQuery( "#donor-outer" ).on( "change", function( event ) {
+		assert.ok( true, "click bubbled to outer div" );
+		assert.equal( event.originalEvent.type, "click", "make sure simulated event is a click" );
+		assert.equal( event.type, "change", "make sure event type is correct" );
+	} );
+
+	jQuery( "#donor-outer" ).on( "click", function( event ) {
+		assert.ok( true, "click bubbled to outer div" );
+		assert.equal( event.originalEvent.type, "click", "make sure originalEvent exist" );
+	} );
+	jQuery( "#donor-input" ).on( "click", function( event ) {
+		assert.ok( true, "got a click event from the input" );
+		assert.ok( !event.isPropagationStopped(), "propagation says it's not stopped" );
+		assert.equal( event.originalEvent.type, "click", "make sure originalEvent exist" );
+		assert.equal( event.type, "click", "make sure event type is correct" );
+	} );
+	jQuery( "#donor-input" ).on( "change", function( event ) {
+		assert.equal( event.originalEvent.type, "click", "make sure originalEvent exist" );
+		assert.equal( event.type, "change", "make sure event type is correct" );
+		assert.ok( true, "got a change event from the input" );
+	} );
+	jQuery( "#donor-input" )[0].click();
+} );
+
+test( "originalEvent property for Chrome, Safari and FF of simualted event", function( assert ) {
+	var userAgent = window.navigator.userAgent;
+
+	if ( !(/chrome/i.test( userAgent ) ||
+	       /firefox/i.test( userAgent ) ||
+	       /safari/i.test( userAgent ) ) ) {
+		assert.expect( 1 );
+		assert.ok( true, "Assertions should run only in Chrome, Safari and FF" );
+		return;
+	}
+
+	assert.expect( 4 );
+
+	var html = "<div id='donor-outer'>" +
+		"<form id='donor-form'>" +
+			"<input id='donor-input' type='radio' />" +
+		"</form>" +
+	"</div>";
+
+	jQuery( "#qunit-fixture" ).append( html );
+
+	jQuery( "#donor-outer" ).on( "focusin", function( event ) {
+		assert.ok( true, "focusin bubbled to outer div" );
+		assert.equal( event.originalEvent.type, "focus",
+		             "make sure originalEvent type is correct" );
+		assert.equal( event.type, "focusin", "make sure type is correct" );
+	} );
+	jQuery( "#donor-input" ).on( "focus", function() {
+		assert.ok( true, "got a focus event from the input" );
+	} );
+	jQuery( "#donor-input" )[0].focus();
+} );
+
 // This tests are unreliable in Firefox
 if ( !(/firefox/i.test( window.navigator.userAgent )) ) {
 	test( "Check order of focusin/focusout events", 2, function() {
