Closed
Bug 325733
Opened 19 years ago
Closed 19 years ago
Unclosed XML tags no longer trigger an error message
Categories
(Core :: XML, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: jruderman, Assigned: peterv)
References
()
Details
(Keywords: regression, testcase)
Attachments
(2 files, 1 obsolete file)
|
10 bytes,
application/xml
|
Details | |
|
6.71 KB,
patch
|
bzbarsky
:
review+
bzbarsky
:
superreview+
|
Details | Diff | Splinter Review |
Steps to reproduce:
1. Load data:text/xml,<foo>.
Expected: XML parsing error.
Result: Shows a tree with a <foo> element.
Mac, trunk. Regressed between 2006-01-26 and 2006-01-30.
|
Reporter | |
Comment 1•19 years ago
|
||
|
Assignee | |
Comment 2•19 years ago
|
||
This fixes the bug. I need to check that it's the right thing to do.
Assignee: xml → peterv
Status: NEW → ASSIGNED
Flags: blocking1.9a1?
|
|
Assignee | |
Comment 3•19 years ago
|
||
This also fixes bug 326741.
Attachment #210605 -
Attachment is obsolete: true
Attachment #213745 -
Flags: superreview?(bzbarsky)
Attachment #213745 -
Flags: review?(bzbarsky)
|
||
Comment 4•19 years ago
|
||
Comment on attachment 213745 [details] [diff] [review]
v1.1
r+sr=bzbarsky.
It would be good to add some unit tests testing these various conditions if we can. Doing chunk boundaries could be tough, but perhaps a multiplex input stream passed to DOMParser could do the trick? Or something?
Attachment #213745 -
Flags: superreview?(bzbarsky)
Attachment #213745 -
Flags: superreview+
Attachment #213745 -
Flags: review?(bzbarsky)
Attachment #213745 -
Flags: review+
|
|
Assignee | |
Updated•19 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Flags: blocking1.9a1?
You need to log in
before you can comment on or make changes to this bug.
Description
•