...
Created attachment 453176 [details] Patch
Created attachment 453242 [details] Patch
Created attachment 453243 [details] Patch
Comment on attachment 453243 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=453243&action=review > Source/WebKit/ChangeLog:7 > + No explanation in the change log (or in an associated radar if it cannot be public). Makes it hard to understand why we need to do this. > Source/WebKit/UIProcess/WebsiteData/WebsiteDataStoreConfiguration.cpp:61 > + m_shouldUseCustomStoragePaths = WebsiteDataStore::defaultShouldUseCustomStoragePaths(); Why isn’t this in the initialization list?
rdar://89521373
Created attachment 453324 [details] Patch
(In reply to Chris Dumez from comment #4) > Comment on attachment 453243 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=453243&action=review > > > Source/WebKit/ChangeLog:7 > > + > > No explanation in the change log (or in an associated radar if it cannot be > public). Makes it hard to understand why we need to do this. Added radar. > > > Source/WebKit/UIProcess/WebsiteData/WebsiteDataStoreConfiguration.cpp:61 > > + m_shouldUseCustomStoragePaths = WebsiteDataStore::defaultShouldUseCustomStoragePaths(); > > Why isn’t this in the initialization list? Moved.
Committed r290582 (?): <https://commits.webkit.org/r290582> All reviewed patches have been landed. Closing bug and clearing flags on attachment 453324 [details].