Reboot device after configuring in provision script.

Reboot device after configuring in provision script.

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

[navabi, 2014-06-09 20:54:22]

This adds 30 extra seconds with local testing of one device attached. Because
the devices are rebooted in parallel it should not take longer with extra
devices. I am currently charging extra devices to test locally and confirm it
does not add significantly more time for more devices.

[tonyg, 2014-06-09 20:56:29]

lgtm

Thanks for taking care of this.

https://codereview.chromium.org/327573003/diff/1/build/android/provision_devi...
File build/android/provision_devices.py (right):

https://codereview.chromium.org/327573003/diff/1/build/android/provision_devi...
build/android/provision_devices.py:151: .old_interface.Reboot(True))
Nit: I think this and the above copy can fit on a single line.

[jbudorick, 2014-06-09 20:57:16]

lgtm

https://codereview.chromium.org/327573003/diff/1/build/android/provision_devi...
File build/android/provision_devices.py (right):

https://codereview.chromium.org/327573003/diff/1/build/android/provision_devi...
build/android/provision_devices.py:149: try:
Can you add a TODO for me to parallelize this after switching from
AndroidCommands?

[navabi, 2014-06-09 22:56:19]

https://codereview.chromium.org/327573003/diff/1/build/android/provision_devi...
File build/android/provision_devices.py (right):

https://codereview.chromium.org/327573003/diff/1/build/android/provision_devi...
build/android/provision_devices.py:149: try:
On 2014/06/09 20:57:15, jbudorick wrote:
> Can you add a TODO for me to parallelize this after switching from
> AndroidCommands?

Done.

https://codereview.chromium.org/327573003/diff/1/build/android/provision_devi...
build/android/provision_devices.py:151: .old_interface.Reboot(True))
On 2014/06/09 20:56:29, tonyg wrote:
> Nit: I think this and the above copy can fit on a single line.

Done.

[navabi, 2014-06-09 22:57:22]

The CQ bit was checked by navabi@google.com

[commit-bot: I haz the power, 2014-06-09 22:58:49]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/navabi@google.com/327573003/40001

[commit-bot: I haz the power, 2014-06-10 05:02:34]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  mac_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/mac_gpu/builds/14707)

[commit-bot: I haz the power, 2014-06-10 14:45:45]

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

[commit-bot: I haz the power, 2014-06-10 14:45:46]

Try jobs failed on following builders:
  win_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_rel/buil...)

[navabi, 2014-06-10 19:21:55]

The CQ bit was checked by navabi@google.com

[commit-bot: I haz the power, 2014-06-10 19:26:09]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/navabi@google.com/327573003/40001

[commit-bot: I haz the power, 2014-06-10 21:35:58]

Change committed as 276162