297814 - Report XML parsing errors to the error (JS) console

Status: RESOLVED FIXED

(Core :: XML, defect)

Description

[Benjamin Smedberg]

Currently XML parsing errors are displayed on-screen with the yellow error page,
but the error is not reported to the error (JS) console. If developers are using
XMLHttpRequest or some kinds of overlays, it is much more helpful if XML parsing
errors are reported to the error console.

Comment 1

[Boris Zbarsky [:bzbarsky]]

We have a bug on not using the yellow thing for data documents (and reporting
the error in some other way) already...

Comment 2

[Benjamin Smedberg]

Boris, can you point me at that bug, I can't find it; the closest thing I found
was bug 52422. I don't really want to touch how we do the yellow error pages
(yet), just add an additional message-to-console.

Comment 3

[Boris Zbarsky [:bzbarsky]]

Bug 289714, I believe.

Comment 4

[Benjamin Smedberg]

Attachment: Report XML parsing errors to the console, rev. 1

Comment 5

[Mike Shaver (:shaver emeritus)]

Comment on attachment 186991 [details] [diff] [review]
Report XML parsing errors to the console, rev. 1

I don't _really_ like using nsScriptError for this, because of the JavaScript
references in toString and so forth, but I sure do like reporting errors to the
console.  sr=shaver

Are we going to want a pref to suppress this category of errors, as many now
crave for the extremely common CSS error messages?

Comment 6

[Benjamin Smedberg]

I hope that these errors will be much less common than the CSS messages, as
there's not a lot of XML out there on the web yet. It sounds to me like people
want a console manager UI flag to suppress the display of certain types of
errors, which would be cool but is a different bug.

Comment 7

[Benjamin Smedberg]

Comment on attachment 186991 [details] [diff] [review]
Report XML parsing errors to the console, rev. 1

a=chofmann

Comment 8

[Benjamin Smedberg]

Fixed for 1.8b3

Comment 9

[Benjamin Smedberg]

Steps for QA testing of this bug are simple: visit a malformed XML file (e.g.
data:text/xml,<a ) and check to see that the error appears in the JS console
with the correct column and line number.