Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(92)

Issue 1159693005: NaCl: Update revision in DEPS, e255676 -> bcaef62 (Closed)

Created:
5 years, 6 months ago by Mark Seaborn
Modified:
5 years, 6 months ago
CC:
chromium-reviews, bradnelson, jvoung (off chromium), Mark Seaborn, native-client-reviews_googlegroups.com, rickyz (no longer on Chrome), Sam Clegg, scottmg, Sergiy Byelozyorov
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

NaCl: Update revision in DEPS, e255676 -> bcaef62 This pulls in the following Native Client changes: 1da5d86: (sbc@chromium.org) Fix for gyp build of nacl_bootstrap with target_arch=mips f404a83: (scottmg@chromium.org) Reland of 'vs2015: avoid redefining UINT8_MAX et al, some warning fixes' 3b1b3cf: (rickyz@chromium.org) Add SIGHUP and SIGTERM handlers in NaCl. 12ab428: (sergiyb@chromium.org) Added CQ config to the nacl repo ae55d9f: (bradnelson@chromium.org) Don't run asan toolchain build through runtest.py 23b1555: (jvoung@chromium.org) Update revision for PNaCl bcaef62: (mseaborn@chromium.org) SelLdrLauncher: Allow using a subset of SetupCommand(): Make some methods public Changes: https://chromium.googlesource.com/native_client/src/native_client/+log/e255676bce441a5a54072548cd1cf1ed8eaaecad..bcaef620a70f2e84e3660bd9098c4af811528f47 BUG=none TEST=browser_tests and nacl_integration CQ_EXTRA_TRYBOTS=tryserver.chromium.linux:linux_chromium_dbg_32_ng,linux_arm_compile,linux_nacl_sdk_build Committed: https://crrev.com/602c2ca843a3542928df1bae4f1fc7858f9e520a Cr-Commit-Position: refs/heads/master@{#333172}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M DEPS View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 7 (2 generated)
Mark Seaborn
5 years, 6 months ago (2015-06-05 22:26:12 UTC) #2
jvoung (off chromium)
lgtm
5 years, 6 months ago (2015-06-05 22:38:56 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1159693005/1
5 years, 6 months ago (2015-06-05 22:55:44 UTC) #5
commit-bot: I haz the power
Committed patchset #1 (id:1)
5 years, 6 months ago (2015-06-05 23:26:24 UTC) #6
commit-bot: I haz the power
5 years, 6 months ago (2015-06-05 23:27:04 UTC) #7
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/602c2ca843a3542928df1bae4f1fc7858f9e520a
Cr-Commit-Position: refs/heads/master@{#333172}

Powered by Google App Engine
This is Rietveld 408576698