122534 - Ability in non-debug builds to send Error Console stuff to system console

Status: RESOLVED INCOMPLETE

(Core :: XPCOM, enhancement)

Description

[Matthew Cline]

Currently, the stuff that goes to the JavaScript console only goes to the
system console for debug builds.  I'd find it useful to be able to activate
this for non-debug builds, in the same way that you can now turn on the "dump()"
function, which previously only worked in the debug builds.

Comment 1

[Boris Zbarsky [:bzbarsky]]

Please measure the performance effects of any changes you make here (both with
the pref on and off)....

Comment 2

[andreww]

*** This bug has been marked as a duplicate of 84204 ***

Comment 3

[andreww]

argh. At first glance these bugs looked identical, but now I see that they are 
not. Ignore dupe message.

Comment 4

[Michiel van Leeuwen (email: mvl+moz@)]

Attachment: patch

This patch uses an ENV var to turn on logging to stderr.

Comment 5

[Mike Shaver (:shaver emeritus)]

Comment on attachment 210352 [details] [diff] [review]
patch

r=shaver! r=shaver!

I'd kinda like another name for the variable, but I'll let bsmedberg pick it.

Comment 6

[Benjamin Smedberg]

Comment on attachment 210352 [details] [diff] [review]
patch

You need to check

const char *env = PR_GetEnv("var");
if (env && *env)...

because using the PR functions an empty environment variable is supposed to act the same as an unset var.

Is XPCOM_CONSOLE_TO_STDERR too long?