Diff
Modified: trunk/LayoutTests/ChangeLog (285029 => 285030)
--- trunk/LayoutTests/ChangeLog 2021-10-29 15:44:26 UTC (rev 285029)
+++ trunk/LayoutTests/ChangeLog 2021-10-29 15:47:24 UTC (rev 285030)
@@ -1,3 +1,16 @@
+2021-10-29 Ayumi Kojima <ayumi_koj...@apple.com>
+
+ Unreviewed, reverting r285007.
+
+ Reverting r285007 because it caused storage/filesystemaccess
+ /sync-access-handle-basics-worker.html to constantly fail.
+
+ Reverted changeset:
+
+ "Release FileSystemStorageHandle when it is not in use"
+ https://bugs.webkit.org/show_bug.cgi?id=232363
+ https://commits.webkit.org/r285007
+
2021-10-29 Martin Robinson <mrobin...@webkit.org>
opacity should flatten when combined with transform-style: preserve-3d
Modified: trunk/LayoutTests/storage/filesystemaccess/resources/sync-access-handle-basics.js (285029 => 285030)
--- trunk/LayoutTests/storage/filesystemaccess/resources/sync-access-handle-basics.js 2021-10-29 15:44:26 UTC (rev 285029)
+++ trunk/LayoutTests/storage/filesystemaccess/resources/sync-access-handle-basics.js 2021-10-29 15:47:24 UTC (rev 285030)
@@ -17,11 +17,7 @@
function getDirectory() {
navigator.storage.getDirectory().then((handle) => {
rootHandle = handle;
- rootHandle.removeEntry("sync-access-handle.txt").then(() => {
- createFileHandle(rootHandle);
- }).catch((error) => {
- finishTest(error);
- });
+ createFileHandle(rootHandle);
}).catch((error) => {
finishTest(error);
});
Modified: trunk/Source/WebCore/ChangeLog (285029 => 285030)
--- trunk/Source/WebCore/ChangeLog 2021-10-29 15:44:26 UTC (rev 285029)
+++ trunk/Source/WebCore/ChangeLog 2021-10-29 15:47:24 UTC (rev 285030)
@@ -1,3 +1,16 @@
+2021-10-29 Ayumi Kojima <ayumi_koj...@apple.com>
+
+ Unreviewed, reverting r285007.
+
+ Reverting r285007 because it caused storage/filesystemaccess
+ /sync-access-handle-basics-worker.html to constantly fail.
+
+ Reverted changeset:
+
+ "Release FileSystemStorageHandle when it is not in use"
+ https://bugs.webkit.org/show_bug.cgi?id=232363
+ https://commits.webkit.org/r285007
+
2021-10-29 Youenn Fablet <you...@apple.com>
Ensure synchronized rendering of incoming audio tracks
Modified: trunk/Source/WebCore/Modules/filesystemaccess/FileSystemDirectoryHandle.cpp (285029 => 285030)
--- trunk/Source/WebCore/Modules/filesystemaccess/FileSystemDirectoryHandle.cpp 2021-10-29 15:44:26 UTC (rev 285029)
+++ trunk/Source/WebCore/Modules/filesystemaccess/FileSystemDirectoryHandle.cpp 2021-10-29 15:47:24 UTC (rev 285030)
@@ -36,12 +36,12 @@
WTF_MAKE_ISO_ALLOCATED_IMPL(FileSystemDirectoryHandle);
-Ref<FileSystemDirectoryHandle> FileSystemDirectoryHandle::create(ScriptExecutionContext& context, String&& name, FileSystemHandleIdentifier identifier, Ref<FileSystemStorageConnection>&& connection)
+Ref<FileSystemDirectoryHandle> FileSystemDirectoryHandle::create(ScriptExecutionContext* context, String&& name, FileSystemHandleIdentifier identifier, Ref<FileSystemStorageConnection>&& connection)
{
return adoptRef(*new FileSystemDirectoryHandle(context, WTFMove(name), identifier, WTFMove(connection)));
}
-FileSystemDirectoryHandle::FileSystemDirectoryHandle(ScriptExecutionContext& context, String&& name, FileSystemHandleIdentifier identifier, Ref<FileSystemStorageConnection>&& connection)
+FileSystemDirectoryHandle::FileSystemDirectoryHandle(ScriptExecutionContext* context, String&& name, FileSystemHandleIdentifier identifier, Ref<FileSystemStorageConnection>&& connection)
: FileSystemHandle(context, FileSystemHandle::Kind::Directory, WTFMove(name), identifier, WTFMove(connection))
{
}
@@ -56,14 +56,8 @@
if (result.hasException())
return promise.reject(result.releaseException());
- auto identifier = result.returnValue();
auto* context = weakThis ? weakThis->scriptExecutionContext() : nullptr;
- if (!context) {
- connection->closeHandle(identifier);
- return promise.reject(Exception { InvalidStateError, "Context has stopped"_s });
- }
-
- promise.settle(FileSystemFileHandle::create(*context, String { name }, result.returnValue(), WTFMove(connection)));
+ promise.settle(FileSystemFileHandle::create(context, String { name }, result.returnValue(), WTFMove(connection)));
});
}
@@ -77,14 +71,8 @@
if (result.hasException())
return promise.reject(result.releaseException());
- auto identifier = result.returnValue();
auto* context = weakThis ? weakThis->scriptExecutionContext() : nullptr;
- if (!context) {
- connection->closeHandle(identifier);
- return promise.reject(Exception { InvalidStateError, "Context has stopped"_s });
- }
-
- promise.settle(FileSystemDirectoryHandle::create(*context, String { name }, identifier, WTFMove(connection)));
+ promise.settle(FileSystemDirectoryHandle::create(context, String { name }, result.returnValue(), WTFMove(connection)));
});
}
@@ -131,17 +119,12 @@
auto [identifier, isDirectory] = result.releaseReturnValue();
auto* context = weakThis ? weakThis->scriptExecutionContext() : nullptr;
- if (!context) {
- connection->closeHandle(identifier);
- return completionHandler(Exception { InvalidStateError, "Context has stopped"_s });
- }
-
if (isDirectory) {
- Ref<FileSystemHandle> handle = FileSystemDirectoryHandle::create(*context, String { name }, identifier, WTFMove(connection));
+ Ref<FileSystemHandle> handle = FileSystemDirectoryHandle::create(context, String { name }, identifier, WTFMove(connection));
return completionHandler(WTFMove(handle));
}
- Ref<FileSystemHandle> handle = FileSystemFileHandle::create(*context, String { name }, identifier, WTFMove(connection));
+ Ref<FileSystemHandle> handle = FileSystemFileHandle::create(context, String { name }, identifier, WTFMove(connection));
completionHandler(WTFMove(handle));
});
}
Modified: trunk/Source/WebCore/Modules/filesystemaccess/FileSystemDirectoryHandle.h (285029 => 285030)
--- trunk/Source/WebCore/Modules/filesystemaccess/FileSystemDirectoryHandle.h 2021-10-29 15:44:26 UTC (rev 285029)
+++ trunk/Source/WebCore/Modules/filesystemaccess/FileSystemDirectoryHandle.h 2021-10-29 15:47:24 UTC (rev 285030)
@@ -47,7 +47,7 @@
bool recursive { false };
};
- WEBCORE_EXPORT static Ref<FileSystemDirectoryHandle> create(ScriptExecutionContext&, String&&, FileSystemHandleIdentifier, Ref<FileSystemStorageConnection>&&);
+ WEBCORE_EXPORT static Ref<FileSystemDirectoryHandle> create(ScriptExecutionContext*, String&&, FileSystemHandleIdentifier, Ref<FileSystemStorageConnection>&&);
void getFileHandle(const String& name, std::optional<GetFileOptions>, DOMPromiseDeferred<IDLInterface<FileSystemFileHandle>>&&);
void getDirectoryHandle(const String& name, std::optional<GetDirectoryOptions>, DOMPromiseDeferred<IDLInterface<FileSystemDirectoryHandle>>&&);
void removeEntry(const String& name, std::optional<RemoveOptions>, DOMPromiseDeferred<void>&&);
@@ -77,7 +77,7 @@
Ref<Iterator> createIterator();
private:
- FileSystemDirectoryHandle(ScriptExecutionContext&, String&&, FileSystemHandleIdentifier, Ref<FileSystemStorageConnection>&&);
+ FileSystemDirectoryHandle(ScriptExecutionContext*, String&&, FileSystemHandleIdentifier, Ref<FileSystemStorageConnection>&&);
};
} // namespace WebCore
Modified: trunk/Source/WebCore/Modules/filesystemaccess/FileSystemFileHandle.cpp (285029 => 285030)
--- trunk/Source/WebCore/Modules/filesystemaccess/FileSystemFileHandle.cpp 2021-10-29 15:44:26 UTC (rev 285029)
+++ trunk/Source/WebCore/Modules/filesystemaccess/FileSystemFileHandle.cpp 2021-10-29 15:47:24 UTC (rev 285030)
@@ -36,12 +36,12 @@
WTF_MAKE_ISO_ALLOCATED_IMPL(FileSystemFileHandle);
-Ref<FileSystemFileHandle> FileSystemFileHandle::create(ScriptExecutionContext& context, String&& name, FileSystemHandleIdentifier identifier, Ref<FileSystemStorageConnection>&& connection)
+Ref<FileSystemFileHandle> FileSystemFileHandle::create(ScriptExecutionContext* context, String&& name, FileSystemHandleIdentifier identifier, Ref<FileSystemStorageConnection>&& connection)
{
return adoptRef(*new FileSystemFileHandle(context, WTFMove(name), identifier, WTFMove(connection)));
}
-FileSystemFileHandle::FileSystemFileHandle(ScriptExecutionContext& context, String&& name, FileSystemHandleIdentifier identifier, Ref<FileSystemStorageConnection>&& connection)
+FileSystemFileHandle::FileSystemFileHandle(ScriptExecutionContext* context, String&& name, FileSystemHandleIdentifier identifier, Ref<FileSystemStorageConnection>&& connection)
: FileSystemHandle(context, FileSystemHandle::Kind::File, WTFMove(name), identifier, WTFMove(connection))
{
}
@@ -60,18 +60,11 @@
if (result.hasException())
return promise.reject(result.releaseException());
- auto [identifier, file] = result.releaseReturnValue();
- if (file == FileSystem::invalidPlatformFileHandle)
+ auto resultValue = result.releaseReturnValue();
+ if (resultValue.second == FileSystem::invalidPlatformFileHandle)
return promise.reject(Exception { UnknownError, "Invalid platform file handle"_s });
- auto* context = protectedThis->scriptExecutionContext();
- if (!context) {
- FileSystem::closeFile(file);
- protectedThis->close(identifier, { });
- return promise.reject(Exception { InvalidStateError, "Context has stopped"_s });
- }
-
- promise.settle(FileSystemSyncAccessHandle::create(*context, protectedThis.get(), identifier, file));
+ promise.settle(FileSystemSyncAccessHandle::create(protectedThis->scriptExecutionContext(), protectedThis.get(), resultValue.first, resultValue.second));
});
}
Modified: trunk/Source/WebCore/Modules/filesystemaccess/FileSystemFileHandle.h (285029 => 285030)
--- trunk/Source/WebCore/Modules/filesystemaccess/FileSystemFileHandle.h 2021-10-29 15:44:26 UTC (rev 285029)
+++ trunk/Source/WebCore/Modules/filesystemaccess/FileSystemFileHandle.h 2021-10-29 15:47:24 UTC (rev 285030)
@@ -37,7 +37,7 @@
class FileSystemFileHandle final : public FileSystemHandle {
WTF_MAKE_ISO_ALLOCATED(FileSystemFileHandle);
public:
- WEBCORE_EXPORT static Ref<FileSystemFileHandle> create(ScriptExecutionContext&, String&&, FileSystemHandleIdentifier, Ref<FileSystemStorageConnection>&&);
+ WEBCORE_EXPORT static Ref<FileSystemFileHandle> create(ScriptExecutionContext*, String&&, FileSystemHandleIdentifier, Ref<FileSystemStorageConnection>&&);
void getFile(DOMPromiseDeferred<IDLInterface<File>>&&);
void createSyncAccessHandle(DOMPromiseDeferred<IDLInterface<FileSystemSyncAccessHandle>>&&);
@@ -47,7 +47,7 @@
void close(FileSystemSyncAccessHandleIdentifier, CompletionHandler<void(ExceptionOr<void>&&)>&&);
private:
- FileSystemFileHandle(ScriptExecutionContext&, String&&, FileSystemHandleIdentifier, Ref<FileSystemStorageConnection>&&);
+ FileSystemFileHandle(ScriptExecutionContext*, String&&, FileSystemHandleIdentifier, Ref<FileSystemStorageConnection>&&);
};
} // namespace WebCore
Modified: trunk/Source/WebCore/Modules/filesystemaccess/FileSystemHandle.cpp (285029 => 285030)
--- trunk/Source/WebCore/Modules/filesystemaccess/FileSystemHandle.cpp 2021-10-29 15:44:26 UTC (rev 285029)
+++ trunk/Source/WebCore/Modules/filesystemaccess/FileSystemHandle.cpp 2021-10-29 15:47:24 UTC (rev 285030)
@@ -34,8 +34,8 @@
WTF_MAKE_ISO_ALLOCATED_IMPL(FileSystemHandle);
-FileSystemHandle::FileSystemHandle(ScriptExecutionContext& context, FileSystemHandle::Kind kind, String&& name, FileSystemHandleIdentifier identifier, Ref<FileSystemStorageConnection>&& connection)
- : ActiveDOMObject(&context)
+FileSystemHandle::FileSystemHandle(ScriptExecutionContext* context, FileSystemHandle::Kind kind, String&& name, FileSystemHandleIdentifier identifier, Ref<FileSystemStorageConnection>&& connection)
+ : ActiveDOMObject(context)
, m_kind(kind)
, m_name(WTFMove(name))
, m_identifier(identifier)
Modified: trunk/Source/WebCore/Modules/filesystemaccess/FileSystemHandle.h (285029 => 285030)
--- trunk/Source/WebCore/Modules/filesystemaccess/FileSystemHandle.h 2021-10-29 15:44:26 UTC (rev 285029)
+++ trunk/Source/WebCore/Modules/filesystemaccess/FileSystemHandle.h 2021-10-29 15:47:24 UTC (rev 285030)
@@ -54,7 +54,7 @@
void move(FileSystemHandle&, const String& newName, DOMPromiseDeferred<void>&&);
protected:
- FileSystemHandle(ScriptExecutionContext&, Kind, String&& name, FileSystemHandleIdentifier, Ref<FileSystemStorageConnection>&&);
+ FileSystemHandle(ScriptExecutionContext*, Kind, String&& name, FileSystemHandleIdentifier, Ref<FileSystemStorageConnection>&&);
FileSystemStorageConnection& connection() { return m_connection.get(); }
private:
Modified: trunk/Source/WebCore/Modules/filesystemaccess/FileSystemStorageConnection.h (285029 => 285030)
--- trunk/Source/WebCore/Modules/filesystemaccess/FileSystemStorageConnection.h 2021-10-29 15:44:26 UTC (rev 285029)
+++ trunk/Source/WebCore/Modules/filesystemaccess/FileSystemStorageConnection.h 2021-10-29 15:47:24 UTC (rev 285030)
@@ -50,7 +50,6 @@
using GetHandleNamesCallback = CompletionHandler<void(ExceptionOr<Vector<String>>&&)>;
using GetHandleWithTypeCallback = CompletionHandler<void(ExceptionOr<std::pair<FileSystemHandleIdentifier, bool>>&&)>;
- virtual void closeHandle(FileSystemHandleIdentifier) = 0;
virtual void isSameEntry(FileSystemHandleIdentifier, FileSystemHandleIdentifier, SameEntryCallback&&) = 0;
virtual void move(FileSystemHandleIdentifier, FileSystemHandleIdentifier, const String& newName, VoidCallback&&) = 0;
virtual void getFileHandle(FileSystemHandleIdentifier, const String& name, bool createIfNecessary, GetHandleCallback&&) = 0;
Modified: trunk/Source/WebCore/Modules/filesystemaccess/FileSystemSyncAccessHandle.cpp (285029 => 285030)
--- trunk/Source/WebCore/Modules/filesystemaccess/FileSystemSyncAccessHandle.cpp 2021-10-29 15:44:26 UTC (rev 285029)
+++ trunk/Source/WebCore/Modules/filesystemaccess/FileSystemSyncAccessHandle.cpp 2021-10-29 15:47:24 UTC (rev 285030)
@@ -33,13 +33,13 @@
namespace WebCore {
-Ref<FileSystemSyncAccessHandle> FileSystemSyncAccessHandle::create(ScriptExecutionContext& context, FileSystemFileHandle& source, FileSystemSyncAccessHandleIdentifier identifier, FileSystem::PlatformFileHandle file)
+Ref<FileSystemSyncAccessHandle> FileSystemSyncAccessHandle::create(ScriptExecutionContext* context, FileSystemFileHandle& source, FileSystemSyncAccessHandleIdentifier identifier, FileSystem::PlatformFileHandle file)
{
return adoptRef(*new FileSystemSyncAccessHandle(context, source, identifier, file));
}
-FileSystemSyncAccessHandle::FileSystemSyncAccessHandle(ScriptExecutionContext& context, FileSystemFileHandle& source, FileSystemSyncAccessHandleIdentifier identifier, FileSystem::PlatformFileHandle file)
- : ActiveDOMObject(&context)
+FileSystemSyncAccessHandle::FileSystemSyncAccessHandle(ScriptExecutionContext* context, FileSystemFileHandle& source, FileSystemSyncAccessHandleIdentifier identifier, FileSystem::PlatformFileHandle file)
+ : ActiveDOMObject(context)
, m_source(source)
, m_identifier(identifier)
, m_file(file)
Modified: trunk/Source/WebCore/Modules/filesystemaccess/FileSystemSyncAccessHandle.h (285029 => 285030)
--- trunk/Source/WebCore/Modules/filesystemaccess/FileSystemSyncAccessHandle.h 2021-10-29 15:44:26 UTC (rev 285029)
+++ trunk/Source/WebCore/Modules/filesystemaccess/FileSystemSyncAccessHandle.h 2021-10-29 15:47:24 UTC (rev 285030)
@@ -44,7 +44,7 @@
unsigned long long at;
};
- static Ref<FileSystemSyncAccessHandle> create(ScriptExecutionContext&, FileSystemFileHandle&, FileSystemSyncAccessHandleIdentifier, FileSystem::PlatformFileHandle);
+ static Ref<FileSystemSyncAccessHandle> create(ScriptExecutionContext*, FileSystemFileHandle&, FileSystemSyncAccessHandleIdentifier, FileSystem::PlatformFileHandle);
~FileSystemSyncAccessHandle();
void truncate(unsigned long long size, DOMPromiseDeferred<void>&&);
@@ -56,7 +56,7 @@
ExceptionOr<unsigned long long> write(BufferSource&&, FilesystemReadWriteOptions);
private:
- FileSystemSyncAccessHandle(ScriptExecutionContext&, FileSystemFileHandle&, FileSystemSyncAccessHandleIdentifier, FileSystem::PlatformFileHandle);
+ FileSystemSyncAccessHandle(ScriptExecutionContext*, FileSystemFileHandle&, FileSystemSyncAccessHandleIdentifier, FileSystem::PlatformFileHandle);
bool isClosingOrClosed() const;
using CloseCallback = CompletionHandler<void(ExceptionOr<void>&&)>;
void closeInternal(CloseCallback&&);
Modified: trunk/Source/WebCore/Modules/filesystemaccess/WorkerFileSystemStorageConnection.cpp (285029 => 285030)
--- trunk/Source/WebCore/Modules/filesystemaccess/WorkerFileSystemStorageConnection.cpp 2021-10-29 15:44:26 UTC (rev 285029)
+++ trunk/Source/WebCore/Modules/filesystemaccess/WorkerFileSystemStorageConnection.cpp 2021-10-29 15:47:24 UTC (rev 285030)
@@ -29,7 +29,6 @@
#include "WorkerGlobalScope.h"
#include "WorkerLoaderProxy.h"
#include "WorkerThread.h"
-#include <wtf/Scope.h>
namespace WebCore {
@@ -76,14 +75,6 @@
m_scope = nullptr;
}
-void WorkerFileSystemStorageConnection::closeHandle(FileSystemHandleIdentifier identifier)
-{
- callOnMainThread([mainThreadConnection = m_mainThreadConnection, identifier]() mutable {
- if (mainThreadConnection)
- mainThreadConnection->closeHandle(identifier);
- });
-}
-
void WorkerFileSystemStorageConnection::isSameEntry(FileSystemHandleIdentifier identifier, FileSystemHandleIdentifier otherIdentifier, FileSystemStorageConnection::SameEntryCallback&& callback)
{
if (!m_scope)
Modified: trunk/Source/WebCore/Modules/filesystemaccess/WorkerFileSystemStorageConnection.h (285029 => 285030)
--- trunk/Source/WebCore/Modules/filesystemaccess/WorkerFileSystemStorageConnection.h 2021-10-29 15:44:26 UTC (rev 285029)
+++ trunk/Source/WebCore/Modules/filesystemaccess/WorkerFileSystemStorageConnection.h 2021-10-29 15:47:24 UTC (rev 285030)
@@ -57,7 +57,6 @@
WorkerFileSystemStorageConnection(WorkerGlobalScope&, Ref<FileSystemStorageConnection>&&);
// FileSystemStorageConnection
- void closeHandle(FileSystemHandleIdentifier) final;
void isSameEntry(FileSystemHandleIdentifier, FileSystemHandleIdentifier, FileSystemStorageConnection::SameEntryCallback&&) final;
void move(FileSystemHandleIdentifier, FileSystemHandleIdentifier, const String& newName, VoidCallback&&) final;
void getFileHandle(FileSystemHandleIdentifier, const String& name, bool createIfNecessary, FileSystemStorageConnection::GetHandleCallback&&) final;
Modified: trunk/Source/WebCore/Modules/storage/StorageManager.cpp (285029 => 285030)
--- trunk/Source/WebCore/Modules/storage/StorageManager.cpp 2021-10-29 15:44:26 UTC (rev 285029)
+++ trunk/Source/WebCore/Modules/storage/StorageManager.cpp 2021-10-29 15:47:24 UTC (rev 285030)
@@ -119,14 +119,9 @@
if (result.hasException())
return promise.reject(result.releaseException());
- auto [identifier, connection] = result.releaseReturnValue();
auto* context = weakNavigator ? weakNavigator->scriptExecutionContext() : nullptr;
- if (!context) {
- connection->closeHandle(identifier);
- return promise.reject(Exception { InvalidStateError, "Context has stopped"_s });
- }
-
- promise.resolve(FileSystemDirectoryHandle::create(*context, { }, identifier, Ref { *connection }));
+ auto identifierConnectionPair = result.releaseReturnValue();
+ promise.resolve(FileSystemDirectoryHandle::create(context, { }, identifierConnectionPair.first, Ref { * identifierConnectionPair.second }));
});
}
Modified: trunk/Source/WebKit/ChangeLog (285029 => 285030)
--- trunk/Source/WebKit/ChangeLog 2021-10-29 15:44:26 UTC (rev 285029)
+++ trunk/Source/WebKit/ChangeLog 2021-10-29 15:47:24 UTC (rev 285030)
@@ -1,3 +1,16 @@
+2021-10-29 Ayumi Kojima <ayumi_koj...@apple.com>
+
+ Unreviewed, reverting r285007.
+
+ Reverting r285007 because it caused storage/filesystemaccess
+ /sync-access-handle-basics-worker.html to constantly fail.
+
+ Reverted changeset:
+
+ "Release FileSystemStorageHandle when it is not in use"
+ https://bugs.webkit.org/show_bug.cgi?id=232363
+ https://commits.webkit.org/r285007
+
2021-10-29 Takashi Komori <takashi.kom...@sony.com>
WKAPI does not have any APIs to set cookie path to WKWebsiteDataStoreConfigurationRef
Modified: trunk/Source/WebKit/NetworkProcess/storage/FileSystemStorageHandle.cpp (285029 => 285030)
--- trunk/Source/WebKit/NetworkProcess/storage/FileSystemStorageHandle.cpp 2021-10-29 15:44:26 UTC (rev 285029)
+++ trunk/Source/WebKit/NetworkProcess/storage/FileSystemStorageHandle.cpp 2021-10-29 15:47:24 UTC (rev 285030)
@@ -69,14 +69,6 @@
FileSystem::closeFile(m_handle);
}
-void FileSystemStorageHandle::close()
-{
- if (!m_manager)
- return;
-
- m_manager->closeHandle(*this);
-}
-
bool FileSystemStorageHandle::isSameEntry(WebCore::FileSystemHandleIdentifier identifier)
{
auto path = m_manager->getPath(identifier);
Modified: trunk/Source/WebKit/NetworkProcess/storage/FileSystemStorageHandle.h (285029 => 285030)
--- trunk/Source/WebKit/NetworkProcess/storage/FileSystemStorageHandle.h 2021-10-29 15:44:26 UTC (rev 285029)
+++ trunk/Source/WebKit/NetworkProcess/storage/FileSystemStorageHandle.h 2021-10-29 15:47:24 UTC (rev 285030)
@@ -50,7 +50,6 @@
const String& path() const { return m_path; }
Type type() const { return m_type; }
- void close();
bool isSameEntry(WebCore::FileSystemHandleIdentifier);
std::optional<FileSystemStorageError> move(WebCore::FileSystemHandleIdentifier, const String& newName);
Expected<WebCore::FileSystemHandleIdentifier, FileSystemStorageError> getFileHandle(IPC::Connection::UniqueID, String&& name, bool createIfNecessary);
Modified: trunk/Source/WebKit/NetworkProcess/storage/FileSystemStorageManager.cpp (285029 => 285030)
--- trunk/Source/WebKit/NetworkProcess/storage/FileSystemStorageManager.cpp 2021-10-29 15:44:26 UTC (rev 285029)
+++ trunk/Source/WebKit/NetworkProcess/storage/FileSystemStorageManager.cpp 2021-10-29 15:47:24 UTC (rev 285030)
@@ -94,14 +94,6 @@
return handle == m_handles.end() ? FileSystemStorageHandle::Type::Any : handle->value->type();
}
-void FileSystemStorageManager::closeHandle(FileSystemStorageHandle& handle)
-{
- auto identifier = handle.identifier();
- auto takenHandle = m_handles.take(identifier);
- ASSERT(takenHandle.get() == &handle);
- m_registry.unregisterHandle(identifier);
-}
-
void FileSystemStorageManager::connectionClosed(IPC::Connection::UniqueID connection)
{
ASSERT(!RunLoop::isMain());
@@ -112,7 +104,7 @@
auto identifiers = connectionHandles->value;
for (auto identifier : identifiers) {
- m_handles.remove(identifier);
+ auto handle = m_handles.take(identifier);
m_registry.unregisterHandle(identifier);
}
Modified: trunk/Source/WebKit/NetworkProcess/storage/FileSystemStorageManager.h (285029 => 285030)
--- trunk/Source/WebKit/NetworkProcess/storage/FileSystemStorageManager.h 2021-10-29 15:44:26 UTC (rev 285029)
+++ trunk/Source/WebKit/NetworkProcess/storage/FileSystemStorageManager.h 2021-10-29 15:47:24 UTC (rev 285030)
@@ -42,7 +42,6 @@
Expected<WebCore::FileSystemHandleIdentifier, FileSystemStorageError> createHandle(IPC::Connection::UniqueID, FileSystemStorageHandle::Type, String&& path, String&& name, bool createIfNecessary);
const String& getPath(WebCore::FileSystemHandleIdentifier);
FileSystemStorageHandle::Type getType(WebCore::FileSystemHandleIdentifier);
- void closeHandle(FileSystemStorageHandle&);
void connectionClosed(IPC::Connection::UniqueID);
Expected<WebCore::FileSystemHandleIdentifier, FileSystemStorageError> getDirectory(IPC::Connection::UniqueID);
bool acquireLockForFile(const String& path, WebCore::FileSystemHandleIdentifier);
Modified: trunk/Source/WebKit/NetworkProcess/storage/NetworkStorageManager.cpp (285029 => 285030)
--- trunk/Source/WebKit/NetworkProcess/storage/NetworkStorageManager.cpp 2021-10-29 15:44:26 UTC (rev 285029)
+++ trunk/Source/WebKit/NetworkProcess/storage/NetworkStorageManager.cpp 2021-10-29 15:47:24 UTC (rev 285030)
@@ -171,14 +171,6 @@
completionHandler(localOriginStorageManager(origin).fileSystemStorageManager(*m_fileSystemStorageHandleRegistry).getDirectory(connection.uniqueID()));
}
-void NetworkStorageManager::closeHandle(WebCore::FileSystemHandleIdentifier identifier)
-{
- ASSERT(!RunLoop::isMain());
-
- if (auto handle = m_fileSystemStorageHandleRegistry->getHandle(identifier))
- handle->close();
-}
-
void NetworkStorageManager::isSameEntry(WebCore::FileSystemHandleIdentifier identifier, WebCore::FileSystemHandleIdentifier targetIdentifier, CompletionHandler<void(bool)>&& completionHandler)
{
ASSERT(!RunLoop::isMain());
Modified: trunk/Source/WebKit/NetworkProcess/storage/NetworkStorageManager.h (285029 => 285030)
--- trunk/Source/WebKit/NetworkProcess/storage/NetworkStorageManager.h 2021-10-29 15:44:26 UTC (rev 285029)
+++ trunk/Source/WebKit/NetworkProcess/storage/NetworkStorageManager.h 2021-10-29 15:47:24 UTC (rev 285030)
@@ -69,7 +69,6 @@
void persisted(const WebCore::ClientOrigin&, CompletionHandler<void(bool)>&&);
void persist(const WebCore::ClientOrigin&, CompletionHandler<void(bool)>&&);
void fileSystemGetDirectory(IPC::Connection&, const WebCore::ClientOrigin&, CompletionHandler<void(Expected<WebCore::FileSystemHandleIdentifier, FileSystemStorageError>)>&&);
- void closeHandle(WebCore::FileSystemHandleIdentifier);
void isSameEntry(WebCore::FileSystemHandleIdentifier, WebCore::FileSystemHandleIdentifier, CompletionHandler<void(bool)>&&);
void move(WebCore::FileSystemHandleIdentifier, WebCore::FileSystemHandleIdentifier, const String& newName, CompletionHandler<void(std::optional<FileSystemStorageError>)>&&);
void getFileHandle(IPC::Connection&, WebCore::FileSystemHandleIdentifier, String&& name, bool createIfNecessary, CompletionHandler<void(Expected<WebCore::FileSystemHandleIdentifier, FileSystemStorageError>)>&&);
Modified: trunk/Source/WebKit/NetworkProcess/storage/NetworkStorageManager.messages.in (285029 => 285030)
--- trunk/Source/WebKit/NetworkProcess/storage/NetworkStorageManager.messages.in 2021-10-29 15:44:26 UTC (rev 285029)
+++ trunk/Source/WebKit/NetworkProcess/storage/NetworkStorageManager.messages.in 2021-10-29 15:47:24 UTC (rev 285030)
@@ -27,7 +27,6 @@
Persisted(struct WebCore::ClientOrigin origin) -> (bool persisted) Async
Persist(struct WebCore::ClientOrigin origin) -> (bool persisted) Async
FileSystemGetDirectory(struct WebCore::ClientOrigin origin) -> (Expected<WebCore::FileSystemHandleIdentifier, WebKit::FileSystemStorageError> result) Async WantsConnection
- CloseHandle(WebCore::FileSystemHandleIdentifier identifier)
IsSameEntry(WebCore::FileSystemHandleIdentifier identifier, WebCore::FileSystemHandleIdentifier targetIdentifier) -> (bool result) Async
GetFileHandle(WebCore::FileSystemHandleIdentifier identifier, String name, bool createIfNecessary) -> (Expected<WebCore::FileSystemHandleIdentifier, WebKit::FileSystemStorageError> result) Async WantsConnection
GetDirectoryHandle(WebCore::FileSystemHandleIdentifier identifier, String name, bool createIfNecessary) -> (Expected<WebCore::FileSystemHandleIdentifier, WebKit::FileSystemStorageError> result) Async WantsConnection
Modified: trunk/Source/WebKit/WebProcess/WebCoreSupport/WebFileSystemStorageConnection.cpp (285029 => 285030)
--- trunk/Source/WebKit/WebProcess/WebCoreSupport/WebFileSystemStorageConnection.cpp 2021-10-29 15:44:26 UTC (rev 285029)
+++ trunk/Source/WebKit/WebProcess/WebCoreSupport/WebFileSystemStorageConnection.cpp 2021-10-29 15:47:24 UTC (rev 285030)
@@ -48,14 +48,6 @@
m_connection = nullptr;
}
-void WebFileSystemStorageConnection::closeHandle(WebCore::FileSystemHandleIdentifier identifier)
-{
- if (!m_connection)
- return;
-
- m_connection->send(Messages::NetworkStorageManager::CloseHandle(identifier), 0);
-}
-
void WebFileSystemStorageConnection::isSameEntry(WebCore::FileSystemHandleIdentifier identifier, WebCore::FileSystemHandleIdentifier otherIdentifier, WebCore::FileSystemStorageConnection::SameEntryCallback&& completionHandler)
{
if (!m_connection)
Modified: trunk/Source/WebKit/WebProcess/WebCoreSupport/WebFileSystemStorageConnection.h (285029 => 285030)
--- trunk/Source/WebKit/WebProcess/WebCoreSupport/WebFileSystemStorageConnection.h 2021-10-29 15:44:26 UTC (rev 285029)
+++ trunk/Source/WebKit/WebProcess/WebCoreSupport/WebFileSystemStorageConnection.h 2021-10-29 15:47:24 UTC (rev 285030)
@@ -72,7 +72,6 @@
explicit WebFileSystemStorageConnection(IPC::Connection&);
// FileSystemStorageConnection
- void closeHandle(WebCore::FileSystemHandleIdentifier) final;
void isSameEntry(WebCore::FileSystemHandleIdentifier, WebCore::FileSystemHandleIdentifier, WebCore::FileSystemStorageConnection::SameEntryCallback&&) final;
void move(WebCore::FileSystemHandleIdentifier, WebCore::FileSystemHandleIdentifier, const String& newName, VoidCallback&&) final;
void getFileHandle(WebCore::FileSystemHandleIdentifier, const String& name, bool createIfNecessary, WebCore::FileSystemStorageConnection::GetHandleCallback&&) final;
