Created attachment 455775 [details]
WIP Patch

Comment on attachment 455775 [details]
WIP Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=455775&action=review

> Source/WebKit/UIProcess/WebsiteData/playstation/WebsiteDataStorePlayStation.cpp:109
> +    return FileSystem::pathByAppendingComponent(FileSystem::userCacheDirectory(), directoryName);

Is this correct for cases where we're not in the USE(WPE_BACKEND_PLAYSTATION)? It seems like you'd get whatever the result of appending directoryName to an empty string was, which doesn't seem like it'd retain the previous behavior.

Comment on attachment 455775 [details]
WIP Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=455775&action=review

> Source/WebKit/UIProcess/WebsiteData/playstation/WebsiteDataStorePlayStation.cpp:114
> +    return FileSystem::pathByAppendingComponent(FileSystem::userDataDirectory(), directoryName);

As with cacheDirectoryFileSystemRepresentation, is this going to do what we want in cases userDataDirectory returns an empty string?

Created attachment 455777 [details]
WIP Patch

This should address the issues mentioned.

Going to work on upstreaming libwpe and adding to our WPE backend before marking it for review.

Created attachment 456042 [details]
WIP Patch

<rdar://problem/91167092>

Pull request: https://github.com/WebKit/WebKit/pull/4931

Committed 255192@main (cbc7d03e34bb): <https://commits.webkit.org/255192@main>

Reviewed commits have been landed. Closing PR #4931 and removing active labels.