The default bug view has changed. See this FAQ.

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

Status

()

Core
JavaScript Engine
P1
normal
RESOLVED FIXED
12 years ago
11 years ago

People

(Reporter: Jesse Ruderman, Assigned: brendan)

Tracking

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

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

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(2 attachments)

(Reporter)

Description

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

Steps to reproduce:
1. Hover one of the equipment icons in
http://www.thottbot.com/index.cgi?p=Aydsnizzeth.

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
http://forums.mozillazine.org/viewtopic.php?t=321123.
(Reporter)

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?
(Reporter)

Comment 2

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

Copy of http://i73.thottbot.com/js/wow.js.
(Assignee)

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.

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

Updated

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

Bah!

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

Updated

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+
(Assignee)

Updated

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

Comment 6

12 years ago
Fixed on trunk and branch.

/be
Status: NEW → RESOLVED
Last Resolved: 12 years ago
Keywords: fixed1.8
Resolution: --- → FIXED
(Assignee)

Comment 7

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

Comment 8

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

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.