Provide nanoseconds precision for base::PlatformFileInfo on POSIX

Provide nanoseconds precision for base::PlatformFileInfo on POSIX

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

[apavlov, 2013-06-04 16:36:11]

Please take a look. Sorry if the code style is a bit off - it's been a while
since I wrote non-Blink code last time...

[apavlov, 2013-06-04 21:08:13]

Folks, I need some guidance on the approach to increasing the file lastModified
timestamp resolution for all platforms as much as possible. Please speak up if
you have better ideas or know someone knowledgeable in this area.

[tzik, 2013-06-05 04:00:17]

https://codereview.chromium.org/16093026/diff/1/base/file_util_posix.cc
File base/file_util_posix.cc (right):

https://codereview.chromium.org/16093026/diff/1/base/file_util_posix.cc#newco...
base/file_util_posix.cc:599: base::Time::kNanosecondsPerSecond);
Can we add Time::FromTimeSpec(const timespec&) and use it for these three?
Or, just FromTimeT(tv_sec) +
FromMicroseconds(tv_nsec/kNanosecondsPerMicroseconds) may work?

[apavlov, 2013-06-05 17:03:41]

https://codereview.chromium.org/16093026/diff/1/base/file_util_posix.cc
File base/file_util_posix.cc (right):

https://codereview.chromium.org/16093026/diff/1/base/file_util_posix.cc#newco...
base/file_util_posix.cc:599: base::Time::kNanosecondsPerSecond);
On 2013/06/05 04:00:17, tzik wrote:
> Can we add Time::FromTimeSpec(const timespec&) and use it for these three?

Done.

[tzik, 2013-06-06 06:21:18]

lgtm

[commit-bot: I haz the power, 2013-06-06 07:13:57]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/apavlov@chromium.org/16093026/12001

[commit-bot: I haz the power, 2013-06-06 07:25:48]

Retried try job too often on chromium_presubmit for step(s) presubmit
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=chromium_p...

[apavlov, 2013-06-06 07:29:49]

On 2013/06/06 07:25:48, I haz the power (commit-bot) wrote:
> Retried try job too often on chromium_presubmit for step(s) presubmit
>
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=chromium_p...

This needs an OWNER's approval to commit.

[brettw, 2013-06-07 15:55:11]

lgtm

[commit-bot: I haz the power, 2013-06-07 15:55:57]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/apavlov@chromium.org/16093026/12001

[commit-bot: I haz the power, 2013-06-07 16:18:01]

Sorry for I got bad news for ya.
Compile failed with a clobber build on android_clang_dbg.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android_cl...
Your code is likely broken or HEAD is junk. Please ensure your
code is not broken then alert the build sheriffs.
Look at the try server FAQ for more details.

[commit-bot: I haz the power, 2013-06-10 08:20:10]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/apavlov@chromium.org/16093026/36001

[commit-bot: I haz the power, 2013-06-10 09:41:36]

Change committed as 205178