Don't pass ProcessHandle through ChildProcessHostDelegate.

base/process/process_win.cc

 // Copyright (c) 2011 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 "base/process/process.h"
 
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/win/windows_version.h"
 
@@ skipping 21 matching lines @@
 }
 
 // static
 Process Process::Current() {
   Process process;
   process.is_current_process_ = true;
   return process.Pass();
 }
 
 // static
+Process Process::DeprecatedGetProcessFromHandle(ProcessHandle handle) {
+  DCHECK_NE(handle, ::GetCurrentProcess());
+  ProcessHandle out_handle;
+  if (!::DuplicateHandle(GetCurrentProcess(), handle,
+                         GetCurrentProcess(), &out_handle,
+                         0, FALSE, DUPLICATE_SAME_ACCESS)) {
+    return Process();

[cpu_(ooo_6.6-7.5), 2014/11/18 01:40:42]

no dcheck before returning at 48?

[rvargas (doing something else), 2014/11/18 01:52:34]

I don't think so. I'm mostly going by what we do at line 78, but I think there
are valid reasons (access related?) to fail here. dunno

+  }
+  return Process(out_handle);
+}
+
+// static
 bool Process::CanBackgroundProcesses() {
   return true;
 }
 
 bool Process::IsValid() const {
   return process_.IsValid() || is_current();
 }
 
 ProcessHandle Process::Handle() const {
   return is_current_process_ ? GetCurrentProcess() : process_.Get();
@@ skipping 70 matching lines @@
 
   return (::SetPriorityClass(Handle(), priority) != 0);
 }
 
 int Process::GetPriority() const {
   DCHECK(IsValid());
   return ::GetPriorityClass(Handle());
 }
 
 }  // namespace base