Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(393)

Unified Diff: src/processor/minidump_processor.cc

Issue 2160373002: Add new exception code for OOM generated from Chromium. (Closed) Base URL: https://chromium.googlesource.com/breakpad/breakpad.git@master
Patch Set: Created 4 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: src/processor/minidump_processor.cc
diff --git a/src/processor/minidump_processor.cc b/src/processor/minidump_processor.cc
index 3ff19518ce4c7828a73d564dec85cea5f36a137f..ddd8ae103bb588caac7e9c1d737c0a4f8aa1b940 100644
--- a/src/processor/minidump_processor.cc
+++ b/src/processor/minidump_processor.cc
@@ -1186,6 +1186,9 @@ string MinidumpProcessor::GetCrashReason(Minidump *dump, uint64_t *address) {
case MD_EXCEPTION_CODE_WIN_HEAP_CORRUPTION:
reason = "EXCEPTION_HEAP_CORRUPTION";
break;
+ case MD_EXCEPTION_OUT_OF_MEMORY:
+ reason = "EXCEPTION_OUT_OF_MEMORY";
Mark Mentovai 2016/07/19 20:30:50 This makes it sound like it’s a constant from the
Will Harris 2016/07/19 21:30:35 yes I agree, I've changed this to "Out of Memory".
+ break;
case MD_EXCEPTION_CODE_WIN_UNHANDLED_CPP_EXCEPTION:
reason = "Unhandled C++ Exception";
break;

Powered by Google App Engine
This is Rietveld 408576698