Make test-results upload URL configurable.

Make test-results upload URL configurable.

and also make the chromium.android testers upload to their gtest results to
test-results.

BUG=

Committed: https://chromium.googlesource.com/chromium/tools/build/+/cde40737c6fdc82d5923314b858aee7af9413aed

[jbudorick, 2016-08-10 00:38:30]

jbudorick@chromium.org changed reviewers:
+ estaab@chromium.org, martiniss@chromium.org, stip@chromium.org

[jbudorick, 2016-08-10 00:39:48]

On 2016/08/10 00:38:31, jbudorick wrote:

This is intentionally limited in scope. I'm planning on landing it, seeing if it
works, then rolling it out to the rest of chromium_tests if it does.

[martiniss, 2016-08-10 00:44:27]

lgtm

[estaab, 2016-08-10 01:19:05]

lgtm

[ghost stip (do not use), 2016-08-10 19:10:51]

rs-lgtm

[jbudorick, 2016-08-10 19:11:40]

The CQ bit was checked by jbudorick@chromium.org

[commit-bot: I haz the power, 2016-08-10 19:11:50]

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-08-10 19:13:59]

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

[commit-bot: I haz the power, 2016-08-10 19:14:00]

Try jobs failed on following builders:
  Build Presubmit on luci.infra.try (JOB_FAILED,
https://luci-milo.appspot.com/swarming/task/308eafe1c1479c10)

[jbudorick, 2016-08-10 19:34:19]

The CQ bit was checked by jbudorick@chromium.org

[jbudorick, 2016-08-10 19:34:20]

The patchset sent to the CQ was uploaded after l-g-t-m from estaab@chromium.org,
martiniss@chromium.org, stip@chromium.org
Link to the patchset: https://codereview.chromium.org/2230043002/#ps60001
(title: "rebase")

[commit-bot: I haz the power, 2016-08-10 19:34:30]

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-08-10 19:42:34]

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

[commit-bot: I haz the power, 2016-08-10 19:42:35]

Try jobs failed on following builders:
  Build Presubmit on luci.infra.try (JOB_FAILED,
https://luci-milo.appspot.com/swarming/task/308ec490ac669b10)

[jbudorick, 2016-08-10 19:44:00]

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

[jbudorick, 2016-08-10 19:44:01]

+dpranke for recipes/chromium.py owners.

[Dirk Pranke, 2016-08-11 00:11:53]

lgtm w/ nit/question.

https://codereview.chromium.org/2230043002/diff/60001/scripts/slave/recipe_mo...
File scripts/slave/recipe_modules/chromium_tests/steps.py (right):

https://codereview.chromium.org/2230043002/diff/60001/scripts/slave/recipe_mo...
scripts/slave/recipe_modules/chromium_tests/steps.py:359: 'got_revision_cp',
'x@{#0}'))
should we just accept the exception here if the key is missing?

[jbudorick, 2016-08-11 02:45:48]

https://codereview.chromium.org/2230043002/diff/60001/scripts/slave/recipe_mo...
File scripts/slave/recipe_modules/chromium_tests/steps.py (right):

https://codereview.chromium.org/2230043002/diff/60001/scripts/slave/recipe_mo...
scripts/slave/recipe_modules/chromium_tests/steps.py:359: 'got_revision_cp',
'x@{#0}'))
On 2016/08/11 00:11:53, Dirk Pranke wrote:
> should we just accept the exception here if the key is missing?

I'm not sure, but this is what we're doing in the other call sites in this
module.

https://codereview.chromium.org/2230043002/diff/60001/scripts/slave/recipe_mo...
scripts/slave/recipe_modules/chromium_tests/steps.py:924: 'got_revision_cp',
'x@{#0}')
i.e., here

https://codereview.chromium.org/2230043002/diff/60001/scripts/slave/recipe_mo...
scripts/slave/recipe_modules/chromium_tests/steps.py:1373: 'got_revision_cp',
'x@{#0}'),
and here

[jbudorick, 2016-08-15 14:31:12]

The CQ bit was checked by jbudorick@chromium.org

[jbudorick, 2016-08-15 14:31:13]

The patchset sent to the CQ was uploaded after l-g-t-m from
dpranke@chromium.org, estaab@chromium.org, martiniss@chromium.org,
stip@chromium.org
Link to the patchset: https://codereview.chromium.org/2230043002/#ps80001
(title: "rebase")

[commit-bot: I haz the power, 2016-08-15 14:31:17]

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-08-15 14:35:20]

Description was changed from

==========
Make test-results upload URL configurable.

and also make the chromium.android testers upload to their gtest results to
test-results.

BUG=
==========

to

==========
Make test-results upload URL configurable.

and also make the chromium.android testers upload to their gtest results to
test-results.

BUG=

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

[commit-bot: I haz the power, 2016-08-15 14:35:21]

Committed patchset #5 (id:80001) as
https://chromium.googlesource.com/chromium/tools/build/+/cde40737c6fdc82d5923...