1151515 - Add a Components.utils.writeToStderr

Status: RESOLVED INACTIVE

(Core :: XPConnect, defect)

Description

[(not currently active) Ted Mielczarek]

Right now there's no easy way to write to stderr from chrome JS. I considered changing the implementation(s) of dump to allow sending it to stderr, but given how many implementations there are that was not appealing. I settled for adding a Components.utils.writeToStderr instead.

Comment 1

[Boris Zbarsky [:bzbarsky]]

I'd vote for just merging the umpteen Dump implemetations, because what we have right now is nuts.

Comment 2

[(not currently active) Ted Mielczarek]

I briefly considered that but they're all pretty different. :-( window.dump is using WebIDL-generated DOM bindings, the JS shell/xpcshell ones are using raw JSAPI, and there's at least one (message manager maybe?) that I don't fully understand.

Comment 3

[Boris Zbarsky [:bzbarsky]]

Merging would basically imply having some unified backend that takes UTF-8 or UTF-16 strings (however we produce those at the various callsites).  But yeah, I can totally understand not wanting to take this on.

Comment 4

[(not currently active) Ted Mielczarek]

Right, so:
* JS components/modules have a raw JSAPI impl:
https://hg.mozilla.org/mozilla-central/annotate/4fe763cbe196/js/xpconnect/loader/mozJSComponentLoader.cpp#l89

* nsFrameMessageManager has an XPConnect-based impl:
https://hg.mozilla.org/mozilla-central/annotate/4fe763cbe196/dom/base/nsFrameMessageManager.cpp#l825

* nsGlobalWindow is using DOM bindings:
https://hg.mozilla.org/mozilla-central/annotate/4fe763cbe196/dom/base/nsGlobalWindow.cpp#l6066

* xpcshell is using raw JSAPI:
https://hg.mozilla.org/mozilla-central/annotate/4fe763cbe196/js/xpconnect/src/XPCShellImpl.cpp#l280

(Is that all of them? I'm not even sure nowadays.)

Comment 5

[Boris Zbarsky [:bzbarsky]]

I think that should be all of them, yes.

Comment 6

[Chris Manchester (limited bugmail, email directly)]

Also SandboxDump: 
https://hg.mozilla.org/mozilla-central/annotate/dd32e3ff3717/js/xpconnect/src/Sandbox.cpp#l94

Comment 7

[Ben Turner (not reading bugmail, use the needinfo flag!)]

And Workers! http://mxr.mozilla.org/mozilla-central/source/dom/workers/WorkerScope.cpp#302

Comment 8

[BMO Automation]

Per policy at https://wiki.mozilla.org/Bug_Triage/Projects/Bug_Handling/Bug_Husbandry#Inactive_Bugs. If this bug is not an enhancement request or a bug not present in a supported release of Firefox, then it may be reopened.