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

Issue 1037953003: NaCl: Update revision in DEPS, 2d8b0bb -> b7793fe (Closed)

Created:
5 years, 9 months ago by JF
Modified:
5 years, 9 months ago
Reviewers:
Derek Schuff
CC:
chromium-reviews, Derek Schuff, hidehiko, JF, Mircea Trofin, native-client-reviews_googlegroups.com, Sam Clegg, Jim Stichnoth
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, 2d8b0bb -> b7793fe This pulls in the following Native Client changes: c7ebcbc: (hidehiko@chromium.org) Non-SFI mode: Update headers for nacl_helper_nonsfi. 2312a39: (stichnot@chromium.org) PNaCl: Update Subzero revision in pnacl/COMPONENT_REVISIONS e0ca648: (stichnot@chromium.org) Add Subzero to test scripts. 316f3e5: (dschuff@chromium.org) Build the IRT with nacl-clang for x86 70e1707: (dschuff@chromium.org) Revert of Build the IRT with nacl-clang for x86 (patchset #16 id:290001 of https://codereview.chromium.org/940993003/) 55bd3af: (jfb@chromium.org) PNaCl: Update LLVM revision in pnacl/COMPONENT_REVISIONS 715ffb5: (mtrofin@chromium.org) Using LLVM FileCheck exclusively. 5995d5f: (sbc@chromium.org) Implement pthread_rwlock functions for NaCl newlib f7533e5: (jfb@chromium.org) Update revision for PNaCl b7793fe: (jfb@chromium.org) Test bswap with large offset Changes: https://chromium.googlesource.com/native_client/src/native_client/+log/2d8b0bb6ffaf871fdc1f902d38778b2df8c6ff0d..b7793fedbadcf1c2c10c7f1a2ffb0f41342750ad BUG=none TEST=browser_tests and nacl_integration CQ_EXTRA_TRYBOTS=tryserver.chromium.linux:linux_chromium_dbg_32_ng,linux_nacl_sdk_build R= dschuff@chromium.org Committed: https://crrev.com/5f791b3ca5d8f45fa0f3723e9998875378ac4e2e Cr-Commit-Position: refs/heads/master@{#322417}

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: 9 (3 generated)
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1037953003/1
5 years, 9 months ago (2015-03-26 15:59:06 UTC) #2
commit-bot: I haz the power
No LGTM from a valid reviewer yet. Only full committers are accepted. Even if an ...
5 years, 9 months ago (2015-03-26 15:59:07 UTC) #4
Derek Schuff
lgtm
5 years, 9 months ago (2015-03-26 16:10:00 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1037953003/1
5 years, 9 months ago (2015-03-26 16:10:12 UTC) #7
commit-bot: I haz the power
Committed patchset #1 (id:1)
5 years, 9 months ago (2015-03-26 17:44:58 UTC) #8
commit-bot: I haz the power
5 years, 9 months ago (2015-03-26 17:45:52 UTC) #9
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/5f791b3ca5d8f45fa0f3723e9998875378ac4e2e
Cr-Commit-Position: refs/heads/master@{#322417}

Powered by Google App Engine
This is Rietveld 408576698