Refactor CrashDump*Manager to use a shared CrashDumpObserver singleton.

chromecast/browser/cast_browser_main_parts.h

Index: chromecast/browser/cast_browser_main_parts.h
diff --git a/chromecast/browser/cast_browser_main_parts.h b/chromecast/browser/cast_browser_main_parts.h
index d60960472b4ea21bd601075112f6ed06c70f17d9..6ed065a05ba784ce0d3f0af4eef41ee378fc30c7 100644
--- a/chromecast/browser/cast_browser_main_parts.h
+++ b/chromecast/browser/cast_browser_main_parts.h
@@ -17,6 +17,12 @@ class SingleThreadTaskRunner;
 class Thread;
 }  // namespace base
 
+#if defined(OS_ANDROID)

[Bernhard Bauer, 2016/10/14 10:41:01]

Nit: in the interest of simplicity, you might want to leave out the #ifdef. A
superfluous forward-declaration is not going to hurt anyone.

[Tobias Sargeant, 2016/12/08 16:41:41]

Done.

+namespace breakpad {
+class CrashDumpManager;
+}
+#endif
+
 namespace net {
 class NetLog;
 }
@@ -82,6 +88,10 @@ class CastBrowserMainParts : public content::BrowserMainParts {
   std::unique_ptr<CastMemoryPressureMonitor> memory_pressure_monitor_;
 #endif
 
+#if defined(OS_ANDROID)
+  scoped_refptr<breakpad::CrashDumpManager> crash_dump_manager_;
+#endif
+
   DISALLOW_COPY_AND_ASSIGN(CastBrowserMainParts);
 };