RESOLVED FIXED232127
Followup to r284652: ensure file handle is closed in web process 
https://bugs.webkit.org/show_bug.cgi?id=232127
Summary Followup to r284652: ensure file handle is closed in web process
Sihui Liu
Reported 2021-10-21 19:08:50 PDT
...
Attachments
Patch (12.16 KB, patch)
2021-10-21 23:30 PDT, Sihui Liu 		no flags
DetailsFormatted DiffDiff
Patch (45.47 KB, patch)
2021-10-21 23:47 PDT, Sihui Liu 		no flags
DetailsFormatted DiffDiff
Patch for landing (15.22 KB, patch)
2021-10-22 09:21 PDT, Sihui Liu 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Sihui Liu
Comment 1 2021-10-21 23:30:05 PDT
Created attachment 442126 [details] Patch
Sihui Liu
Comment 2 2021-10-21 23:47:28 PDT
Created attachment 442129 [details] Patch
youenn fablet
Comment 3 2021-10-22 00:45:28 PDT
Comment on attachment 442129 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=442129&action=review > Source/WebCore/Modules/filesystemaccess/FileSystemSyncAccessHandle.cpp:65 > + FileSystem::closeFile(m_file); We are calling closeFile in WebProcess and NetworkProcess. Are both actually needed? I would hope we can move to a place where network process is only responsible for opening the file handle, sending it to web process, locking/unlocking the sync handle and doing quota checks. WebProcess would be fully responsible to do everything else: getSize, flush, read, write, close, truncate... > LayoutTests/storage/filesystemaccess/sync-access-handle-close-worker-expected.txt:537 > +[Worker] Create and close access handle No.511 successfully Not sure we need that big of an expected file. Can we reduce its size?
Sihui Liu
Comment 4 2021-10-22 09:21:51 PDT
Created attachment 442169 [details] Patch for landing
EWS
Comment 5 2021-10-22 09:52:33 PDT
Committed r284692 (243411@main): <https://commits.webkit.org/243411@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 442169 [details].
Radar WebKit Bug Importer
Comment 6 2021-10-22 09:53:18 PDT
<rdar://problem/84553142>
Note You need to log in before you can comment on or make changes to this bug.