#11632 closed bug (invalid)
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.
Change History (3)
comment:1 Changed 10 years ago by
| Owner: | set to snandy |
|---|---|
| Priority: | undecided → low |
| Status: | new → pending |
comment:2 Changed 10 years ago by
| 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/
Note: See
TracTickets for help on using
tickets.
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.