Non-SFI: move socketpair() from plugin process to browser process.

components/nacl/browser/nacl_process_host.h

Index: components/nacl/browser/nacl_process_host.h
diff --git a/components/nacl/browser/nacl_process_host.h b/components/nacl/browser/nacl_process_host.h
index b244e08bc3d5511979a82471c8b39496c9cfb557..fab653cfd5c80b7e9d72673ec798f7719be8e5cd 100644
--- a/components/nacl/browser/nacl_process_host.h
+++ b/components/nacl/browser/nacl_process_host.h
@@ -166,6 +166,15 @@ class NaClProcessHost : public content::BrowserChildProcessHostDelegate {
       const base::FilePath& file_path,
       base::File nexe_file);
 
+#if defined(OS_LINUX)
+  // Creates a pair of IPC::ChannelHandle. Returns true on success.
+  static bool CreateChannelHandlePair(ScopedChannelHandle* channel_handle1,
+                                      ScopedChannelHandle* channel_handle2);
+#endif
+
+  // Starts browser PPAPI proxy. Returns true on success. Otherwise false.

[Mark Seaborn, 2015/05/12 23:24:06]

Nit: You could drop "Otherwise false" for consistency with the comment above.

[hidehiko, 2015/05/13 06:39:20]

Done.

+  bool StartPPAPIProxy(ScopedChannelHandle channel_handle);
+
   // Does post-process-launching tasks for starting the NaCl process once
   // we have a connection.
   //
@@ -191,7 +200,7 @@ class NaClProcessHost : public content::BrowserChildProcessHostDelegate {
   // Called when the PPAPI IPC channels to the browser/renderer have been
   // created.
   void OnPpapiChannelsCreated(
-      const IPC::ChannelHandle& browser_channel_handle,
+      const IPC::ChannelHandle& ppapi_browser_channel_handle,
       const IPC::ChannelHandle& ppapi_renderer_channel_handle,
       const IPC::ChannelHandle& trusted_renderer_channel_handle,
       const IPC::ChannelHandle& manifest_service_channel_handle);