Don't pass ProcessHandle through ChildProcessHostDelegate.

base/process/process_win.cc

Index: base/process/process_win.cc
diff --git a/base/process/process_win.cc b/base/process/process_win.cc
index 05041b20ddaa7e8f908acc242dce2e45df066046..3e0d79fa8d39dc769f79eb241efaafeee397bb0c 100644
--- a/base/process/process_win.cc
+++ b/base/process/process_win.cc
@@ -39,6 +39,18 @@ Process Process::Current() {
 }
 
 // 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;
 }