Add IRT interface for PNaCl's sandboxed linker to use to talk to Chromium

Add IRT interface for PNaCl's sandboxed linker to use to talk to Chromium

This is a private IRT interface that PNaCl's sandboxed linker (Gold)
will use for communicating with Chromium's renderer process.

Initially, this is implemented using SRPC, using the same SRPC
interface that Gold and Chromium currently use.  This will allow
direct use of SRPC to be removed from Gold.

This IRT interface must be implemented in the NaCl repo currently
because PNaCl's sel_universal-based testing uses this SRPC interface.

Later, Chromium can be changed to provide its own implementation of
this IRT interface, based on Chrome IPC.

BUG=https://code.google.com/p/chromium/issues/detail?id=302078
TEST=tested with change to gold/nacl_file.cc (PNaCl linker)
NOPRESUBMIT=true

Committed: https://chromium.googlesource.com/native_client/src/native_client/+/6c1154966d275482620a31d6947e3e4863274b5d

[Mark Seaborn, 2015-02-05 18:03:32]

mseaborn@chromium.org changed reviewers:
+ jvoung@chromium.org

[Mark Seaborn, 2015-02-05 18:15:36]

Depends on a Chromium-side change: https://codereview.chromium.org/901053008/

Gold change: https://codereview.chromium.org/904583002/

[jvoung (off chromium), 2015-02-05 19:14:11]

LGTM

https://codereview.chromium.org/883393005/diff/60001/src/untrusted/irt/irt_pn...
File src/untrusted/irt/irt_pnacl_translator.c (right):

https://codereview.chromium.org/883393005/diff/60001/src/untrusted/irt/irt_pn...
src/untrusted/irt/irt_pnacl_translator.c:30: for (i = 0; i < obj_file_count;
i++) {
nit: could scope the "int i;" in the loop?

[Mark Seaborn, 2015-02-05 19:46:52]

New patchsets have been uploaded after l-g-t-m from jvoung@chromium.org

[Mark Seaborn, 2015-02-05 22:01:33]

https://codereview.chromium.org/883393005/diff/60001/src/untrusted/irt/irt_pn...
File src/untrusted/irt/irt_pnacl_translator.c (right):

https://codereview.chromium.org/883393005/diff/60001/src/untrusted/irt/irt_pn...
src/untrusted/irt/irt_pnacl_translator.c:30: for (i = 0; i < obj_file_count;
i++) {
On 2015/02/05 19:14:11, jvoung wrote:
> nit: could scope the "int i;" in the loop?

Done.  I didn't actually realise that is allowed in C99 as well as C++.

[Mark Seaborn, 2015-02-05 22:02:58]

The CQ bit was checked by mseaborn@chromium.org

[commit-bot: I haz the power, 2015-02-05 22:03:34]

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

[commit-bot: I haz the power, 2015-02-05 22:03:38]

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

[commit-bot: I haz the power, 2015-02-05 22:03:39]

Presubmit check for 883393005-80001 failed and returned exit status 1.

Running presubmit commit checks ...

** Presubmit Warnings **
/b/infra_internal/commit_queue/workdir/nacl/src/untrusted/irt/irt_dev.h: cpplint
  /b/infra_internal/commit_queue/workdir/nacl/src/untrusted/irt/irt_dev.h:14: 
Include the directory when naming .h files  [build/include] [4]
Done processing
/b/infra_internal/commit_queue/workdir/nacl/src/untrusted/irt/irt_dev.h
Total errors found: 1

[Mark Seaborn, 2015-02-05 22:44:45]

The CQ bit was checked by mseaborn@chromium.org

[commit-bot: I haz the power, 2015-02-05 22:45:44]

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

[commit-bot: I haz the power, 2015-02-05 23:14:11]

Committed patchset #5 (id:80001) as
https://chromium.googlesource.com/native_client/src/native_client/+/6c1154966...