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

Unified Diff: chrome/app/breakpad_linux.cc

Issue 11142010: Specify a file size limit for minidumps (Closed) Base URL: http://src.chromium.org/svn/trunk/src/
Patch Set: Created 8 years, 1 month 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
« no previous file with comments | « chrome/app/breakpad_linux.h ('k') | chrome/browser/crash_handler_host_linux.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/app/breakpad_linux.cc
===================================================================
--- chrome/app/breakpad_linux.cc (revision 168922)
+++ chrome/app/breakpad_linux.cc (working copy)
@@ -528,12 +528,14 @@
strncpy(g_crash_log_path, logfile_str.c_str(), crash_log_path_len);
}
DCHECK(!g_breakpad);
+ MinidumpDescriptor minidump_descriptor(dumps_path.value());
+ minidump_descriptor.set_size_limit(kMaxMinidumpFileSize);
#if defined(OS_ANDROID)
unattended = true; // Android never uploads directly.
#endif
if (unattended) {
g_breakpad = new ExceptionHandler(
- MinidumpDescriptor(dumps_path.value()),
+ minidump_descriptor,
NULL,
CrashDoneNoUpload,
NULL,
@@ -545,7 +547,7 @@
#if !defined(OS_ANDROID)
// Attended mode
g_breakpad = new ExceptionHandler(
- MinidumpDescriptor(dumps_path.value()),
+ minidump_descriptor,
NULL,
CrashDoneUpload,
NULL,
« no previous file with comments | « chrome/app/breakpad_linux.h ('k') | chrome/browser/crash_handler_host_linux.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698