[Android] Clean up gtest stdout handling.

[Android] Clean up gtest stdout handling.

including:
 - capturing partial stdout for failed shards
 - removing vestigial fifo support
 - removing on-device results parsing

BUG=646223
Committed: https://crrev.com/cea883202f2e33a6b48b643b3d57221a2ef6187e
Cr-Commit-Position: refs/heads/master@{#434982}

[jbudorick, 2016-11-18 20:11:25]

jbudorick@chromium.org changed reviewers:
+ mikecase@chromium.org, shenghuazhang@chromium.org

[jbudorick, 2016-11-18 20:11:26]

This should also help the case where an individual gtest shard times out, which
has been a problem in multiple instances lately.

[jbudorick, 2016-11-18 20:12:00]

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

[commit-bot: I haz the power, 2016-11-18 20:12:57]

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

[jbudorick, 2016-11-21 16:37:43]

ping

[mikecase (-- gone --), 2016-11-21 18:26:23]

lgtm w/ one unused constant you can delete. Also, was the Java gtest parsing
logic ever used?

https://codereview.chromium.org/2514943002/diff/20001/testing/android/native_...
File
testing/android/native_test/java/src/org/chromium/native_test/NativeTestInstrumentationTestRunner.java
(right):

https://codereview.chromium.org/2514943002/diff/20001/testing/android/native_...
testing/android/native_test/java/src/org/chromium/native_test/NativeTestInstrumentationTestRunner.java:58:
private static final Pattern RE_TEST_OUTPUT =
This looks like you can delete it as well
Edit

[jbudorick, 2016-11-21 18:56:01]

Java gtest parsing was used for appurify.

https://codereview.chromium.org/2514943002/diff/20001/testing/android/native_...
File
testing/android/native_test/java/src/org/chromium/native_test/NativeTestInstrumentationTestRunner.java
(right):

https://codereview.chromium.org/2514943002/diff/20001/testing/android/native_...
testing/android/native_test/java/src/org/chromium/native_test/NativeTestInstrumentationTestRunner.java:58:
private static final Pattern RE_TEST_OUTPUT =
On 2016/11/21 18:26:22, mikecase wrote:
> This looks like you can delete it as well
> Edit

Done.

[jbudorick, 2016-11-21 18:56:55]

The CQ bit was checked by jbudorick@chromium.org

[jbudorick, 2016-11-21 18:56:56]

The patchset sent to the CQ was uploaded after l-g-t-m from
mikecase@chromium.org
Link to the patchset: https://codereview.chromium.org/2514943002/#ps40001
(title: "mikecase comment")

[commit-bot: I haz the power, 2016-11-21 18:57:41]

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-11-21 20:21:19]

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

[commit-bot: I haz the power, 2016-11-21 20:21:20]

Try jobs failed on following builders:
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)

[jbudorick, 2016-11-28 23:57:58]

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

[commit-bot: I haz the power, 2016-11-28 23:58:55]

Dry run: 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-11-29 00:59:05]

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

[commit-bot: I haz the power, 2016-11-29 00:59:06]

Dry run: Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[jbudorick, 2016-11-29 01:07:18]

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

[commit-bot: I haz the power, 2016-11-29 01:08:55]

Dry run: 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-11-29 03:03:25]

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

[commit-bot: I haz the power, 2016-11-29 03:03:26]

Dry run: This issue passed the CQ dry run.

[jbudorick, 2016-11-29 13:56:08]

The CQ bit was checked by jbudorick@chromium.org

[jbudorick, 2016-11-29 13:56:09]

The patchset sent to the CQ was uploaded after l-g-t-m from
mikecase@chromium.org
Link to the patchset: https://codereview.chromium.org/2514943002/#ps80001
(title: "write/update -> append/update")

[commit-bot: I haz the power, 2016-11-29 13:56:25]

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-11-29 14:00:11]

CQ is committing da patch.

Bot data: {"patchset_id": 80001, "attempt_start_ts": 1480427768497640,
"parent_rev": "17b65c7698f73041d47d4e4f1f906cbe0ae99790", "commit_rev":
"a590852d1cbf1ddccfe5927d29fe5e9acaadf633"}

[commit-bot: I haz the power, 2016-11-29 14:00:35]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-11-29 14:03:37]

Description was changed from

==========
[Android] Clean up gtest stdout handling.

including:
 - capturing partial stdout for failed shards
 - removing vestigial fifo support
 - removing on-device results parsing

BUG=646223
==========

to

==========
[Android] Clean up gtest stdout handling.

including:
 - capturing partial stdout for failed shards
 - removing vestigial fifo support
 - removing on-device results parsing

BUG=646223

Committed: https://crrev.com/cea883202f2e33a6b48b643b3d57221a2ef6187e
Cr-Commit-Position: refs/heads/master@{#434982}
==========

[commit-bot: I haz the power, 2016-11-29 14:03:38]

Patchset 5 (id:??) landed as
https://crrev.com/cea883202f2e33a6b48b643b3d57221a2ef6187e
Cr-Commit-Position: refs/heads/master@{#434982}