Add provision_devices step to all Android testers.

Add provision_devices step to all Android testers.

This CL adds --auto-reconnect as an option to the
provisioning script. This option is not being used.

BUG=164180
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=192668

[Isaac (away), 2013-04-05 05:41:18]

this looks good.  I have a couple nits but generally looks solid.

https://chromiumcodereview.appspot.com/13649010/diff/1/build/android/buildbot...
File build/android/buildbot/bb_device_steps.py (right):

https://chromiumcodereview.appspot.com/13649010/diff/1/build/android/buildbot...
build/android/buildbot/bb_device_steps.py:276:
RunCmd(['build/android/provision_devices.py'])
Let's pass target, even though it's currently used.

https://chromiumcodereview.appspot.com/13649010/diff/1/build/android/provisio...
File build/android/provision_devices.py (right):

https://chromiumcodereview.appspot.com/13649010/diff/1/build/android/provisio...
build/android/provision_devices.py:73: if options.target:
I think you can remove this check if we have a default target.

https://chromiumcodereview.appspot.com/13649010/diff/1/build/android/provisio...
build/android/provision_devices.py:84: parser.add_option('-t', '--target',
help='The build target')
let's default this to 'Debug'

https://chromiumcodereview.appspot.com/13649010/diff/1/build/android/provisio...
build/android/provision_devices.py:87: help='Specify that the adb_reboot binary
should be pushed to the devices')
'Push binary which will reboot the device on adb disconnections.'

[Isaac (away), 2013-04-05 05:41:47]

https://chromiumcodereview.appspot.com/13649010/diff/1/build/android/buildbot...
File build/android/buildbot/bb_device_steps.py (right):

https://chromiumcodereview.appspot.com/13649010/diff/1/build/android/buildbot...
build/android/buildbot/bb_device_steps.py:276:
RunCmd(['build/android/provision_devices.py'])
On 2013/04/05 05:41:18, Isaac wrote:
> Let's pass target, even though it's currently used.

*even though it's not currently used.

[Siva Chandra, 2013-04-05 20:55:31]

PTAL

https://chromiumcodereview.appspot.com/13649010/diff/1/build/android/buildbot...
File build/android/buildbot/bb_device_steps.py (right):

https://chromiumcodereview.appspot.com/13649010/diff/1/build/android/buildbot...
build/android/buildbot/bb_device_steps.py:276:
RunCmd(['build/android/provision_devices.py'])
On 2013/04/05 05:41:47, Isaac wrote:
> On 2013/04/05 05:41:18, Isaac wrote:
> > Let's pass target, even though it's currently used.
> 
> *even though it's not currently used.

Done.

https://chromiumcodereview.appspot.com/13649010/diff/1/build/android/provisio...
File build/android/provision_devices.py (right):

https://chromiumcodereview.appspot.com/13649010/diff/1/build/android/provisio...
build/android/provision_devices.py:73: if options.target:
On 2013/04/05 05:41:18, Isaac wrote:
> I think you can remove this check if we have a default target.

Done.

https://chromiumcodereview.appspot.com/13649010/diff/1/build/android/provisio...
build/android/provision_devices.py:84: parser.add_option('-t', '--target',
help='The build target')
On 2013/04/05 05:41:18, Isaac wrote:
> let's default this to 'Debug'

Done.

https://chromiumcodereview.appspot.com/13649010/diff/1/build/android/provisio...
build/android/provision_devices.py:87: help='Specify that the adb_reboot binary
should be pushed to the devices')
On 2013/04/05 05:41:18, Isaac wrote:
> 'Push binary which will reboot the device on adb disconnections.'

Done.

[Isaac (away), 2013-04-05 21:17:34]

LGTM, thanks for making the changes.

[commit-bot: I haz the power, 2013-04-05 21:40:30]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/sivachandra@chromium.org/13649010/8001

[commit-bot: I haz the power, 2013-04-06 00:25:21]

Change committed as 192668