[telemetry] Don't use time.sleep in text entry tests

[telemetry] Don't use time.sleep in text entry tests

This patch modifies KeyPressActionTest.testTextEntry and
ActionRunnerTest.testEnterText to use
`WaitForJavaScript(Condition|Expression)` instead of `time.sleep`.

BUG=chromium:630017
Committed: https://chromium.googlesource.com/external/github.com/catapult-project/catapult/+/2befef4332bccf830c80b0c1806ad44409bb1b9c

[petrcermak, 2016-07-25 11:38:40]

Description was changed from

==========
[telemetry] Don't use time.sleep in KeyPressActionTest.testTextEntry and
ActionRunnerTest.testEnterText

BUG=chromium:630017
==========

to

==========
[telemetry] Don't use time.sleep in text entry tests

This patch modifies KeyPressActionTest.testTextEntry and
ActionRunnerTest.testEnterText to use
`WaitForJavaScript(Condition|Expression)` instead of `time.sleep`.

BUG=chromium:630017
==========

[petrcermak, 2016-07-25 11:38:57]

petrcermak@chromium.org changed reviewers:
+ achuith@chromium.org, nednguyen@google.com

[petrcermak, 2016-07-25 11:38:58]

PTAL.

Thanks,
Petr

[nednguyen, 2016-07-25 11:43:06]

lgtm

[petrcermak, 2016-07-25 11:43:44]

The CQ bit was checked by petrcermak@chromium.org

[commit-bot: I haz the power, 2016-07-25 11:43:50]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-25 12:08:39]

Description was changed from

==========
[telemetry] Don't use time.sleep in text entry tests

This patch modifies KeyPressActionTest.testTextEntry and
ActionRunnerTest.testEnterText to use
`WaitForJavaScript(Condition|Expression)` instead of `time.sleep`.

BUG=chromium:630017
==========

to

==========
[telemetry] Don't use time.sleep in text entry tests

This patch modifies KeyPressActionTest.testTextEntry and
ActionRunnerTest.testEnterText to use
`WaitForJavaScript(Condition|Expression)` instead of `time.sleep`.

BUG=chromium:630017

Committed:
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...
==========

[commit-bot: I haz the power, 2016-07-25 12:08:40]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...

[achuithb, 2016-07-25 18:44:34]

lgtm. Thanks for taking care of the comments in the other CL

https://codereview.chromium.org/2174273002/diff/1/telemetry/telemetry/interna...
File telemetry/telemetry/internal/actions/action_runner_unittest.py (right):

https://codereview.chromium.org/2174273002/diff/1/telemetry/telemetry/interna...
telemetry/telemetry/internal/actions/action_runner_unittest.py:283: timeout=1)
You could consider increasing the timeout to a few seconds since there's no real
cost to this.