Closed
Bug 39466
Opened 24 years ago
Closed 24 years ago
[regression] Unable to handle XML errors.
Categories
(Core :: DOM: HTML Parser, defect, P3)
Tracking
()
VERIFIED
FIXED
M16
People
(Reporter: harishd, Assigned: harishd)
Details
(Whiteboard: [nsbeta2-])
I introduced this regression ( yesterday ) while fixing a MLK. Test case: <?xml version="1.0"?> <?xml-stylesheet href="chrome://global/skin/" type="text/css"?> <!DOCTYPE window> <window style="height: 100%; width: 100%;" xmlns:html="http://www.w3.org/TR/REC-html40" xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul" onload="MyOnLoad();"> <!-- <html:script language="JavaScript" src="resource:/res/samples/test2.js" /> --> <html:script> function MyOnLoad() { for ( var i = 0; i < 100; i++ ) dump( "Hello World " + i + "\n" ); } </html:script> <html:div> Hello World </html:div> </window> Expected: Mozilla should display an XML error. Actual: Error not getting displayed ( Mozilla might crash too! ).
Setting M16...and nominating for nsbeta2.
Comment 2•24 years ago
|
||
Updating status whiteboard. I just reviewed and approved Harish's fix. Great work, Harish. You identified this regression yourself, assigned yourself a bug for it, fixed the problem and got it reviewed, all in the space of 1 day!
Whiteboard: Fix in hand.
Marking as crash, and recommend for PDT approval today.
Keywords: crash
Making sure that the XML errors are handled before calling DidBuildModel(). Fixed.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Comment 5•24 years ago
|
||
Not fixed 2000-07-07-10-M17 : WinNT Saved the test code to a file and loaded it. Browser doesn't hang, but rendering stops and no error message is displayed. reopening.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Whiteboard: Fix in hand.
Comment 6•24 years ago
|
||
Putting on [nsbeta2-] radar. Not critical to beta2. Removing crash keyword.
Keywords: crash
Whiteboard: [nsbeta2-]
Comment 7•24 years ago
|
||
oops, my bad, saved this as an html file! This bug is verified fixed. (sorry Harish!) 2000-07-07-10-M17 : WinNT
Status: REOPENED → RESOLVED
Closed: 24 years ago → 24 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•