Add RequestOSFileHandle as a private PPAPI

content/renderer/pepper/renderer_ppapi_host_impl.cc

Index: content/renderer/pepper/renderer_ppapi_host_impl.cc
diff --git a/content/renderer/pepper/renderer_ppapi_host_impl.cc b/content/renderer/pepper/renderer_ppapi_host_impl.cc
index 50b4fc7b14952b7a0d95e3b79e5d08e10146751b..d371e475148ae5f433d9b8cd51794bc1434443b3 100644
--- a/content/renderer/pepper/renderer_ppapi_host_impl.cc
+++ b/content/renderer/pepper/renderer_ppapi_host_impl.cc
@@ -6,6 +6,8 @@
 
 #include "base/files/file_path.h"
 #include "base/logging.h"
+#include "base/process_util.h"
+#include "content/common/sandbox_util.h"
 #include "content/renderer/pepper/pepper_graphics_2d_host.h"
 #include "content/renderer/pepper/pepper_in_process_resource_creation.h"
 #include "content/renderer/pepper/pepper_in_process_router.h"
@@ -245,9 +247,9 @@ IPC::PlatformFileForTransit RendererPpapiHostImpl::ShareHandleWithRemote(
     base::PlatformFile handle,
     bool should_close_source) {
   if (!dispatcher_) {
-    if (should_close_source)
-      base::ClosePlatformFile(handle);
-    return IPC::InvalidPlatformFileForTransit();
+    return BrokerGetFileHandleForProcess(handle,

[dmichael (off chromium), 2013/03/27 19:40:53]

This might deserve a comment that what you're doing is sharing the handle back
to this process to support in-process.

[yzshen1, 2013/03/27 20:47:58]

Could you please DCHECK here that is_running_in_process_ is true? It will serve
as a comment and a warning when things change.

[hamaji, 2013/03/28 00:17:29]

Done.

[hamaji, 2013/03/28 00:17:29]

Done.

[yzshen1, 2013/03/28 20:59:49]

I don't see the change. Maybe it hasn't been uploaded?
On 2013/03/28 00:17:29, hamaji wrote:

[hamaji, 2013/03/28 21:37:15]

Oops, I was adding comments/DCHECK against a wrong tree.

+                                         base::GetCurrentProcId(),
+                                         should_close_source);
   }
   return dispatcher_->ShareHandleWithRemote(handle, should_close_source);
 }