Configure serviceWorkerRegistrationDirectory on the web site data store and move it to a Caches subfolder as a default
<rdar://problem/36673358>
Created attachment 332902 [details] Patch
Created attachment 332929 [details] Patch
It'd be good if Alex or Brady took a look at this one.
Comment on attachment 332929 [details] Patch Clearing flags on attachment: 332929 Committed r228019: <https://trac.webkit.org/changeset/228019>
All reviewed patches have been landed. Closing bug.
Reopening to attach new patch.
Created attachment 333014 [details] Follow-up fix
Comment on attachment 333014 [details] Follow-up fix Clearing flags on attachment: 333014 Committed r228033: <https://trac.webkit.org/changeset/228033>
After this change, it seems all WebKitTestRunner instances share the same registration database direction, thus introducing flakiness :/
Comment on attachment 332929 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=332929&action=review > Source/WebKit/UIProcess/WebsiteData/WebsiteDataStore.h:111 > + void setServiceWorkerRegistrationDirectory(String&& directory) { m_resolvedConfiguration.serviceWorkerRegistrationDirectory = WTFMove(directory); } WebsiteDataStore::resolveDirectoriesIfNecessary() overrides the path you set here in the resolvedConfiguration with the default path that is in the configuration :/
(In reply to Chris Dumez from comment #12) > Comment on attachment 332929 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=332929&action=review > > > Source/WebKit/UIProcess/WebsiteData/WebsiteDataStore.h:111 > > + void setServiceWorkerRegistrationDirectory(String&& directory) { m_resolvedConfiguration.serviceWorkerRegistrationDirectory = WTFMove(directory); } > > WebsiteDataStore::resolveDirectoriesIfNecessary() overrides the path you set > here in the resolvedConfiguration with the default path that is in the > configuration :/ Filed https://bugs.webkit.org/show_bug.cgi?id=182514