HTML-style comments in the middle of .js file cause part of script to be ignored (treated as XML data)

RESOLVED FIXED in mozilla1.8beta5



JavaScript Engine
12 years ago
12 years ago


(Reporter: Jesse Ruderman, Assigned: brendan)


({fixed1.8, js1.6, regression})

fixed1.8, js1.6, regression
Bug Flags:
blocking1.8b5 +
in-testsuite +

Firefox Tracking Flags

(Not tracked)




(2 attachments)



12 years ago
Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.8b5) Gecko/20050922

Steps to reproduce:
1. Hover one of the equipment icons in

Result: Nothing happens.

Expected: Info box should appear.

Regression from bug 309242, "E4X should be on by default, while preserving the
comment hiding hack".

Identified by MozillaZine Forum member bizarrojack in

Comment 1

12 years ago
I guess we can...

(1) Back out bug 309242 and go back to requiring opt-in for E4X.

(2) Ask sites that use HTML-style comments in strange ways to change their code.

(3) Change the heuristic introduced by bug 309242 to handle this case as the
author intended.  For example, treat "<!--" as the beginning of XML comment data
iff it does not occur at the beginning of a statement.

Brendan, is (3) realistic?

Comment 2

12 years ago
Created attachment 197150 [details]
wow.js, the file with the HTML-style in the middle

Copy of

Comment 3

12 years ago
Created attachment 197152 [details] [diff] [review]
Use a more conservative heuristic

The heuristic is there just to spare people from having to select an E4X
version, via e4x=1, version=1.6, or better than these ineffective MIME type
parameters on application/javascript, perhaps the application/e4x MIME type
that Chris Lilley is proposing.

So if the heuristic is hurting, make it conservative, which may at the margin
require some E4X users to do the right thing, anyway, and request the MIME type
they require.

Attachment #197152 - Flags: superreview?(shaver)
Attachment #197152 - Flags: review?(mrbkap)


12 years ago
Assignee: general → brendan
Flags: blocking1.8b5+
Comment on attachment 197152 [details] [diff] [review]
Use a more conservative heuristic


Attachment #197152 - Flags: review?(mrbkap) → review+


12 years ago
Keywords: js1.6
OS: MacOS X → All
Priority: -- → P1
Hardware: Macintosh → All
Target Milestone: --- → mozilla1.8beta5
Comment on attachment 197152 [details] [diff] [review]
Use a more conservative heuristic

sr=shaver, though I'll also ping the Thottbot guys to fix their script!
Attachment #197152 - Flags: superreview?(shaver) → superreview+


12 years ago
Attachment #197152 - Flags: approval1.8b5+

Comment 6

12 years ago
Fixed on trunk and branch.

Last Resolved: 12 years ago
Keywords: fixed1.8
Resolution: --- → FIXED

Comment 7

12 years ago
*** Bug 309806 has been marked as a duplicate of this bug. ***

Comment 8

12 years ago
*** Bug 309778 has been marked as a duplicate of this bug. ***

Comment 9

12 years ago
*** Bug 309802 has been marked as a duplicate of this bug. ***

Comment 10

12 years ago
test covered by js1_6/Regress/regress-309242.js
Flags: testcase+
You need to log in before you can comment on or make changes to this bug.