Skip to content
Permalink
Browse files
Fix #13223. Re-allow leading space in HTML. Close gh-1264.
(cherry picked from commit 00eafdf)
  • Loading branch information
dmethvin committed May 9, 2013
1 parent c9267ab commit 9fdbc8b
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 14 deletions.
@@ -52,7 +52,7 @@ var
// A simple way to check for HTML strings
// Prioritize #id over <tag> to avoid XSS via location.hash (#9521)
// Strict HTML recognition (#11290: must start with <)
rquickExpr = /^(?:(<[\w\W]+>)[^>]*|#([\w-]*))$/,
rquickExpr = /^(?:\s*(<[\w\W]+>)[^>]*|#([\w-]*))$/,

// Match a standalone tag
rsingleTag = /^<(\w+)\s*\/?>(?:<\/\1>|)$/,
@@ -38,7 +38,7 @@ test("jQuery()", function() {
div = jQuery("<div/><hr/><code/><b/>"),
exec = false,
lng = "",
expected = 20,
expected = 22,
attrObj = {
"text": "test",
"class": "test2",
@@ -113,6 +113,12 @@ test("jQuery()", function() {

equal( jQuery(document.body).get(0), jQuery("body").get(0), "Test passing an html node to the factory" );

elem = jQuery(" <em>hello</em>")[0];
equal( elem.nodeName.toLowerCase(), "em", "leading space" );

elem = jQuery("\n\n<em>world</em>")[0];
equal( elem.nodeName.toLowerCase(), "em", "leading newlines" );

elem = jQuery("<div/>", attrObj );

if ( jQuery.fn.width ) {
@@ -73,18 +73,6 @@ test("disconnected nodes - jQuery only", function() {
equal( $opt.is(":selected"), true, "selected option" );
});

test("jQuery only - broken", 1, function() {
raises(function() {
// Setting context to null here somehow avoids QUnit's window.error handling
// making the e & e.message correct
// For whatever reason, without this,
// Sizzle.error will be called but no error will be seen in oldIE
jQuery.call( null, " <div/> " );
}, function( e ) {
return (/syntax.err/i).test( e.message );
}, "leading space invalid: $(' <div/> ')" );
});

testIframe("selector/html5_selector", "attributes - jQuery.attr", function( jQuery, window, document ) {
expect( 35 );

0 comments on commit 9fdbc8b

Please sign in to comment.