Linux: Handle renderer and plugin crashes on a separate thread.

Also set retries / timeouts for wget to attempt to limit the duration of each crash handler. Move some code out of headers while we're at it.

BUG=54071
TEST=change /usr/bin/wget to sleep forever, visit about:crash in a Breakpad-enabled build, make sure other renderers continue to work.
Review URL: http://codereview.chromium.org/3308007

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58403 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 14 insertions, 16 deletions

diff --git a/chrome/browser/crash_handler_host_linux.h b/chrome/browser/crash_handler_host_linux.h
index 625ac8e..7aa8dbe 100644
--- a/chrome/browser/crash_handler_host_linux.h
+++ b/chrome/browser/crash_handler_host_linux.h
@@ -1,4 +1,4 @@
-// Copyright (c) 2009 The Chromium Authors. All rights reserved.
+// Copyright (c) 2010 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
@@ -8,8 +8,13 @@
 
 #include <string>
 
-#include "base/singleton.h"
 #include "base/message_loop.h"
+#include "base/scoped_ptr.h"
+#include "base/singleton.h"
+
+namespace base {
+class Thread;
+}
 
 // This is the base class for singleton objects which crash dump renderers and
 // plugins on Linux. We perform the crash dump from the browser because it
@@ -51,6 +56,7 @@ class CrashHandlerHostLinux : public MessageLoopForIO::Watcher,
   int process_socket_;
   int browser_socket_;
   MessageLoopForIO::FileDescriptorWatcher file_descriptor_watcher_;
+  scoped_ptr<base::Thread> uploader_thread_;
 
   DISALLOW_COPY_AND_ASSIGN(CrashHandlerHostLinux);
 };
@@ -58,14 +64,10 @@ class CrashHandlerHostLinux : public MessageLoopForIO::Watcher,
 class PluginCrashHandlerHostLinux : public CrashHandlerHostLinux {
  private:
   friend struct DefaultSingletonTraits<PluginCrashHandlerHostLinux>;
-  PluginCrashHandlerHostLinux() {
-    SetProcessType();
-  }
-  virtual ~PluginCrashHandlerHostLinux() {}
+  PluginCrashHandlerHostLinux();
+  virtual ~PluginCrashHandlerHostLinux();
 
-  virtual void SetProcessType() {
-    process_type_ = "plugin";
-  }
+  virtual void SetProcessType();
 
   DISALLOW_COPY_AND_ASSIGN(PluginCrashHandlerHostLinux);
 };
@@ -73,14 +75,10 @@ class PluginCrashHandlerHostLinux : public CrashHandlerHostLinux {
 class RendererCrashHandlerHostLinux : public CrashHandlerHostLinux {
  private:
   friend struct DefaultSingletonTraits<RendererCrashHandlerHostLinux>;
-  RendererCrashHandlerHostLinux() {
-    SetProcessType();
-  }
-  virtual ~RendererCrashHandlerHostLinux() {}
+  RendererCrashHandlerHostLinux();
+  virtual ~RendererCrashHandlerHostLinux();
 
-  virtual void SetProcessType() {
-    process_type_ = "renderer";
-  }
+  virtual void SetProcessType();
 
   DISALLOW_COPY_AND_ASSIGN(RendererCrashHandlerHostLinux);
 };