176174 2017-08-31 11:50:10 -0700 [GTK][WPE] Test media/event-queue-crash is flaky (race condition on load events) after r220052 2020-06-19 00:34:07 -0700 1 1 1 Unclassified WebKit WebKitGTK WebKit Nightly Build Unspecified Unspecified RESOLVED FIXED https://bugs.webkit.org/show_bug.cgi?id=130653 P2 Normal --- 1 clopez webkit-unassigned ap bugs-noreply darin dpino oldest_to_newest 1344045 0 clopez 2017-08-31 11:50:10 -0700 r220052 <https://trac.webkit.org/r220052> implemented some improvements regarding style resolution and layout, and (quoting the changelog) "Also fixes a bug where load events are delivered prematurely in some cases when an object, embed, frame, or iframe element is still loading." As a result of this, the expectation for the test media/event-queue-crash was changed to log a syntax error that before was not triggering: --- a/LayoutTests/media/event-queue-crash-expected.txt +++ b/LayoutTests/media/event-queue-crash-expected.txt @@ -1,3 +1,4 @@ +CONSOLE MESSAGE: line 34: SyntaxError: Unexpected token '}'. Expected '(' to start an 'if' condition. When an element containing video is removed, WebKit should not crash. PASS. WebKit didn't crash. On the GTK+/WPE ports this test is now flaky and gives different results depending on the bot: 1) On the GTK+ release test, this test is passing fine. 2) On the GTK+ wayland bot this test is giving the failure below (like the fix on r220052 didn't worked here) https://build.webkit.org/results/GTK%20Linux%2064-bit%20Release%20Wayland%20(Tests)/r220052%20(2288)/media/event-queue-crash-pretty-diff.html 3) On the WPE release test bot, this test is skipped (general skip of media/ tests), but running it manually happens the same than on GTK+ wayland. 4) On my laptop with GTK+ release, this test is either giving the same result than on GTK+ wayland, passing or crashing. So my conclusion is that there is still some race condition with this test that affects the GTK+/WPE ports 1344053 1 clopez 2017-08-31 12:03:41 -0700 History from the flakiness dashboard : https://webkit-test-results.webkit.org/dashboards/flakiness_dashboard.html#tests=media%2Fevent-queue-crash.html I'm marking it as flaky for the text diff (it will still check that no crashes but won't complain about the text diff due to the race condition in load events) 1344054 2 clopez 2017-08-31 12:05:27 -0700 Committed r221432: <http://trac.webkit.org/changeset/221432> 1664233 3 dpino 2020-06-19 00:34:07 -0700 The test(s) filed under this bug have been consistently passing for the last 4000 revisions. Marking bug as fixed. Committed r263254: <https://trac.webkit.org/changeset/263254>