Fix race condition in utility process clients

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 54f8fcca35e74b950db880b29273448e5d47a4f1..7cc10f141c17194990396ae67b9cd32b3adc0b97 100644
--- a/content/browser/browser_child_process_host_impl.cc
+++ b/content/browser/browser_child_process_host_impl.cc
@@ -215,7 +215,7 @@ void BrowserChildProcessHostImpl::OnChildDisconnected() {
   switch (status) {
     case base::TERMINATION_STATUS_PROCESS_CRASHED:
     case base::TERMINATION_STATUS_ABNORMAL_TERMINATION: {
-      delegate_->OnProcessCrashed(exit_code);
+      delegate_->OnProcessCrashedOrWasKilled(exit_code);
       // Report that this child process crashed.
       Notify(content::NOTIFICATION_CHILD_PROCESS_CRASHED);
       UMA_HISTOGRAM_ENUMERATION("ChildProcess.Crashed",
@@ -229,6 +229,7 @@ void BrowserChildProcessHostImpl::OnChildDisconnected() {
       break;
     }
     case base::TERMINATION_STATUS_PROCESS_WAS_KILLED: {
+      delegate_->OnProcessCrashedOrWasKilled(exit_code);

[jam, 2012/01/26 03:56:18]

it seems that consumers don't care if it crashed because it crashes or was
killed. so why not just leave it as OnProcessCrashed here?

[dgrogan, 2012/01/26 04:17:39]

I didn't want to blur the line between crashed and killed that is made here. But
you're right, that distinction doesn't matter to the consumers and keeping the
name makes this change smaller, so: changed back to OnProcessCrashed.

       // Report that this child process was killed.
       UMA_HISTOGRAM_ENUMERATION("ChildProcess.Killed",
                                 data_.type,