Let timer test retry for 20ms to reduce flakiness.

Let timer test retry for 20ms to reduce flakiness.

BUG=
R=jkummerow@chromium.org

Committed: https://code.google.com/p/v8/source/detail?r=16948

[Michael Achenbach, 2013-09-25 15:34:14]

PTAL

[Jakob Kummerow, 2013-09-25 16:16:23]

LGTM with comments. Thanks for fixing this!

https://codereview.chromium.org/24484003/diff/1/test/mjsunit/timer.js
File test/mjsunit/timer.js (right):

https://codereview.chromium.org/24484003/diff/1/test/mjsunit/timer.js#newcode30
test/mjsunit/timer.js:30: (function run(){
nit: space before '{'

https://codereview.chromium.org/24484003/diff/1/test/mjsunit/timer.js#newcode33
test/mjsunit/timer.js:33: for (var start = Date.now(); start - start_test  < 20;
start = Date.now()) {
I think we can afford a higher upper limit. Maybe 100 ms?

And a nit: only one space before '<'

https://codereview.chromium.org/24484003/diff/1/test/mjsunit/timer.js#newcode38
test/mjsunit/timer.js:38: if (end - start <= 1) {
I'd prefer '==' here. 0 can't happen, negative values are undesirable.

[Michael Achenbach, 2013-09-25 16:48:07]

Committed patchset #2 manually as r16948 (presubmit successful).

[Jakob Kummerow, 2013-09-25 17:11:55]

https://codereview.chromium.org/24484003/diff/6001/test/mjsunit/timer.js
File test/mjsunit/timer.js (right):

https://codereview.chromium.org/24484003/diff/6001/test/mjsunit/timer.js#newc...
test/mjsunit/timer.js:32: // Let the retry run for maximum 20ms to reduce
flakiness.
Should have updated this comment (and the CL description) too ;-)