Open Bug 636081 Opened 11 years ago Updated 11 years ago

x11 error messages repeated in crash reports


(Toolkit :: Crash Reporting, defect)

Not set




(Reporter: karlt, Assigned: karlt)




(1 file, 2 obsolete files)

NS_RUNTIMEABORT appends the error string to crash report App Notes in
libxul-builds/apps.  As we've already added the string, it ends up repeated,
and we get app notes such as

 "X_CloseDevice: XI_BadDevice (invalid Device parameter); 9 requests agoxpcom_runtime_abort(###!!! ABORT: X_CloseDevice: XI_BadDevice (invalid Device parameter); 9 requests ago: file /builds/slave/cen-lnx64-ntly/build/toolkit/xre/nsX11ErrorHandler.cpp, line 190)"
Oops!  Unless there are existing tools that break when trying to parse the error string inside of the "xpcom_runtime_abort()", the simplest fix would be to switch the RUNTIMEABORT() in nsX11ErrorHandler to a direct mozalloc_abort().
Comment on attachment 514416 [details] [diff] [review]
't add X11 error app notes when they will be added during NS_RUNTIMEABORT

cjones pointed out that NS_RUNTIMEABORT doesn't handle forwarding from plugin processes, so this (on its own, at least) is not good.
Attachment #514416 - Attachment is obsolete: true
Attachment #514416 - Flags: review?(jones.chris.g)
As suggested by cjones in comment 1.
This also happens to workaround the double message output of NS_DebugBreak and in debug builds the double stack dump.
It loses the "###!!! ABORT", but I'm not too worried about that.
Attachment #516784 - Flags: review?(jones.chris.g)
Attachment #516784 - Flags: review?(jones.chris.g) → review+
Depends on: post2.0
Attached patch updated patchSplinter Review
Some platforms (maemo, linux64 opt + debug) need the header explicitly included now.
Attachment #516784 - Attachment is obsolete: true
Bug 581341 is making progress and, with that fixed, it would make sense to use NS_RUNTIMEABORT and fix the problems with NS_DebugBreak for all clients.
Depends on: 581341
No longer depends on: post2.0
You need to log in before you can comment on or make changes to this bug.