Return partial output when shell.run_and_get_output time limit is hit.

Return partial output when shell.run_and_get_output time limit is hit.

This patch fixes run_and_get_output() of the Linux shell abstraction so
that if the process times out and is forcefully terminated, any output
produced by it before termination is still returned to the caller,
aligning it with the Android shell abstraction.

We also grow simple tests for run_and_get_output(), using 'echo' and
'cat' instead of the real shell binary.

R=etiennej@chromium.org

Committed: https://chromium.googlesource.com/external/mojo/+/fdfd6dfc9d529b63f46034e3197cd7932b869acf

[ppi, 2015-08-17 16:23:00]

ppi@chromium.org changed reviewers:
+ etiennej@chromium.org

[ppi, 2015-08-17 16:23:16]

Hi Etienne,

ptal.

[etiennej, 2015-08-18 09:58:43]

https://codereview.chromium.org/1301613003/diff/1/mojo/devtools/common/devtoo...
File mojo/devtools/common/devtoolslib/shell_unittest.py (right):

https://codereview.chromium.org/1301613003/diff/1/mojo/devtools/common/devtoo...
mojo/devtools/common/devtoolslib/shell_unittest.py:5: """Tests for the shell
abstraction."""
This is actually about LinuxShell. Maybe rename the file into
linux_shell_unittest.py?

[ppi, 2015-08-18 10:11:05]

https://codereview.chromium.org/1301613003/diff/1/mojo/devtools/common/devtoo...
File mojo/devtools/common/devtoolslib/shell_unittest.py (right):

https://codereview.chromium.org/1301613003/diff/1/mojo/devtools/common/devtoo...
mojo/devtools/common/devtoolslib/shell_unittest.py:5: """Tests for the shell
abstraction."""
On 2015/08/18 09:58:43, etiennej wrote:
> This is actually about LinuxShell. Maybe rename the file into
> linux_shell_unittest.py?

Done.

[etiennej, 2015-08-18 10:54:23]

lgtm

[ppi, 2015-08-18 10:55:00]

Committed patchset #2 (id:20001) manually as
fdfd6dfc9d529b63f46034e3197cd7932b869acf (presubmit successful).