Closed Bug 484033 Opened 13 years ago Closed 13 years ago
ICrash Reporter to allow crashreporter to be used without going through XRE _Main
It would be nice to be able to get stack traces from xpcshell tests when they crash (bug 483062). xpcshell can't currently use the crashreporter code because the only way to initialize it is either to startup through XRE_Main, or to call it from C++. I've added a few things to nsICrashReporter to make it usable from xpcshell.
Needs a pass through the try server, but works fine on my mac. This extends nsICrashReporter, adding the following properties: .enabled .serverURL .minidumpPath It also adds an xpcshell test (yay!) I'll probably build on this and add a test that actually forces a crash and checks the dump later.
Comment on attachment 368074 [details] [diff] [review] extend nsICrashReporter Passes the try server.
Attachment #368074 - Flags: review?(benjamin)
Comment on attachment 368074 [details] [diff] [review] extend nsICrashReporter Do you want to sanity-check the URL at all, to force it to be HTTP for example?
Attachment #368074 - Flags: review?(benjamin) → review+
Yeah, I'll fix that. I also realized that this test might fail on --disable-libxul builds that have crash reporting enabled, since they won't be able to get the crashreporter service, so I'll tweak the test to accomodate.
Pushed to m-c: http://hg.mozilla.org/mozilla-central/rev/8e464707cf5b
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
I'd like to take this on 1.9.1 to enable bug 483062, but I guess I'll have to move the interface changes to a new branch-only interface. Should be feasible, although we should sort out bug 488596 first.
Depends on: 488596
Whiteboard: [Would want this on 1.9.1: see comment 6]
You need to log in before you can comment on or make changes to this bug.