Refactor CrashDump*Manager to use a shared CrashDumpObserver singleton.

chromecast/browser/cast_browser_main_parts.h

 // Copyright 2014 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 CHROMECAST_BROWSER_CAST_BROWSER_MAIN_PARTS_H_
 #define CHROMECAST_BROWSER_CAST_BROWSER_MAIN_PARTS_H_
 
 #include <memory>
 
 #include "base/macros.h"
 #include "base/memory/ref_counted.h"
 #include "content/public/browser/browser_main_parts.h"
 #include "content/public/common/main_function_params.h"
 
 namespace base {
 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;
 }
 
 namespace chromecast {
 class CastMemoryPressureMonitor;
 
 namespace media {
 class MediaCapsImpl;
 class MediaPipelineBackendManager;
@@ skipping 45 matching lines @@
   // Tracks usage of media resource by e.g. CMA pipeline, CDM.
   media::MediaResourceTracker* media_resource_tracker_;
 
   // Tracks all media pipeline backends.
   std::unique_ptr<media::MediaPipelineBackendManager>
       media_pipeline_backend_manager_;
 
   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);
 };
 
 }  // namespace shell
 }  // namespace chromecast
 
 #endif  // CHROMECAST_BROWSER_CAST_BROWSER_MAIN_PARTS_H_