Add cs-base-url, store_tombstones, master_name for android inst tests.

Add --cs-base-url and --master-name for android inst tests.

--cs-base-url is an argument that specifies the code search base url in
the test results presentation html generated by the results of android
instrumentation tests.

--master-name is an argument that specifies master name.

--store-tombstones specified whether to store tombstones when a testcase  crashes.

In this cl, these two arguments are added to instrumentation test run
and are default to none.

BUG=605572
Committed: https://chromium.googlesource.com/chromium/tools/build/+/4e323b91fca31f987f70a57b5ac3e3d96fdeb26c

[BigBossZhiling, 2016-08-15 23:17:34]

hzl@google.com changed reviewers:
+ jbudorick@chromium.org

[BigBossZhiling, 2016-08-15 23:18:24]

Is there a train/simulate command that I should run for this?

[jbudorick, 2016-08-16 00:32:06]

Add a sample use to recipe_modules/chromium_android/example.py and retrain
expectations.

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

https://codereview.chromium.org/2252553002/diff/1/scripts/slave/recipe_module...
scripts/slave/recipe_modules/chromium_android/api.py:853: master_name = None,
nit: no spaces around =

[BigBossZhiling, 2016-08-16 00:46:25]

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

https://codereview.chromium.org/2252553002/diff/1/scripts/slave/recipe_module...
scripts/slave/recipe_modules/chromium_android/api.py:853: master_name = None,
On 2016/08/16 00:32:06, jbudorick wrote:
> nit: no spaces around =

Done.

[jbudorick, 2016-08-17 04:15:28]

https://codereview.chromium.org/2252553002/diff/20001/scripts/slave/recipe_mo...
File scripts/slave/recipe_modules/chromium_android/api.py (right):

https://codereview.chromium.org/2252553002/diff/20001/scripts/slave/recipe_mo...
scripts/slave/recipe_modules/chromium_android/api.py:853: master_name=None,
Remove this argument.

https://codereview.chromium.org/2252553002/diff/20001/scripts/slave/recipe_mo...
scripts/slave/recipe_modules/chromium_android/api.py:922:
presentation_args.extend(['--master-name', master_name])
Just use self.m.properties.get('mastername') as master_name.

[BigBossZhiling, 2016-08-17 22:38:27]

https://codereview.chromium.org/2252553002/diff/20001/scripts/slave/recipe_mo...
File scripts/slave/recipe_modules/chromium_android/api.py (right):

https://codereview.chromium.org/2252553002/diff/20001/scripts/slave/recipe_mo...
scripts/slave/recipe_modules/chromium_android/api.py:853: master_name=None,
On 2016/08/17 04:15:28, jbudorick wrote:
> Remove this argument.

Done.

https://codereview.chromium.org/2252553002/diff/20001/scripts/slave/recipe_mo...
scripts/slave/recipe_modules/chromium_android/api.py:922:
presentation_args.extend(['--master-name', master_name])
On 2016/08/17 04:15:28, jbudorick wrote:
> Just use self.m.properties.get('mastername') as master_name.

Done.

[BigBossZhiling, 2016-08-17 23:27:52]

Description was changed from

==========
Add --cs-base-url and --master-name for android inst tests.

--cs-base-url is an argument that specifies the code search base url in
the test results presentation html generated by the results of android
instrumentation tests.

--master-name is an argument that specifies master name.

In this cl, these two arguments are added to instrumentation test run
and are default to none.

BUG=605572
==========

to

==========
Add --cs-base-url and --master-name for android inst tests.

--cs-base-url is an argument that specifies the code search base url in
the test results presentation html generated by the results of android
instrumentation tests.

--master-name is an argument that specifies master name.

--store-tombstones specified whether to store tombstones when a testcase 
crashes.

In this cl, these two arguments are added to instrumentation test run
and are default to none.

BUG=605572
==========

[BigBossZhiling, 2016-08-17 23:28:33]

added --store-tombstones argument

[jbudorick, 2016-08-18 13:57:53]

lgtm

[BigBossZhiling, 2016-08-18 16:11:12]

The CQ bit was checked by hzl@google.com

[commit-bot: I haz the power, 2016-08-18 16:11:18]

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-18 16:18:15]

Description was changed from

==========
Add --cs-base-url and --master-name for android inst tests.

--cs-base-url is an argument that specifies the code search base url in
the test results presentation html generated by the results of android
instrumentation tests.

--master-name is an argument that specifies master name.

--store-tombstones specified whether to store tombstones when a testcase 
crashes.

In this cl, these two arguments are added to instrumentation test run
and are default to none.

BUG=605572
==========

to

==========
Add --cs-base-url and --master-name for android inst tests.

--cs-base-url is an argument that specifies the code search base url in
the test results presentation html generated by the results of android
instrumentation tests.

--master-name is an argument that specifies master name.

--store-tombstones specified whether to store tombstones when a testcase 
crashes.

In this cl, these two arguments are added to instrumentation test run
and are default to none.

BUG=605572

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

[commit-bot: I haz the power, 2016-08-18 16:18:16]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/chromium/tools/build/+/4e323b91fca31f987f70...