deac/jquery
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Skip id regex check when large html strings are passed to the jQuery constructor (#7990).

Browse source
This commit is contained in:
carpie 2011-01-19 17:37:31 -06:00 committed by Dave Methvin
parent ceaf093942
commit e0856738e6
2 changed files with 20 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
src/core.js
View file

@ -96,7 +96,12 @@ jQuery.fn = jQuery.prototype = {
// Handle HTML strings
if ( typeof selector === "string" ) {
// Are we dealing with HTML string or an ID?
match = quickExpr.exec( selector );
if ( selector.length > 1024 ) {
// Assume very large strings are HTML and skip the regex check
match = [ null, selector, null ];
} else {
match = quickExpr.exec( selector );
}
// Verify a match, and that no context was specified for #id
if ( match && (match[1] || !context) ) {

15
test/unit/core.js
View file

@ -469,7 +469,7 @@ test("isWindow", function() {
});
test("jQuery('html')", function() {
expect(15);
expect(18);
QUnit.reset();
jQuery.foo = false;
@ -501,6 +501,19 @@ test("jQuery('html')", function() {
ok( jQuery("<div></div>")[0], "Create a div with closing tag." );
ok( jQuery("<table></table>")[0], "Create a table with closing tag." );
// Test very large html string #7990
var i;
var li = '<li>very large html string</li>';
var html = ['<ul>'];
for ( i = 0; i < 50000; i += 1 ) {
html.push(li);
}
html.push('</ul>');
html = jQuery(html.join(''))[0];
equals( html.nodeName.toUpperCase(), 'UL');
equals( html.firstChild.nodeName.toUpperCase(), 'LI');
equals( html.childNodes.length, 50000 );
});
test("jQuery('html', context)", function() {