Skip to content
Permalink
Browse files

Event: Simplify the logic behind blacklisting Android 2.3 on one test

  • Loading branch information
mgol committed Mar 18, 2014
1 parent 14e0a63 commit 55c196e8377221f1b417eda5cda74a56c5cae96a
Showing with 4 additions and 7 deletions.
  1. +4 −7 test/unit/event.js
@@ -387,12 +387,7 @@ test("on immediate propagation", function() {
});

test("on bubbling, isDefaultPrevented, stopImmediatePropagation", function() {
// Support: Android 2.3
if ( /android 2\.3/i.test( navigator.userAgent ) ) {
expect( 2 );
} else {
expect( 3 );
}
expect( 3 );

var $anchor2 = jQuery( "#anchor2" ),
$main = jQuery( "#qunit-fixture" ),
@@ -427,7 +422,9 @@ test("on bubbling, isDefaultPrevented, stopImmediatePropagation", function() {
// Android 2.3 doesn't support stopImmediatePropagation; jQuery fallbacks to stopPropagation
// in such a case.
// Support: Android 2.3
if ( !/android 2\.3/i.test( navigator.userAgent ) ) {
if ( /android 2\.3/i.test( navigator.userAgent ) ) {
ok( true, "Android 2.3, skipping native stopImmediatePropagation check" );
} else {
$anchor2.on( "click", function( e ) {
e.stopImmediatePropagation();
ok( true, "anchor was clicked and prop stopped" );

0 comments on commit 55c196e

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