#12133 closed defect (fixed)
dojox/html/_base.js, snarfScripts: Hangs if there are empty HTML comments in the content
Reported by: | Jared Jurkiewicz | Owned by: | |
---|---|---|---|
Priority: | high | Milestone: | 1.6 |
Component: | DojoX Layout | Version: | 1.5 |
Keywords: | Cc: | uhop Douglas Hays | |
Blocked By: | Blocking: |
Description
This is a weird one and came up at my place of work. A long document had <!-- --> in it, just an empty comment ... and dojox.layout.ContentPane? hung and the dreaded 'this script is taking too long' showed up.
The problem was tracked down to 'dojox/html/_base.js, snarfScripts, the function that looks for commented out script tags. For some reason that regexp hangs. I can produce this in a very simple testcase. Will be attaching it
Attachments (3)
Change History (6)
Changed 10 years ago by
Attachment: | testComment.html added |
---|
comment:2 Changed 10 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
comment:3 Changed 10 years ago by
Milestone: | tbd → 1.6 |
---|
bulk update: bugs fixed in past few months, presumably milestone is 1.6
Note: See
TracTickets for help on using
tickets.
Test file that loks a simple HTML file with an empty comment