Reviewers: Søren Gjesse, Description: Run script in ProfLazyMode test longer to improve stability.
Prior to this change debug version of the test crashed 2 of 1000 times. After the change no crashes (out of 1000 runs) occured. Please review this at http://codereview.chromium.org/115772 Affected files: M test/cctest/test-log.cc Index: test/cctest/test-log.cc diff --git a/test/cctest/test-log.cc b/test/cctest/test-log.cc index ada66119d2aca9e8868bff16d939b478645015cf..780a01617caa1405abae96ea9ce0313393fa4c87 100644 --- a/test/cctest/test-log.cc +++ b/test/cctest/test-log.cc @@ -165,9 +165,9 @@ static int CheckThatProfilerWorks(int log_pos) { "for (var i = 0; i < 1000; ++i) { " "(function(x) { return %d * x; })(i); }", log_pos); - // Run code for 100 msecs to get some ticks. + // Run code for 200 msecs to get some ticks. const int64_t started_us = i::OS::Ticks(); - while (i::OS::Ticks() - started_us < 100 * 1000) { + while (i::OS::Ticks() - started_us < 200 * 1000) { CompileAndRunScript(script_src.start()); } --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
