Unreviewed, rolling out r119680.
authorkinuko@chromium.org <kinuko@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 7 Jun 2012 04:52:05 +0000 (04:52 +0000)
committerkinuko@chromium.org <kinuko@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 7 Jun 2012 04:52:05 +0000 (04:52 +0000)
http://trac.webkit.org/changeset/119680
https://bugs.webkit.org/show_bug.cgi?id=88498

It broke mac build (Requested by kinuko on #webkit).

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

* 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):

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

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

index 8f4fc1f..d333550 100644 (file)
@@ -1,3 +1,28 @@
+2012-06-06  Sheriff Bot  <webkit.review.bot@gmail.com>
+
+        Unreviewed, rolling out r119680.
+        http://trac.webkit.org/changeset/119680
+        https://bugs.webkit.org/show_bug.cgi?id=88498
+
+        It broke mac build (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):
+
 2012-06-06  Kinuko Yasuda  <kinuko@chromium.org>
 
         File::lastModifiedDate should use NaN or separate boolean flag for null Date value
index baa84d5..86d5989 100644 (file)
@@ -30,7 +30,6 @@
 #include "FileSystem.h"
 #include "MIMETypeRegistry.h"
 #include <wtf/CurrentTime.h>
-#include <wtf/MathExtras.h>
 #include <wtf/text/WTFString.h>
 
 namespace WebCore {
@@ -87,7 +86,7 @@ File::File(const String& path)
     , m_name(pathGetFileName(path))
 #if ENABLE(FILE_SYSTEM)
     , m_snapshotSize(-1)
-    , m_snapshotModificationTime(invalidTime)
+    , m_snapshotModificationTime(0)
 #endif
 {
 }
@@ -97,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(invalidTime)
+    , m_snapshotModificationTime(0)
 #endif
 {
     m_name = pathGetFileName(path);
@@ -112,7 +111,7 @@ File::File(const String& path, const String& name)
     , m_name(name)
 #if ENABLE(FILE_SYSTEM)
     , m_snapshotSize(-1)
-    , m_snapshotModificationTime(invalidTime)
+    , m_snapshotModificationTime(0)
 #endif
 {
 }
@@ -137,12 +136,18 @@ double File::lastModifiedDate() const
 
     time_t modificationTime;
     if (!getFileModificationTime(m_path, modificationTime))
-        return invalidTime;
+        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)
@@ -173,7 +178,7 @@ void File::captureSnapshot(long long& snapshotSize, double& snapshotModification
     FileMetadata metadata;
     if (!getFileMetadata(m_path, metadata)) {
         snapshotSize = 0;
-        snapshotModificationTime = invalidTime;
+        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 af9cc7b..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(invalidTime), length(-1), type(TypeUnknown) { }
+    FileMetadata() : modificationTime(0.0), length(-1), type(TypeUnknown) { }
 };
 
 } // namespace WebCore
index 6c09161..21ece11 100644 (file)
@@ -145,8 +145,6 @@ enum FileSeekOrigin {
     SeekFromEnd
 };
 
-const double invalidTime = std::numeric_limits<double>::quiet_NaN();
-
 #if OS(WINDOWS)
 static const char PlatformFilePathSeparator = '\\';
 #else
index 9c22fa2..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 = invalidTime;
+    result.modificationTime = 0.0;
     result.blobURL = KURL();
 
     switch (element.m_type) {
index 291a1f8..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,7 +40,7 @@
 namespace WebCore {
 
 const long long BlobDataItem::toEndOfFile = -1;
-const double BlobDataItem::doNotCheckFileChange = invalidTime;
+const double BlobDataItem::doNotCheckFileChange = 0;
 
 RawData::RawData()
 {