Closed Bug 325500 Opened 14 years ago Closed 14 years ago

ChatZilla should be closing its <script> tag in outputwindow.html

Categories

(Other Applications :: ChatZilla, defect, critical)

x86
Windows XP
defect
Not set
critical

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: Gijs, Assigned: rginda)

References

Details

(Whiteboard: [cz-0.9.71])

Attachments

(1 file)

Testcase coming up. This breaks ChatZilla.
Blocks: 305873
No longer depends on: 305873
Assignee: mrbkap → rginda
No longer blocks: 305873
Status: UNCONFIRMED → NEW
Component: HTML: Parser → ChatZilla
Ever confirmed: true
Keywords: regression
Product: Core → Other Applications
QA Contact: parser → samuel
Summary: HTML parser puts everything after <script /> in a script node (in an XHTML doctyped doc) → ChatZilla should be closing its <script> tag in outputwindow.html
Attached patch add </script>Splinter Review
Add back the </script> tag. Whether we want to keep the output window as html or change to xml is for later discussion.
Attachment #210411 - Flags: review?(samuel)
Attachment #210411 - Flags: review?(samuel) → review+
Checked in, so hopefully FIXED. (assuming that for once today, I've gotten something right...)
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
FWIW, I disagree with the fix of bug 305873, but I bet no-one gives a fuck.
Depends on: 305873
Well, if you state your case in a sensible manner in that bug i'm sure people will listen.
*** Bug 326032 has been marked as a duplicate of this bug. ***
*** Bug 327225 has been marked as a duplicate of this bug. ***
Whiteboard: [cz-0.9.71]
You need to log in before you can comment on or make changes to this bug.