Refactor CrashDump*Manager to use a shared CrashDumpObserver.

content/shell/browser/shell_content_browser_client.cc

Index: content/shell/browser/shell_content_browser_client.cc
diff --git a/content/shell/browser/shell_content_browser_client.cc b/content/shell/browser/shell_content_browser_client.cc
index 2f522b40a170f9f3114f70b9f8ebe073237fdd0c..5fe4483d0466b298b80cedd8bb3cac0c8e8d583b 100644
--- a/content/shell/browser/shell_content_browser_client.cc
+++ b/content/shell/browser/shell_content_browser_client.cc
@@ -297,18 +297,8 @@ void ShellContentBrowserClient::GetAdditionalMappedFilesForChildProcess(
   regions->insert(std::make_pair(
       kShellPakDescriptor,
       base::GlobalDescriptors::GetInstance()->GetRegion(kShellPakDescriptor)));
-
-  if (breakpad::IsCrashReporterEnabled()) {

[Torne, 2016/08/03 16:33:48]

Where did this condition (and the equivalent in the other files)go? I can't spot
it in the new code, maybe I'm missing something?

-    base::File f(breakpad::CrashDumpManager::GetInstance()->CreateMinidumpFile(
-        child_process_id));
-    if (!f.IsValid()) {
-      LOG(ERROR) << "Failed to create file for minidump, crash reporting will "
-                 << "be disabled for this process.";
-    } else {
-      mappings->Transfer(kAndroidMinidumpDescriptor,
-                         base::ScopedFD(f.TakePlatformFile()));
-    }
-  }
+  breakpad::CrashDumpObserver::GetInstance()->BrowserChildProcessStarted(
+      child_process_id, mappings);
 }
 #elif defined(OS_POSIX) && !defined(OS_MACOSX)
 void ShellContentBrowserClient::GetAdditionalMappedFilesForChildProcess(