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

mgol · mgol · commit b3eb2a13cd51 · 2015-07-28T13:24:32.000+02:00
Android 2.3 doesn't fire the window.onerror handler, just accept the reality there and skip the test. (cherry-picked from 6044fb6) Refs gh-1573 Refs gh-1786 Refs jquery/jquery.com#108 Closes gh-2458
diff --git a/test/unit/ajax.js b/test/unit/ajax.js
@@ -1485,6 +1485,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" );
diff --git a/test/unit/manipulation.js b/test/unit/manipulation.js
@@ -2304,6 +2304,16 @@ 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;
 
