Closed
Bug 320585
Opened 19 years ago
Closed 19 years ago
View source can miss a final = sign
Categories
(Core :: DOM: HTML Parser, defect, P4)
Tracking
()
RESOLVED
FIXED
mozilla1.9alpha1
People
(Reporter: mrbkap, Assigned: mrbkap)
References
()
Details
Attachments
(1 file)
|
(deleted),
patch
|
jst
:
review+
jst
:
superreview+
jst
:
approval-branch-1.8.1+
|
Details | Diff | Splinter Review |
STEPS TO REPRODUCE:
1) Go to the URL in the URL field.
2) View the source
ACTUAL RESULTS:
We show: <f />
EXPECTED RESULTS:
We show: <f /=>
|
Assignee | |
Updated•19 years ago
|
Status: NEW → ASSIGNED
Priority: -- → P4
|
|
Assignee | |
Comment 1•19 years ago
|
||
If we have an equals sign without a value, then we're not useless.
Attachment #206136 -
Flags: superreview?(jst)
Attachment #206136 -
Flags: review?(jst)
|
|
Assignee | |
Comment 2•19 years ago
|
||
Comment on attachment 206136 [details] [diff] [review]
Fix
Please pretend that there are two &s in that changed line.
|
||
Comment 3•19 years ago
|
||
Comment on attachment 206136 [details] [diff] [review]
Fix
r+sr=jst
Attachment #206136 -
Flags: superreview?(jst)
Attachment #206136 -
Flags: superreview+
Attachment #206136 -
Flags: review?(jst)
Attachment #206136 -
Flags: review+
|
|
Assignee | |
Comment 4•19 years ago
|
||
Fixed on trunk.
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
|
|
Assignee | |
Comment 5•19 years ago
|
||
Comment on attachment 206136 [details] [diff] [review]
Fix
This is a trivial patch that makes view source not lie. I don't think this is 1.8.0.1 fodder, but I'd love to see this go into 1.8.1.
Attachment #206136 -
Flags: approval1.8.1?
|
||
Updated•19 years ago
|
Attachment #206136 -
Flags: approval1.8.1? → branch-1.8.1?(jst)
|
|
||
Updated•19 years ago
|
Attachment #206136 -
Flags: branch-1.8.1?(jst) → branch-1.8.1+
You need to log in
before you can comment on or make changes to this bug.
Description
•