Attachment 356337 Details for Bug 192287 – Patch

[patch] Patch

bug-192287-20181203002048.patch (text/plain), 46.38 KB, created by Yusuke Suzuki on 2018-12-02 07:20:49 PST

(hide)

Description:

Filename:

MIME Type:

Creator: Yusuke Suzuki

Created: 2018-12-02 07:20:49 PST

Size: 46.38 KB

patch

obsolete

>Subversion Revision: 238789
>diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
>index dcf602f7fb289c5520d07d02d5c7f5ad5f597a3e..a94529a9f0a97f2ab75f18a1f67cd9d300a1755a 100644
>--- a/Source/WebCore/ChangeLog
>+++ b/Source/WebCore/ChangeLog
>@@ -1,3 +1,70 @@
>+2018-12-02  Yusuke Suzuki  <yusukesuzuki@slowstart.org>
>+
>+        Use WallTime for file time
>+        https://bugs.webkit.org/show_bug.cgi?id=192287
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        This patch changes a type of file time from double to WallTime to use strongly typed file time.
>+
>+        No behavior change.
>+
>+        * Modules/webdatabase/Database.cpp:
>+        (WebCore::Database::details const):
>+        * Modules/webdatabase/DatabaseDetails.h:
>+        (WebCore::DatabaseDetails::DatabaseDetails):
>+        (WebCore::DatabaseDetails::creationTime const):
>+        (WebCore::DatabaseDetails::modificationTime const):
>+        * Modules/webdatabase/DatabaseManager.cpp:
>+        (WebCore::DatabaseManager::ProposedDatabase::ProposedDatabase):
>+        * Modules/webdatabase/DatabaseTracker.cpp:
>+        (WebCore::DatabaseTracker::detailsForNameAndOrigin):
>+        * fileapi/AsyncFileStream.cpp:
>+        (WebCore::AsyncFileStream::getSize):
>+        * fileapi/AsyncFileStream.h:
>+        * fileapi/File.cpp:
>+        (WebCore::File::lastModified const):
>+        * page/Page.cpp:
>+        (WebCore::Page::userStyleSheetLocationChanged):
>+        (WebCore::Page::userStyleSheet const):
>+        * page/Page.h:
>+        * platform/FileMetadata.h:
>+        * platform/FileStream.cpp:
>+        (WebCore::FileStream::getSize):
>+        * platform/FileStream.h:
>+        * platform/FileSystem.cpp:
>+        (WebCore::FileSystem::getFileModificationTime): Deleted.
>+        * platform/FileSystem.h:
>+        (WebCore::FileSystem::invalidFileTime):
>+        (WebCore::FileSystem::isValidFileTime):
>+        * platform/glib/FileSystemGlib.cpp:
>+        (WebCore::FileSystem::getFileCreationTime):
>+        (WebCore::FileSystem::getFileModificationTime):
>+        (WebCore::FileSystem::fileMetadataUsingFunction):
>+        * platform/network/BlobDataFileReference.cpp:
>+        (WebCore::BlobDataFileReference::expectedModificationTime):
>+        * platform/network/BlobDataFileReference.h:
>+        * platform/network/FormData.cpp:
>+        (WebCore::FormData::appendFileRange):
>+        * platform/network/FormData.h:
>+        (WebCore::FormDataElement::FormDataElement):
>+        (WebCore::FormDataElement::EncodedFileData::decode):
>+        * platform/network/cf/FormDataStreamCFNet.cpp:
>+        (WebCore::advanceCurrentStream):
>+        * platform/network/soup/ResourceRequestSoup.cpp:
>+        (WebCore::appendEncodedBlobItemToSoupMessageBody):
>+        * platform/posix/FileSystemPOSIX.cpp:
>+        (WebCore::FileSystem::getFileCreationTime):
>+        (WebCore::FileSystem::getFileModificationTime):
>+        * platform/sql/SQLiteFileSystem.cpp:
>+        (WebCore::SQLiteFileSystem::databaseCreationTime):
>+        (WebCore::SQLiteFileSystem::databaseModificationTime):
>+        * platform/sql/SQLiteFileSystem.h:
>+        * platform/win/FileSystemWin.cpp:
>+        (WebCore::FileSystem::getFileModificationTime):
>+        (WebCore::FileSystem::getFileCreationTime):
>+        (WebCore::FileSystem::findDataToFileMetadata):
>+
> 2018-12-01  Brent Fulgham  <bfulgham@apple.com>
> 
>         Lifetime of HTMLMediaElement is not properly handled in asynchronous actions
>diff --git a/Source/WebKit/ChangeLog b/Source/WebKit/ChangeLog
>index 9783723a2578e59a8bac716a1d67f7d3ce1485e0..97fd2a185596471bc214da4cf1a2cdf7f79809f9 100644
>--- a/Source/WebKit/ChangeLog
>+++ b/Source/WebKit/ChangeLog
>@@ -1,3 +1,30 @@
>+2018-12-02  Yusuke Suzuki  <yusukesuzuki@slowstart.org>
>+
>+        Use WallTime for file time
>+        https://bugs.webkit.org/show_bug.cgi?id=192287
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        * Shared/SessionState.h:
>+        * Shared/WebCoreArgumentCoders.cpp:
>+        (IPC::ArgumentCoder<DatabaseDetails>::decode):
>+        * UIProcess/API/glib/WebKitWebViewSessionState.cpp:
>+        (decodeHTTPBody):
>+        * UIProcess/Plugins/gtk/PluginInfoCache.cpp:
>+        (WebKit::PluginInfoCache::getPluginInfo):
>+        (WebKit::PluginInfoCache::updatePluginInfo):
>+        * UIProcess/ResourceLoadStatisticsPersistentStorage.cpp:
>+        (WebKit::hasFileChangedSince):
>+        * UIProcess/WebStorage/LocalStorageDatabaseTracker.cpp:
>+        (WebKit::LocalStorageDatabaseTracker::databasesModifiedSince):
>+        (WebKit::LocalStorageDatabaseTracker::originDetails):
>+        * WebProcess/MediaCache/WebMediaKeyStorageManager.cpp:
>+        (WebKit::removeAllMediaKeyStorageForOriginPath):
>+        (WebKit::WebMediaKeyStorageManager::deleteMediaKeyEntriesForOrigin):
>+        (WebKit::WebMediaKeyStorageManager::deleteMediaKeyEntriesModifiedBetweenDates):
>+        (WebKit::WebMediaKeyStorageManager::deleteAllMediaKeyEntries):
>+        * WebProcess/MediaCache/WebMediaKeyStorageManager.h:
>+
> 2018-12-01  Chris Dumez  <cdumez@apple.com>
> 
>         [PSON] process-swapping may occur even though opener has handle to openee
>diff --git a/Source/WebKitLegacy/win/ChangeLog b/Source/WebKitLegacy/win/ChangeLog
>index 3a77e7f06804b767015914abd8fc348588e860b6..114129054adb44259c68f0ea0a7c9f80cfdb2b8d 100644
>--- a/Source/WebKitLegacy/win/ChangeLog
>+++ b/Source/WebKitLegacy/win/ChangeLog
>@@ -1,3 +1,13 @@
>+2018-12-02  Yusuke Suzuki  <yusukesuzuki@slowstart.org>
>+
>+        Use WallTime for file time
>+        https://bugs.webkit.org/show_bug.cgi?id=192287
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        * Plugins/PluginDatabase.cpp:
>+        (WebCore::PluginDatabase::refresh):
>+
> 2018-11-30  Alex Christensen  <achristensen@webkit.org>
> 
>         Fix Windows build.
>diff --git a/Source/WebCore/Modules/webdatabase/Database.cpp b/Source/WebCore/Modules/webdatabase/Database.cpp
>index e9f2501ef84bdd898f341c8a415bd6118e51b66a..5e9b4adbd19846e6db9b2561635d62771f007b18 100644
>--- a/Source/WebCore/Modules/webdatabase/Database.cpp
>+++ b/Source/WebCore/Modules/webdatabase/Database.cpp
>@@ -638,7 +638,7 @@ String Database::fileName() const
> DatabaseDetails Database::details() const
> {
>     // This code path is only used for database quota delegate calls, so file dates are irrelevant and left uninitialized.
>-    return DatabaseDetails(stringIdentifier(), displayName(), estimatedSize(), 0, 0, 0);
>+    return DatabaseDetails(stringIdentifier(), displayName(), estimatedSize(), 0, WallTime::fromRawSeconds(0), WallTime::fromRawSeconds(0));
> }
> 
> void Database::disableAuthorizer()
>diff --git a/Source/WebCore/Modules/webdatabase/DatabaseDetails.h b/Source/WebCore/Modules/webdatabase/DatabaseDetails.h
>index 6a96e23f0755f05d306acdcf91d6b5b5a26d5896..0ef5cc94b28bdfddd749be9ab8fdca42aca185b1 100644
>--- a/Source/WebCore/Modules/webdatabase/DatabaseDetails.h
>+++ b/Source/WebCore/Modules/webdatabase/DatabaseDetails.h
>@@ -29,6 +29,7 @@
> #pragma once
> 
> #include <wtf/Threading.h>
>+#include <wtf/WallTime.h>
> #include <wtf/text/WTFString.h>
> 
> namespace WebCore {
>@@ -64,7 +65,7 @@ class DatabaseDetails {
>         return *this;
>     }
> 
>-    DatabaseDetails(const String& databaseName, const String& displayName, unsigned long long expectedUsage, unsigned long long currentUsage, double creationTime, double modificationTime)
>+    DatabaseDetails(const String& databaseName, const String& displayName, unsigned long long expectedUsage, unsigned long long currentUsage, WallTime creationTime, WallTime modificationTime)
>         : m_name(databaseName)
>         , m_displayName(displayName)
>         , m_expectedUsage(expectedUsage)
>@@ -78,8 +79,8 @@ class DatabaseDetails {
>     const String& displayName() const { return m_displayName; }
>     uint64_t expectedUsage() const { return m_expectedUsage; }
>     uint64_t currentUsage() const { return m_currentUsage; }
>-    double creationTime() const { return m_creationTime; }
>-    double modificationTime() const { return m_modificationTime; }
>+    WallTime creationTime() const { return m_creationTime; }
>+    WallTime modificationTime() const { return m_modificationTime; }
> #ifndef NDEBUG
>     Thread& thread() const { return m_thread.get(); }
> #endif
>@@ -89,8 +90,8 @@ class DatabaseDetails {
>     String m_displayName;
>     uint64_t m_expectedUsage { 0 };
>     uint64_t m_currentUsage { 0 };
>-    double m_creationTime { 0.0 };
>-    double m_modificationTime { 0.0 };
>+    WallTime m_creationTime;
>+    WallTime m_modificationTime;
> #ifndef NDEBUG
>     Ref<Thread> m_thread { Thread::current() };
> #endif
>diff --git a/Source/WebCore/Modules/webdatabase/DatabaseManager.cpp b/Source/WebCore/Modules/webdatabase/DatabaseManager.cpp
>index ffb7e86a2895ff2dd56b080985fa7d916eecf962..190de001e4ed465cba9045e4362c04355ac5d616 100644
>--- a/Source/WebCore/Modules/webdatabase/DatabaseManager.cpp
>+++ b/Source/WebCore/Modules/webdatabase/DatabaseManager.cpp
>@@ -59,7 +59,7 @@ class DatabaseManager::ProposedDatabase {
> DatabaseManager::ProposedDatabase::ProposedDatabase(DatabaseManager& manager, SecurityOrigin& origin, const String& name, const String& displayName, unsigned long estimatedSize)
>     : m_manager(manager)
>     , m_origin(origin.isolatedCopy())
>-    , m_details(name.isolatedCopy(), displayName.isolatedCopy(), estimatedSize, 0, 0, 0)
>+    , m_details(name.isolatedCopy(), displayName.isolatedCopy(), estimatedSize, 0, WallTime::fromRawSeconds(0), WallTime::fromRawSeconds(0))
> {
>     m_manager.addProposedDatabase(*this);
> }
>diff --git a/Source/WebCore/Modules/webdatabase/DatabaseTracker.cpp b/Source/WebCore/Modules/webdatabase/DatabaseTracker.cpp
>index 0d912913a023d237c3c8145df0924e6393771f43..83c3f76b3f8857952eb613c96e6b492d45429e04 100644
>--- a/Source/WebCore/Modules/webdatabase/DatabaseTracker.cpp
>+++ b/Source/WebCore/Modules/webdatabase/DatabaseTracker.cpp
>@@ -472,7 +472,7 @@ DatabaseDetails DatabaseTracker::detailsForNameAndOrigin(const String& name, con
> 
>     String path = fullPathForDatabase(origin, name, false);
>     if (path.isEmpty())
>-        return DatabaseDetails(name, displayName, expectedUsage, 0, 0, 0);
>+        return DatabaseDetails(name, displayName, expectedUsage, 0, WallTime::fromRawSeconds(0), WallTime::fromRawSeconds(0));
>     return DatabaseDetails(name, displayName, expectedUsage, SQLiteFileSystem::getDatabaseFileSize(path), SQLiteFileSystem::databaseCreationTime(path), SQLiteFileSystem::databaseModificationTime(path));
> }
> 
>diff --git a/Source/WebCore/fileapi/AsyncFileStream.cpp b/Source/WebCore/fileapi/AsyncFileStream.cpp
>index 73a365228799c16b9a988d9a1189e0cf9380fe9d..8cb22261e662f2c10984089b07633ab14d2ffe31 100644
>--- a/Source/WebCore/fileapi/AsyncFileStream.cpp
>+++ b/Source/WebCore/fileapi/AsyncFileStream.cpp
>@@ -134,7 +134,7 @@ void AsyncFileStream::perform(WTF::Function<WTF::Function<void(FileStreamClient&
>     });
> }
> 
>-void AsyncFileStream::getSize(const String& path, double expectedModificationTime)
>+void AsyncFileStream::getSize(const String& path, WallTime expectedModificationTime)
> {
>     // FIXME: Explicit return type here and in all the other cases like this below is a workaround for a deficiency
>     // in the Windows compiler at the time of this writing. Could remove it if that is resolved.
>diff --git a/Source/WebCore/fileapi/AsyncFileStream.h b/Source/WebCore/fileapi/AsyncFileStream.h
>index 62caffe2d98ad1d5d10f4ed68817da90494ea178..4978b49f669d4c7b74d76473a35fe024805eb392 100644
>--- a/Source/WebCore/fileapi/AsyncFileStream.h
>+++ b/Source/WebCore/fileapi/AsyncFileStream.h
>@@ -33,6 +33,7 @@
> 
> #include <wtf/Forward.h>
> #include <wtf/Function.h>
>+#include <wtf/WallTime.h>
> 
> namespace WebCore {
> 
>@@ -44,7 +45,7 @@ class WEBCORE_EXPORT AsyncFileStream {
>     explicit AsyncFileStream(FileStreamClient&);
>     ~AsyncFileStream();
> 
>-    void getSize(const String& path, double expectedModificationTime);
>+    void getSize(const String& path, WallTime expectedModificationTime);
>     void openForRead(const String& path, long long offset, long long length);
>     void close();
>     void read(char* buffer, int length);
>diff --git a/Source/WebCore/fileapi/File.cpp b/Source/WebCore/fileapi/File.cpp
>index e7b0d3ec92f48e2a5c8e7618f61cc9850e8fe901..9c5ec12f8a2f756cabf28a1619b0abe1e23aaffe 100644
>--- a/Source/WebCore/fileapi/File.cpp
>+++ b/Source/WebCore/fileapi/File.cpp
>@@ -112,11 +112,11 @@ int64_t File::lastModified() const
>     // FIXME: This does sync-i/o on the main thread and also recalculates every time the method is called.
>     // The i/o should be performed on a background thread,
>     // and the result should be cached along with an asynchronous monitor for changes to the file.
>-    time_t modificationTime;
>-    if (FileSystem::getFileModificationTime(m_path, modificationTime) && FileSystem::isValidFileTime(modificationTime))
>-        result = modificationTime * msPerSecond;
>+    auto modificationTime = FileSystem::getFileModificationTime(m_path);
>+    if (modificationTime && FileSystem::isValidFileTime(modificationTime.value()))
>+        result = modificationTime->secondsSinceEpoch().millisecondsAs<int64_t>();
>     else
>-        result = WallTime::now().secondsSinceEpoch().milliseconds();
>+        result = WallTime::now().secondsSinceEpoch().millisecondsAs<int64_t>();
> 
>     return WTF::timeClip(result);
> }
>diff --git a/Source/WebCore/page/Page.cpp b/Source/WebCore/page/Page.cpp
>index bd0347f76d725f834d90633f8317819683bedb7e..aa726ef2cc5b2fbf822b23d3facaddebd13da955 100644
>--- a/Source/WebCore/page/Page.cpp
>+++ b/Source/WebCore/page/Page.cpp
>@@ -1347,7 +1347,7 @@ void Page::userStyleSheetLocationChanged()
> 
>     m_didLoadUserStyleSheet = false;
>     m_userStyleSheet = String();
>-    m_userStyleSheetModificationTime = 0;
>+    m_userStyleSheetModificationTime = WallTime::fromRawSeconds(0);
> 
>     // Data URLs with base64-encoded UTF-8 style sheets are common. We can process them
>     // synchronously and avoid using a loader. 
>@@ -1370,8 +1370,8 @@ const String& Page::userStyleSheet() const
>     if (m_userStyleSheetPath.isEmpty())
>         return m_userStyleSheet;
> 
>-    time_t modTime;
>-    if (!FileSystem::getFileModificationTime(m_userStyleSheetPath, modTime)) {
>+    auto modificationTime = FileSystem::getFileModificationTime(m_userStyleSheetPath);
>+    if (!modificationTime) {
>         // The stylesheet either doesn't exist, was just deleted, or is
>         // otherwise unreadable. If we've read the stylesheet before, we should
>         // throw away that data now as it no longer represents what's on disk.
>@@ -1381,12 +1381,12 @@ const String& Page::userStyleSheet() const
> 
>     // If the stylesheet hasn't changed since the last time we read it, we can
>     // just return the old data.
>-    if (m_didLoadUserStyleSheet && modTime <= m_userStyleSheetModificationTime)
>+    if (m_didLoadUserStyleSheet && modificationTime <= m_userStyleSheetModificationTime)
>         return m_userStyleSheet;
> 
>     m_didLoadUserStyleSheet = true;
>     m_userStyleSheet = String();
>-    m_userStyleSheetModificationTime = modTime;
>+    m_userStyleSheetModificationTime = modificationTime.value();
> 
>     // FIXME: It would be better to load this asynchronously to avoid blocking
>     // the process, but we will first need to create an asynchronous loading
>diff --git a/Source/WebCore/page/Page.h b/Source/WebCore/page/Page.h
>index 411487f8375efbe79d28ae1d3244783891f580c2..498ee0570b7d9ec1d80af982bdbd1eecf4cac02d 100644
>--- a/Source/WebCore/page/Page.h
>+++ b/Source/WebCore/page/Page.h
>@@ -811,7 +811,7 @@ class Page : public Supplementable<Page> {
>     String m_userStyleSheetPath;
>     mutable String m_userStyleSheet;
>     mutable bool m_didLoadUserStyleSheet { false };
>-    mutable time_t m_userStyleSheetModificationTime { 0 };
>+    mutable WallTime m_userStyleSheetModificationTime;
> 
>     String m_captionUserPreferencesStyleSheet;
> 
>diff --git a/Source/WebCore/platform/FileMetadata.h b/Source/WebCore/platform/FileMetadata.h
>index 8164d6db8a08dd72008195cc19277fb8307ab026..7d884953e7675d92c0638decad857cd41a59c561 100644
>--- a/Source/WebCore/platform/FileMetadata.h
>+++ b/Source/WebCore/platform/FileMetadata.h
>@@ -30,11 +30,13 @@
> 
> #pragma once
> 
>+#include <wtf/WallTime.h>
>+
> namespace WebCore {
> 
> struct FileMetadata {
>     // The last modification time of the file, in seconds.
>-    double modificationTime;
>+    WallTime modificationTime;
> 
>     // The length of the file in bytes.
>     long long length;
>diff --git a/Source/WebCore/platform/FileStream.cpp b/Source/WebCore/platform/FileStream.cpp
>index ab48c04aceca39d0b77553dce75cadb5d4b335a4..c97bd1a247e5e39e89b1e5b80f69d6929b93c36c 100644
>--- a/Source/WebCore/platform/FileStream.cpp
>+++ b/Source/WebCore/platform/FileStream.cpp
>@@ -49,14 +49,14 @@ FileStream::~FileStream()
>     close();
> }
> 
>-long long FileStream::getSize(const String& path, double expectedModificationTime)
>+long long FileStream::getSize(const String& path, WallTime expectedModificationTime)
> {
>     // Check the modification time for the possible file change.
>-    time_t modificationTime;
>-    if (!FileSystem::getFileModificationTime(path, modificationTime))
>+    auto modificationTime = FileSystem::getFileModificationTime(path);
>+    if (!modificationTime)
>         return -1;
>     if (FileSystem::isValidFileTime(expectedModificationTime)) {
>-        if (static_cast<time_t>(expectedModificationTime) != modificationTime)
>+        if (expectedModificationTime.secondsSinceEpoch().secondsAs<time_t>() != modificationTime->secondsSinceEpoch().secondsAs<time_t>())
>             return -1;
>     }
> 
>diff --git a/Source/WebCore/platform/FileStream.h b/Source/WebCore/platform/FileStream.h
>index b2aa284479f7b1226ebf7aaa9b7fdcac30c33761..16996cd5200629c7db4bfbe2c18f682e034d5e94 100644
>--- a/Source/WebCore/platform/FileStream.h
>+++ b/Source/WebCore/platform/FileStream.h
>@@ -43,7 +43,7 @@ class FileStream {
> 
>     // Gets the size of a file. Also validates if the file has been changed or not if the expected modification time is provided, i.e. non-zero.
>     // Returns total number of bytes if successful. -1 otherwise.
>-    long long getSize(const String& path, double expectedModificationTime);
>+    long long getSize(const String& path, WallTime expectedModificationTime);
> 
>     // Opens a file for reading. The reading starts at the specified offset and lasts till the specified length.
>     // Returns true on success. False otherwise.
>diff --git a/Source/WebCore/platform/FileSystem.cpp b/Source/WebCore/platform/FileSystem.cpp
>index 2fdb260f829362bbcac6597670b0388319fd246a..a77998ccf2070e4d1219d41f12cf927cba0a9de7 100644
>--- a/Source/WebCore/platform/FileSystem.cpp
>+++ b/Source/WebCore/platform/FileSystem.cpp
>@@ -361,13 +361,5 @@ bool fileIsDirectory(const String& path, ShouldFollowSymbolicLinks shouldFollowS
>     return metadata.value().type == FileMetadata::Type::Directory;
> }
> 
>-std::optional<WallTime> getFileModificationTime(const String& path)
>-{
>-    time_t modificationTime = 0;
>-    if (!getFileModificationTime(path, modificationTime))
>-        return std::nullopt;
>-    return WallTime::fromRawSeconds(modificationTime);
>-}
>-
> } // namespace FileSystem
> } // namespace WebCore
>diff --git a/Source/WebCore/platform/FileSystem.h b/Source/WebCore/platform/FileSystem.h
>index e140b2457e1ae6ae8abef1e2b41470a8471ef811..b1e1bc2dff73b849b29a1fa781732b105ae46698 100644
>--- a/Source/WebCore/platform/FileSystem.h
>+++ b/Source/WebCore/platform/FileSystem.h
>@@ -104,9 +104,8 @@ WEBCORE_EXPORT bool deleteEmptyDirectory(const String&);
> WEBCORE_EXPORT bool moveFile(const String& oldPath, const String& newPath);
> WEBCORE_EXPORT bool getFileSize(const String&, long long& result);
> WEBCORE_EXPORT bool getFileSize(PlatformFileHandle, long long& result);
>-WEBCORE_EXPORT bool getFileModificationTime(const String&, time_t& result);
> WEBCORE_EXPORT std::optional<WallTime> getFileModificationTime(const String&);
>-WEBCORE_EXPORT bool getFileCreationTime(const String&, time_t& result); // Not all platforms store file creation time.
>+WEBCORE_EXPORT std::optional<WallTime> getFileCreationTime(const String&); // Not all platforms store file creation time.
> WEBCORE_EXPORT std::optional<FileMetadata> fileMetadata(const String& path);
> WEBCORE_EXPORT std::optional<FileMetadata> fileMetadataFollowingSymlinks(const String& path);
> WEBCORE_EXPORT bool fileIsDirectory(const String&, ShouldFollowSymbolicLinks);
>@@ -133,8 +132,8 @@ String stringFromFileSystemRepresentation(const char*);
> 
> 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 std::isfinite(time); }
>+inline WallTime invalidFileTime() { return WallTime::nan(); }
>+inline bool isValidFileTime(WallTime time) { return std::isnan(time); }
> 
> // Prefix is what the filename should be prefixed with, not the full path.
> WEBCORE_EXPORT String openTemporaryFile(const String& prefix, PlatformFileHandle&);
>diff --git a/Source/WebCore/platform/glib/FileSystemGlib.cpp b/Source/WebCore/platform/glib/FileSystemGlib.cpp
>index f940815f51248015f73b0051782dd5b8256588af..8fc6864cec67adde29b312bb810eba5373b18831 100644
>--- a/Source/WebCore/platform/glib/FileSystemGlib.cpp
>+++ b/Source/WebCore/platform/glib/FileSystemGlib.cpp
>@@ -151,20 +151,19 @@ bool getFileSize(PlatformFileHandle, long long&)
>     return false;
> }
> 
>-bool getFileCreationTime(const String&, time_t&)
>+std::optional<WallTime> getFileCreationTime(const String&)
> {
>     // FIXME: Is there a way to retrieve file creation time with Gtk on platforms that support it?
>-    return false;
>+    return std::nullopt;
> }
> 
>-bool getFileModificationTime(const String& path, time_t& modifiedTime)
>+std::optional<WallTime> getFileModificationTime(const String& path)
> {
>     GStatBuf statResult;
>     if (!getFileStat(path, &statResult))
>-        return false;
>+        return std::nullopt;
> 
>-    modifiedTime = statResult.st_mtime;
>-    return true;
>+    return WallTime::fromRawSeconds(statResult.st_mtime);
> }
> 
> static FileMetadata::Type toFileMetataType(GStatBuf statResult)
>@@ -186,7 +185,7 @@ static std::optional<FileMetadata> fileMetadataUsingFunction(const String& path,
>     bool isHidden = !filename.isEmpty() && filename[0] == '.';
> 
>     return FileMetadata {
>-        static_cast<double>(statResult.st_mtime),
>+        WallTime::fromRawSeconds(statResult.st_mtime),
>         statResult.st_size,
>         isHidden,
>         toFileMetataType(statResult)
>diff --git a/Source/WebCore/platform/network/BlobDataFileReference.cpp b/Source/WebCore/platform/network/BlobDataFileReference.cpp
>index 6a5f12e84d747d85c09e05fa143093f481006d31..84efecfc246bf9c1e2835e25bf977e5e4c088aaf 100644
>--- a/Source/WebCore/platform/network/BlobDataFileReference.cpp
>+++ b/Source/WebCore/platform/network/BlobDataFileReference.cpp
>@@ -73,7 +73,7 @@ unsigned long long BlobDataFileReference::size()
>     return m_size;
> }
> 
>-double BlobDataFileReference::expectedModificationTime()
>+WallTime BlobDataFileReference::expectedModificationTime()
> {
> #if ENABLE(FILE_REPLACEMENT)
>     // We do not currently track modifications for generated files, because we have a snapshot.
>diff --git a/Source/WebCore/platform/network/BlobDataFileReference.h b/Source/WebCore/platform/network/BlobDataFileReference.h
>index a53f77551258b48f3ebb8aa47019b8e4e47dfafb..5aff6767d7f5e70e6538cfe84986c0a54fc8cbc9 100644
>--- a/Source/WebCore/platform/network/BlobDataFileReference.h
>+++ b/Source/WebCore/platform/network/BlobDataFileReference.h
>@@ -27,6 +27,7 @@
> #define BlobDataFileReference_h
> 
> #include <wtf/RefCounted.h>
>+#include <wtf/WallTime.h>
> #include <wtf/text/WTFString.h>
> 
> namespace WebCore {
>@@ -44,7 +45,7 @@ class WEBCORE_EXPORT BlobDataFileReference : public RefCounted<BlobDataFileRefer
> 
>     const String& path();
>     unsigned long long size();
>-    double expectedModificationTime();
>+    WallTime expectedModificationTime();
> 
>     virtual void prepareForFileAccess();
>     virtual void revokeFileAccess();
>@@ -63,7 +64,7 @@ class WEBCORE_EXPORT BlobDataFileReference : public RefCounted<BlobDataFileRefer
>     bool m_replacementShouldBeGenerated;
> #endif
>     unsigned long long m_size;
>-    double m_expectedModificationTime;
>+    WallTime m_expectedModificationTime;
> };
> 
> }
>diff --git a/Source/WebCore/platform/network/FormData.cpp b/Source/WebCore/platform/network/FormData.cpp
>index 8db3dec6d95432f68f23c40269ce345b82245416..0e4a4b537487d96711ebf4883e8f36088fdf1675 100644
>--- a/Source/WebCore/platform/network/FormData.cpp
>+++ b/Source/WebCore/platform/network/FormData.cpp
>@@ -190,7 +190,7 @@ void FormData::appendFile(const String& filename, bool shouldGenerateFile)
>     m_lengthInBytes = std::nullopt;
> }
> 
>-void FormData::appendFileRange(const String& filename, long long start, long long length, double expectedModificationTime, bool shouldGenerateFile)
>+void FormData::appendFileRange(const String& filename, long long start, long long length, WallTime expectedModificationTime, bool shouldGenerateFile)
> {
>     m_elements.append(FormDataElement(filename, start, length, expectedModificationTime, shouldGenerateFile));
>     m_lengthInBytes = std::nullopt;
>diff --git a/Source/WebCore/platform/network/FormData.h b/Source/WebCore/platform/network/FormData.h
>index bbaade185b79f54f8c0707539632166346ec2dec..8919c3feed5a1d2a35b6f93b9cfc342e9b4d28c3 100644
>--- a/Source/WebCore/platform/network/FormData.h
>+++ b/Source/WebCore/platform/network/FormData.h
>@@ -45,7 +45,7 @@ struct FormDataElement {
>         : data(WTFMove(data)) { }
>     explicit FormDataElement(Vector<char>&& array)
>         : data(WTFMove(array)) { }
>-    FormDataElement(const String& filename, int64_t fileStart, int64_t fileLength, double expectedFileModificationTime, bool shouldGenerateFile)
>+    FormDataElement(const String& filename, int64_t fileStart, int64_t fileLength, WallTime expectedFileModificationTime, bool shouldGenerateFile)
>         : data(EncodedFileData { filename, fileStart, fileLength, expectedFileModificationTime, { }, shouldGenerateFile, false }) { }
>     explicit FormDataElement(const URL& blobURL)
>         : data(EncodedBlobData { blobURL }) { }
>@@ -71,7 +71,7 @@ struct FormDataElement {
>         String filename;
>         int64_t fileStart { 0 };
>         int64_t fileLength { 0 };
>-        double expectedFileModificationTime { 0 };
>+        WallTime expectedFileModificationTime { WallTime::fromRawSeconds(0) };
>         String generatedFilename;
>         bool shouldGenerateFile { false };
>         bool ownsGeneratedFile { false };
>@@ -116,7 +116,7 @@ struct FormDataElement {
>             if (!fileLength)
>                 return std::nullopt;
>             
>-            std::optional<double> expectedFileModificationTime;
>+            std::optional<WallTime> expectedFileModificationTime;
>             decoder >> expectedFileModificationTime;
>             if (!expectedFileModificationTime)
>                 return std::nullopt;
>@@ -218,7 +218,7 @@ class FormData : public RefCounted<FormData> {
> 
>     WEBCORE_EXPORT void appendData(const void* data, size_t);
>     void appendFile(const String& filePath, bool shouldGenerateFile = false);
>-    WEBCORE_EXPORT void appendFileRange(const String& filename, long long start, long long length, double expectedModificationTime, bool shouldGenerateFile = false);
>+    WEBCORE_EXPORT void appendFileRange(const String& filename, long long start, long long length, WallTime expectedModificationTime, bool shouldGenerateFile = false);
>     WEBCORE_EXPORT void appendBlob(const URL& blobURL);
> 
>     Vector<char> flatten() const; // omits files
>diff --git a/Source/WebCore/platform/network/cf/FormDataStreamCFNet.cpp b/Source/WebCore/platform/network/cf/FormDataStreamCFNet.cpp
>index 45e31e5437af8d9a412494d426fd2dc380582994..42fb7eac46e6e823d75c3211a8897e68f20e59d7 100644
>--- a/Source/WebCore/platform/network/cf/FormDataStreamCFNet.cpp
>+++ b/Source/WebCore/platform/network/cf/FormDataStreamCFNet.cpp
>@@ -146,8 +146,8 @@ static bool advanceCurrentStream(FormStreamFields* form)
>         }, [form] (const FormDataElement::EncodedFileData& fileData) {
>             // Check if the file has been changed or not if required.
>             if (FileSystem::isValidFileTime(fileData.expectedFileModificationTime)) {
>-                time_t fileModificationTime;
>-                if (!FileSystem::getFileModificationTime(fileData.filename, fileModificationTime) || fileModificationTime != static_cast<time_t>(fileData.expectedFileModificationTime))
>+                auto fileModificationTime = FileSystem::getFileModificationTime(fileData.filename);
>+                if (!fileModificationTime || fileModificationTime->secondsSinceEpoch().secondsAs<time_t>() != fileData.expectedFileModificationTime.secondsSinceEpoch().secondsAs<time_t>())
>                     return false;
>             }
>             const String& path = fileData.shouldGenerateFile ? fileData.generatedFilename : fileData.filename;
>diff --git a/Source/WebCore/platform/network/soup/ResourceRequestSoup.cpp b/Source/WebCore/platform/network/soup/ResourceRequestSoup.cpp
>index 229c53cf5720276e525e9a6fb6f762b8ff58ee07..fb4a2326cebb23535990c695f7e94f1660194f93 100644
>--- a/Source/WebCore/platform/network/soup/ResourceRequestSoup.cpp
>+++ b/Source/WebCore/platform/network/soup/ResourceRequestSoup.cpp
>@@ -44,9 +44,8 @@ static uint64_t appendEncodedBlobItemToSoupMessageBody(SoupMessage* soupMessage,
>         if (!FileSystem::isValidFileTime(blobItem.file()->expectedModificationTime()))
>             return 0;
> 
>-        time_t fileModificationTime;
>-        if (!FileSystem::getFileModificationTime(blobItem.file()->path(), fileModificationTime)
>-            || fileModificationTime != static_cast<time_t>(blobItem.file()->expectedModificationTime()))
>+        auto fileModificationTime = FileSystem::getFileModificationTime(blobItem.file()->path());
>+        if (!fileModificationTime || fileModificationTime->secondsSinceEpoch().secondsAs<time_t>() != blobItem.file()->expectedModificationTime().secondsSinceEpoch().secondsAs<time_t>())
>             return 0;
> 
>         if (RefPtr<SharedBuffer> buffer = SharedBuffer::createWithContentsOfFile(blobItem.file()->path())) {
>diff --git a/Source/WebCore/platform/posix/FileSystemPOSIX.cpp b/Source/WebCore/platform/posix/FileSystemPOSIX.cpp
>index 1e6823171d2f52cc924b3c978b43f302a861d12a..8d284c25fa01057786c73c056c27784c80f5b7bc 100644
>--- a/Source/WebCore/platform/posix/FileSystemPOSIX.cpp
>+++ b/Source/WebCore/platform/posix/FileSystemPOSIX.cpp
>@@ -202,42 +202,39 @@ bool getFileSize(PlatformFileHandle handle, long long& result)
>     return true;
> }
> 
>-bool getFileCreationTime(const String& path, time_t& result)
>+std::optional<WallTime> getFileCreationTime(const String& path)
> {
> #if OS(DARWIN) || OS(OPENBSD) || OS(NETBSD) || OS(FREEBSD)
>     CString fsRep = fileSystemRepresentation(path);
> 
>     if (!fsRep.data() || fsRep.data()[0] == '\0')
>-        return false;
>+        return std::nullopt;
> 
>     struct stat fileInfo;
> 
>     if (stat(fsRep.data(), &fileInfo))
>-        return false;
>+        return std::nullopt;
> 
>-    result = fileInfo.st_birthtime;
>-    return true;
>+    return WallTime::fromRawSeconds(fileInfo.st_birthtime);
> #else
>     UNUSED_PARAM(path);
>-    UNUSED_PARAM(result);
>-    return false;
>+    return std::nullopt;
> #endif
> }
> 
>-bool getFileModificationTime(const String& path, time_t& result)
>+std::optional<WallTime> getFileModificationTime(const String& path)
> {
>     CString fsRep = fileSystemRepresentation(path);
> 
>     if (!fsRep.data() || fsRep.data()[0] == '\0')
>-        return false;
>+        return std::nullopt;
> 
>     struct stat fileInfo;
> 
>     if (stat(fsRep.data(), &fileInfo))
>-        return false;
>+        return std::nullopt;
> 
>-    result = fileInfo.st_mtime;
>-    return true;
>+    return WallTime::fromRawSeconds(fileInfo.st_mtime);
> }
> 
> static FileMetadata::Type toFileMetataType(struct stat fileInfo)
>@@ -263,7 +260,7 @@ static std::optional<FileMetadata> fileMetadataUsingFunction(const String& path,
>     String filename = pathGetFileName(path);
>     bool isHidden = !filename.isEmpty() && filename[0] == '.';
>     return FileMetadata {
>-        static_cast<double>(fileInfo.st_mtime),
>+        WallTime::fromRawSeconds(fileInfo.st_mtime),
>         fileInfo.st_size,
>         isHidden,
>         toFileMetataType(fileInfo)
>diff --git a/Source/WebCore/platform/sql/SQLiteFileSystem.cpp b/Source/WebCore/platform/sql/SQLiteFileSystem.cpp
>index 9ef1868707e030703f690199a12c666274754b52..a50a14f8c0b0a6fde77334815a18ab0c585034d7 100644
>--- a/Source/WebCore/platform/sql/SQLiteFileSystem.cpp
>+++ b/Source/WebCore/platform/sql/SQLiteFileSystem.cpp
>@@ -108,16 +108,20 @@ long long SQLiteFileSystem::getDatabaseFileSize(const String& fileName)
>     return FileSystem::getFileSize(fileName, size) ? size : 0;
> }
> 
>-double SQLiteFileSystem::databaseCreationTime(const String& fileName)
>+WallTime SQLiteFileSystem::databaseCreationTime(const String& fileName)
> {
>-    time_t time;
>-    return FileSystem::getFileCreationTime(fileName, time) ? time : 0;
>+    auto result = FileSystem::getFileCreationTime(fileName);
>+    if (result)
>+        return result.value();
>+    return WallTime::fromRawSeconds(0);
> }
> 
>-double SQLiteFileSystem::databaseModificationTime(const String& fileName)
>+WallTime SQLiteFileSystem::databaseModificationTime(const String& fileName)
> {
>-    time_t time;
>-    return FileSystem::getFileModificationTime(fileName, time) ? time : 0;
>+    auto result = FileSystem::getFileModificationTime(fileName);
>+    if (result)
>+        return result.value();
>+    return WallTime::fromRawSeconds(0);
> }
> 
> } // namespace WebCore
>diff --git a/Source/WebCore/platform/sql/SQLiteFileSystem.h b/Source/WebCore/platform/sql/SQLiteFileSystem.h
>index c8a4f940e8168e71470234011abb28fe36f1e087..f6c69a8bc79fc7242aa8d405c41bd1ad226bd802 100644
>--- a/Source/WebCore/platform/sql/SQLiteFileSystem.h
>+++ b/Source/WebCore/platform/sql/SQLiteFileSystem.h
>@@ -97,8 +97,8 @@ class SQLiteFileSystem {
> #endif
>     
>     static long long getDatabaseFileSize(const String& fileName);
>-    WEBCORE_EXPORT static double databaseCreationTime(const String& fileName);
>-    WEBCORE_EXPORT static double databaseModificationTime(const String& fileName);
>+    WEBCORE_EXPORT static WallTime databaseCreationTime(const String& fileName);
>+    WEBCORE_EXPORT static WallTime databaseModificationTime(const String& fileName);
> 
> private:
>     // do not instantiate this class
>diff --git a/Source/WebCore/platform/win/FileSystemWin.cpp b/Source/WebCore/platform/win/FileSystemWin.cpp
>index 49fa14412a96520f83486ee70cdf45019953e34f..42a27fd5872ee74113acf3762477e39f09991383 100644
>--- a/Source/WebCore/platform/win/FileSystemWin.cpp
>+++ b/Source/WebCore/platform/win/FileSystemWin.cpp
>@@ -124,24 +124,25 @@ bool getFileSize(PlatformFileHandle fileHandle, long long& size)
>     return getFileSizeFromByHandleFileInformationStructure(fileInformation, size);
> }
> 
>-bool getFileModificationTime(const String& path, time_t& time)
>+std::optional<WallTime> getFileModificationTime(const String& path)
> {
>     WIN32_FIND_DATAW findData;
>     if (!getFindData(path, findData))
>-        return false;
>+        return std::nullopt;
> 
>+    time_t time = 0;
>     getFileModificationTimeFromFindData(findData, time);
>-    return true;
>+    return WallTime::fromRawSeconds(time);
> }
> 
>-bool getFileCreationTime(const String& path, time_t& time)
>+std::optional<WallTime> getFileCreationTime(const String& path)
> {
>     WIN32_FIND_DATAW findData;
>     if (!getFindData(path, findData))
>-        return false;
>+        return std::nullopt;
> 
>     getFileCreationTimeFromFindData(findData, time);
>-    return true;
>+    return WallTime::fromRawSeconds(time);
> }
> 
> static String getFinalPathName(const String& path)
>@@ -185,7 +186,7 @@ static std::optional<FileMetadata> findDataToFileMetadata(WIN32_FIND_DATAW findD
>     getFileModificationTimeFromFindData(findData, modificationTime);
> 
>     return FileMetadata {
>-        static_cast<double>(modificationTime),
>+        WallTime::fromRawSeconds(modificationTime),
>         length,
>         static_cast<bool>(findData.dwFileAttributes & FILE_ATTRIBUTE_HIDDEN),
>         toFileMetadataType(findData)
>diff --git a/Source/WebKit/Shared/SessionState.h b/Source/WebKit/Shared/SessionState.h
>index fe81a5d80955ece5d2c625f896962ea9e6c336bd..92afdc598a2b37f91197adf897c61cf594165c3b 100644
>--- a/Source/WebKit/Shared/SessionState.h
>+++ b/Source/WebKit/Shared/SessionState.h
>@@ -69,7 +69,7 @@ struct HTTPBody {
>         String filePath;
>         int64_t fileStart;
>         std::optional<int64_t> fileLength;
>-        std::optional<double> expectedFileModificationTime;
>+        std::optional<WallTime> expectedFileModificationTime;
> 
>         // Blob.
>         String blobURLString;
>diff --git a/Source/WebKit/Shared/WebCoreArgumentCoders.cpp b/Source/WebKit/Shared/WebCoreArgumentCoders.cpp
>index 66ffe9194b047e1cf5e608a297fae52f28892340..0a2b88f5ed7b99c21504f34766b28e1953120257 100644
>--- a/Source/WebKit/Shared/WebCoreArgumentCoders.cpp
>+++ b/Source/WebKit/Shared/WebCoreArgumentCoders.cpp
>@@ -1622,11 +1622,11 @@ bool ArgumentCoder<DatabaseDetails>::decode(Decoder& decoder, DatabaseDetails& d
>     if (!decoder.decode(currentUsage))
>         return false;
> 
>-    double creationTime;
>+    WallTime creationTime;
>     if (!decoder.decode(creationTime))
>         return false;
> 
>-    double modificationTime;
>+    WallTime modificationTime;
>     if (!decoder.decode(modificationTime))
>         return false;
> 
>diff --git a/Source/WebKit/UIProcess/API/glib/WebKitWebViewSessionState.cpp b/Source/WebKit/UIProcess/API/glib/WebKitWebViewSessionState.cpp
>index c79b5678732d2014c2ee85a6b2e7bcd275d01644..7ee045dcc5876cf7b8d1c9b627d942f85c6017a9 100644
>--- a/Source/WebKit/UIProcess/API/glib/WebKitWebViewSessionState.cpp
>+++ b/Source/WebKit/UIProcess/API/glib/WebKitWebViewSessionState.cpp
>@@ -271,7 +271,7 @@ static inline bool decodeHTTPBody(GVariant* httpBodyVariant, HTTPBody& httpBody)
>         if (hasFileLength)
>             element.fileLength = fileLength;
>         if (hasFileModificationTime)
>-            element.expectedFileModificationTime = fileModificationTime;
>+            element.expectedFileModificationTime = WallTime::fromRawSeconds(fileModificationTime);
>         element.blobURLString = String::fromUTF8(blobURLString);
> 
>         httpBody.elements.uncheckedAppend(WTFMove(element));
>diff --git a/Source/WebKit/UIProcess/Plugins/gtk/PluginInfoCache.cpp b/Source/WebKit/UIProcess/Plugins/gtk/PluginInfoCache.cpp
>index b9b0c2eae360b4e01ff44448cb26b43948e9c6ca..80647eb12ee52511a119e85fde2f59c4e2377885 100644
>--- a/Source/WebKit/UIProcess/Plugins/gtk/PluginInfoCache.cpp
>+++ b/Source/WebKit/UIProcess/Plugins/gtk/PluginInfoCache.cpp
>@@ -112,11 +112,11 @@ bool PluginInfoCache::getPluginInfo(const String& pluginPath, PluginModuleInfo&
>     if (!g_key_file_has_group(m_cacheFile.get(), pluginGroup.data()))
>         return false;
> 
>-    time_t lastModified;
>-    if (!WebCore::FileSystem::getFileModificationTime(pluginPath, lastModified))
>+    auto lastModifiedTime = WebCore::FileSystem::getFileModificationTime(pluginPath);
>+    if (!lastModifiedTime)
>         return false;
>     time_t cachedLastModified = static_cast<time_t>(g_key_file_get_uint64(m_cacheFile.get(), pluginGroup.data(), "mtime", nullptr));
>-    if (lastModified != cachedLastModified)
>+    if (lastModifiedTime->secondsSinceEpoch().secondsAs<time_t>() != cachedLastModified)
>         return false;
> 
>     plugin.path = pluginPath;
>@@ -140,12 +140,12 @@ bool PluginInfoCache::getPluginInfo(const String& pluginPath, PluginModuleInfo&
> 
> void PluginInfoCache::updatePluginInfo(const String& pluginPath, const PluginModuleInfo& plugin)
> {
>-    time_t lastModified;
>-    if (!WebCore::FileSystem::getFileModificationTime(pluginPath, lastModified))
>+    auto lastModifiedTime = WebCore::FileSystem::getFileModificationTime(pluginPath);
>+    if (!lastModifiedTime)
>         return;
> 
>     CString pluginGroup = pluginPath.utf8();
>-    g_key_file_set_uint64(m_cacheFile.get(), pluginGroup.data(), "mtime", static_cast<guint64>(lastModified));
>+    g_key_file_set_uint64(m_cacheFile.get(), pluginGroup.data(), "mtime", lastModifiedTime->secondsSinceEpoch().secondsAs<guint64>());
>     g_key_file_set_string(m_cacheFile.get(), pluginGroup.data(), "name", plugin.info.name.utf8().data());
>     g_key_file_set_string(m_cacheFile.get(), pluginGroup.data(), "description", plugin.info.desc.utf8().data());
> 
>diff --git a/Source/WebKit/UIProcess/ResourceLoadStatisticsPersistentStorage.cpp b/Source/WebKit/UIProcess/ResourceLoadStatisticsPersistentStorage.cpp
>index 6f85d36f947ac216450f75c05358565686784afc..770e837857063370a47ede1c0fd2747fefbcad18 100644
>--- a/Source/WebKit/UIProcess/ResourceLoadStatisticsPersistentStorage.cpp
>+++ b/Source/WebKit/UIProcess/ResourceLoadStatisticsPersistentStorage.cpp
>@@ -45,11 +45,11 @@ using namespace WebCore;
> static bool hasFileChangedSince(const String& path, WallTime since)
> {
>     ASSERT(!RunLoop::isMain());
>-    time_t modificationTime;
>-    if (!FileSystem::getFileModificationTime(path, modificationTime))
>+    auto modificationTime = FileSystem::getFileModificationTime(path);
>+    if (!modificationTime)
>         return true;
> 
>-    return WallTime::fromRawSeconds(modificationTime) > since;
>+    return modificationTime.value() > since;
> }
> 
> static std::unique_ptr<KeyedDecoder> createDecoderForFile(const String& path)
>diff --git a/Source/WebKit/UIProcess/WebStorage/LocalStorageDatabaseTracker.cpp b/Source/WebKit/UIProcess/WebStorage/LocalStorageDatabaseTracker.cpp
>index 0a0a1560c54e57ce0363a804d307bf0d41ee21c8..f104b042e34ca3d71ca72d79437f9cdd8759811e 100644
>--- a/Source/WebKit/UIProcess/WebStorage/LocalStorageDatabaseTracker.cpp
>+++ b/Source/WebKit/UIProcess/WebStorage/LocalStorageDatabaseTracker.cpp
>@@ -102,7 +102,7 @@ Vector<SecurityOriginData> LocalStorageDatabaseTracker::databasesModifiedSince(W
>     for (auto origin : databaseOrigins) {
>         auto path = databasePath(origin);
>         
>-        auto modificationTime = WallTime::fromRawSeconds(SQLiteFileSystem::databaseModificationTime(path));
>+        auto modificationTime = SQLiteFileSystem::databaseModificationTime(path);
>         if (modificationTime >= time)
>             databaseOriginsModified.append(origin);
>     }
>@@ -139,8 +139,8 @@ Vector<LocalStorageDatabaseTracker::OriginDetails> LocalStorageDatabaseTracker::
> 
>         OriginDetails details;
>         details.originIdentifier = origin.databaseIdentifier();
>-        details.creationTime = WallTime::fromRawSeconds(SQLiteFileSystem::databaseCreationTime(path));
>-        details.modificationTime = WallTime::fromRawSeconds(SQLiteFileSystem::databaseModificationTime(path));
>+        details.creationTime = SQLiteFileSystem::databaseCreationTime(path);
>+        details.modificationTime = SQLiteFileSystem::databaseModificationTime(path);
>         result.uncheckedAppend(details);
>     }
> 
>diff --git a/Source/WebKit/WebProcess/MediaCache/WebMediaKeyStorageManager.cpp b/Source/WebKit/WebProcess/MediaCache/WebMediaKeyStorageManager.cpp
>index 903c6d012c1db288fba238b578bd06563448d814..4ed638b41c06e529209144425392459b1d2ee9c5 100644
>--- a/Source/WebKit/WebProcess/MediaCache/WebMediaKeyStorageManager.cpp
>+++ b/Source/WebKit/WebProcess/MediaCache/WebMediaKeyStorageManager.cpp
>@@ -79,7 +79,7 @@ Vector<SecurityOriginData> WebMediaKeyStorageManager::getMediaKeyOrigins()
>     return results;
> }
> 
>-static void removeAllMediaKeyStorageForOriginPath(const String& originPath, double startDate, double endDate)
>+static void removeAllMediaKeyStorageForOriginPath(const String& originPath, WallTime startDate, WallTime endDate)
> {
>     Vector<String> mediaKeyPaths = FileSystem::listDirectory(originPath, "*");
> 
>@@ -89,10 +89,10 @@ static void removeAllMediaKeyStorageForOriginPath(const String& originPath, doub
>         if (!FileSystem::fileExists(mediaKeyFile))
>             continue;
> 
>-        time_t modTime;
>-        FileSystem::getFileModificationTime(mediaKeyFile, modTime);
>-
>-        if (modTime < startDate || modTime > endDate)
>+        auto modificationTime = FileSystem::getFileModificationTime(mediaKeyFile);
>+        if (!modificationTime)
>+            continue;
>+        if (modificationTime < startDate || modificationTime > endDate)
>             continue;
> 
>         FileSystem::deleteFile(mediaKeyFile);
>@@ -108,10 +108,10 @@ void WebMediaKeyStorageManager::deleteMediaKeyEntriesForOrigin(const SecurityOri
>         return;
> 
>     String originPath = mediaKeyStorageDirectoryForOrigin(originData);
>-    removeAllMediaKeyStorageForOriginPath(originPath, std::numeric_limits<double>::lowest(), std::numeric_limits<double>::max());
>+    removeAllMediaKeyStorageForOriginPath(originPath, WallTime::fromRawSeconds(std::numeric_limits<double>::lowest()), WallTime::fromRawSeconds(std::numeric_limits<double>::max()));
> }
> 
>-void WebMediaKeyStorageManager::deleteMediaKeyEntriesModifiedBetweenDates(double startDate, double endDate)
>+void WebMediaKeyStorageManager::deleteMediaKeyEntriesModifiedBetweenDates(WallTime startDate, WallTime endDate)
> {
>     if (m_mediaKeyStorageDirectory.isEmpty())
>         return;
>@@ -128,7 +128,7 @@ void WebMediaKeyStorageManager::deleteAllMediaKeyEntries()
> 
>     Vector<String> originPaths = FileSystem::listDirectory(m_mediaKeyStorageDirectory, "*");
>     for (auto& originPath : originPaths)
>-        removeAllMediaKeyStorageForOriginPath(originPath, std::numeric_limits<double>::lowest(), std::numeric_limits<double>::max());
>+        removeAllMediaKeyStorageForOriginPath(originPath, WallTime::fromRawSeconds(std::numeric_limits<double>::lowest()), WallTime::fromRawSeconds(std::numeric_limits<double>::max()));
> }
> 
> }
>diff --git a/Source/WebKit/WebProcess/MediaCache/WebMediaKeyStorageManager.h b/Source/WebKit/WebProcess/MediaCache/WebMediaKeyStorageManager.h
>index 119ece834e44dce89a3b37c157de293f9b3ff942..cc0bd14c91d0d50ca537af1a11591ffa98b58d82 100644
>--- a/Source/WebKit/WebProcess/MediaCache/WebMediaKeyStorageManager.h
>+++ b/Source/WebKit/WebProcess/MediaCache/WebMediaKeyStorageManager.h
>@@ -50,7 +50,7 @@ class WebMediaKeyStorageManager : public WebProcessSupplement {
> 
>     Vector<WebCore::SecurityOriginData> getMediaKeyOrigins();
>     void deleteMediaKeyEntriesForOrigin(const WebCore::SecurityOriginData&);
>-    void deleteMediaKeyEntriesModifiedBetweenDates(double startDate, double endDate);
>+    void deleteMediaKeyEntriesModifiedBetweenDates(WallTime startDate, WallTime endDate);
>     void deleteAllMediaKeyEntries();
> 
> private:
>diff --git a/Source/WebKitLegacy/win/Plugins/PluginDatabase.cpp b/Source/WebKitLegacy/win/Plugins/PluginDatabase.cpp
>index 9cdcc145b71999cda698414e52fe22c37a897b84..f7e3edf4dca128879ab7c2a750331babb676a359 100644
>--- a/Source/WebKitLegacy/win/Plugins/PluginDatabase.cpp
>+++ b/Source/WebKitLegacy/win/Plugins/PluginDatabase.cpp
>@@ -125,9 +125,10 @@ bool PluginDatabase::refresh()
> 
>     auto pathsEnd = paths.end();
>     for (auto it = paths.begin(); it != pathsEnd; ++it) {
>-        time_t lastModified;
>-        if (!FileSystem::getFileModificationTime(*it, lastModified))
>+        auto lastModifiedTime = FileSystem::getFileModificationTime(*it);
>+        if (!lastModifiedTime)
>             continue;
>+        time_t lastModified = lastModifiedTime.secondsSinceEpoch().secondsAs<time_t>();
> 
>         pathsWithTimes.add(*it, lastModified);
>

Actions: View | Formatted Diff | Diff

Attachments on bug 192287: 356337 | 356368 | 356370 | 356376