Wait until rebooting is done if java assert is not set.

Wait until rebooting is done if java assert is not set.

When we turn the device on, java assert flag would have been unset.
In this status if we execute test_runner.py, this script
will exit with -1 because the device is still rebooting.
so we should wait for adb connection until the device is fully rebooted.

BUG=None

Committed: https://crrev.com/585d2566e78f683f180583ba8e6f3844d44c14c6
Cr-Commit-Position: refs/heads/master@{#332559}

[JungJik, 2015-05-29 04:51:45]

jungjik.lee@samsung.com changed reviewers:
+ jbudorick@chromium.org

[JungJik, 2015-05-29 04:51:45]

Please take a look this patch.

[jbudorick, 2015-05-29 13:06:15]

https://codereview.chromium.org/1161023003/diff/1/build/android/pylib/instrum...
File build/android/pylib/instrumentation/test_runner.py (right):

https://codereview.chromium.org/1161023003/diff/1/build/android/pylib/instrum...
build/android/pylib/instrumentation/test_runner.py:102:
device_settings.SetLockScreenSettings(self.device)
Why would we set the lock screen settings here?

[JungJik, 2015-06-01 14:44:51]

https://codereview.chromium.org/1161023003/diff/1/build/android/pylib/instrum...
File build/android/pylib/instrumentation/test_runner.py (right):

https://codereview.chromium.org/1161023003/diff/1/build/android/pylib/instrum...
build/android/pylib/instrumentation/test_runner.py:102:
device_settings.SetLockScreenSettings(self.device)
On 2015/05/29 13:06:15, jbudorick wrote:
> Why would we set the lock screen settings here?

When I execute the instrumentation,
if the lock screen is enable, a test which has some touch events does not work.
so I add this unlock code before restarting the device.

[jbudorick, 2015-06-01 14:50:56]

https://codereview.chromium.org/1161023003/diff/1/build/android/pylib/instrum...
File build/android/pylib/instrumentation/test_runner.py (right):

https://codereview.chromium.org/1161023003/diff/1/build/android/pylib/instrum...
build/android/pylib/instrumentation/test_runner.py:102:
device_settings.SetLockScreenSettings(self.device)
On 2015/06/01 at 14:44:51, JungJik wrote:
> On 2015/05/29 13:06:15, jbudorick wrote:
> > Why would we set the lock screen settings here?
> 
> When I execute the instrumentation,
> if the lock screen is enable, a test which has some touch events does not
work.
> so I add this unlock code before restarting the device.

This isn't in the scope of the test runner. We handle this in
provision_devices.py:
https://code.google.com/p/chromium/codesearch#chromium/src/build/android/prov...

[JungJik, 2015-06-02 16:09:45]

https://codereview.chromium.org/1161023003/diff/1/build/android/pylib/instrum...
File build/android/pylib/instrumentation/test_runner.py (right):

https://codereview.chromium.org/1161023003/diff/1/build/android/pylib/instrum...
build/android/pylib/instrumentation/test_runner.py:102:
device_settings.SetLockScreenSettings(self.device)
On 2015/06/01 14:50:56, jbudorick wrote:
> On 2015/06/01 at 14:44:51, JungJik wrote:
> > On 2015/05/29 13:06:15, jbudorick wrote:
> > > Why would we set the lock screen settings here?
> > 
> > When I execute the instrumentation,
> > if the lock screen is enable, a test which has some touch events does not
> work.
> > so I add this unlock code before restarting the device.
> 
> This isn't in the scope of the test runner. We handle this in
> provision_devices.py:
>
https://code.google.com/p/chromium/codesearch#chromium/src/build/android/prov...

thanks for your help. I've checked the file.
I've uploaded new patch without lockscreen code.

[jbudorick, 2015-06-02 16:50:07]

lgtm

[JungJik, 2015-06-03 03:52:14]

The CQ bit was checked by jungjik.lee@samsung.com

[commit-bot: I haz the power, 2015-06-03 03:53:31]

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

[commit-bot: I haz the power, 2015-06-03 05:36:53]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-06-03 05:37:38]

Patchset 2 (id:??) landed as
https://crrev.com/585d2566e78f683f180583ba8e6f3844d44c14c6
Cr-Commit-Position: refs/heads/master@{#332559}