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

Issue 11411326: Valgrind: enable Linux Sandbox tests under Valgrind (Closed)

Created:
8 years ago by jln (very slow on Chromium)
Modified:
8 years ago
CC:
chromium-reviews, glider+watch_chromium.org, pam+watch_chromium.org, timurrrr+watch_chromium.org, bruening+watch_chromium.org, Markus (顧孟勤)
Visibility:
Public.

Description

Valgrind: enable Linux Sandbox tests under Valgrind Most of these tests will actually not really run, since the probe process will fail / crash under Valgrind. At least this allows us to test syscall.cc and in the future, the compiler (once policies get compiled even when we lack kernel support). BUG=163996 NOTRY=true Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=170833

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -0 lines) Patch
M tools/valgrind/chrome_tests.py View 2 chunks +4 lines, -0 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
jln (very slow on Chromium)
Timur, does this look right? I tested this on 64 bits. Is the try bot ...
8 years ago (2012-12-03 20:57:29 UTC) #1
jln (very slow on Chromium)
Adding Alexander as reviewer, since he is in the same timezone currently.
8 years ago (2012-12-03 20:59:40 UTC) #2
Alexander Potapenko
On 2012/12/03 20:59:40, Julien Tinnes wrote: > Adding Alexander as reviewer, since he is in ...
8 years ago (2012-12-03 21:01:45 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/jln@chromium.org/11411326/1
8 years ago (2012-12-03 22:48:26 UTC) #4
commit-bot: I haz the power
8 years ago (2012-12-03 22:50:51 UTC) #5
Message was sent while issue was closed.
Change committed as 170833

Powered by Google App Engine
This is Rietveld 408576698