crash: Fixes following r413431

components/crash/content/browser/crash_dump_observer_android.cc

Index: components/crash/content/browser/crash_dump_observer_android.cc
diff --git a/components/crash/content/browser/crash_dump_observer_android.cc b/components/crash/content/browser/crash_dump_observer_android.cc
index a7acdd89e51be855804d61aeb8f18473adb31c34..6cfaab7924e3e9a5d3f1cba71af519212155361b 100644
--- a/components/crash/content/browser/crash_dump_observer_android.cc
+++ b/components/crash/content/browser/crash_dump_observer_android.cc
@@ -32,6 +32,7 @@ CrashDumpObserver::CrashDumpObserver() {
   notification_registrar_.Add(this,
                               content::NOTIFICATION_RENDERER_PROCESS_CLOSED,
                               content::NotificationService::AllSources());
+  BrowserChildProcessObserver::Add(this);

[Lei Zhang, 2016/09/01 23:17:58]

Does the dtor need a BrowserChildProcessObserver::Remove(this) call to balance
this? Or does it not matter because this class is a Singleton?

[boliu, 2016/09/01 23:26:18]

Ahh, good point. Done

(Although.. this is android, globals are never going to be cleaned up..)

[Lei Zhang, 2016/09/01 23:30:44]

Right, just wondering what the policy is on Android.

 }
 
 CrashDumpObserver::~CrashDumpObserver() {}