Closed Bug 325500 Opened 19 years ago Closed 19 years ago

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

Categories

(Other Applications :: ChatZilla, defect)

Product:
Other Applications
Component:
ChatZilla
Platform:
x86
Windows XP
Type:
defect
Priority:
Not set
Severity:
critical

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: Gijs, Assigned: rginda)

References

Details

(Whiteboard: [cz-0.9.71])

Attachments

(1 file)

add </script>
979 bytes, patch
samuel
: review+
Details | Diff | Splinter Review 
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: 19 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.

Attachment

General

Creator:
Created:
Updated:
Size: