Non-SFI mode: Use dummy PID for NaCl's IPC channel for nacl_helper_nonsfi.

Non-SFI mode: Use dummy PID for NaCl's IPC channel for nacl_helper_nonsfi.

In nacl_helper_nonsfi, getpid() is prohibited to be called by seccomp-bpf
(will be implemented somehow soon). So, base::GetCurrentProcID() used in IPC
library, which uses getpid(), would cause a SIGSYS crashing.
As, in nacl_helper_nonsfi, PID is actually not used, so this CL replaces it
with -1 (dummy PID).

Note that the more generic CL crrev.com/695353005, which replaces PID with
dummy also on Linux platforms, was landed once, but reverted due to it was
still in use (crbug.com/441312). This CL extracts only Non-SFI related
part to avoid breakage.

TEST=Ran bots.
BUG=358465, 441312
CQ_EXTRA_TRYBOTS=tryserver.chromium.linux:linux_rel_precise32,linux_arm

Committed: https://crrev.com/763f8be26fd888e0ae7070a1a977238d2d65433b
Cr-Commit-Position: refs/heads/master@{#314283}

[hidehiko, 2015-02-02 12:33:50]

hidehiko@chromium.org changed reviewers:
+ dmichael@chromium.org, jln@chromium.org, mdempsky@chromium.org,
mseaborn@chromium.org

[hidehiko, 2015-02-02 12:33:50]

This CL is extracted only Non-SFI parts from
https://codereview.chromium.org/695353005/, which was reverted (i.e. I dropped
Linux code clean up), because of crbug.com/441312.

PTAL.

[Mark Seaborn, 2015-02-02 20:59:18]

LGTM

[dmichael (off chromium), 2015-02-02 23:43:38]

lgtm

https://codereview.chromium.org/879303004/diff/1/ipc/ipc_channel_posix.cc
File ipc/ipc_channel_posix.cc (right):

https://codereview.chromium.org/879303004/diff/1/ipc/ipc_channel_posix.cc#new...
ipc/ipc_channel_posix.cc:778: // allowed and would cause a SIGSYS crashing
because of the seccomp sandbox.
s/crashing/crash

[mdempsky, 2015-02-03 03:32:09]

lgtm

[hidehiko, 2015-02-03 04:53:13]

Thank you for quick review! Submitting.

https://codereview.chromium.org/879303004/diff/1/ipc/ipc_channel_posix.cc
File ipc/ipc_channel_posix.cc (right):

https://codereview.chromium.org/879303004/diff/1/ipc/ipc_channel_posix.cc#new...
ipc/ipc_channel_posix.cc:778: // allowed and would cause a SIGSYS crashing
because of the seccomp sandbox.
On 2015/02/02 23:43:38, dmichael wrote:
> s/crashing/crash

Done.

[hidehiko, 2015-02-03 04:53:18]

The CQ bit was checked by hidehiko@chromium.org

[commit-bot: I haz the power, 2015-02-03 04:55:00]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/879303004/20001

[commit-bot: I haz the power, 2015-02-03 07:25:03]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-02-03 07:25:48]

Patchset 2 (id:??) landed as
https://crrev.com/763f8be26fd888e0ae7070a1a977238d2d65433b
Cr-Commit-Position: refs/heads/master@{#314283}