Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(72)

Unified Diff: public/platform/WebFileInfo.h

Issue 882343002: Upgrade Blink to milliseconds-based last modified filetimes, part 5. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@sof-fileinfo-modtime-in-ms-3
Patch Set: rebased Created 5 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « Source/web/WebFileChooserCompletionImpl.cpp ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: public/platform/WebFileInfo.h
diff --git a/public/platform/WebFileInfo.h b/public/platform/WebFileInfo.h
index 3e78971589834cafa9437104ed0fa4bb04f54214..6cdc125c2089b66bcb30079f1f72fce979787401 100644
--- a/public/platform/WebFileInfo.h
+++ b/public/platform/WebFileInfo.h
@@ -36,15 +36,12 @@
namespace blink {
struct WebFileInfo {
- // The last modification time of the file, in seconds since Epoch,
+ // The last modification time of the file, in milliseconds since Epoch,
// with a quiet NaN value representing "not known."
- // FIXME: change to be milliseconds-based once the transition to a
- // WebFileInfo with milliseconds-based modification is complete.
double modificationTime;
- // The last modification time of the file, in milliseconds since Epoch,
- // with a quiet NaN value representing "not known."
- // FIXME: remove once the milliseconds transition is complete.
+ // FIXME: unused, remove once embedder has stopped updating (and expecting)
+ // this field.
double modificationTimeMS;
// The length of the file in bytes.
« no previous file with comments | « Source/web/WebFileChooserCompletionImpl.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698