Enable ouputting results for passed tests on AMP.

Enable ouputting results for passed tests on AMP.

AMP had a bug outputting results if the output stream was too
large. Therefore, we only were outputting test results for failed
tests. This issue has been fixed (b/18981674) and this CL is to
enable full test result outputting on AMP.

BUG=

Committed: https://crrev.com/803ff930a0556a59087e24c2a5461325c01185b9
Cr-Commit-Position: refs/heads/master@{#330892}

[mikecase (-- gone --), 2015-03-19 21:04:40]

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

[mikecase (-- gone --), 2015-03-19 21:18:38]

mikecase@chromium.org changed reviewers:
+ tedchoc@chromium.org

[mikecase (-- gone --), 2015-03-19 21:18:39]

Adding tedchoc@ for a review of testing/android

[jbudorick, 2015-03-19 21:43:14]

lgtm

[Ted C, 2015-03-19 22:09:56]

lgtm

[mikecase (-- gone --), 2015-03-19 22:11:10]

The CQ bit was checked by mikecase@chromium.org

[commit-bot: I haz the power, 2015-03-19 22:11:34]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1023803002/1

[mikecase (-- gone --), 2015-03-19 22:12:24]

The CQ bit was unchecked by mikecase@chromium.org

[mikecase (-- gone --), 2015-05-20 22:39:34]

I uploaded this CL a month or two ago when I mistakenly thought the appurify bug
was fixed (b/18981674) and therefore had to hold off on committing it.

The bug was recently fixed for real. I updated this CL and would appreciate if
you could take a another look to make sure it looks ok.

Thanks.

[jbudorick, 2015-05-20 22:42:50]

lgtm

https://codereview.chromium.org/1023803002/diff/60001/testing/android/native_...
File
testing/android/native_test/java/src/org/chromium/native_test/NativeTestInstrumentationTestRunner.java
(left):

https://codereview.chromium.org/1023803002/diff/60001/testing/android/native_...
testing/android/native_test/java/src/org/chromium/native_test/NativeTestInstrumentationTestRunner.java:156:
sendStatus(0, mLogBundle);
hm, why was this sending two failure bundles...?

[Ted C, 2015-05-20 23:21:13]

On 2015/05/20 22:42:50, jbudorick wrote:
> lgtm
> 
>
https://codereview.chromium.org/1023803002/diff/60001/testing/android/native_...
> File
>
testing/android/native_test/java/src/org/chromium/native_test/NativeTestInstrumentationTestRunner.java
> (left):
> 
>
https://codereview.chromium.org/1023803002/diff/60001/testing/android/native_...
>
testing/android/native_test/java/src/org/chromium/native_test/NativeTestInstrumentationTestRunner.java:156:
> sendStatus(0, mLogBundle);
> hm, why was this sending two failure bundles...?

still lgtm

[mikecase (-- gone --), 2015-05-20 23:27:07]

The CQ bit was checked by mikecase@chromium.org

[mikecase (-- gone --), 2015-05-20 23:27:12]

The CQ bit was unchecked by mikecase@chromium.org

[mikecase (-- gone --), 2015-05-20 23:28:03]

https://codereview.chromium.org/1023803002/diff/60001/testing/android/native_...
File
testing/android/native_test/java/src/org/chromium/native_test/NativeTestInstrumentationTestRunner.java
(left):

https://codereview.chromium.org/1023803002/diff/60001/testing/android/native_...
testing/android/native_test/java/src/org/chromium/native_test/NativeTestInstrumentationTestRunner.java:156:
sendStatus(0, mLogBundle);
On 2015/05/20 22:42:50, jbudorick wrote:
> hm, why was this sending two failure bundles...?

well. fixed now.

[mikecase (-- gone --), 2015-05-20 23:28:06]

The CQ bit was checked by mikecase@chromium.org

[commit-bot: I haz the power, 2015-05-20 23:28:36]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1023803002/60001

[commit-bot: I haz the power, 2015-05-20 23:34:42]

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

[commit-bot: I haz the power, 2015-05-20 23:34:44]

Try jobs failed on following builders:
  linux_chromium_compile_dbg_32_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  ios_dbg_simulator_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...)
  ios_rel_device_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_ni...)

[mikecase (-- gone --), 2015-05-21 00:06:13]

The CQ bit was checked by mikecase@chromium.org

[mikecase (-- gone --), 2015-05-21 00:06:13]

The patchset sent to the CQ was uploaded after l-g-t-m from
tedchoc@chromium.org, jbudorick@chromium.org
Link to the patchset: https://codereview.chromium.org/1023803002/#ps100001
(title: "Rebase")

[commit-bot: I haz the power, 2015-05-21 00:07:33]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1023803002/100001

[commit-bot: I haz the power, 2015-05-21 04:41:15]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2015-05-21 04:42:32]

Patchset 6 (id:??) landed as
https://crrev.com/803ff930a0556a59087e24c2a5461325c01185b9
Cr-Commit-Position: refs/heads/master@{#330892}