...
<rdar://problem/85022453>
Created attachment 444354 [details] Patch
Created attachment 444401 [details] Patch
Comment on attachment 444401 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=444401&action=review > Source/WebCore/Modules/filesystemaccess/FileSystemFileHandle.cpp:62 > + promise.settle(File::create(context, result.returnValue(), { }, protectedThis->name())); Should be resolve, not settle. > Source/WebCore/fileapi/Blob.h:86 > + static Ref<Blob> deserialize(ScriptExecutionContext* context, const URL& srcURL, const String& type, std::optional<long long> size, const String& fileBackedPath) Why is this needed? > Source/WebCore/platform/network/BlobRegistryImpl.cpp:235 > + return nullptr; Wy is this needed?
Created attachment 444464 [details] Patch for landing
(In reply to youenn fablet from comment #4) > Comment on attachment 444401 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=444401&action=review > > > Source/WebCore/Modules/filesystemaccess/FileSystemFileHandle.cpp:62 > > + promise.settle(File::create(context, result.returnValue(), { }, protectedThis->name())); > > Should be resolve, not settle. > > > Source/WebCore/fileapi/Blob.h:86 > > + static Ref<Blob> deserialize(ScriptExecutionContext* context, const URL& srcURL, const String& type, std::optional<long long> size, const String& fileBackedPath) > > Why is this needed? Not needed. Removed. > > > Source/WebCore/platform/network/BlobRegistryImpl.cpp:235 > > + return nullptr; > > Wy is this needed? Not needed. Removed.
Committed r285912 (244321@main): <https://commits.webkit.org/244321@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 444464 [details].