Drive: Show localtime instead of UTC at "last modified" field of GCache in drive-internals.

Drive: Show local time instead of UTC at "last modified" field of GCache in drive-internals.

This CL show "last modified" date of GCache in local time instead of in UTC.
- Before: 2012-08-03T07:28:26.000Z
- After: 2012-08-03T00:28:26.000

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

[yoshiki, 2012-08-03 23:04:59]

satorux: It was useful for me on debbuing and may be useful for us.

[satorux1, 2012-08-03 23:06:28]

https://chromiumcodereview.appspot.com/10852014/diff/1/chrome/browser/resourc...
File chrome/browser/resources/chromeos/drive_internals.js (right):

https://chromiumcodereview.appspot.com/10852014/diff/1/chrome/browser/resourc...
chrome/browser/resources/chromeos/drive_internals.js:36: '[' + last_modified +
']';
Can we instead pass the local time from C++ side?

[yoshiki, 2012-08-03 23:29:24]

I tried it at Patch Set 3. But it is currently lacking the timezone suffix (like
"+09:00"). We have to add some code to get the timezone.  

I think it is better and simple that converting timezone is on JavaScript.

On 2012/08/03 23:06:28, satorux1 wrote:
>
https://chromiumcodereview.appspot.com/10852014/diff/1/chrome/browser/resourc...
> File chrome/browser/resources/chromeos/drive_internals.js (right):
> 
>
https://chromiumcodereview.appspot.com/10852014/diff/1/chrome/browser/resourc...
> chrome/browser/resources/chromeos/drive_internals.js:36: '[' + last_modified +
> ']';
> Can we instead pass the local time from C++ side?

[satorux1, 2012-08-03 23:52:24]

LGTM. Let's go with the C++ solution. Tweaking the time string in JavaScript
sounds like a hack.

Please update the patch description.

[Yoshiki IGUCHI, 2012-08-03 23:55:27]

Thanks for comment. I've updated the patch description and I'll commit it.

[commit-bot: I haz the power, 2012-08-04 18:20:59]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/yoshiki@chromium.org/10852014/6002

[commit-bot: I haz the power, 2012-08-04 18:37:06]

Try job failure for 10852014-6002 (retry) on win_rel for step "compile" (clobber
build).
It's a second try, previously, step "compile" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...