Skip to content
Permalink
Browse files

Ajax: don’t test jQuery.ajax() on beforeunload in Chrome 31

Chrome 31 doesn't fire Ajax requests in beforeunload event handler.
There is no way for us to workaround it and it's been fixed in Chrome 32
so let's just blacklist Chrome 31 as long as it's in TestSwarm.
See https://code.google.com/p/chromium/issues/detail?id=321241
  • Loading branch information
mgol committed Dec 18, 2013
1 parent 8467710 commit fd0bce07584b73fb755574c4d462e0b175d1c6b6
Showing with 11 additions and 4 deletions.
  1. +11 −4 test/unit/ajax.js
@@ -1532,10 +1532,17 @@ module( "ajax", {
}
} );

testIframeWithCallback( "#14379 - jQuery.ajax() on unload", "ajax/onunload.html", function( status ) {
expect( 1 );
strictEqual( status, "success", "Request completed" );
});
// Support: Chrome 31.
// Chrome 31 doesn't fire Ajax requests in beforeunload event handler.
// There is no way for us to workaround it and it's been fixed in Chrome 32
// so let's just blacklist Chrome 31 as long as it's in TestSwarm.
// See https://code.google.com/p/chromium/issues/detail?id=321241
if ( navigator.userAgent.indexOf( " Chrome/31." ) === -1 ) {
testIframeWithCallback( "#14379 - jQuery.ajax() on unload", "ajax/onunload.html", function( status ) {
expect( 1 );
strictEqual( status, "success", "Request completed" );
});
}

//----------- jQuery.ajaxPrefilter()

0 comments on commit fd0bce0

Please sign in to comment.
You can’t perform that action at this time.