diff options
author | jeremy@chromium.org <jeremy@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-05-19 00:13:08 +0000 |
---|---|---|
committer | jeremy@chromium.org <jeremy@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-05-19 00:13:08 +0000 |
commit | 74d87f776a6e6103bc66b599a6e0f4c67b7c33b7 (patch) | |
tree | adbaafaab7adc99d11465de1352a0630280b905e /base/debug_util_mac.cc | |
parent | fdbb399c36ce791a053a0afcb035e30556f11e27 (diff) | |
download | chromium_src-74d87f776a6e6103bc66b599a6e0f4c67b7c33b7.zip chromium_src-74d87f776a6e6103bc66b599a6e0f4c67b7c33b7.tar.gz chromium_src-74d87f776a6e6103bc66b599a6e0f4c67b7c33b7.tar.bz2 |
Fix for breakpad not generating a minidump in certain cases on OSX.
1)Fix logic inversion in IsCrashReporterEnabled().
2)Don't intercept SIGPIPE in non-branded builds since it isn't fatal.
3)Roll DEPS to pickup a bunch of Mac Breakpad fixes.
When breakpad is disabled, we intercept a bunch of signals so that we can crash fast, without waiting for Apple's crash reporter. The problem was that the function we where using to test whether breakpad was enabled was wrong so we were always installing these signal handlers which where just calling exit().
By fixing the IsCrashReporterEnabled() call, we no longer install these signal handlers if Breakpad is enabled.
In any case SIGPIPE is non-fatal so we remove it from the list of signals we intercept.
There have been a number of fixes to the OSX version of Breakpad recently, so we pull those in as well.
BUG=11929
Review URL: http://codereview.chromium.org/115493
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@16346 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'base/debug_util_mac.cc')
-rw-r--r-- | base/debug_util_mac.cc | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/base/debug_util_mac.cc b/base/debug_util_mac.cc index 3a1b184..5a52387 100644 --- a/base/debug_util_mac.cc +++ b/base/debug_util_mac.cc @@ -25,10 +25,9 @@ void DebugUtil::DisableOSCrashDumps() { SIGBUS, SIGSEGV, SIGSYS, - SIGPIPE, SIGXCPU, SIGXFSZ}; - // For all these signals, just wire thing sup so we exit immediately. + // For all these signals, just wire things up so we exit immediately. for (size_t i = 0; i < arraysize(signals_to_intercept); ++i) { signal(signals_to_intercept[i], ExitSignalHandler); } |