RESOLVED FIXED 180870
API test WebKit.WebsiteDataStoreCustomPaths is failing 
https://bugs.webkit.org/show_bug.cgi?id=180870
Summary API test WebKit.WebsiteDataStoreCustomPaths is failing
Ryan Haddad
Reported 2017-12-15 10:54:36 PST
API test WebKit.WebsiteDataStoreCustomPaths is failing on Sierra and High Sierra bots. FAIL WebKit.WebsiteDataStoreCustomPaths /Volumes/Data/slave/elcapitan-release/build/Tools/TestWebKitAPI/Tests/WebKitCocoa/WebsiteDataStoreCustomPaths.mm:219 Value of: [WKWebsiteDataStore _defaultDataStoreExists] Actual: true Expected: false https://build.webkit.org/builders/Apple%20High%20Sierra%20Release%20WK2%20(Tests)/builds/1789
Attachments
Patch (15.27 KB, patch)
2017-12-15 11:36 PST, Brady Eidson 		cdumez: review+
DetailsFormatted DiffDiff
Fixed for landing (15.22 KB, patch)
2017-12-15 12:13 PST, Brady Eidson 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Ryan Haddad
Comment 1 2017-12-15 10:55:10 PST
The test started failing after https://trac.webkit.org/changeset/225789/webkit landed, and continues to fail after https://bugs.webkit.org/show_bug.cgi?id=180722
Brady Eidson
Comment 2 2017-12-15 11:36:24 PST
Created attachment 329502 [details] Patch
Chris Dumez
Comment 3 2017-12-15 11:38:21 PST
Comment on attachment 329502 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=329502&action=review r=me with suggestion. > Source/WebKit/UIProcess/ServiceWorkerProcessProxy.cpp:66 > + send(Messages::WebProcess::EstablishWorkerContextConnectionToStorageProcess { m_serviceWorkerPageID, store, idToUse }, 0); initialSessionID.value_or(PAL::SessionID::defaultSessionID()) ?
Brady Eidson
Comment 4 2017-12-15 12:13:48 PST
Created attachment 329505 [details] Fixed for landing
WebKit Commit Bot
Comment 5 2017-12-15 13:11:57 PST
Comment on attachment 329505 [details] Fixed for landing Clearing flags on attachment: 329505 Committed r225980: <https://trac.webkit.org/changeset/225980>
Radar WebKit Bug Importer
Comment 6 2018-01-02 13:18:24 PST
<rdar://problem/36261368>
Note You need to log in before you can comment on or make changes to this bug.