Disable a PPAPI crash test under Valgrind.

Disable a PPAPI crash test under Valgrind.

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=107110

[Nick Bray, 2011-10-21 17:06:21]

LGTM w/ comment.

http://codereview.chromium.org/8360030/diff/1/ppapi/native_client/tests/ppapi...
File ppapi/native_client/tests/ppapi_browser/bad/nacl.scons (right):

http://codereview.chromium.org/8360030/diff/1/ppapi/native_client/tests/ppapi...
ppapi/native_client/tests/ppapi_browser/bad/nacl.scons:75:
env.Bit('running_on_valgrind'))
Why?  Please add a comment.

[Evgeniy Stepanov, 2011-10-24 12:07:38]

On 2011/10/21 17:06:21, Nick Bray wrote:
> LGTM w/ comment.
> 
>
http://codereview.chromium.org/8360030/diff/1/ppapi/native_client/tests/ppapi...
> File ppapi/native_client/tests/ppapi_browser/bad/nacl.scons (right):
> 
>
http://codereview.chromium.org/8360030/diff/1/ppapi/native_client/tests/ppapi...
> ppapi/native_client/tests/ppapi_browser/bad/nacl.scons:75:
> env.Bit('running_on_valgrind'))
> Why?  Please add a comment.

Done. Limitations of Valgrind/NaCl.

[commit-bot: I haz the power, 2011-10-24 12:07:56]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/eugenis@google.com/8360030/5001

[commit-bot: I haz the power, 2011-10-24 13:11:01]

The commit queue went berserk retrying too often for a
seemingly flaky test. Builder is linux_rel, revision is 106906, job name
was 8360030-5001 (previous was lost) (previous was lost) (previous was lost)
(previous was lost).

[commit-bot: I haz the power, 2011-10-25 12:33:29]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/eugenis@google.com/8360030/5001

[commit-bot: I haz the power, 2011-10-25 13:53:25]

Change committed as 107110