From 503e54564a8a88b7e968b64a920f2eeceffb0b74 Mon Sep 17 00:00:00 2001 From: Oleg Gaidarenko Date: Wed, 3 Dec 2014 12:35:21 +0300 Subject: [PATCH] Manipulation: don't test data-URI with script element in IE8 Since, apparently, it doesn't support it. Couldn't find more relevant info then this - http://msdn.microsoft.com/en-us/library/cc848897(v=vs.85).aspx No guard for older IE, since support for them will be removed soon anyway --- test/unit/manipulation.js | 20 ++++++++++++-------- 1 file changed, 12 insertions(+), 8 deletions(-) diff --git a/test/unit/manipulation.js b/test/unit/manipulation.js index f4918b59a..7e38d25af 100644 --- a/test/unit/manipulation.js +++ b/test/unit/manipulation.js @@ -2462,11 +2462,15 @@ test( "Make sure jQuery.fn.remove can work on elements in documentFragment", 1, equal( fragment.childNodes.length, 0, "div element was removed from documentFragment" ); }); -asyncTest( "Insert script with data-URI (gh-1887)", 1, function() { - Globals.register( "testFoo" ); - jQuery( "#qunit-fixture" ).append( "" ); - setTimeout(function (){ - strictEqual( window[ "testFoo" ], "foo", "data-URI script executed" ); - start(); - }, 100 ); -}); +// IE8 doesn't support data-URI in src attribute of script element +// Relevant - http://msdn.microsoft.com/en-us/library/cc848897(v=vs.85).aspx +if ( !/msie 8\.0/i.test( navigator.userAgent ) ) { + asyncTest( "Insert script with data-URI (gh-1887)", 1, function() { + Globals.register( "testFoo" ); + jQuery( "#qunit-fixture" ).append( "" ); + setTimeout(function (){ + strictEqual( window[ "testFoo" ], "foo", "data-URI script executed" ); + start(); + }, 100 ); + }); +} -- 2.39.5