Use the right paths for website data
Created attachment 232044 [details] Patch
Committed r169325: <http://trac.webkit.org/changeset/169325>
Comment on attachment 232044 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=232044&action=review > Source/WebKit2/UIProcess/API/Cocoa/WKProcessPool.mm:156 > + webContextConfiguration.localStorageDirectory = websiteDataDirectoryURL(@"LocalStorage").absoluteURL.path.fileSystemRepresentation; Another possible fix would be: websiteDataDirectoryURL(@"LocalStorage").absoluteURL.fileSystemRepresentation I don’t think there’s any need to call “path” explicitly. And unless it’s value to have these be relative URLs, I would expect that just: websiteDataDirectoryURL(@"LocalStorage").fileSystemRepresentation would work too.
(In reply to comment #3) > unless it’s value I think I meant "valid", or "expected", here.
(In reply to comment #3) > (From update of attachment 232044 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=232044&action=review > > > Source/WebKit2/UIProcess/API/Cocoa/WKProcessPool.mm:156 > > + webContextConfiguration.localStorageDirectory = websiteDataDirectoryURL(@"LocalStorage").absoluteURL.path.fileSystemRepresentation; > > Another possible fix would be: > > websiteDataDirectoryURL(@"LocalStorage").absoluteURL.fileSystemRepresentation > > I don’t think there’s any need to call “path” explicitly. And unless it’s value to have these be relative URLs, I would expect that just: > > websiteDataDirectoryURL(@"LocalStorage").fileSystemRepresentation > > would work too. It would, except -[NSURL fileSystemRepresentation] isn't available on 10.8.
Comment on attachment 232044 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=232044&action=review >>> Source/WebKit2/UIProcess/API/Cocoa/WKProcessPool.mm:156 >>> + webContextConfiguration.localStorageDirectory = websiteDataDirectoryURL(@"LocalStorage").absoluteURL.path.fileSystemRepresentation; >> >> Another possible fix would be: >> >> websiteDataDirectoryURL(@"LocalStorage").absoluteURL.fileSystemRepresentation >> >> I don’t think there’s any need to call “path” explicitly. And unless it’s value to have these be relative URLs, I would expect that just: >> >> websiteDataDirectoryURL(@"LocalStorage").fileSystemRepresentation >> >> would work too. > > It would, except -[NSURL fileSystemRepresentation] isn't available on 10.8. Oh, got it.