Remove hardcoding of appurify address information.

Remove hardcoding of appurify address information.

Add AMP support to test runner.

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

[rnephew (Reviews Here), 2014-12-12 20:36:28]

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

[jbudorick, 2014-12-12 20:39:35]

https://codereview.chromium.org/803643002/diff/20001/build/android/test_runne...
File build/android/test_runner.py (right):

https://codereview.chromium.org/803643002/diff/20001/build/android/test_runne...
build/android/test_runner.py:137: group.add_argument('--api-protocol',
default='',
remove default= and add required=True. You can then remove the changes in
remote_device_environment.

https://codereview.chromium.org/803643002/diff/20001/build/android/test_runne...
build/android/test_runner.py:139: group.add_argument('--api-address',
default='',
same

https://codereview.chromium.org/803643002/diff/20001/build/android/test_runne...
build/android/test_runner.py:141: group.add_argument('--api-port', default='',
same

[rnephew (Reviews Here), 2014-12-12 20:44:00]

https://codereview.chromium.org/803643002/diff/20001/build/android/test_runne...
File build/android/test_runner.py (right):

https://codereview.chromium.org/803643002/diff/20001/build/android/test_runne...
build/android/test_runner.py:137: group.add_argument('--api-protocol',
default='',
On 2014/12/12 20:39:35, jbudorick wrote:
> remove default= and add required=True. You can then remove the changes in
> remote_device_environment.

Done.

https://codereview.chromium.org/803643002/diff/20001/build/android/test_runne...
build/android/test_runner.py:139: group.add_argument('--api-address',
default='',
On 2014/12/12 20:39:35, jbudorick wrote:
> same

Done.

https://codereview.chromium.org/803643002/diff/20001/build/android/test_runne...
build/android/test_runner.py:141: group.add_argument('--api-port', default='',
On 2014/12/12 20:39:35, jbudorick wrote:
> same

Done.

[jbudorick, 2014-12-12 20:44:53]

lgtm

[rnephew (Reviews Here), 2014-12-12 20:45:29]

The CQ bit was checked by rnephew@chromium.org

[jbudorick, 2014-12-12 20:45:31]

On 2014/12/12 20:44:53, jbudorick wrote:
> lgtm

wait, not lgtm

[rnephew (Reviews Here), 2014-12-12 20:45:43]

The CQ bit was unchecked by rnephew@chromium.org

[jbudorick, 2014-12-12 20:46:33]

On 2014/12/12 20:45:31, jbudorick wrote:
> On 2014/12/12 20:44:53, jbudorick wrote:
> > lgtm
> 
> wait, not lgtm

to clarify: with the way the options are handled now, I think I was wrong to
suggest using required=True.

go back to doing the checking in remote_device_environment, but leave the
default='' out

sorry :/

[rnephew (Reviews Here), 2014-12-12 20:52:32]

Added it back.

[jbudorick, 2014-12-12 20:55:06]

ok, lgtm

[rnephew (Reviews Here), 2014-12-12 20:55:59]

The CQ bit was checked by rnephew@chromium.org

[commit-bot: I haz the power, 2014-12-12 20:57:12]

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

[commit-bot: I haz the power, 2014-12-12 23:00:19]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2014-12-12 23:01:00]

Patchset 5 (id:??) landed as
https://crrev.com/7f1e2059f0db3edc339b04c4141051913db2a87f
Cr-Commit-Position: refs/heads/master@{#308188}