Removed the ProcessHandle from the RendererClosedDetails payload

content/browser/child_process_launcher.cc

 // Copyright (c) 2012 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.
 
 #include "content/browser/child_process_launcher.h"
 
+#include <string>

[Charlie Reis, 2014/11/10 19:57:25]

Is this necessary?

[sramajay, 2014/11/11 14:16:05]

Done.

 #include <utility>  // For std::pair.
 
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/files/file_util.h"
 #include "base/files/scoped_file.h"
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/metrics/histogram.h"
 #include "base/process/process.h"
@@ skipping 295 matching lines @@
       if (!launched)
         handle = base::kNullProcessHandle;
 
 #if defined(OS_MACOSX)
       if (ShouldEnableBootstrapSandbox() &&
           bootstrap_sandbox_policy != SANDBOX_TYPE_INVALID) {
         GetBootstrapSandbox()->FinishedFork(handle);
       }
 
       if (launched)
-        broker->AddPlaceholderForPid(handle);
+        broker->AddPlaceholderForPid(handle, child_process_id);
 
       // After updating the broker, release the lock and let the child's
       // messasge be processed on the broker's thread.
       broker->GetLock().Release();
 #endif  // defined(OS_MACOSX)
     }
 #endif  // else defined(OS_POSIX)
 #if !defined(OS_ANDROID)
   if (handle)
     RecordHistograms(begin_launch_time);
@@ skipping 191 matching lines @@
   context_->SetProcessBackgrounded(background);
 }
 
 void ChildProcessLauncher::SetTerminateChildOnShutdown(
     bool terminate_on_shutdown) {
   if (context_.get())
     context_->set_terminate_child_on_shutdown(terminate_on_shutdown);
 }
 
 }  // namespace content