Stop early exit watcher in BrowserChildProcessHostImpl::OnChildDisconnected

content/browser/browser_child_process_host_impl.cc

Index: content/browser/browser_child_process_host_impl.cc
diff --git a/content/browser/browser_child_process_host_impl.cc b/content/browser/browser_child_process_host_impl.cc
index de94d33dcb825047862e2ddec9153f05f9337ea9..d2d1faf46e32f37a5d2e3852f35290894812673d 100644
--- a/content/browser/browser_child_process_host_impl.cc
+++ b/content/browser/browser_child_process_host_impl.cc
@@ -270,6 +270,10 @@ bool BrowserChildProcessHostImpl::CanShutdown() {
 
 void BrowserChildProcessHostImpl::OnChildDisconnected() {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
+#if defined(OS_WIN)
+  DeleteProcessWaitableEvent(early_exit_watcher_.GetWatchedEvent());
+  early_exit_watcher_.StopWatching();

[rvargas (doing something else), 2014/10/17 01:00:07]

How does this work after https://codereview.chromium.org/651253002/ lands?

Do we still have to call StopWatching here? We actually stopped watching when
OnChannelConnected was received (which tells me nothing else is needed).

+#endif
   if (child_process_.get() || data_.handle) {
     DCHECK(data_.handle != base::kNullProcessHandle);
     int exit_code;