We need you to sign our CLA before accepting a PR.

done

Did you invoke npm install? The Git commit hook would tell you the commit message is incorrect, it's missing a component Manipulation. The format for marking the bug closed is also different, please read https://contribute.jquery.org/commits-and-pull-requests/#commit-guidelines.

Otherwise, the code looks good.

I don't have node installed. I have updated the commit message as requested.

Component:Manipulation should be Manipulation: COMMIT_MESSAGE, for example:

Manipulation: Switch rnoInnerhtml to a version more performant in IE

The rest of the commit message looks good.

I have added additional detail to the commit message. Similar regular expressions exist but I didn't find one that will be used on large strings that might have the same kind of impact.

I have added additional detail to the commit message.

Could you add it beside the regexp itself? Otherwise we might forget that it is slow in Edge and remove it sometime in the future.

Could you add it beside the regexp itself? Otherwise we might forget that it is slow in Edge and remove it sometime in the future.

@SeanHenderson For reference, it should look something like that:

    // 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,

Similar regular expressions exist but I didn't find one that will be used on large strings that might have the same kind of impact.

I'm not sure about that. The above rxhtmlTag regex is used in jQuery.htmlPrefilter which, in turn is used on most inputs to jQuery.fn.html. It seems this may be causing exactly the same slowdowns (although it would be good to perf-test it to make sure).

Maybe there are others?

Sigh, I work with TFS still getting used to GIT. I have created a new branch and a new pull request so I am closing off this one.