Retry Set RequestContextObject for PNaCl pexe fetches, to fix On-Demand update.
commitefd4b3c8968ae61cdfcdbce0f8068e98b5cea6ff
authorjvoung <jvoung@chromium.org>
Sat, 30 Aug 2014 21:47:59 +0000 (30 14:47 -0700)
committerCommit bot <commit-bot@chromium.org>
Sat, 30 Aug 2014 21:50:49 +0000 (30 21:50 +0000)
treee88cc2051a8f9f13a8b09ddbef19db7e6cf0b6f7
parent00b31698ea813d2769d7a4f0f0442db203571b07
Retry Set RequestContextObject for PNaCl pexe fetches, to fix On-Demand update.

There was a race condition in the added test check in the previous
attempt. That made the test Super Flaky. This is more noticeable in the
Release builds than the Debug build that I was working with.
(see https://codereview.chromium.org/471233003/)

With a Release build I was able to repro the flakiness (40% failure
over 200 runs). After the fix, I tested 500 runs and the success rate
is now 100%. I can't say it won't flake for other reasons, but
this fixes the main flake that was introduced. Historically,
the test does ocassionally need to be retried due to timeouts,
according to the bot log history. We may want to split the
CORS and the non-CORS test case to see if that will help w/ timeouts.

I didn't want to do that for this patch, since it would get
much more complicated and this is intended for merging to M38.

Retry with the race condition fixed. Also re-enable the test since
it got disabled by another CL for being flaky.

BUG=401755
BUG=315328

Review URL: https://codereview.chromium.org/524683002

Cr-Commit-Position: refs/heads/master@{#292807}
chrome/test/nacl/pnacl_header_test.cc
chrome/test/nacl/pnacl_header_test.h
components/nacl/renderer/ppb_nacl_private_impl.cc