File::lastModifiedDate should use NaN or separate boolean flag for null Date value
authorkinuko@chromium.org <kinuko@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 14 Jun 2012 06:17:26 +0000 (06:17 +0000)
committerkinuko@chromium.org <kinuko@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 14 Jun 2012 06:17:26 +0000 (06:17 +0000)
https://bugs.webkit.org/show_bug.cgi?id=87826

Reviewed by Kent Tamura.

Test: http/tests/local/fileapi/file-last-modified-after-delete.html

* fileapi/File.cpp:
(WebCore::File::File):
(WebCore::File::captureSnapshot):
(WebCore::File::lastModifiedDate):
(WebCore::File::lastModifiedDateForBinding): Removed.
* fileapi/File.h:
(File):
* fileapi/File.idl:
* platform/FileMetadata.h:
(WebCore::FileMetadata::FileMetadata):
* platform/FileSystem.h:
* platform/FileStream.cpp:
(WebCore::FileStream::getSize):
* platform/chromium/support/WebHTTPBody.cpp:
(WebKit::WebHTTPBody::elementAt):
* platform/network/BlobData.h:
(WebCore::BlobDataItem::doNotCheckFileChange): Removed.
* platform/network/BlobData.cpp:
* platform/network/cf/FormDataStreamCFNet.cpp:
(WebCore::advanceCurrentStream):
* platform/network/soup/ResourceHandleSoup.cpp:
(WebCore::blobIsOutOfDate):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@120282 268f45cc-cd09-0410-ab3c-d52691b4dbfc

13 files changed:
Source/WebCore/ChangeLog
Source/WebCore/fileapi/File.cpp
Source/WebCore/fileapi/File.h
Source/WebCore/fileapi/File.idl
Source/WebCore/platform/FileMetadata.h
Source/WebCore/platform/FileStream.cpp
Source/WebCore/platform/FileSystem.h
Source/WebCore/platform/chromium/support/WebHTTPBody.cpp
Source/WebCore/platform/network/BlobData.cpp
Source/WebCore/platform/network/BlobData.h
Source/WebCore/platform/network/FormData.cpp
Source/WebCore/platform/network/cf/FormDataStreamCFNet.cpp
Source/WebCore/platform/network/soup/ResourceHandleSoup.cpp

index cafcf91..b4acacb 100644 (file)
@@ -1,3 +1,35 @@
+2012-06-07  Kinuko Yasuda  <kinuko@chromium.org>
+
+        File::lastModifiedDate should use NaN or separate boolean flag for null Date value
+        https://bugs.webkit.org/show_bug.cgi?id=87826
+
+        Reviewed by Kent Tamura.
+
+        Test: http/tests/local/fileapi/file-last-modified-after-delete.html
+
+        * fileapi/File.cpp:
+        (WebCore::File::File):
+        (WebCore::File::captureSnapshot):
+        (WebCore::File::lastModifiedDate):
+        (WebCore::File::lastModifiedDateForBinding): Removed.
+        * fileapi/File.h:
+        (File):
+        * fileapi/File.idl:
+        * platform/FileMetadata.h:
+        (WebCore::FileMetadata::FileMetadata):
+        * platform/FileSystem.h:
+        * platform/FileStream.cpp:
+        (WebCore::FileStream::getSize):
+        * platform/chromium/support/WebHTTPBody.cpp:
+        (WebKit::WebHTTPBody::elementAt):
+        * platform/network/BlobData.h:
+        (WebCore::BlobDataItem::doNotCheckFileChange): Removed.
+        * platform/network/BlobData.cpp:
+        * platform/network/cf/FormDataStreamCFNet.cpp:
+        (WebCore::advanceCurrentStream):
+        * platform/network/soup/ResourceHandleSoup.cpp:
+        (WebCore::blobIsOutOfDate):
+
 2012-06-13  Eli Fidler  <efidler@rim.com>
 
         Fix CMake build with SMOOTH_SCROLLING enabled
index 86d5989..5f8b93b 100644 (file)
@@ -86,7 +86,7 @@ File::File(const String& path)
     , m_name(pathGetFileName(path))
 #if ENABLE(FILE_SYSTEM)
     , m_snapshotSize(-1)
-    , m_snapshotModificationTime(0)
+    , m_snapshotModificationTime(invalidFileTime())
 #endif
 {
 }
@@ -96,7 +96,7 @@ File::File(const String& path, const KURL& url, const String& type)
     , m_path(path)
 #if ENABLE(FILE_SYSTEM)
     , m_snapshotSize(-1)
-    , m_snapshotModificationTime(0)
+    , m_snapshotModificationTime(invalidFileTime())
 #endif
 {
     m_name = pathGetFileName(path);
@@ -111,7 +111,7 @@ File::File(const String& path, const String& name)
     , m_name(name)
 #if ENABLE(FILE_SYSTEM)
     , m_snapshotSize(-1)
-    , m_snapshotModificationTime(0)
+    , m_snapshotModificationTime(invalidFileTime())
 #endif
 {
 }
@@ -136,18 +136,12 @@ double File::lastModifiedDate() const
 
     time_t modificationTime;
     if (!getFileModificationTime(m_path, modificationTime))
-        return 0;
+        return invalidFileTime();
 
     // Needs to return epoch time in milliseconds for Date.
     return modificationTime * 1000.0;
 }
 
-double File::lastModifiedDateForBinding() const
-{
-    double value = lastModifiedDate();
-    return (!value) ? std::numeric_limits<double>::quiet_NaN() : value;
-}
-
 unsigned long long File::size() const
 {
 #if ENABLE(FILE_SYSTEM)
@@ -178,7 +172,7 @@ void File::captureSnapshot(long long& snapshotSize, double& snapshotModification
     FileMetadata metadata;
     if (!getFileMetadata(m_path, metadata)) {
         snapshotSize = 0;
-        snapshotModificationTime = 0;
+        snapshotModificationTime = invalidFileTime();
         return;
     }
 
index fc4384c..e3ca2a5 100644 (file)
@@ -77,12 +77,9 @@ public:
     const String& path() const { return m_path; }
     const String& name() const { return m_name; }
 
-    // This may return zero if getFileModificationTime() platform call has failed or zero snapshot lastModifiedTime is given at construction time.
+    // This may return NaN (which is converted to null Date in javascript layer) if getFileModificationTime() platform call has failed or the information is not available.
     double lastModifiedDate() const;
 
-    // For binding. We want to return null Date if we get the value 0 Date (which is used to indicate the information is unavailable).
-    double lastModifiedDateForBinding() const;
-
 #if ENABLE(DIRECTORY_UPLOAD)
     // Returns the relative path of this file in the context of a directory selection.
     const String& webkitRelativePath() const { return m_relativePath; }
index 00bdde6..7c0de43 100644 (file)
@@ -32,7 +32,7 @@ module html {
     ] File : Blob {
         readonly attribute DOMString name;
 #if !defined(LANGUAGE_GOBJECT) || !LANGUAGE_GOBJECT
-        readonly attribute [ImplementedAs=lastModifiedDateForBinding] Date lastModifiedDate;
+        readonly attribute Date lastModifiedDate;
 #endif
 #if defined(ENABLE_DIRECTORY_UPLOAD) && ENABLE_DIRECTORY_UPLOAD
         readonly attribute DOMString webkitRelativePath;
index 04df92c..8290d67 100644 (file)
@@ -31,6 +31,7 @@
 #ifndef FileMetadata_h
 #define FileMetadata_h
 
+#include "FileSystem.h"
 #include <wtf/text/WTFString.h>
 
 namespace WebCore {
@@ -56,7 +57,7 @@ struct FileMetadata {
     String platformPath;
 #endif
 
-    FileMetadata() : modificationTime(0.0), length(-1), type(TypeUnknown) { }
+    FileMetadata() : modificationTime(invalidFileTime()), length(-1), type(TypeUnknown) { }
 };
 
 } // namespace WebCore
index e0b29e0..b85667d 100644 (file)
@@ -33,7 +33,7 @@
 #if ENABLE(BLOB)
 
 #include "FileStream.h"
-
+#include "FileSystem.h"
 #include "PlatformString.h"
 
 namespace WebCore {
@@ -66,7 +66,7 @@ long long FileStream::getSize(const String& path, double expectedModificationTim
     time_t modificationTime;
     if (!getFileModificationTime(path, modificationTime))
         return -1;
-    if (expectedModificationTime) {
+    if (isValidFileTime(expectedModificationTime)) {
         if (static_cast<time_t>(expectedModificationTime) != modificationTime)
             return -1;
     }
index cc05b02..6d7e05e 100644 (file)
@@ -33,6 +33,7 @@
 #include "PlatformString.h"
 #include <time.h>
 #include <wtf/Forward.h>
+#include <wtf/MathExtras.h>
 #include <wtf/Vector.h>
 
 #if USE(CF)
@@ -174,6 +175,9 @@ CString fileSystemRepresentation(const String&);
 
 inline bool isHandleValid(const PlatformFileHandle& handle) { return handle != invalidPlatformFileHandle; }
 
+inline double invalidFileTime() { return std::numeric_limits<double>::quiet_NaN(); }
+inline bool isValidFileTime(double time) { return isfinite(time); }
+
 // Prefix is what the filename should be prefixed with, not the full path.
 String openTemporaryFile(const String& prefix, PlatformFileHandle&);
 PlatformFileHandle openFile(const String& path, FileOpenMode);
index d183af1..40b0fcd 100644 (file)
@@ -31,6 +31,7 @@
 #include "config.h"
 #include <public/WebHTTPBody.h>
 
+#include "FileSystem.h"
 #include "FormData.h"
 
 using namespace WebCore;
@@ -77,7 +78,7 @@ bool WebHTTPBody::elementAt(size_t index, Element& result) const
     result.filePath.reset();
     result.fileStart = 0;
     result.fileLength = 0;
-    result.modificationTime = 0.0;
+    result.modificationTime = invalidFileTime();
     result.blobURL = KURL();
 
     switch (element.m_type) {
index d4f885f..a1cc313 100644 (file)
@@ -40,7 +40,6 @@
 namespace WebCore {
 
 const long long BlobDataItem::toEndOfFile = -1;
-const double BlobDataItem::doNotCheckFileChange = 0;
 
 RawData::RawData()
 {
index f919d64..5195cc9 100644 (file)
@@ -31,6 +31,7 @@
 #ifndef BlobData_h
 #define BlobData_h
 
+#include "FileSystem.h"
 #include "KURL.h"
 #include "PlatformString.h"
 #include <wtf/Forward.h>
@@ -59,14 +60,13 @@ private:
 
 struct BlobDataItem {
     static const long long toEndOfFile;
-    static const double doNotCheckFileChange;
 
     // Default constructor.
     BlobDataItem()
         : type(Data)
         , offset(0)
         , length(toEndOfFile)
-        , expectedModificationTime(doNotCheckFileChange)
+        , expectedModificationTime(invalidFileTime())
     {
     }
 
@@ -76,7 +76,7 @@ struct BlobDataItem {
         , data(data)
         , offset(0)
         , length(toEndOfFile)
-        , expectedModificationTime(doNotCheckFileChange)
+        , expectedModificationTime(invalidFileTime())
     {
     }
 
@@ -86,7 +86,7 @@ struct BlobDataItem {
         , path(path)
         , offset(0)
         , length(toEndOfFile)
-        , expectedModificationTime(doNotCheckFileChange)
+        , expectedModificationTime(invalidFileTime())
     {
     }
 
@@ -99,14 +99,14 @@ struct BlobDataItem {
         , expectedModificationTime(expectedModificationTime)
     {
     }
-    
+
     // Constructor for Blob type.
     BlobDataItem(const KURL& url, long long offset, long long length)
         : type(Blob)
         , url(url)
         , offset(offset)
         , length(length)
-        , expectedModificationTime(doNotCheckFileChange)
+        , expectedModificationTime(invalidFileTime())
     {
     }
 
@@ -114,7 +114,7 @@ struct BlobDataItem {
     void detachFromCurrentThread();
 
     enum { Data, File, Blob } type;
-    
+
     // For Data type.
     RefPtr<RawData> data;
 
@@ -137,7 +137,7 @@ private:
         , data(data)
         , offset(offset)
         , length(length)
-        , expectedModificationTime(doNotCheckFileChange)
+        , expectedModificationTime(invalidFileTime())
     {
     }
 };
index ebf273c..5f5b5df 100644 (file)
@@ -164,7 +164,7 @@ void FormData::appendData(const void* data, size_t size)
 void FormData::appendFile(const String& filename, bool shouldGenerateFile)
 {
 #if ENABLE(BLOB)
-    m_elements.append(FormDataElement(filename, 0, BlobDataItem::toEndOfFile, BlobDataItem::doNotCheckFileChange, shouldGenerateFile));
+    m_elements.append(FormDataElement(filename, 0, BlobDataItem::toEndOfFile, invalidFileTime(), shouldGenerateFile));
 #else
     m_elements.append(FormDataElement(filename, shouldGenerateFile));
 #endif
@@ -355,7 +355,7 @@ static void encode(Encoder& encoder, const FormDataElement& element)
 #else
         encoder.encodeInt64(0);
         encoder.encodeInt64(0);
-        encoder.encodeDouble(0);
+        encoder.encodeDouble(invalidFileTime());
 #endif
         return;
 
index dec9dc2..68f0633 100644 (file)
@@ -156,7 +156,7 @@ static bool advanceCurrentStream(FormStreamFields* form)
     } else {
 #if ENABLE(BLOB)
         // Check if the file has been changed or not if required.
-        if (nextInput.m_expectedFileModificationTime != BlobDataItem::doNotCheckFileChange) {
+        if (isValidFileTime(nextInput.m_expectedFileModificationTime)) {
             time_t fileModificationTime;
             if (!getFileModificationTime(nextInput.m_filename, fileModificationTime) || fileModificationTime != static_cast<time_t>(nextInput.m_expectedFileModificationTime))
                 return false;
index dc0cb62..10c6fa9 100644 (file)
@@ -428,7 +428,7 @@ static bool addFileToSoupMessageBody(SoupMessage* message, const String& fileNam
 static bool blobIsOutOfDate(const BlobDataItem& blobItem)
 {
     ASSERT(blobItem.type == BlobDataItem::File);
-    if (blobItem.expectedModificationTime == BlobDataItem::doNotCheckFileChange)
+    if (!isValidFileTime(blobItem.expectedModificationTime))
         return false;
 
     time_t fileModificationTime;