Update the isolate interrupt test

Update the isolate interrupt test

* Space out the interrupts by short sleeps
* Use a constant instead of the magic number 3 for the number of interrups

R=iposva@google.com

BUG=dart:906
TEST=VM test IsolateInterrupt

Committed: https://code.google.com/p/dart/source/detail?r=2740

[Ivan Posva, 2011-12-21 17:27:14]

LGTM with comment.

-Ivan

http://codereview.chromium.org/9016007/diff/1/runtime/vm/dart_api_impl_test.cc
File runtime/vm/dart_api_impl_test.cc (right):

http://codereview.chromium.org/9016007/diff/1/runtime/vm/dart_api_impl_test.c...
runtime/vm/dart_api_impl_test.cc:3104: OS::Sleep(i + 1);
If you move OS::Sleep(i) before the call to Dart_InterruptIsolate you will not
waste" time waiting here after delivering the last interrupt.

[Søren Gjesse, 2011-12-22 08:21:26]

http://codereview.chromium.org/9016007/diff/1/runtime/vm/dart_api_impl_test.cc
File runtime/vm/dart_api_impl_test.cc (right):

http://codereview.chromium.org/9016007/diff/1/runtime/vm/dart_api_impl_test.c...
runtime/vm/dart_api_impl_test.cc:3104: OS::Sleep(i + 1);
On 2011/12/21 17:27:14, Ivan Posva wrote:
> If you move OS::Sleep(i) before the call to Dart_InterruptIsolate you will not
> waste" time waiting here after delivering the last interrupt.

Done.

[Ivan Posva, 2011-12-22 08:23:10]

http://codereview.chromium.org/9016007/diff/2001/runtime/vm/dart_api_impl_tes...
File runtime/vm/dart_api_impl_test.cc (right):

http://codereview.chromium.org/9016007/diff/2001/runtime/vm/dart_api_impl_tes...
runtime/vm/dart_api_impl_test.cc:3104: // Space out the interrupts a bit.
Comment could have moved with the corresponding code...

[Søren Gjesse, 2011-12-22 15:33:45]

http://codereview.chromium.org/9016007/diff/2001/runtime/vm/dart_api_impl_tes...
File runtime/vm/dart_api_impl_test.cc (right):

http://codereview.chromium.org/9016007/diff/2001/runtime/vm/dart_api_impl_tes...
runtime/vm/dart_api_impl_test.cc:3104: // Space out the interrupts a bit.
On 2011/12/22 08:23:10, Ivan Posva wrote:
> Comment could have moved with the corresponding code...

Done, http://codereview.chromium.org/9025014/.