Opened 10 years ago

Closed 9 years ago

#15770 closed defect (fixed)

[patch] [cla] dojox/html/_base (snarfStyles) picks up commented out styles, link tags, and @imports.

Reported by: Jared Jurkiewicz Owned by: Jared Jurkiewicz
Priority: undecided Milestone: 1.8.1
Component: HTML Version: 1.7.3
Keywords: Cc:
Blocked By: Blocking:

Description

dojox/html/_base (snarfStyles) picks up commented out styles, link tags, and @imports.

This is similar to bug: http://bugs.dojotoolkit.org/ticket/12133

If it processes a commented out block of links, etc, it will load them. And worse, if the comment block then ends up empty, the browser can actually lock up. This is seen with Google Chrome especially, but also can occur with IE 9. The snarfStyles function needs to escape those too, to skip them. Patch forthcoming.

Attachments (1)

15770.patch (4.7 KB) - added by Jared Jurkiewicz 10 years ago.
Patch for this issue. (Includes test)

Download all attachments as: .zip

Change History (4)

Changed 10 years ago by Jared Jurkiewicz

Attachment: 15770.patch added

Patch for this issue. (Includes test)

comment:1 Changed 10 years ago by bill

Component: GeneralHTML
Owner: set to Jared Jurkiewicz
Status: newassigned
Summary: dojox/html/_base (snarfStyles) picks up commented out styles, link tags, and @imports.[patch] [cla] dojox/html/_base (snarfStyles) picks up commented out styles, link tags, and @imports.

comment:2 Changed 10 years ago by Jared Jurkiewicz

Milestone: tbd1.8.1

comment:3 Changed 9 years ago by Jared Jurkiewicz

Resolution: fixed
Status: assignedclosed

In [29425]:

Fix snarfStyles to not choke on emptyish comments around style tags. \!strict fixes #15770

Note: See TracTickets for help on using tickets.