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

Issue 7566045: NaCl: Update revision in DEPS, r6303 -> r6322 (Closed)

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

Description

NaCl: Update revision in DEPS, r6303 -> r6322 This pulls in the following Native Client changes: r6304: (mcgrathr) Remove magic to get IRT-compatible linking layout for browser-targetted nexes r6305: (ncbray) Add a hard timeout for the browser tester to stop runaway tests. r6306: (mcgrathr) Define private __aeabi_read_tp for IRT r6307: (mcgrathr) Load IRT for spec2k runs r6308: (jvoung) Bump toolchain deps. Pull in pdox's windows changes: r6309: (mcgrathr) Remove -lppapi hack for nacl_irt_test r6310: (sehr) Make __launchExecutableFromFd take a continuation rather than returning success r6311: (sehr) Remove possible use-after-free attack from console logging. r6312: (mseaborn) Cleanup: Remove obsolete reference to SDL from comment in sel_main.c r6313: (bradnelson) Allow branch to get either native_client (as on the buildbots) r6314: (jvoung) Fix pnacl glibc builder -- no more platform libs in sdk() after removal of link_segment_gap.o: r6315: (mseaborn) Cleanup: Fix typo in comment r6316: (mseaborn) Cleanup: Fix spelling: "seperate" -> "separate" r6318: (mcgrathr) Clean up buildbot context tests, get browser tests re-enabled on trybots r6319: (ncbray) Fixed how NaCl's build system used asserts. r6320: (ncbray) Added a stress test for determining now many nexes can be loaded simultaneously. r6321: (ncbray) Fixed breakage introduced in r6320. r6322: (mseaborn) Only set SDKROOT for standalone builds - redo r4926. Also, disable a thread safety stress test that is known to be flaky on Windows (run_ppapi_ppb_var_browser_test). BUG=none TEST=trybots Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=95562

Patch Set 1 #

Patch Set 2 : Disable a test #

Unified diffs Side-by-side diffs Delta from patch set Stats (+7 lines, -3 lines) Patch
M DEPS View 1 chunk +3 lines, -3 lines 0 comments Download
M chrome/test/nacl_test_injection/buildbot_nacl_integration.py View 1 1 chunk +4 lines, -0 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
Mark Seaborn
Today's DEPS update - a little earlier than usual in order to get Nico's Mac ...
9 years, 4 months ago (2011-08-04 21:08:00 UTC) #1
Nico
LGTM stamp
9 years, 4 months ago (2011-08-04 21:12:20 UTC) #2
Mark Seaborn
run_ppapi_ppb_var_browser_test failed on the trybot run. It's known to be flaky, so I'm disabling it. ...
9 years, 4 months ago (2011-08-04 23:02:38 UTC) #3
sehr (please use chromium)
On 2011/08/04 23:02:38, Mark Seaborn wrote: > run_ppapi_ppb_var_browser_test failed on the trybot run. It's known ...
9 years, 4 months ago (2011-08-04 23:09:55 UTC) #4
commit-bot: I haz the power
9 years, 4 months ago (2011-08-05 03:59:29 UTC) #5
Change committed as 95562

Powered by Google App Engine
This is Rietveld 408576698