Bug 176174 – [GTK][WPE] Test media/event-queue-crash is flaky (race condition on load events) after r220052

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKitGTK

[Carlos Alberto Lopez Perez]

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

[Carlos Alberto Lopez Perez]

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)

[Carlos Alberto Lopez Perez]

Committed r221432: <http://trac.webkit.org/changeset/221432>

[Diego Pino]

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>