Reviewers: Erik Corry, Description: Revert change to LongRunningApply, but allow the test case to time out for now.
Please review this at http://codereview.chromium.org/125241 SVN Base: http://v8.googlecode.com/svn/branches/bleeding_edge/ Affected files: M test/cctest/cctest.status M test/cctest/test-api.cc Index: test/cctest/test-api.cc =================================================================== --- test/cctest/test-api.cc (revision 2196) +++ test/cctest/test-api.cc (working copy) @@ -6366,7 +6366,7 @@ "function do_very_little(bar) {" " this.foo = bar;" "}" - "for (var i = 0; i < 10000; i++) {" + "for (var i = 0; i < 100000; i++) {" " do_very_little.apply(this, ['bar']);" "}"; Local<String> source = String::New(c_source); Index: test/cctest/cctest.status =================================================================== --- test/cctest/cctest.status (revision 2195) +++ test/cctest/cctest.status (working copy) @@ -30,6 +30,10 @@ # BUG(281): This test fails on some Linuxes. test-debug/DebuggerAgent: PASS, (PASS || FAIL) if $system == linux +# BUG(382): Weird test. Can't guarantee that it never times out. +test-api/ApplyInterruption: PASS || TIMEOUT + + [ $arch == arm ] test-debug: SKIP --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
