Bug 131545 - [GTK] Layout Test media/media-event-listeners.html is failing
Summary: [GTK] Layout Test media/media-event-listeners.html is failing
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Other Linux
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on: 83704
Blocks:
  Show dependency treegraph
 
Reported: 2014-04-11 10:31 PDT by Carlos Alberto Lopez Perez
Modified: 2021-07-02 03:52 PDT (History)
7 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Carlos Alberto Lopez Perez 2014-04-11 10:31:21 PDT
The following layout test is failing on platform GTK since it was added on r164142 <http://trac.webkit.org/r164142>

media/media-event-listeners.html

The tests usually times out, but if you run it with a high enough timeout it gives the following failure:

--- /stuff/webkit/webkit/layout-test-results/media/media-event-listeners-expected.txt
+++ /stuff/webkit/webkit/layout-test-results/media/media-event-listeners-actual.txt
@@ -1,15 +1,2 @@
+FAIL: Timed out waiting for notifyDone to be called
 
-LISTENER(durationchange)
-LISTENER(loadedmetadata)
-LISTENER(loadeddata)
-LISTENER(canplay)
-LISTENER(canplaythrough)
-LISTENER(play)
-LISTENER(playing)
-LISTENER(ratechange)
-LISTENER(timeupdate)
-LISTENER(pause)
-LISTENER(ended)
-LISTENER(volumechange)
-END OF TEST
-


I will mark the test accordingly on the expectations.
Comment 1 Marcos Chavarría Teijeiro (irc: chavaone) 2015-04-07 02:47:11 PDT
This test is no longer timing out after r178418 (https://trac.webkit.org/changeset/178418). Now, it's failing. Updating expectations on bugs.webkit.org/show_bug.cgi?id=143467.
Comment 2 Michael Catanzaro 2015-04-08 14:48:35 PDT
It occasionally passes fine, so I marked it as flaky.
Comment 3 Diego Pino 2020-09-21 03:18:11 PDT
It seems this test was passing, although it was marked flaky. However, since r267162 the test is timing out. The failure happened in the following interval:

[r267156-r267159]             PASS (Expected: PASS FAIL)
[r267160-r267161]             UNKNOWN
r267162                       TIMEOUT (Expected: PASS FAIL)

I cannot determine the cause within the range [r267160-r267162].

The test result looks the same as the output described in https://bugs.webkit.org/show_bug.cgi?id=131545#c0

Diff: https://build.webkit.org/results/GTK%20Linux%2064-bit%20Release%20(Tests)/r267333%20(15940)/media/media-event-listeners-diff.txt

--- /home/buildbot/worker/gtk-linux-64-release-tests/build/layout-test-results/media/media-event-listeners-expected.txt
+++ /home/buildbot/worker/gtk-linux-64-release-tests/build/layout-test-results/media/media-event-listeners-actual.txt
@@ -1,15 +1,2 @@
+FAIL: Timed out waiting for notifyDone to be called
 
-LISTENER(durationchange)
-LISTENER(loadedmetadata)
-LISTENER(loadeddata)
-LISTENER(canplay)
-LISTENER(canplaythrough)
-LISTENER(play)
-LISTENER(playing)
-LISTENER(ratechange)
-LISTENER(timeupdate)
-LISTENER(pause)
-LISTENER(ended)
-LISTENER(volumechange)
-END OF TEST
-

Update expectations accordingly.
Comment 4 Lauro Moura 2020-09-21 12:22:55 PDT
It started in r267162, which changed WTR's TestController.cpp:

> - WKPageSetMuted(m_mainWebView->page(), true);
> + WKPageSetMediaVolume(m_mainWebView->page(), 0);

With this change, after setting the video source the player changes the volume from 1 to 0 before playing, which happens to cause the following 'video.volume = 0' become a NOP and the volumechange event ends up not being triggered.
Comment 5 Lauro Moura 2020-09-22 20:31:01 PDT
Ditto for media/video-volume.html
Comment 6 Xabier Rodríguez Calvar 2021-07-02 03:52:02 PDT
Fixed in r279499
Comment 7 Radar WebKit Bug Importer 2021-07-02 03:52:30 PDT
<rdar://problem/80076065>