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

Issue 7583017: NaCl: Update revision in DEPS, r6322 -> r6339 (Closed)

Created:
9 years, 4 months ago by Mark Seaborn
Modified:
9 years, 4 months ago
CC:
chromium-reviews, native-client-reviews_googlegroups.com, jvoung - send to chromium..., krasin1, Roland McGrath, Mark Seaborn, Nick Bray, robertm, Nico
Visibility:
Public.

Description

NaCl: Update revision in DEPS, r6322 -> r6339 This pulls in the following Native Client changes: r6323: (mseaborn) sel_universal: Clean up usage of threads a little r6324: (jvoung) Fix pnacl_coordinator's usage of undefined var_arg behavior (broke on Windows) r6325: (krasin) service_runtime: remove nacl_bottom_half because it's a dead code. r6326: (robertm) This will allow us to do proper book-keeping of PPB_Core_AddRefResource and r6327: (robertm) Make the pepper event loop more versatile, i.e. allow for polling and time limits. r6328: (krasin) Revert r6325 due to build break (service_runtime: remove nacl_bottom_half because it's a dead code) r6329: (robertm) Revert 6326 which fails memcheck r6330: (mseaborn) Cleanup: Remove unused function NaClThreadKill() r6331: (mseaborn) Cleanup: Remove help string for sel_ldr's '-I' option (which was removed in r6280) r6332: (thakis) Fix a compiler warning. r6333: (robertm) Add a few more tests to url_loader and some cleanup r6334: (robertm) Resubmit of rev6326 after fixing memcheck issue r6335: (mcgrathr) Parameterize in-tree test links' use of libraries r6336: (krasin) service_runtime: remove nacl_bottom_half because it's a dead code. r6337: (jvoung) Update compiletime perf expectations for benchmarks that compile faster r6338: (mseaborn) Cleanup: Rename ARM's NaClGetSp() to NaClGetStackPtr() for consistency r6339: (ncbray) Refactored ppapi_bad test to dynamically create the embed tags it uses as it runs. BUG=none TEST=trybots Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=95726

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: 3 (0 generated)
Mark Seaborn
9 years, 4 months ago (2011-08-05 23:33:26 UTC) #1
bradn
LGTM
9 years, 4 months ago (2011-08-05 23:53:50 UTC) #2
commit-bot: I haz the power
9 years, 4 months ago (2011-08-06 03:24:39 UTC) #3
Change committed as 95726

Powered by Google App Engine
This is Rietveld 408576698