Skip to main content

Bug Tracker

Side navigation

#11632 closed bug (invalid)

Opened April 24, 2012 09:01AM UTC

Closed April 24, 2012 04:36PM UTC

Last modified April 25, 2012 01:59AM UTC

FUNCTION CREATESAFEFRAGMENT HAVE SOME REDUNDANT CODE

Reported by: snandy Owned by: snandy
Priority: low Milestone: None
Component: unfiled Version: 1.7.2
Keywords: Cc:
Blocked by: Blocking:
Description

Only document object has createElement method. so "if ( safeFrag.createElement )"in createSafeFragment is redundant.

Attachments (0)
Change History (3)

Changed April 24, 2012 09:06AM UTC by sindresorhus comment:1

owner: → snandy
priority: undecidedlow
status: newpending

Can you link to some references?

Line number?

There's probably a reason it's there. Try to remove it, run the testsuite in supported browsers, and report back if it still passes.

Changed April 24, 2012 04:36PM UTC by rwaldron comment:2

resolution: → invalid
status: pendingclosed

That's how we avoid the loop in modern browsers. In oldIE, there is a createElement method on DocumentFragment: http://jsfiddle.net/rwaldron/ZGgGs/

Changed April 25, 2012 01:59AM UTC by snandy comment:3

thanks!