Make EXC_BAD_ACCESS / EXC_I386_GPFLT print nicely in the processor

src/processor/minidump_processor.cc

Index: src/processor/minidump_processor.cc
diff --git a/src/processor/minidump_processor.cc b/src/processor/minidump_processor.cc
index 914685602035d07139e32a679deb1cd9d68804a6..decdf884e959ddb5f50b5d130b7b06d138d939ed 100644
--- a/src/processor/minidump_processor.cc
+++ b/src/processor/minidump_processor.cc
@@ -745,6 +745,19 @@ string MinidumpProcessor::GetCrashReason(Minidump *dump, uint64_t *address) {
                     BPLOG(INFO) << "Unknown exception reason " << reason;
                     break;
                 }
+              } else if (raw_system_info->processor_architecture ==
+                         MD_CPU_ARCHITECTURE_X86 ||
+                         raw_system_info->processor_architecture ==
+                         MD_CPU_ARCHITECTURE_AMD64) {
+                switch (exception_flags) {
+                  case MD_EXCEPTION_CODE_MAC_X86_GENERAL_PROTECTION_FAULT:
+                    reason.append("EXC_GPFLT");

[Mark Mentovai, 2016/03/25 14:52:24]

EXC_I386_GPFLT. Fix the change description too.

[Ted Mielczarek, 2016/03/25 15:46:44]

I copied this from:
https://chromium.googlesource.com/breakpad/breakpad/+/master/src/processor/mi...

Do you want that changed or should we leave it?

+                    break;
+                  default:
+                    reason.append(flags_string);
+                    BPLOG(INFO) << "Unknown exception reason " << reason;
+                    break;
+                }
               } else {
                 reason.append(flags_string);
                 BPLOG(INFO) << "Unknown exception reason " << reason;