Allow provision_devices to work on the L-release preview.

Allow provision_devices to work on the L-release preview.

We weren't respecting the full timeout specified by the Reboot() method. And
apparently timing is different enough on the L-release preview that reboots
get stuck forever without the timeout.

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

[jbudorick, 2014-07-08 20:00:45]

Thankfully, we won't have to deal with android_testrunner for much longer...

https://codereview.chromium.org/375973002/diff/1/third_party/android_testrunn...
File third_party/android_testrunner/adb_interface.py (right):

https://codereview.chromium.org/375973002/diff/1/third_party/android_testrunn...
third_party/android_testrunner/adb_interface.py:321:
self.SendCommand("wait-for-device")
I think we should pass the wait time in here, too, with no retries.

https://codereview.chromium.org/375973002/diff/1/third_party/android_testrunn...
third_party/android_testrunner/adb_interface.py:413: timeout_time=wait_time/2)
Why wait_time/2?

[tonyg, 2014-07-08 20:17:11]

https://codereview.chromium.org/375973002/diff/1/third_party/android_testrunn...
File third_party/android_testrunner/adb_interface.py (right):

https://codereview.chromium.org/375973002/diff/1/third_party/android_testrunn...
third_party/android_testrunner/adb_interface.py:321:
self.SendCommand("wait-for-device")
On 2014/07/08 20:00:45, jbudorick wrote:
> I think we should pass the wait time in here, too, with no retries.

Done.

https://codereview.chromium.org/375973002/diff/1/third_party/android_testrunn...
third_party/android_testrunner/adb_interface.py:413: timeout_time=wait_time/2)
On 2014/07/08 20:00:45, jbudorick wrote:
> Why wait_time/2?

Since we'll run this twice (1 retry), I initially thought each should get half
the time. But upon further thought, that doesn't make a lot of sense, so I just
made it wait_time.

[jbudorick, 2014-07-08 20:44:38]

lgtm

[tonyg, 2014-07-08 21:01:32]

The CQ bit was checked by tonyg@chromium.org

[commit-bot: I haz the power, 2014-07-08 21:03:30]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/tonyg@chromium.org/375973002/20001

[frankf, 2014-07-08 23:54:56]

lgtm

[commit-bot: I haz the power, 2014-07-09 01:03:44]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  chromium_presubmit on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/chromium_presubmit/bu...)
  mac_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/mac_chromium_rel/buil...)

[commit-bot: I haz the power, 2014-07-09 04:25:42]

Change committed as 281927