Index: components/crash/content/browser/crash_dump_manager_android.h |
diff --git a/components/crash/content/browser/crash_dump_manager_android.h b/components/crash/content/browser/crash_dump_manager_android.h |
index e3f2391efe775851903cdfa88b6617599c646109..8e23e65453ee4f6ebb080f2b1e0c70de80fd455d 100644 |
--- a/components/crash/content/browser/crash_dump_manager_android.h |
+++ b/components/crash/content/browser/crash_dump_manager_android.h |
@@ -14,6 +14,7 @@ |
#include "content/public/browser/browser_child_process_observer.h" |
#include "content/public/browser/notification_observer.h" |
#include "content/public/browser/notification_registrar.h" |
+#include "content/public/common/process_type.h" |
namespace content { |
class RenderProcessHost; |
@@ -48,8 +49,18 @@ class CrashDumpManager : public content::BrowserChildProcessObserver, |
private: |
typedef std::map<int, base::FilePath> ChildProcessIDToMinidumpPath; |
+ // This enum is used to back a UMA histogram, and must be treated as |
+ // append-only. |
+ enum ExitStatus { |
+ EMPTY_MINIDUMP = 0, |
+ VALID_MINIDUMP = 1, |
+ COUNT = 2 |
+ }; |
+ |
static void ProcessMinidump(const base::FilePath& minidump_path, |
- base::ProcessHandle pid); |
+ base::ProcessHandle pid, |
+ content::ProcessType process_type, |
+ bool record_exit_status); |
// content::BrowserChildProcessObserver implementation: |
void BrowserChildProcessHostDisconnected( |
@@ -64,7 +75,10 @@ class CrashDumpManager : public content::BrowserChildProcessObserver, |
const content::NotificationDetails& details) override; |
// Called on child process exit (including crash). |
- void OnChildExit(int child_process_id, base::ProcessHandle pid); |
+ void OnChildExit(int child_process_id, |
+ base::ProcessHandle pid, |
+ content::ProcessType process_type, |
+ bool record_exit_status); |
content::NotificationRegistrar notification_registrar_; |