Use -s to pass string date to 'adb shell date' command to set date in provision.

Use -s to pass string date to 'adb shell date' command to set date in provision.

The 'date -u' was not working on N5 devices. The -s option seems to work across
devices. I've tested on N4 and N5. This change implements an earlier change that
was reverted. See the change here: https://codereview.chromium.org/668013003/
The difference in this change is that it sets the time zone to UTC.

BUG=406848
Committed: https://crrev.com/3fad747a3352fb78e4beaf51cbf839e72168a4cb
Cr-Commit-Position: refs/heads/master@{#300803}

[navabi, 2014-10-22 08:26:02]

navabi@google.com changed reviewers:
+ jam@chromium.org, jbudorick@chromium.org, yfriedman@chromium.org,
zty@chromium.org

[navabi, 2014-10-22 08:26:02]

Added jam@ in case he has any ideas on how to test this on the try bots. The try
bots do not test this change properly (as seen in the reverted CL). The reason
is because changes to provision_devices.py does not trigger tests. Do manually
triggered try jobs also use analyze?

[navabi, 2014-10-22 08:34:24]

> Do manually triggered try jobs also use analyze?

I answered that by doing a try job. It uses analyze and thus does not properly
test this change. Is there an easy way to tell it to run all test steps?

[jbudorick, 2014-10-22 15:47:03]

On 2014/10/22 08:34:24, navabi wrote:
> > Do manually triggered try jobs also use analyze?
> 
> I answered that by doing a try job. It uses analyze and thus does not properly
> test this change. Is there an easy way to tell it to run all test steps?

We'll probably have to expand this:

https://codereview.chromium.org/586813003/diff/1/testing/buildbot/trybot_anal...

[jbudorick, 2014-10-22 15:52:11]

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

https://codereview.chromium.org/671833003/diff/1/build/android/provision_devi...
build/android/provision_devices.py:196: device.RunShellCommand('date -s %s' %
time.strftime('%Y%m%d.%I%M%S'),
Can you just do time.strftime(..., time.gmtime()) and drop the preceding two
lines?

[navabi, 2014-10-22 16:17:35]

Thanks for the suggestion jbudorick. PTAL.

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

https://codereview.chromium.org/671833003/diff/1/build/android/provision_devi...
build/android/provision_devices.py:196: device.RunShellCommand('date -s %s' %
time.strftime('%Y%m%d.%I%M%S'),
On 2014/10/22 15:52:11, jbudorick wrote:
> Can you just do time.strftime(..., time.gmtime()) and drop the preceding two
> lines?

Done.

[jbudorick, 2014-10-22 19:06:30]

change lgtm, but it'd be nice to have the trybot coverage in place before
landing.

[navabi, 2014-10-22 20:19:31]

On 2014/10/22 19:06:30, jbudorick wrote:
> change lgtm, but it'd be nice to have the trybot coverage in place before
> landing.

I agree. So I submitted this to land first:
https://codereview.chromium.org/676533002/

[navabi, 2014-10-22 22:45:52]

The CQ bit was checked by navabi@google.com

[commit-bot: I haz the power, 2014-10-22 22:47:53]

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

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

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2014-10-23 00:04:08]

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