Unreviewed, rolling out r119821.
authorkinuko@chromium.org <kinuko@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 8 Jun 2012 12:46:06 +0000 (12:46 +0000)
committerkinuko@chromium.org <kinuko@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 8 Jun 2012 12:46:06 +0000 (12:46 +0000)
http://trac.webkit.org/changeset/119821
https://bugs.webkit.org/show_bug.cgi?id=88648

Seems to have regression on Mac (Requested by kinuko on
#webkit).

Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2012-06-08

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

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

12 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/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 3be496b..6002108 100644 (file)
@@ -1,3 +1,38 @@
+2012-06-08  Sheriff Bot  <webkit.review.bot@gmail.com>
+
+        Unreviewed, rolling out r119821.
+        http://trac.webkit.org/changeset/119821
+        https://bugs.webkit.org/show_bug.cgi?id=88648
+
+        Seems to have regression on Mac (Requested by kinuko on
+        #webkit).
+
+        * fileapi/File.cpp:
+        (WebCore::File::File):
+        (WebCore::File::lastModifiedDate):
+        (WebCore::File::lastModifiedDateForBinding):
+        (WebCore):
+        (WebCore::File::captureSnapshot):
+        * fileapi/File.h:
+        (File):
+        * fileapi/File.idl:
+        * platform/FileMetadata.h:
+        (WebCore::FileMetadata::FileMetadata):
+        * platform/FileSystem.h:
+        * platform/chromium/support/WebHTTPBody.cpp:
+        (WebKit::WebHTTPBody::elementAt):
+        * platform/network/BlobData.cpp:
+        (WebCore):
+        * platform/network/BlobData.h:
+        (BlobDataItem):
+        (WebCore::BlobDataItem::BlobDataItem):
+        * platform/network/FormData.cpp:
+        (WebCore::FormData::appendFile):
+        * platform/network/cf/FormDataStreamCFNet.cpp:
+        (WebCore::advanceCurrentStream):
+        * platform/network/soup/ResourceHandleSoup.cpp:
+        (WebCore::blobIsOutOfDate):
+
 2012-06-08  Kinuko Yasuda  <kinuko@chromium.org>
 
         Unreviewed, removing duplicated entries in gyp.
index 5f8b93b..86d5989 100644 (file)
@@ -86,7 +86,7 @@ File::File(const String& path)
     , m_name(pathGetFileName(path))
 #if ENABLE(FILE_SYSTEM)
     , m_snapshotSize(-1)
-    , m_snapshotModificationTime(invalidFileTime())
+    , m_snapshotModificationTime(0)
 #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(invalidFileTime())
+    , m_snapshotModificationTime(0)
 #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(invalidFileTime())
+    , m_snapshotModificationTime(0)
 #endif
 {
 }
@@ -136,12 +136,18 @@ double File::lastModifiedDate() const
 
     time_t modificationTime;
     if (!getFileModificationTime(m_path, modificationTime))
-        return invalidFileTime();
+        return 0;
 
     // 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)
@@ -172,7 +178,7 @@ void File::captureSnapshot(long long& snapshotSize, double& snapshotModification
     FileMetadata metadata;
     if (!getFileMetadata(m_path, metadata)) {
         snapshotSize = 0;
-        snapshotModificationTime = invalidFileTime();
+        snapshotModificationTime = 0;
         return;
     }
 
index cb7c41e..67f31f6 100644 (file)
@@ -77,9 +77,12 @@ public:
     const String& path() const { return m_path; }
     const String& name() const { return m_name; }
 
-    // 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.
+    // This may return zero if getFileModificationTime() platform call has failed or zero snapshot lastModifiedTime is given at construction time.
     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 7c0de43..00bdde6 100644 (file)
@@ -32,7 +32,7 @@ module html {
     ] File : Blob {
         readonly attribute DOMString name;
 #if !defined(LANGUAGE_GOBJECT) || !LANGUAGE_GOBJECT
-        readonly attribute Date lastModifiedDate;
+        readonly attribute [ImplementedAs=lastModifiedDateForBinding] Date lastModifiedDate;
 #endif
 #if defined(ENABLE_DIRECTORY_UPLOAD) && ENABLE_DIRECTORY_UPLOAD
         readonly attribute DOMString webkitRelativePath;
index 8290d67..04df92c 100644 (file)
@@ -31,7 +31,6 @@
 #ifndef FileMetadata_h
 #define FileMetadata_h
 
-#include "FileSystem.h"
 #include <wtf/text/WTFString.h>
 
 namespace WebCore {
@@ -57,7 +56,7 @@ struct FileMetadata {
     String platformPath;
 #endif
 
-    FileMetadata() : modificationTime(invalidFileTime()), length(-1), type(TypeUnknown) { }
+    FileMetadata() : modificationTime(0.0), length(-1), type(TypeUnknown) { }
 };
 
 } // namespace WebCore
index 97f4b0f..21ece11 100644 (file)
@@ -33,7 +33,6 @@
 #include "PlatformString.h"
 #include <time.h>
 #include <wtf/Forward.h>
-#include <wtf/MathExtras.h>
 #include <wtf/Vector.h>
 
 #if USE(CF)
@@ -175,9 +174,6 @@ 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 40b0fcd..d183af1 100644 (file)
@@ -31,7 +31,6 @@
 #include "config.h"
 #include <public/WebHTTPBody.h>
 
-#include "FileSystem.h"
 #include "FormData.h"
 
 using namespace WebCore;
@@ -78,7 +77,7 @@ bool WebHTTPBody::elementAt(size_t index, Element& result) const
     result.filePath.reset();
     result.fileStart = 0;
     result.fileLength = 0;
-    result.modificationTime = invalidFileTime();
+    result.modificationTime = 0.0;
     result.blobURL = KURL();
 
     switch (element.m_type) {
index 3698b42..d4f885f 100644 (file)
@@ -30,7 +30,6 @@
 
 #include "config.h"
 #include "BlobData.h"
-#include "FileSystem.h"
 
 #include <wtf/OwnPtr.h>
 #include <wtf/PassOwnPtr.h>
@@ -41,6 +40,7 @@
 namespace WebCore {
 
 const long long BlobDataItem::toEndOfFile = -1;
+const double BlobDataItem::doNotCheckFileChange = 0;
 
 RawData::RawData()
 {
index 5195cc9..f919d64 100644 (file)
@@ -31,7 +31,6 @@
 #ifndef BlobData_h
 #define BlobData_h
 
-#include "FileSystem.h"
 #include "KURL.h"
 #include "PlatformString.h"
 #include <wtf/Forward.h>
@@ -60,13 +59,14 @@ private:
 
 struct BlobDataItem {
     static const long long toEndOfFile;
+    static const double doNotCheckFileChange;
 
     // Default constructor.
     BlobDataItem()
         : type(Data)
         , offset(0)
         , length(toEndOfFile)
-        , expectedModificationTime(invalidFileTime())
+        , expectedModificationTime(doNotCheckFileChange)
     {
     }
 
@@ -76,7 +76,7 @@ struct BlobDataItem {
         , data(data)
         , offset(0)
         , length(toEndOfFile)
-        , expectedModificationTime(invalidFileTime())
+        , expectedModificationTime(doNotCheckFileChange)
     {
     }
 
@@ -86,7 +86,7 @@ struct BlobDataItem {
         , path(path)
         , offset(0)
         , length(toEndOfFile)
-        , expectedModificationTime(invalidFileTime())
+        , expectedModificationTime(doNotCheckFileChange)
     {
     }
 
@@ -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(invalidFileTime())
+        , expectedModificationTime(doNotCheckFileChange)
     {
     }
 
@@ -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(invalidFileTime())
+        , expectedModificationTime(doNotCheckFileChange)
     {
     }
 };
index 07bebb2..ebf273c 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, invalidFileTime(), shouldGenerateFile));
+    m_elements.append(FormDataElement(filename, 0, BlobDataItem::toEndOfFile, BlobDataItem::doNotCheckFileChange, shouldGenerateFile));
 #else
     m_elements.append(FormDataElement(filename, shouldGenerateFile));
 #endif
index 68f0633..dec9dc2 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 (isValidFileTime(nextInput.m_expectedFileModificationTime)) {
+        if (nextInput.m_expectedFileModificationTime != BlobDataItem::doNotCheckFileChange) {
             time_t fileModificationTime;
             if (!getFileModificationTime(nextInput.m_filename, fileModificationTime) || fileModificationTime != static_cast<time_t>(nextInput.m_expectedFileModificationTime))
                 return false;
index 10c6fa9..dc0cb62 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 (!isValidFileTime(blobItem.expectedModificationTime))
+    if (blobItem.expectedModificationTime == BlobDataItem::doNotCheckFileChange)
         return false;
 
     time_t fileModificationTime;