762519 - Send xpconnect stack dump and javascript dump output to OutputDebugString

Status: RESOLVED FIXED

(Core :: General, defect)

Description

[Jim Mathies [:jimm]]

I'd like to add a simple build config option that enables routing debug output to the Windows local/remote debugger via OutputDebugString. Something like "--enable-win-debug". I'm particularly interested seeing dump() and XPCDebug output, although maybe there's a way to capture everything we normally dump to the win console and send it over.

Comment 1

[Jim Mathies [:jimm]]

http://hg.mozilla.org/projects/elm/rev/d850e7827a2a

doh!

Comment 2

[Jim Mathies [:jimm]]

(In reply to Jim Mathies [:jimm] from comment #1)
> http://hg.mozilla.org/projects/elm/rev/d850e7827a2a
> 
> doh!

oops, wrong bug. double doh!

Comment 3

[Jim Mathies [:jimm]]

Attachment: js dump patch

Comment 4

[Jim Mathies [:jimm]]

Attachment: XPCDebug patch

Comment 5

[Jim Mathies [:jimm]]

Attachment: XPCDebug patch

use printf instead of puts in DebugDump.

Comment 6

[Blake Kaplan (:mrbkap) (inactive)]

Comment on attachment 642907 [details] [diff] [review]
XPCDebug patch

Review of attachment 642907 [details] [diff] [review]:
-----------------------------------------------------------------

Nits only.

::: js/xpconnect/src/XPCDebug.cpp
@@ +277,5 @@
>  {
>      char* buf;
>      JSExceptionState *state = JS_SaveExceptionState(cx);
>      if (!state)
> +        DebugDump("Call to a debug function modifying state!\n");

It'd be slightly faster and less error-prone to make this: DebugDump("%s", "...") and below, also. Failing that, it'd be a decent idea to use the gcc warning flags to get some error coverage for bad uses.

@@ +406,1 @@
>                 (void *)obj, js::GetObjectClass(obj)->name);

Nit: re-indent the second line.

Comment 7

[Jim Mathies [:jimm]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/a85081e2fbff

Comment 9

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/a85081e2fbff

Comment 10

[[e-mail]]

Comment on attachment 642902 [details] [diff] [review]
js dump patch

>@@ -4558,16 +4558,21 @@ nsGlobalWindow::Dump(const nsAString& aStr)
>+      OutputDebugStringA(cstr);
Nit: I would have preferred OutputDebugStringW(PromiseFlatString(aStr).get());

Comment 11

[[e-mail]]

Comment on attachment 642907 [details] [diff] [review]
XPCDebug patch

>+  char buffer[2048];
DumpJSStack can try to dump more than 2048 characters, unfortunately.

>+  printf(buffer);
printf(buffer); is potentially unsafe; use fputs(stdout, buffer); or printf("%s", buffer);

>-        fputs(buf, stdout);
>+        DebugDump("%s\n", buf);
Confusingly, although puts adds a newline, fputs does not.