diff options
| author | Dave Methvin <dave.methvin@gmail.com> | 2013-09-17 18:51:54 -0400 |
|---|---|---|
| committer | Dave Methvin <dave.methvin@gmail.com> | 2013-11-12 21:24:20 -0500 |
| commit | 97d53838e00f27d7dcf51f6a5627d8675f998284 (patch) | |
| tree | 84c1119c20892199e8870d8a7bee3965b8d5b09c | |
| parent | 60a6178131afec97b68c9a45bc24459f7b8bd905 (diff) | |
| download | jquery-97d53838e00f27d7dcf51f6a5627d8675f998284.tar.gz jquery-97d53838e00f27d7dcf51f6a5627d8675f998284.zip | |
Fix #14180. Allow cross-frame use of focusin/out. Close gh-1369.
(cherry picked from commit ebdb467761d756d4e52608a0df4a4d9b17da8092)
(conflicts with .data() resolved manually)
| -rw-r--r-- | src/event.js | 23 | ||||
| -rw-r--r-- | test/data/event/focusinCrossFrame.html | 18 | ||||
| -rw-r--r-- | test/unit/event.js | 33 |
3 files changed, 67 insertions, 7 deletions
diff --git a/src/event.js b/src/event.js index 2ca50aba4..3f19bab30 100644 --- a/src/event.js +++ b/src/event.js @@ -892,21 +892,30 @@ if ( !support.changeBubbles ) { if ( !support.focusinBubbles ) { jQuery.each({ focus: "focusin", blur: "focusout" }, function( orig, fix ) { - // Attach a single capturing handler while someone wants focusin/focusout - var attaches = 0, - handler = function( event ) { + // Attach a single capturing handler on the document while someone wants focusin/focusout + var handler = function( event ) { jQuery.event.simulate( fix, event.target, jQuery.event.fix( event ), true ); }; jQuery.event.special[ fix ] = { setup: function() { - if ( attaches++ === 0 ) { - document.addEventListener( orig, handler, true ); + var doc = this.ownerDocument, + attaches = jQuery._data( doc, fix ); + + if ( !attaches ) { + doc.addEventListener( orig, handler, true ); } + jQuery._data( doc, fix, ( attaches || 0 ) + 1 ); }, teardown: function() { - if ( --attaches === 0 ) { - document.removeEventListener( orig, handler, true ); + var doc = this.ownerDocument, + attaches = jQuery._data( doc, fix ) - 1; + + if ( !attaches ) { + doc.removeEventListener( orig, handler, true ); + jQuery._removeData( doc, fix ); + } else { + jQuery._data( doc, fix, attaches ); } } }; diff --git a/test/data/event/focusinCrossFrame.html b/test/data/event/focusinCrossFrame.html new file mode 100644 index 000000000..487f8de8f --- /dev/null +++ b/test/data/event/focusinCrossFrame.html @@ -0,0 +1,18 @@ +<!doctype html> +<html> +<head> + <meta http-equiv="Content-type" content="text/html; charset=utf-8"> + <title>focusin event cross-frame (#14180)</title> + + <script src="../../jquery.js"></script> +</head> +<body> + <input type="text" id="frame-input" /> + <script> + // Call parent when this frame is fully loaded, it will mess with #frame-input + jQuery( window ).one( "load", function() { + window.parent.iframeCallback( document ); + }); + </script> +</body> +</html> diff --git a/test/unit/event.js b/test/unit/event.js index b099ef2b9..454943718 100644 --- a/test/unit/event.js +++ b/test/unit/event.js @@ -2462,6 +2462,39 @@ test("fixHooks extensions", function() { jQuery.event.fixHooks.click = saved; }); +testIframeWithCallback( "focusin from an iframe", "event/focusinCrossFrame.html", function( frameDoc ) { + expect(1); + + var input = jQuery( frameDoc ).find( "#frame-input" ); + + if ( input.length ) { + // Create a focusin handler on the parent; shouldn't affect the iframe's fate + jQuery ( "body" ).on( "focusin.iframeTest", function() { + ok( false, "fired a focusin event in the parent document" ); + }); + + input.on( "focusin", function() { + ok( true, "fired a focusin event in the iframe" ); + }); + + // Avoid a native event; Chrome can't force focus to another frame + input.trigger( "focusin" ); + + // Must manually remove handler to avoid leaks in our data store + input.remove(); + + // Be sure it was removed; nothing should happen + input.trigger( "focusin" ); + + // Remove body handler manually since it's outside the fixture + jQuery( "body" ).off( "focusin.iframeTest" ); + + } else { + // Opera 12 (pre-Blink) doesn't select anything + ok( true, "SOFTFAIL: no focus event fired in the iframe" ); + } +}); + testIframeWithCallback( "jQuery.ready promise", "event/promiseReady.html", function( isOk ) { expect(1); ok( isOk, "$.when( $.ready ) works" ); |