[tools] Explicit timeout keyword in WaitForJavaScriptCondition

[tools] Explicit timeout keyword in WaitForJavaScriptCondition

Explicitly use the timeout_in_seconds keyword argument in
action_runner.WaitForJavaScriptCondition; this is to allow an API change
in the signature of that method.

TBR=bustamante@chromium.org
BUG=catapult:#3028

Review-Url: https://codereview.chromium.org/2617483006
Cr-Commit-Position: refs/heads/master@{#442238}
Committed: https://chromium.googlesource.com/chromium/src/+/027bfe2aaf9b681c4c484ff411211591101247c9

[perezju, 2017-01-06 14:52:30]

The CQ bit was checked by perezju@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-06 14:52:45]

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

[perezju, 2017-01-06 14:58:27]

perezju@chromium.org changed reviewers:
+ bustamante@chromium.org, nednguyen@google.com

[perezju, 2017-01-06 14:58:28]

bustamante: tools/chrome_proxy/* - 1 file
nednguyen: tools/perf/*

[nednguyen, 2017-01-06 15:16:40]

lgtm

[commit-bot: I haz the power, 2017-01-06 15:59:52]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-01-06 15:59:53]

Dry run: This issue passed the CQ dry run.

[perezju, 2017-01-09 11:22:09]

Description was changed from

==========
[tools] Explicit timeout keyword in WaitForJavaScriptCondition

Explicitly use the timeout_in_seconds keyword argument in
action_runner.WaitForJavaScriptCondition; this is to allow an API change
in the signature of that method.

BUG=catapult:#3028
==========

to

==========
[tools] Explicit timeout keyword in WaitForJavaScriptCondition

Explicitly use the timeout_in_seconds keyword argument in
action_runner.WaitForJavaScriptCondition; this is to allow an API change
in the signature of that method.

TBR=bustamante@chromium.org
BUG=catapult:#3028
==========

[perezju, 2017-01-09 11:22:45]

TBR'ing bustamante as this is only a tiny change and is blocking other CLs.

[perezju, 2017-01-09 11:22:50]

The CQ bit was checked by perezju@chromium.org

[commit-bot: I haz the power, 2017-01-09 11:23:05]

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, 2017-01-09 12:26:26]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1483960970417960, "parent_rev":
"161af0077dffbdd6aa5846677da04c4e3d3d6677", "commit_rev":
"027bfe2aaf9b681c4c484ff411211591101247c9"}

[commit-bot: I haz the power, 2017-01-09 12:26:53]

Description was changed from

==========
[tools] Explicit timeout keyword in WaitForJavaScriptCondition

Explicitly use the timeout_in_seconds keyword argument in
action_runner.WaitForJavaScriptCondition; this is to allow an API change
in the signature of that method.

TBR=bustamante@chromium.org
BUG=catapult:#3028
==========

to

==========
[tools] Explicit timeout keyword in WaitForJavaScriptCondition

Explicitly use the timeout_in_seconds keyword argument in
action_runner.WaitForJavaScriptCondition; this is to allow an API change
in the signature of that method.

TBR=bustamante@chromium.org
BUG=catapult:#3028

Review-Url: https://codereview.chromium.org/2617483006
Cr-Commit-Position: refs/heads/master@{#442238}
Committed:
https://chromium.googlesource.com/chromium/src/+/027bfe2aaf9b681c4c484ff41121...
==========

[commit-bot: I haz the power, 2017-01-09 12:26:54]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/027bfe2aaf9b681c4c484ff41121...