Refactor CrashDump*Manager to use a shared CrashDumpObserver.

android_webview/browser/aw_browser_terminator.h

Index: android_webview/browser/aw_browser_terminator.h
diff --git a/android_webview/browser/aw_browser_terminator.h b/android_webview/browser/aw_browser_terminator.h
new file mode 100644
index 0000000000000000000000000000000000000000..9cf83819f4a55466647f51d8a9ed5730d3e2db27
--- /dev/null
+++ b/android_webview/browser/aw_browser_terminator.h
@@ -0,0 +1,49 @@
+// Copyright 2016 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.
+
+#ifndef COMPONENTS_CRASH_CONTENT_BROWSER_CRASH_DUMP_BROWSER_TERMINATOR_H_
+#define COMPONENTS_CRASH_CONTENT_BROWSER_CRASH_DUMP_BROWSER_TERMINATOR_H_
+
+#include <map>
+
+#include "base/sync_socket.h"

[Bernhard Bauer, 2016/08/04 16:01:32]

Could you forward-declare this instead of including the header?

[Tobias Sargeant, 2016/08/05 14:59:21]

Done.

+#include "base/synchronization/lock.h"
+#include "components/crash/content/browser/crash_dump_observer_android.h"
+
+namespace android_webview {
+
+// This class manages behavior of the browser on renderer crashes when
+// microdumps are used for capturing the crash stack.  Normally, in this case
+// the browser doesn't need to do much, because a microdump is written into
+// Android log by the renderer process itself.  However, the browser may need to
+// crash itself on a renderer crash.  Since on Android renderers are not child
+// processes of the browser, it can't access the exit code. Instead, the browser
+// uses a dedicated pipe in order to receive the information about the renderer
+// crash status.
+class AwBrowserTerminator : public breakpad::CrashDumpObserver::Client {
+ public:
+  AwBrowserTerminator();
+  ~AwBrowserTerminator() override;
+
+  // CrashDumpObserver::Client implementation.
+  void OnChildStart(int child_process_id,
+                    content::FileDescriptorInfo* mappings) override;
+  void OnChildExit(int child_process_id,
+                   base::ProcessHandle pid,
+                   content::ProcessType process_type,
+                   base::TerminationStatus termination_status,
+                   base::android::ApplicationState app_state) override;
+
+ private:
+  // This map should only be accessed with its lock aquired as it is accessed
+  // from the PROCESS_LAUNCHER, FILE, and UI threads.
+  base::Lock child_process_id_to_pipe_lock_;
+  std::map<int, std::unique_ptr<base::SyncSocket>> child_process_id_to_pipe_;
+
+  DISALLOW_COPY_AND_ASSIGN(AwBrowserTerminator);
+};
+
+}  // namespace breakpad
+
+#endif  // COMPONENTS_CRASH_CONTENT_BROWSER_CRASH_DUMP_BROWSER_TERMINATOR_H_