NaCl: Update revision in DEPS, r13770 -> r13783

NaCl: Update revision in DEPS, r13759 -> r13783

This pulls in the following Native Client changes:

r13760: (mseaborn) untrusted.gypi: Factor out duplicated "--gomadir" option
r13761: (mseaborn) Fix mmap() with length > 0xffff0000 to behave consistently on 32/64-bit
r13762: (teravest) Support file path metadata for the main nexe.
r13763: (teravest) Add CreateNaClDescWithFilePathMetadata fn.
r13764: (dschuff) Switch PNaCl lib, include, and translator directories for new layout
r13765: (dschuff) Pass naclsdk_validate=0 to scons runs run by main waterfall bots
r13768: (dbrazdil) Allowed decrementing program break with sbrk() in MinSFI
r13769: (dyen) Added tests for rename functions in irt_ext.
r13770: (dyen) Added tests for unlink functions irt_ext.
r13771: (dschuff) Update revision for PNaCl r13753->r13769
r13772: (dyen) Added tests for fsync/fdatasync functions in irt_ext.
r13773: (petarj) PNaCl: Update libcxx revision in pnacl/COMPONENT_REVISIONS
r13774: (dyen) Added tests for file truncation functions in irt_ext.
r13775: (sehr) Add Nick Bray and Shyam as owners of the validator
r13776: (jvoung) Remember to va_end in callingconv test.
r13777: (shyamsundarr) a script to generate a trie from our DFA.
r13778: (shyamsundarr) updated version of mseaborn@chromium's old trie code. We will generate
r13779: (dschuff) Allow build of host tools with system compiler
r13780: (dyen) Added support for supplying futex calls under irt_ext.
r13781: (dschuff) Remove archived_frontend test and use gcc build for FYI bots
r13782: (dschuff) Clang and binutils for direct to nacl x86_64
r13783: (dschuff) Change prep_nacl_sdk header locations for PNaCl to match new layout
r13784: (dschuff) Revert r13723, r13730, r13744
r13785: (dschuff) Revert "untrusted.gypi: Factor out duplicated "--gomadir" option"

This also adjusts the SDK header copying script to put the libnacl
headers in the new location for PNaCl.

BUG=none
TEST=browser_tests and nacl_integration
CQ_EXTRA_TRYBOTS=tryserver.chromium.linux:linux_rel_precise32,linux_nacl_sdk,linux_chromium_dbg

Committed: https://crrev.com/24cc138c03e5e58dd3cd1536be0bbcdd6bd9ba49
Cr-Commit-Position: refs/heads/master@{#296781}

[Derek Schuff, 2014-09-24 22:33:54]

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

[Derek Schuff, 2014-09-24 22:36:41]

dschuff@chromium.org changed reviewers:
+ noelallen@chromium.org

[Derek Schuff, 2014-09-24 22:37:08]

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

[Derek Schuff, 2014-09-24 22:37:31]

won't land until after https://codereview.chromium.org/604483002/ does, but
wanted to get a jump on the try jobs

[Derek Schuff, 2014-09-24 23:36:46]

On 2014/09/24 22:37:31, Derek Schuff wrote:
> won't land until after https://codereview.chromium.org/604483002/ does, but
> wanted to get a jump on the try jobs

@noel: I think there might be some kind of dependence problem related to the
prep_nacl_sdk step. the linux_rel_precise32 bot is failing because it's missing
a header, and it's not running
 ACTION prep_nacl_sdk: Prep pnacl toolchain_88bfa6cee4a917a975e4498ed9479300
the way the other bots are. Do you know how that's supposed to work?

[jvoung (off chromium), 2014-09-24 23:36:47]

lgtm

https://codereview.chromium.org/597393003/diff/1/ppapi/native_client/src/untr...
File
ppapi/native_client/src/untrusted/pnacl_support_extension/pnacl_component_crx_gen.py
(right):

https://codereview.chromium.org/597393003/diff/1/ppapi/native_client/src/untr...
ppapi/native_client/src/untrusted/pnacl_support_extension/pnacl_component_crx_gen.py:58:
def StandardArch(arch):
standardarch is now unused

https://codereview.chromium.org/597393003/diff/1/ppapi/native_client/src/untr...
ppapi/native_client/src/untrusted/pnacl_support_extension/pnacl_component_crx_gen.py:173:
return J(PnaclDirs.TranslatorRoot(), 'translator', '%s' % target_arch)
Except for the lib-XYZ -> XYZ, you could just change the commandline passed in
by the .gyp file to have the "translator" part.

[Derek Schuff, 2014-09-24 23:54:25]

https://codereview.chromium.org/597393003/diff/1/ppapi/native_client/src/untr...
File
ppapi/native_client/src/untrusted/pnacl_support_extension/pnacl_component_crx_gen.py
(right):

https://codereview.chromium.org/597393003/diff/1/ppapi/native_client/src/untr...
ppapi/native_client/src/untrusted/pnacl_support_extension/pnacl_component_crx_gen.py:58:
def StandardArch(arch):
On 2014/09/24 23:36:46, jvoung wrote:
> standardarch is now unused

Delorted.

https://codereview.chromium.org/597393003/diff/1/ppapi/native_client/src/untr...
ppapi/native_client/src/untrusted/pnacl_support_extension/pnacl_component_crx_gen.py:173:
return J(PnaclDirs.TranslatorRoot(), 'translator', '%s' % target_arch)
On 2014/09/24 23:36:46, jvoung wrote:
> Except for the lib-XYZ -> XYZ, you could just change the commandline passed in
> by the .gyp file to have the "translator" part.

we could, but then we'd be embedding knowledge of the toolchain's directory
layout there as well as here. might as well have it only in one place.

[Derek Schuff, 2014-09-25 00:01:07]

About the dependence, according to the ninja dependence browser,
the action (pnacl_dyncode_syscall_disabled_test.actions_rules_copies.stamp)
depends on gen/sdk/linux_x86/pnacl_newlib/stamp.prep which seems to be the right
thing. I don't know why the build was running that step without running the prep
step first. I tried a clobber build just to make sure something crazy wasn't
happening.

[binji, 2014-09-25 16:26:49]

binji@chromium.org changed reviewers:
+ binji@chromium.org

[binji, 2014-09-25 16:26:49]

native_client_sdk lgtm

[Derek Schuff, 2014-09-25 16:31:47]

The CQ bit was checked by dschuff@chromium.org

[commit-bot: I haz the power, 2014-09-25 16:32:28]

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

[Derek Schuff, 2014-09-25 20:20:35]

On 2014/09/25 16:32:28, I haz the power (commit-bot) wrote:
> CQ is trying da patch. Follow status at
>  https://chromium-cq-status.appspot.com/patch-status/597393003/80001

arm cross compile seems busted, although i can't reproduce it locally (cross
compile works for me). removing from CQ_EXTRA_TRYBOTS

[Derek Schuff, 2014-09-25 20:20:40]

The CQ bit was unchecked by dschuff@chromium.org

[Derek Schuff, 2014-09-25 20:21:17]

The CQ bit was checked by dschuff@chromium.org

[commit-bot: I haz the power, 2014-09-25 20:22:24]

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

[commit-bot: I haz the power, 2014-09-25 20:23:20]

Committed patchset #5 (id:80001) as c549461e58877f5db52601dcba0b2d7e88d46a53

[commit-bot: I haz the power, 2014-09-25 20:23:51]

Patchset 5 (id:??) landed as
https://crrev.com/24cc138c03e5e58dd3cd1536be0bbcdd6bd9ba49
Cr-Commit-Position: refs/heads/master@{#296781}