NaCl Sandbox: close low-level IPC channel in Non-SFI mode

NaCl Sandbox: close low-level IPC channel in Non-SFI mode

In Non-SFI mode, we close the low-level kSandboxIPCChannel channel.

BUG=367263, 358719
R=mseaborn@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=266375

[jln (very slow on Chromium), 2014-04-25 22:26:26]

Mark: PTAL!

[Mark Seaborn, 2014-04-25 22:59:54]

LGTM

https://codereview.chromium.org/255903004/diff/1/components/nacl/loader/nacl_...
File components/nacl/loader/nacl_helper_linux.cc (right):

https://codereview.chromium.org/255903004/diff/1/components/nacl/loader/nacl_...
components/nacl/loader/nacl_helper_linux.cc:142: // should only be close in
Non-SFI mode.
"closed"

[jln (very slow on Chromium), 2014-04-25 23:40:55]

Thanks!

https://codereview.chromium.org/255903004/diff/1/components/nacl/loader/nacl_...
File components/nacl/loader/nacl_helper_linux.cc (right):

https://codereview.chromium.org/255903004/diff/1/components/nacl/loader/nacl_...
components/nacl/loader/nacl_helper_linux.cc:142: // should only be close in
Non-SFI mode.
On 2014/04/25 22:59:55, Mark Seaborn wrote:
> "closed"

Done.

[jln (very slow on Chromium), 2014-04-26 16:42:47]

The CQ bit was checked by jln@chromium.org

[commit-bot: I haz the power, 2014-04-26 16:43:08]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jln@chromium.org/255903004/20001

[commit-bot: I haz the power, 2014-04-26 17:26:29]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-04-26 17:26:29]

Try jobs failed on following builders:
  tryserver.chromium on chromium_presubmit

[Mark Seaborn, 2014-04-26 20:34:01]

The CQ bit was checked by mseaborn@chromium.org

[commit-bot: I haz the power, 2014-04-26 20:34:46]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jln@chromium.org/255903004/20001

[jln (very slow on Chromium), 2014-04-26 22:07:30]

Committed patchset #2 manually as r266375 (presubmit successful).