Permalink
Browse files

Ajax: Account for Android 2.3 not firing window.onerror on script errors

Android 2.3 doesn't fire the window.onerror handler, just accept the reality
there and skip the test.

Refs gh-1573
Refs gh-1786
Refs jquery/jquery.com#108
Closes gh-2458
  • Loading branch information...
mgol committed May 5, 2014
1 parent 2792845 commit 6044fb6a7384aec85906949835ef9a58114896ce
Showing with 20 additions and 0 deletions.
  1. +9 −0 test/unit/ajax.js
  2. +11 −0 test/unit/manipulation.js
View
@@ -1450,6 +1450,15 @@ module( "ajax", {
});
asyncTest( "#11743 - jQuery.ajax() - script, throws exception", 1, function() {
// Support: Android 2.3 only
// Android 2.3 doesn't fire the window.onerror handler, just accept the reality there.
if ( /android 2\.3/i.test( navigator.userAgent ) ) {
ok( true, "Test skipped, Android 2.3 doesn't fire window.onerror for " +
"errors in dynamically included scripts" );
start();
return;
}
var onerror = window.onerror;
window.onerror = function() {
ok( true, "Exception thrown" );
View
@@ -2241,6 +2241,17 @@ test( "Ensure oldIE creates a new set on appendTo (#8894)", function() {
});
asyncTest( "html() - script exceptions bubble (#11743)", 2, function() {
// Support: Android 2.3 only
// Android 2.3 doesn't fire the window.onerror handler, just accept the reality there.
if ( /android 2\.3/i.test( navigator.userAgent ) ) {
ok( true, "Test skipped, Android 2.3 doesn't fire window.onerror for " +
"errors in dynamically included scripts" );
ok( true, "Test skipped, Android 2.3 doesn't fire window.onerror for " +
"errors in dynamically included scripts" );
start();
return;
}
var onerror = window.onerror;
setTimeout(function() {

0 comments on commit 6044fb6

Please sign in to comment.