[regression] Unable to handle XML errors.

VERIFIED FIXED in M16

Status

()

P3
major
VERIFIED FIXED
19 years ago
18 years ago

People

(Reporter: harishd, Assigned: harishd)

Tracking

Trunk
x86
Windows NT
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [nsbeta2-])

(Assignee)

Description

19 years ago
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! ).
(Assignee)

Comment 1

19 years ago
Setting M16...and nominating for nsbeta2.
Status: NEW → ASSIGNED
Keywords: nsbeta2
Target Milestone: --- → M16

Comment 2

19 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.

Comment 3

19 years ago
Marking as crash, and recommend for PDT approval today.
Keywords: crash
(Assignee)

Comment 4

19 years ago
Making sure that the XML errors are handled before calling DidBuildModel().
Fixed.
Status: ASSIGNED → RESOLVED
Last Resolved: 19 years ago
Resolution: --- → FIXED

Comment 5

18 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

18 years ago
Putting on [nsbeta2-] radar. Not critical to beta2. Removing crash keyword.
Keywords: crash
Whiteboard: [nsbeta2-]

Comment 7

18 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
Last Resolved: 19 years ago18 years ago
Resolution: --- → FIXED

Comment 8

18 years ago
and verified...
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.