312648 - NS_ScriptErrorReporter uses nsIScriptError for console message

Status: RESOLVED INVALID

(Core :: DOM: Core & HTML, defect, P5)

Description

[Alex Vincent [:WeirdAl]]

nsCOMPtr<nsIScriptError> errorObject =
do_CreateInstance("@mozilla.org/scripterror;1");
...
if (!report && message) {
 rv = errorObject->Init(msg.get(), nsnull, nsnull, 0, 0, 0, category);
}

Instead, we should just log the message.

Comment 1

[Boris Zbarsky [:bzbarsky]]

Just logging the message doesn't let you set the category, does it?

Comment 2

[Alex Vincent [:WeirdAl]]

(In reply to comment #1)
> Just logging the message doesn't let you set the category, does it?

Oh.  Hm.  That could be a problem.

Comment 3

[Alex Vincent [:WeirdAl]]

I'm going to replace the assertion I would've added for bug 312630 with a
warning.  There's just too many instances to justify crashing the program.

Comment 4

[Firefox Bug Husbandry Bot]

https://bugzilla.mozilla.org/show_bug.cgi?id=1472046

Move all DOM bugs that haven’t been updated in more than 3 years and has no one currently assigned to P5.

If you have questions, please contact :mdaly.

Comment 5

[Tom S. (please needinfo tschuster)]

NS_ScriptErrorReporter seems to be gone.