Move jQuery(...) selector speed-up logic into Sizzle(...) qSA handling. Additionally add in a new catch for Sizzle('.class') (avoid using qSA and use getElementsByClassName instead, where applicable).
This commit is contained in:
parent
64e1cdbb95
commit
c475222bfb
1 changed files with 16 additions and 62 deletions
54
src/core.js
54
src/core.js
|
@ -15,9 +15,8 @@ var jQuery = function( selector, context ) {
|
||||||
// A central reference to the root jQuery(document)
|
// A central reference to the root jQuery(document)
|
||||||
rootjQuery,
|
rootjQuery,
|
||||||
|
|
||||||
// A simple way to check for HTML strings or ID strings
|
// A simple way to check for HTML strings
|
||||||
// (both of which we optimize for)
|
quickExpr = /^(?:[^<]*(<[\w\W]+>)[^>]*$/,
|
||||||
quickExpr = /^(?:[^<]*(<[\w\W]+>)[^>]*$|#([\w\-]+)$)/,
|
|
||||||
|
|
||||||
// Check if a string has a non-whitespace character in it
|
// Check if a string has a non-whitespace character in it
|
||||||
rnotwhite = /\S/,
|
rnotwhite = /\S/,
|
||||||
|
@ -93,25 +92,10 @@ jQuery.fn = jQuery.prototype = {
|
||||||
return this;
|
return this;
|
||||||
}
|
}
|
||||||
|
|
||||||
// The body element only exists once, optimize finding it
|
|
||||||
if ( selector === "body" && !context && document.body ) {
|
|
||||||
this.context = document;
|
|
||||||
this[0] = document.body;
|
|
||||||
this.selector = "body";
|
|
||||||
this.length = 1;
|
|
||||||
return this;
|
|
||||||
}
|
|
||||||
|
|
||||||
// Handle HTML strings
|
// Handle HTML strings
|
||||||
if ( typeof selector === "string" ) {
|
if ( typeof selector === "string" ) {
|
||||||
// Are we dealing with HTML string or an ID?
|
// Are we dealing with HTML string
|
||||||
match = quickExpr.exec( selector );
|
if ( (match = quickExpr.exec( selector )) ) {
|
||||||
|
|
||||||
// Verify a match, and that no context was specified for #id
|
|
||||||
if ( match && (match[1] || !context) ) {
|
|
||||||
|
|
||||||
// HANDLE: $(html) -> $(array)
|
|
||||||
if ( match[1] ) {
|
|
||||||
context = context instanceof jQuery ? context[0] : context;
|
context = context instanceof jQuery ? context[0] : context;
|
||||||
doc = (context ? context.ownerDocument || context : document);
|
doc = (context ? context.ownerDocument || context : document);
|
||||||
|
|
||||||
|
@ -135,36 +119,6 @@ jQuery.fn = jQuery.prototype = {
|
||||||
|
|
||||||
return jQuery.merge( this, selector );
|
return jQuery.merge( this, selector );
|
||||||
|
|
||||||
// HANDLE: $("#id")
|
|
||||||
} else {
|
|
||||||
elem = document.getElementById( match[2] );
|
|
||||||
|
|
||||||
// Check parentNode to catch when Blackberry 4.6 returns
|
|
||||||
// nodes that are no longer in the document #6963
|
|
||||||
if ( elem && elem.parentNode ) {
|
|
||||||
// Handle the case where IE and Opera return items
|
|
||||||
// by name instead of ID
|
|
||||||
if ( elem.id !== match[2] ) {
|
|
||||||
return rootjQuery.find( selector );
|
|
||||||
}
|
|
||||||
|
|
||||||
// Otherwise, we inject the element directly into the jQuery object
|
|
||||||
this.length = 1;
|
|
||||||
this[0] = elem;
|
|
||||||
}
|
|
||||||
|
|
||||||
this.context = document;
|
|
||||||
this.selector = selector;
|
|
||||||
return this;
|
|
||||||
}
|
|
||||||
|
|
||||||
// HANDLE: $("TAG")
|
|
||||||
} else if ( !context && !rnonword.test( selector ) ) {
|
|
||||||
this.selector = selector;
|
|
||||||
this.context = document;
|
|
||||||
selector = document.getElementsByTagName( selector );
|
|
||||||
return jQuery.merge( this, selector );
|
|
||||||
|
|
||||||
// HANDLE: $(expr, $(...))
|
// HANDLE: $(expr, $(...))
|
||||||
} else if ( !context || context.jquery ) {
|
} else if ( !context || context.jquery ) {
|
||||||
return (context || rootjQuery).find( selector );
|
return (context || rootjQuery).find( selector );
|
||||||
|
|
Loading…
Reference in a new issue