Add and test _WKWebsiteDataStoreConfiguration.deviceIdHashSaltsStorageDirectory SPI
Created attachment 374425 [details] Patch
Comment on attachment 374425 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=374425&action=review > Tools/TestWebKitAPI/Tests/WebKit/GetUserMediaNavigation.mm:106 > + NSURL *tempDir = [NSURL fileURLWithPath:[NSTemporaryDirectory() stringByAppendingPathComponent:@"CustomPathsTest"] isDirectory:YES]; Maybe add EXPECT_FALSE([fileManager fileExistsAtPath:[tempDir URLByAppendingPathComponent:@"1"].path]) > Tools/TestWebKitAPI/Tests/WebKit/GetUserMediaNavigation.mm:120 > + webView.UIDelegate = delegate.get(); Shouldn't we load a page like [webView loadTestPageNamed:@"enumerateMediaDevices"]?
(In reply to youenn fablet from comment #2) > Comment on attachment 374425 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=374425&action=review > > > Tools/TestWebKitAPI/Tests/WebKit/GetUserMediaNavigation.mm:106 > > + NSURL *tempDir = [NSURL fileURLWithPath:[NSTemporaryDirectory() stringByAppendingPathComponent:@"CustomPathsTest"] isDirectory:YES]; > > Maybe add EXPECT_FALSE([fileManager fileExistsAtPath:[tempDir > URLByAppendingPathComponent:@"1"].path]) Will do. > > Tools/TestWebKitAPI/Tests/WebKit/GetUserMediaNavigation.mm:120 > > + webView.UIDelegate = delegate.get(); > > Shouldn't we load a page like [webView > loadTestPageNamed:@"enumerateMediaDevices"]? Strange. I will, but it wasn't necessary to write the "1" directory. We should probably fix that.
Comment on attachment 374425 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=374425&action=review > Source/WebKit/UIProcess/API/Cocoa/_WKWebsiteDataStoreConfiguration.mm:113 > + [NSException raise:NSInvalidArgumentException format:@"Cannot set networkCacheDirectory on a non-persistent _WKWebsiteDataStoreConfiguration."]; I also fixed this string to say deviceIdHashSaltsStorageDirectory. http://trac.webkit.org/r247625