Closed
Bug 39466
Opened 25 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•25 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: 25 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: 25 years ago → 24 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•