Remove some PlatformFile uses from NaCl.

ipc/ipc_platform_file.h

 // 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.
 
 #ifndef IPC_IPC_PLATFORM_FILE_H_
 #define IPC_IPC_PLATFORM_FILE_H_
 
 #include "base/basictypes.h"
 #include "base/files/file.h"
 #include "base/platform_file.h"
@@ skipping 23 matching lines @@
 inline base::PlatformFile PlatformFileForTransitToPlatformFile(
     const PlatformFileForTransit& transit) {
 #if defined(OS_WIN)
   return transit;
 #elif defined(OS_POSIX)
   return transit.fd;
 #endif
 }
 
 // Returns a file handle equivalent to |file| that can be used in |process|.
 IPC_EXPORT PlatformFileForTransit GetFileHandleForProcess(

[Mark Seaborn, 2014/02/14 19:00:08]

Might want a TODO to remove this one in favour of the new function?

[rvargas (doing something else), 2014/02/14 19:37:15]

Initially I was thinking that would be the case, but there's a couple of callers
that don't want the handle closed and in that case passing File would not work,
so I think we'll need both versions long term (probably removing the last
argument from this version)

     base::PlatformFile file,
     base::ProcessHandle process,
     bool close_source_handle);
 
+// Returns a file handle equivalent to |file| that can be used in |process|.
+// Note that this function takes ownership of |file|.
+IPC_EXPORT PlatformFileForTransit TakeFileHandleForProcess(
+    base::File file,
+    base::ProcessHandle process);
+
 }  // namespace IPC
 
 #endif  // IPC_IPC_PLATFORM_FILE_H_