Implementation of TestRunner::Sleep

Implementation of TestRunner::Sleep

Which blocks script execution and loading for the caller for a
given amount of time, like an alert() call.

BUG=418116

[João Eiras, 2014-11-18 15:14:07]

joaoe@opera.com changed reviewers:
+ kbr@chromium.org, tkent@chromium.org

[João Eiras, 2014-11-18 15:14:09]

This API is here to help with testing 
https://codereview.chromium.org/625583002/

I'd like to know your opinion about it.

[Ken Russell (switch to Gerrit), 2014-11-18 21:44:19]

kbr@chromium.org changed reviewers:
+ dpranke@chromium.org, pfeldman@chromium.org

[Ken Russell (switch to Gerrit), 2014-11-18 21:44:20]

+more test_runner OWNERS

This seems basically OK though I would try to find a more descriptive name.

[tkent, 2014-11-19 01:16:21]

We shouldn't add this.
A test with this would be flaky.  If a test has testRunner.sleep(500), it
expects Blink changes its state in 500 msec.  The state change might not happen
in 500 msec if a layout test bot has heavy load.

[João Eiras, 2014-11-19 02:39:43]

On 2014/11/19 01:16:21, tkent wrote:
> We shouldn't add this.
> A test with this would be flaky.  If a test has testRunner.sleep(500), it
> expects Blink changes its state in 500 msec.  The state change might not
happen
> in 500 msec if a layout test bot has heavy load.

Eric asked for a test for https://codereview.chromium.org/625583002/
How do you suggest the defers loading flag to be set and the message queue ran
to verify that the asserts don't trigger ?
The timeout is there so the testcase is not stuck indefinitely. The API is
supposed to behave similar to having an alert() open for a while.

[tkent, 2014-11-19 03:47:33]

On 2014/11/19 02:39:43, João Eiras wrote:
> Eric asked for a test for https://codereview.chromium.org/625583002/
> How do you suggest the defers loading flag to be set and the message queue ran
> to verify that the asserts don't trigger ?
> The timeout is there so the testcase is not stuck indefinitely. The API is
> supposed to behave similar to having an alert() open for a while.

Though I don't understand the detail of what you need, we should add a
testRunner function to cause the state change immediately.