We already fprintf, __android_log_print, etc, we don't need to loop in the logging system as well. Also note NS_DebugBreak can be called very early in startup and very late in shutdown so this will start to lead to possible issues of logging too early / too late as we improve the mozilla logging module.

Lets go ahead and do this before bug 1174785; it looks like it will make things easier.

Comment on attachment 8675803 [details] [diff] [review] Remove nsDebug logger Review of attachment 8675803 [details] [diff] [review]: ----------------------------------------------------------------- r=me, but please post on dev-platform about the removal and wait a day or so before committing to see if anybody yells extremely loudly about it.

(In reply to Nathan Froyd [:froydnj] from comment #2) > Comment on attachment 8675803 [details] [diff] [review] > Remove nsDebug logger > > Review of attachment 8675803 [details] [diff] [review]: > ----------------------------------------------------------------- > > r=me, but please post on dev-platform about the removal and wait a day or so > before committing to see if anybody yells extremely loudly about it. Try push including this built fine: https://treeherder.mozilla.org/#/jobs?repo=try&revision=472a09e30476 Posted to dev platform earlier today: https://groups.google.com/forum/#!topic/mozilla.dev.platform/mcsSEX4yJls

https://hg.mozilla.org/integration/mozilla-inbound/rev/3bc66762ac6ec54351a63f9d18cf4ef9f3d31a54 Bug 1215629 - Remove nsDebug logger. r=froydnj