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

Unified Diff: Source/core/fileapi/File.h

Issue 458953002: Remove custom getters for File.lastModified{Date} (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Preserve comment Created 6 years, 4 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/bindings/core/v8/custom/V8FileCustom.cpp ('k') | Source/core/fileapi/File.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/fileapi/File.h
diff --git a/Source/core/fileapi/File.h b/Source/core/fileapi/File.h
index 9f6faa29a6d621db5980845fda75e3c9dece5df0..9b195a32c8094727e897e7377b7728cf6658831b 100644
--- a/Source/core/fileapi/File.h
+++ b/Source/core/fileapi/File.h
@@ -115,7 +115,12 @@ public:
const String& path() const { ASSERT(hasValidFilePath()); return m_path; }
const String name() const { return m_name; }
- // This returns the current date and time if the file's last modification date is not known (per spec: http://www.w3.org/TR/FileAPI/#dfn-lastModifiedDate).
+ // Getter for the lastModified IDL attribute,
+ // http://dev.w3.org/2006/webapi/FileAPI/#file-attrs
+ long long lastModified() const;
+
+ // Getter for the lastModifiedDate IDL attribute,
+ // http://www.w3.org/TR/FileAPI/#dfn-lastModifiedDate
double lastModifiedDate() const;
UserVisibility userVisibility() const { return m_userVisibility; }
@@ -139,6 +144,10 @@ private:
void invalidateSnapshotMetadata() { m_snapshotSize = -1; }
+ // Returns File's last modified time (in MS since Epoch.)
+ // If the modification time isn't known, the current time is returned.
+ double lastModifiedMS() const;
+
#if ENABLE(ASSERT)
bool hasValidFileSystemURL() const { return hasBackingFile(); }
// Instances not backed by a file must have an empty path set.
« no previous file with comments | « Source/bindings/core/v8/custom/V8FileCustom.cpp ('k') | Source/core/fileapi/File.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698