ASAN timeout on OutOfProcessPPAPITest.Crypto

ASAN timeout on OutOfProcessPPAPITest.Crypto

Disables OutOfProcessPPAPITest.Crypto test on ASAN.

BUG=104832
TEST=ASAN bot runs green
TBR=dmichael@chromium.org, noelallen@chromium.org

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

[dhollowa, 2011-11-19 02:01:26]

Timur, is there a better way to disable ASAN tests?  I tried
http://codereview.chromium.org/8603005 but that didn't work.

Thanks for any info.

[Timur Iskhodzhanov, 2011-11-19 19:02:57]

AFAIK, no.

Alexander has answered you on the other thread;
Also I think he was planning to add the invocation wrappers for ASan but not
sure about gtest_exclude files. IIRC he intentionally wanted NOT to support
them.

btw, I thought memory sheriffs are only "Memory FYI waterfall" sheriffs; the
ASan failures should be handled by the main sheriffs [correct me if I'm wrong].
If they have troubles - please ping kcc@ and glider@ to improve the docs instead
of spending your cycles on two waterfalls at the same time.

[Timur Iskhodzhanov, 2011-11-19 19:04:00]

another PoV: a better way is to file a bug and disable the test unconditionally
:)
If ASan times out then the test is likely broken on hardware slower than your
Z600.

[Alexander Potapenko, 2011-11-19 19:06:59]

ASan defines the ADDRESS_SANITIZER macro, so you can check for it.
We've also a test multiplier for ASan test timeouts in
base/test/test_timeouts.cc, which is currently 2. Maybe we can increase it a bit
more (the typical slowdown of ASan is ~2x, but could sometimes be greater)

[Nico, 2012-04-06 14:50:18]

http://codereview.chromium.org/8550004/diff/1/chrome/test/ui/ppapi_uitest.cc
File chrome/test/ui/ppapi_uitest.cc (right):

http://codereview.chromium.org/8550004/diff/1/chrome/test/ui/ppapi_uitest.cc#...
chrome/test/ui/ppapi_uitest.cc:298: TEST_F(OutOfProcessPPAPITest, Crypto) {
Shouldn't this say MAYE_Crypto?

(Apologies if the file has been updated since this CL already, didn't check.)

[Alexander Potapenko, 2012-04-06 14:53:32]

I think it should.
BTW, does this test run NaCl?