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 comment:1
| owner: | → snandy |
|---|---|
| priority: | undecided → low |
| status: | new → pending |
Changed April 24, 2012 04:36PM UTC by comment:2
| resolution: | → invalid |
|---|---|
| status: | pending → closed |
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 comment:3
thanks!
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.