[Android] Add option to run perf tests in platform mode.

[Android] Add option to run perf tests in platform mode.

BUG=615157
Committed: https://chromium.googlesource.com/chromium/tools/build/+/dfdf1345605f32aad5c842247e94e130c274154b

[rnephew (Reviews Here), 2016-07-13 15:56:32]

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

[rnephew (Reviews Here), 2016-07-13 15:56:33]

Note: This does not turn it on for any perf bots, it just adds the plumbing to
turn it on. After this lands, another CL will be done that turns it on for
specific bots.

[dtu, 2016-07-13 18:04:01]

What's platform mode?

[jbudorick, 2016-07-13 18:06:13]

lgtm w/ nit

@dtu: "platform mode" is the somewhat-nondescript name for the rewrite of the
test runner's architecture.

https://codereview.chromium.org/2146783003/diff/1/scripts/slave/recipe_module...
File scripts/slave/recipe_modules/chromium_android/api.py (right):

https://codereview.chromium.org/2146783003/diff/1/scripts/slave/recipe_module...
scripts/slave/recipe_modules/chromium_android/api.py:646: enable_platform_mode:
If set, will run in new platform mode for android test
nit: "will run using the android test runner's new platform mode" or something

[rnephew (Reviews Here), 2016-07-13 20:45:54]

https://codereview.chromium.org/2146783003/diff/1/scripts/slave/recipe_module...
File scripts/slave/recipe_modules/chromium_android/api.py (right):

https://codereview.chromium.org/2146783003/diff/1/scripts/slave/recipe_module...
scripts/slave/recipe_modules/chromium_android/api.py:646: enable_platform_mode:
If set, will run in new platform mode for android test
On 2016/07/13 18:06:13, jbudorick wrote:
> nit: "will run using the android test runner's new platform mode" or something

Done.

[rnephew (Reviews Here), 2016-07-19 15:27:05]

ping dtu@ I'd like to get your ok before moving ahead on this.

[dtu, 2016-07-19 23:57:34]

lgtm

[rnephew (Reviews Here), 2016-07-20 01:21:47]

The CQ bit was checked by rnephew@chromium.org

[rnephew (Reviews Here), 2016-07-20 01:21:47]

The patchset sent to the CQ was uploaded after l-g-t-m from
jbudorick@chromium.org
Link to the patchset: https://codereview.chromium.org/2146783003/#ps20001
(title: "johns nit")

[commit-bot: I haz the power, 2016-07-20 01:21:51]

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-07-20 01:30:20]

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

[commit-bot: I haz the power, 2016-07-20 01:30:21]

Try jobs failed on following builders:
  Build Presubmit on master.tryserver.infra (JOB_FAILED,
https://build.chromium.org/p/tryserver.infra/builders/Build%20Presubmit/build...)

[rnephew (Reviews Here), 2016-07-20 16:57:28]

rnephew@chromium.org changed reviewers:
+ dpranke@chromium.org, phajdan.jr@chromium.org

[rnephew (Reviews Here), 2016-07-20 16:57:47]

On 2016/07/20 16:57:29, rnephew (Reviews Here) wrote:

MIssing owners for steps.py; added 2.

[rnephew (Reviews Here), 2016-07-20 17:03:41]

rnephew@chromium.org changed reviewers:
+ stip@chromium.org

[ghost stip (do not use), 2016-07-20 17:09:06]

surprised there aren't more expectation changes but lgtm

[rnephew (Reviews Here), 2016-07-20 17:09:46]

The CQ bit was checked by rnephew@chromium.org

[commit-bot: I haz the power, 2016-07-20 17:10:02]

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

[rnephew (Reviews Here), 2016-07-20 17:10:55]

On 2016/07/20 17:09:06, stip wrote:
> surprised there aren't more expectation changes but lgtm

This adds a flag to enable it; no bot is set to use that flag yet so not many
expectation changes. The next flag will enable it on a select bot to test it and
make sure there aren't any bugs found by running on the bots; then It will be
set to default and this CL will be backed out.

[commit-bot: I haz the power, 2016-07-20 17:15:14]

Description was changed from

==========
[Android] Add option to run perf tests in platform mode.

BUG=615157
==========

to

==========
[Android] Add option to run perf tests in platform mode.

BUG=615157

Committed:
https://chromium.googlesource.com/chromium/tools/build/+/dfdf1345605f32aad5c8...
==========

[commit-bot: I haz the power, 2016-07-20 17:15:16]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/tools/build/+/dfdf1345605f32aad5c8...