API test TestWebKitAPI.WKWebView.ProvisionalURLNotChange is failing on Mojave (Release and Debug) sample output: https://build.webkit.org/builders/Apple%20Mojave%20Release%20WK2%20(Tests)/builds/59/steps/run-api-tests/logs/stdio TestWebKitAPI.WKWebView.ProvisionalURLNotChange _RegisterApplication(), FAILED TO establish the default connection to the WindowServer, _CGSDefaultConnection() is NULL. /Volumes/Data/slave/mojave-release/build/Tools/TestWebKitAPI/Tests/WebKitCocoa/ProvisionalURLNotChange.mm:59 Expected equality of these values: [webView URL].absoluteString.UTF8String Which is: "http://www.webkit.org!/" "data:text/html,start" It has been failing since Mojave queues were added to build.webkit.org
<rdar://problem/44925871>
This test was introduced in http://trac.webkit.org/r184151. It seems we have regressed it and [webView URL].absoluteString.UTF8String can now return an invalid URL. It could also be that we recognize "http://www.webkit.org!/" as a valid URL now.
It does not reproduce locally for me or Alex.
"http://www.webkit.org!" is valid URL according to WebKit's URL parser.
Created attachment 351426 [details] Patch
Comment on attachment 351426 [details] Patch This is a speculative fix. r=me
Comment on attachment 351426 [details] Patch Clearing flags on attachment: 351426 Committed r236764: <https://trac.webkit.org/changeset/236764>
All reviewed patches have been landed. Closing bug.
(In reply to Alex Christensen from comment #6) > Comment on attachment 351426 [details] > Patch > > This is a speculative fix. r=me David confirmed that this speculative fix made the test pass on the bot.