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

Issue 286033003: NaCl: Update revision in DEPS, r13167 -> r13181 (Closed)

Created:
6 years, 7 months ago by Mark Seaborn
Modified:
6 years, 7 months ago
Reviewers:
bradn
CC:
chromium-reviews, bradn, Derek Schuff, David Yen, jvoung - send to chromium..., Mark Seaborn, native-client-reviews_googlegroups.com, noelallen_use_chromium, petarj
Visibility:
Public.

Description

NaCl: Update revision in DEPS, r13167 -> r13181 This pulls in the following Native Client changes: r13168: (mseaborn) Non-SFI NaCl: Enable running more tests with nonsfi_loader r13169: (dyen) Make toolchain build upload linux 64 bit hosted toolchain. r13170: (dschuff) Add bot selectors for new PNaCl mac and windows bots r13171: (dyen) Removed old toolchain management files. r13172: (mseaborn) PNaCl: Remove unnecessary use of "-Ttext" option when doing bitcode linking r13173: (petarj) [MIPS] Switch to newer versions of the tools r13174: (noelallen) Create GROUPS for trusted and untrusted targets to avoid using default toolchain... r13175: (noelallen) Pin latest bionic version number. r13176: (jvoung) Remove deprecated NaClDescIoDescMakeFromHandle. r13177: (jvoung) Add more detailed perf markers to clarify perf of NaCl ELF file load. r13178: (petarj) [MIPS] Fix incorrect use of assembly macro in the test r13179: (bradnelson) Use SO_EXCLUSIVEADDRUSE on Windows instead of SO_REUSEADDR. r13180: (mseaborn) command_tester.py: Don't write output file if the test failed r13181: (mseaborn) readlink() syscall: Add missing test and fix to work on larger buffer sizes BUG=none TEST=browser_tests and nacl_integration Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=271081

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: 5 (0 generated)
Mark Seaborn
I intend to CQ this one...
6 years, 7 months ago (2014-05-15 18:14:37 UTC) #1
bradn
lgtm
6 years, 7 months ago (2014-05-15 18:19:06 UTC) #2
Mark Seaborn
The CQ bit was checked by mseaborn@chromium.org
6 years, 7 months ago (2014-05-16 18:34:45 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/mseaborn@chromium.org/286033003/1
6 years, 7 months ago (2014-05-16 18:35:45 UTC) #4
commit-bot: I haz the power
6 years, 7 months ago (2014-05-16 19:35:04 UTC) #5
Message was sent while issue was closed.
Change committed as 271081

Powered by Google App Engine
This is Rietveld 408576698