Add live output for blimp integration script {run} func

Add live output for blimp integration script {run} func

Support for getting console log from the engine.
Print the logs to stdout for {run} positional argument.

BUG=636410
Committed: https://crrev.com/8207e3a60214c97137ccb86ecd714a750124047c
Cr-Commit-Position: refs/heads/master@{#411705}

[shenghuazhang, 2016-08-11 02:54:55]

shenghuazhang@chromium.org changed reviewers:
+ nyquist@chromium.org

[shenghuazhang, 2016-08-11 02:58:42]

This change prints output of engine process as default when using {run} as the
positional argument.

[nyquist, 2016-08-11 23:09:43]

https://codereview.chromium.org/2236143002/diff/20001/blimp/tools/client_engi...
File blimp/tools/client_engine_integration.py (right):

https://codereview.chromium.org/2236143002/diff/20001/blimp/tools/client_engi...
blimp/tools/client_engine_integration.py:179: if engine_process.poll() is None:
If this is not the case, could we log an error? I mean, at that point the engine
failed to start.

[shenghuazhang, 2016-08-12 00:24:36]

Log error when engine fails to start.

https://codereview.chromium.org/2236143002/diff/20001/blimp/tools/client_engi...
File blimp/tools/client_engine_integration.py (right):

https://codereview.chromium.org/2236143002/diff/20001/blimp/tools/client_engi...
blimp/tools/client_engine_integration.py:179: if engine_process.poll() is None:
On 2016/08/11 23:09:43, nyquist wrote:
> If this is not the case, could we log an error? I mean, at that point the
engine
> failed to start.

Done.

[nyquist, 2016-08-12 03:45:26]

assuming the answer to the only comment is yes, this lgtm!

https://codereview.chromium.org/2236143002/diff/40001/blimp/tools/client_engi...
File blimp/tools/client_engine_integration.py (right):

https://codereview.chromium.org/2236143002/diff/40001/blimp/tools/client_engi...
blimp/tools/client_engine_integration.py:181: engine_process.poll())
Is this guaranteed to return the same result across invocations?

[shenghuazhang, 2016-08-12 17:31:20]

The CQ bit was checked by shenghuazhang@chromium.org

[commit-bot: I haz the power, 2016-08-12 17:31:48]

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-08-12 18:25:42]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-08-12 18:28:39]

Description was changed from

==========
Add live output for blimp integration script {run} func

Support for getting console log from the engine.
Print the logs to stdout for {run} positional argument.

BUG=636410
==========

to

==========
Add live output for blimp integration script {run} func

Support for getting console log from the engine.
Print the logs to stdout for {run} positional argument.

BUG=636410

Committed: https://crrev.com/8207e3a60214c97137ccb86ecd714a750124047c
Cr-Commit-Position: refs/heads/master@{#411705}
==========

[commit-bot: I haz the power, 2016-08-12 18:28:40]

Patchset 3 (id:??) landed as
https://crrev.com/8207e3a60214c97137ccb86ecd714a750124047c
Cr-Commit-Position: refs/heads/master@{#411705}