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 /chrome/app | |
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 'chrome/app')
-rw-r--r-- | chrome/app/breakpad_mac.mm | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/chrome/app/breakpad_mac.mm b/chrome/app/breakpad_mac.mm index 445edcb..28afe27 100644 --- a/chrome/app/breakpad_mac.mm +++ b/chrome/app/breakpad_mac.mm @@ -45,7 +45,7 @@ BreakpadRef gBreakpadRef = NULL; } // namespace bool IsCrashReporterEnabled() { - return gBreakpadRef == NULL; + return gBreakpadRef != NULL; } void DestructCrashReporter() { |