]> source.dussan.org Git - jquery.git/commitdiff
Manipulation: Switch rnoInnerhtml to a version more performant in IE
authorSean Henderson <seanh.za@gmail.com>
Thu, 3 Sep 2015 15:48:20 +0000 (17:48 +0200)
committerMichał Gołębiowski <m.goleb@gmail.com>
Tue, 8 Sep 2015 17:31:57 +0000 (19:31 +0200)
IE versions greater than 9 do not handle the old regular expression well
with large html content. This is due to the use of a non-capturing group
after a very common html character (<).

Test suite: http://jsfiddle.net/Lwa0t5rp/3/
Microsoft bug: https://connect.microsoft.com/IE/feedback/details/1736512/

Fixes gh-2563
Closes gh-2574

src/manipulation.js

index 227d0ef9e69dab6d60174962ddff9488294067fa..b85ef1824d2582d5e364012db032891503589404 100644 (file)
@@ -27,7 +27,11 @@ define( [
 
 var
        rxhtmlTag = /<(?!area|br|col|embed|hr|img|input|link|meta|param)(([\w:-]+)[^>]*)\/>/gi,
-       rnoInnerhtml = /<(?:script|style|link)/i,
+
+       // Support: IE 10-11, Edge 10240+
+       // In IE/Edge using regex groups here causes severe slowdowns.
+       // See https://connect.microsoft.com/IE/feedback/details/1736512/
+       rnoInnerhtml = /<script|<style|<link/i,
 
        // checked="checked" or checked
        rchecked = /checked\s*(?:[^=]|=\s*.checked.)/i,