[Android] Switch instrumentation tests to platform mode. (RELAND)

[Android] Switch instrumentation tests to platform mode. (RELAND)

This is a reland of https://codereview.chromium.org/1387933005/

BUG=428729
Committed: https://crrev.com/a46e07369fc36afae9fab7506640eb76ff9b925a
Cr-Commit-Position: refs/heads/master@{#383729}

[jbudorick, 2016-03-28 18:58:54]

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

[rnephew (Reviews Here), 2016-03-28 19:03:36]

On 2016/03/28 18:58:55, jbudorick wrote:

lgtm

[mikecase (-- gone --), 2016-03-28 21:03:47]

lgtm

You already know this, but I have code in infra that will save per-step logcats
for instr tests
in platform mode but does not upload the logcats to GS (they are saved on the
bots).
If these logs overwhelm the bots, I can super easily disable it (or polish my CL
to upload them).

[jbudorick, 2016-03-28 21:22:07]

On 2016/03/28 21:03:47, mikecase wrote:
> lgtm
> 
> You already know this, but I have code in infra that will save per-step
logcats
> for instr tests
> in platform mode but does not upload the logcats to GS (they are saved on the
> bots).
> If these logs overwhelm the bots, I can super easily disable it (or polish my
CL
> to upload them).

yeah, I noticed. This is what motivated my logcat-on-failure CL. adding infra
folks as fyi...

[jbudorick, 2016-03-28 21:22:48]

jbudorick@chromium.org changed reviewers:
+ bpastene@chromium.org, martiniss@chromium.org

[jbudorick, 2016-03-28 21:23:05]

On 2016/03/28 21:03:47, mikecase wrote:
> lgtm
> 
> You already know this, but I have code in infra that will save per-step
logcats
> for instr tests
> in platform mode but does not upload the logcats to GS (they are saved on the
> bots).
> If these logs overwhelm the bots, I can super easily disable it (or polish my
CL
> to upload them).

+bpastene, +martiniss: FYI ^

[jbudorick, 2016-03-28 21:43:46]

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

[jbudorick, 2016-03-28 21:43:47]

+yfriedman for base/android/ OWNERS

[Yaron, 2016-03-29 14:18:52]

lgtm

[jbudorick, 2016-03-29 14:31:04]

The CQ bit was checked by jbudorick@chromium.org

[commit-bot: I haz the power, 2016-03-29 14:31:24]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1836883002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1836883002/20001

[commit-bot: I haz the power, 2016-03-29 15:34:30]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-03-29 15:36:01]

Description was changed from

==========
[Android] Switch instrumentation tests to platform mode. (RELAND)

This is a reland of https://codereview.chromium.org/1387933005/

BUG=428729
==========

to

==========
[Android] Switch instrumentation tests to platform mode. (RELAND)

This is a reland of https://codereview.chromium.org/1387933005/

BUG=428729

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

[commit-bot: I haz the power, 2016-03-29 15:36:03]

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

[jbudorick, 2016-03-29 17:41:36]

A revert of this CL (patchset #2 id:20001) has been created in
https://codereview.chromium.org/1843713002/ by jbudorick@chromium.org.

The reason for reverting is: broke mojo on Android GN:
https://build.chromium.org/p/chromium.linux/builders/Android%20GN/builds/33530.