commit 1544da7123d5daa3216e12118a5874a4fac9155b
author Kent Tamura <tkent@chromium.org> Tue Dec 03 03:41:09 2019
committer Commit Bot <commit-bot@chromium.org> Tue Dec 03 03:41:09 2019
tree 0548b84ecff7c6a65678ff89f57239cfc8db888a
parent c0fe69d8b5a0b11ea5c8e9ecb016e0a0ed16ec36

Remove |double| timestamp code from blink::File

This CL changes:
- all timestamp arguments of File,
- the return value of LastModifiedMS(), and
- the type of snapshot_modification_time_ms_
from double to base::Optional<base::Time>.

This renames
 - LastModifiedMS() to LastModifiedTime() and
 - snapshot_modification_time_ms_ to snapshot_modification_time_
and removes
 - File::LastModifiedDate(),
 - InvalidFileTime() in file_metadata.h,
 - IsValidFileTime() in file_metadata.h,
 - ToJsTimeOrNaN() in file_metadata.h, and
 - JsTimeToOptionalTime() in file_metadata.h.

This CL fixes File.lastModified behavior for timestamps later than
275760-09-13T00:00Z by removing unnecessary double-base:Time
conversions.

* v8_script_value_serializer_test.cc
 Do not wrap test cases with |namespace { ... }| because
FRIEND_TEST_ALL_PREFIXES() doesn't work with it.

Change-Id: I630d869641ea033b68534f17178e78b8fc2d1953
Bug: 988343
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1942671
Commit-Queue: Kent Tamura <tkent@chromium.org>
Reviewed-by: Joshua Bell <jsbell@chromium.org>
Cr-Commit-Position: refs/heads/master@{#720806}