diff options
author | Michał Gołębiowski <m.goleb@gmail.com> | 2016-02-25 01:18:55 +0100 |
---|---|---|
committer | Michał Gołębiowski <m.goleb@gmail.com> | 2016-03-08 23:26:46 +0100 |
commit | 9b086888b803db872e8906cbdbf0de458231f225 (patch) | |
tree | 084539db8297eae7f97f82ffe0da2d318f81251f /test/unit/manipulation.js | |
parent | b43a3685b60b307d61f41f0c94412380ed46ab22 (diff) | |
download | jquery-9b086888b803db872e8906cbdbf0de458231f225.tar.gz jquery-9b086888b803db872e8906cbdbf0de458231f225.zip |
Docs:Tests: Remove obsolete code from tests, update support comments
Support comments that were lacking the final IE/Edge version that exhibits
the bug were checked & updated. Links to the Chromium bug tracker were updated.
Code in tests related to unsupported browsers (like Android 2.3 in non-basic
tests) has been removed.
Fixes gh-2868
Closes gh-2949
Diffstat (limited to 'test/unit/manipulation.js')
-rw-r--r-- | test/unit/manipulation.js | 32 |
1 files changed, 2 insertions, 30 deletions
diff --git a/test/unit/manipulation.js b/test/unit/manipulation.js index fb63e44fc..d439ba11c 100644 --- a/test/unit/manipulation.js +++ b/test/unit/manipulation.js @@ -406,8 +406,7 @@ QUnit.test( "XML DOM manipulation (#9960)", function( assert ) { assert.expect( 5 ); - var scxml1Adopted, - xmlDoc1 = jQuery.parseXML( "<scxml xmlns='http://www.w3.org/2005/07/scxml' version='1.0'><state x='100' y='100' initial='actions' id='provisioning'></state><state x='100' y='100' id='error'></state><state x='100' y='100' id='finished' final='true'></state></scxml>" ), + var xmlDoc1 = jQuery.parseXML( "<scxml xmlns='http://www.w3.org/2005/07/scxml' version='1.0'><state x='100' y='100' initial='actions' id='provisioning'></state><state x='100' y='100' id='error'></state><state x='100' y='100' id='finished' final='true'></state></scxml>" ), xmlDoc2 = jQuery.parseXML( "<scxml xmlns='http://www.w3.org/2005/07/scxml' version='1.0'><state id='provisioning3'></state></scxml>" ), xml1 = jQuery( xmlDoc1 ), xml2 = jQuery( xmlDoc2 ), @@ -415,15 +414,6 @@ QUnit.test( "XML DOM manipulation (#9960)", function( assert ) { scxml2 = jQuery( "scxml", xml2 ), state = scxml2.find( "state" ); - // Android 2.3 doesn't automatically adopt nodes from foreign documents. - // Although technically this is compliant behavior, no other browser - // (including newer Android Browsers) behave in this way so do the adopting - // just for Android 2.3. - // Support: Android 2.3 - if ( /android 2\.3/i.test( navigator.userAgent ) ) { - state = jQuery( xmlDoc1.adoptNode( state[ 0 ] ) ); - } - scxml1.append( state ); assert.strictEqual( scxml1[ 0 ].lastChild, state[ 0 ], "append" ); @@ -436,13 +426,7 @@ QUnit.test( "XML DOM manipulation (#9960)", function( assert ) { scxml1.find( "#provisioning" ).before( state ); assert.strictEqual( scxml1[ 0 ].firstChild, state[ 0 ], "before" ); - // Support: Android 2.3 - if ( /android 2\.3/i.test( navigator.userAgent ) ) { - scxml1Adopted = jQuery( xmlDoc2.adoptNode( scxml1[ 0 ] ) ); - scxml2.replaceWith( scxml1Adopted ); - } else { - scxml2.replaceWith( scxml1 ); - } + scxml2.replaceWith( scxml1 ); assert.deepEqual( jQuery( "state", xml2 ).get(), scxml1.find( "state" ).get(), "replaceWith" ); } ); @@ -2377,18 +2361,6 @@ QUnit.test( "Ensure oldIE creates a new set on appendTo (#8894)", function( asse } ); QUnit.asyncTest( "html() - script exceptions bubble (#11743)", 2, function( assert ) { - - // 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 ) ) { - assert.ok( true, "Test skipped, Android 2.3 doesn't fire window.onerror for " + - "errors in dynamically included scripts" ); - assert.ok( true, "Test skipped, Android 2.3 doesn't fire window.onerror for " + - "errors in dynamically included scripts" ); - QUnit.start(); - return; - } - var onerror = window.onerror; setTimeout( function() { |