Bug 176174 - [GTK][WPE] Test media/event-queue-crash is flaky (race condition on load events) after r220052
Summary: [GTK][WPE] Test media/event-queue-crash is flaky (race condition on load even...
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
Depends on:
Reported: 2017-08-31 11:50 PDT by Carlos Alberto Lopez Perez
Modified: 2020-06-19 00:34 PDT (History)
4 users (show)

See Also:


Note You need to log in before you can comment on or make changes to this bug.
Description Carlos Alberto Lopez Perez 2017-08-31 11:50:10 PDT
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
Comment 1 Carlos Alberto Lopez Perez 2017-08-31 12:03:41 PDT
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)
Comment 2 Carlos Alberto Lopez Perez 2017-08-31 12:05:27 PDT
Committed r221432: <http://trac.webkit.org/changeset/221432>
Comment 3 Diego Pino 2020-06-19 00:34:07 PDT
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>