Non-SFI Mode: Build ipc/ library by PNaCl toolchain for nacl_helper_nonsfi.

ipc/ipc_channel_posix.h

Index: ipc/ipc_channel_posix.h
diff --git a/ipc/ipc_channel_posix.h b/ipc/ipc_channel_posix.h
index 717e43ee05fa710b27db64af7530360d2fcfef8c..c7ed782d35af745ae234a0c10f5e14c5fdef16d3 100644
--- a/ipc/ipc_channel_posix.h
+++ b/ipc/ipc_channel_posix.h
@@ -85,9 +85,9 @@ class IPC_EXPORT ChannelPosix : public Channel,
   void CloseClientFileDescriptor();
 
   static bool IsNamedServerInitialized(const std::string& channel_id);
-#if defined(OS_LINUX)
+#if defined(OS_LINUX) || defined(__native_client_nonsfi__)

[Mark Seaborn, 2014/10/23 03:33:48]

If I understand correctly, these changes to the PID handling won't be needed
until nacl_helper_nonsfi is seccomp-sandboxed.  The non-sandboxed version won't
need these changes.

So would you mind taking out the PID-related changes for now?  It should be
possible to make IPC work without exposing the PID inside the sandbox.

[hidehiko, 2014/10/23 14:19:10]

Ok, reverted. But we need to solve the pid issue.
Please let me start the discussion separately.

   static void SetGlobalPid(int pid);
-#endif  // OS_LINUX
+#endif
 
  private:
   bool CreatePipe(const IPC::ChannelHandle& channel_handle);
@@ -214,10 +214,10 @@ class IPC_EXPORT ChannelPosix : public Channel,
   // True if we are responsible for unlinking the unix domain socket file.
   bool must_unlink_;
 
-#if defined(OS_LINUX)
+#if defined(OS_LINUX) || defined(__native_client_nonsfi__)
   // If non-zero, overrides the process ID sent in the hello message.
   static int global_pid_;
-#endif  // OS_LINUX
+#endif
 
   DISALLOW_IMPLICIT_CONSTRUCTORS(ChannelPosix);
 };