fast/loader/javascript-url-iframe-remove-on-navigate.html is flaky with async policy delegates: --- /Volumes/Data/EWS/WebKit/WebKitBuild/Release/layout-test-results/fast/loader/javascript-url-iframe-remove-on-navigate-expected.txt +++ /Volumes/Data/EWS/WebKit/WebKitBuild/Release/layout-test-results/fast/loader/javascript-url-iframe-remove-on-navigate-actual.txt @@ -1,2 +1 @@ -Blocked access to external URL http://does.not.exist/ PASS - Javascript URL blocked without crashing.
Also: --- /Volumes/Data/WebKit/OpenSource/WebKitBuild/Debug/layout-test-results/fast/loader/javascript-url-iframe-remove-on-navigate-expected.txt +++ /Volumes/Data/WebKit/OpenSource/WebKitBuild/Debug/layout-test-results/fast/loader/javascript-url-iframe-remove-on-navigate-actual.txt @@ -1,2 +1,2 @@ -Blocked access to external URL http://does.not.exist/ +ALERT: FAIL PASS - Javascript URL blocked without crashing.
Created attachment 335701 [details] Patch
Comment on attachment 335701 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=335701&action=review > LayoutTests/ChangeLog:15 > + I checked that this test was printing the "FAIL" alert in both Firefox and Chrome before > + this change. So both Chrome & Firefox run javascript protocol async?
Comment on attachment 335701 [details] Patch Clearing flags on attachment: 335701 Committed r229579: <https://trac.webkit.org/changeset/229579>
All reviewed patches have been landed. Closing bug.
<rdar://problem/38423408>