Opened 9 years ago

Closed 8 years ago

Last modified 8 years ago

#11210 closed defect (fixed)

dojox.layout.ContentPane: content setter runs a long time if the source contains a script tag and an HTML comment pattern

Reported by: Martin Jakobi Owned by: Sam Foster
Priority: high Milestone: 1.6
Component: DojoX Layout Version: 1.5.0b2
Keywords: dojox contentpane content setter Cc:
Blocked By: Blocking:

Description

If a source is containing a script tag and an HTML comment of a very specfic pattern then the dojox.layout.ContentPane?'s setter runs a long time (IE, ~90 seconds) or shows a script message (FF, after some seconds).

The critical comment pattern is:

<!--/div-->

Changing to something like:

<!--/div -->

e.g. inserting a space and it works.

See tcDojoCacheFF.html for FF and/or tcDojoCacheIE.html for IE.
Hint: tcDojoCacheIE.html is the smallest source I found at which the behaviour is reproducible in IE, but this is working in FF.
The order of the script tag and HTML comment doesn't matter.

Attachments (4)

tcDojoCacheIE.html (775 bytes) - added by Martin Jakobi 9 years ago.
test case IE
tcDojoCacheContentIE.html (206 bytes) - added by Martin Jakobi 9 years ago.
content source IE
tcDojoCacheFF.html (813 bytes) - added by Martin Jakobi 9 years ago.
test case FF
tcDojoCacheContentFF.html (808 bytes) - added by Martin Jakobi 9 years ago.
content source FF

Download all attachments as: .zip

Change History (8)

Changed 9 years ago by Martin Jakobi

Attachment: tcDojoCacheIE.html added

test case IE

Changed 9 years ago by Martin Jakobi

Attachment: tcDojoCacheContentIE.html added

content source IE

Changed 9 years ago by Martin Jakobi

Attachment: tcDojoCacheFF.html added

test case FF

Changed 9 years ago by Martin Jakobi

Attachment: tcDojoCacheContentFF.html added

content source FF

comment:1 Changed 9 years ago by dante

Owner: changed from dante to Sam Foster

you're it.

comment:2 Changed 9 years ago by bill

Component: DojoX WidgetsDojoX Layout

comment:3 Changed 8 years ago by Sam Foster

Resolution: fixed
Status: newclosed

Confirmed in 1.4 and 1.5, fixed in 1.6.0 by jaradj in r23480 by a change to the regexp in the snarfScripts function in dojox/html/_base.js

comment:4 Changed 8 years ago by bill

Milestone: tbd1.6
Note: See TracTickets for help on using tickets.