134976 2014-07-16 08:50:40 -0700 REGRESSION(r171033) [GTK] media/media-controls-invalid-url.html and media/video-fast-seek.html fail 2023-01-25 19:35:04 -0800 1 1 1 Unclassified WebKit Tools / Tests 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED InRadar P2 Normal --- 1 clopez webkit-unassigned bugs-noreply calvaris eric.carlson jer.noble mcatanzaro Ms2ger webkit-bug-importer oldest_to_newest 1022732 0 clopez 2014-07-16 08:50:40 -0700 Revision r171033 <http://trac.webkit.org/r171033> has caused the 2 following tests to fail on the GTK port: media/media-controls-invalid-url.html [ Failure ] * Diff: http://build.webkit.org/results/GTK%20Linux%2064-bit%20Release%20%28Tests%29/r171040%20%281383%29/media/media-controls-invalid-url-pretty-diff.html media/video-fast-seek.html [ Failure ] * Diff: http://build.webkit.org/results/GTK%20Linux%2064-bit%20Release%20%28Tests%29/r171040%20%281383%29/media/video-fast-seek-pretty-diff.html It also has caused the following diff on the test http/tests/w3c/webperf/approved/navigation-timing/html/test_timing_xserver_redirect.html : http://build.webkit.org/results/GTK%20Linux%2064-bit%20Release%20%28Tests%29/r171040%20%281383%29/http/tests/w3c/webperf/approved/navigation-timing/html/test_timing_xserver_redirect-pretty-diff.html But I understand this last one only needs to be rebaselined. 1022930 1 clopez 2014-07-16 20:07:37 -0700 I double checked this: I tested to locally revert r171033 and this two test pass again. I have marked this two test as failing on the GTK TestExpectations, as also rebaselined the test http/tests/w3c/webperf/approved/navigation-timing/html/test_timing_xserver_redirect.html on r171169 <http://trac.webkit.org/r171169> 1240835 2 mcatanzaro 2016-10-16 18:33:10 -0700 media/media-controls-invalid-url.html is now timing out as well. Updating expectations accordingly. 1340568 3 Ms2ger 2017-08-21 04:45:07 -0700 (In reply to Carlos Alberto Lopez Perez from comment #0) > Revision r171033 <http://trac.webkit.org/r171033> has caused the 2 following > tests to fail on the GTK port: > > media/media-controls-invalid-url.html [ Failure ] Current result: This tests that media element controls are reset to their default state when the src is changed to an invalid url. This test only runs in DRT! EVENT(canplaythrough) EXPECTED (getTimeLineValue() == '0') OK EVENT(seeked) EXPECTED (getTimeLineValue() == '1') OK EVENT(error) EXPECTED (getTimeLineValue() == '0'), OBSERVED '1' FAIL END OF TEST Two problems here: - We don't update the displayed duration after an error event. - Even if we did update it from the error handler, the failing check runs before mediaControls's error handler. (A setTimeout(0) in the test would probably be enough to fix that, though.) Also note that the test fails the same way on mac: bug 67716. 1340569 4 318627 Ms2ger 2017-08-21 04:46:54 -0700 Created attachment 318627 Reduced test case for media-controls-invalid-url.html 1340664 5 eric.carlson 2017-08-21 11:38:44 -0700 (In reply to Ms2ger from comment #3) > > Two problems here: > > - We don't update the displayed duration after an error event. > - Even if we did update it from the error handler, the failing check runs > before mediaControls's error handler. (A setTimeout(0) in the test would > probably be enough to fix that, though.) > > Also note that the test fails the same way on mac: bug 67716. This test now passes on macOS. 1928425 6 dpino 2023-01-25 19:32:00 -0800 There are no references to this bug in any TestExpectations. It's probable this bug was solved at some point but it wasn't marked as closed. I'm closing this bug now. If you think this bug report is still valid, please reopen it and add an entry to TestExpectations. 1928455 7 webkit-bug-importer 2023-01-25 19:35:04 -0800 <rdar://problem/104678737> 318627 2017-08-21 04:46:54 -0700 2017-08-21 04:46:54 -0700 Reduced test case for media-controls-invalid-url.html test.html text/html 270 Ms2ger PCFET0NUWVBFIGh0bWw+Cjx2aWRlbyBzcmM9aHR0cDovL3czYy10ZXN0Lm9yZy9tZWRpYS9ncmVl bi1hdC0xNS5tcDQgY29udHJvbHMgaGVpZ2h0PTEwMCB3aWR0aD01MDA+PC92aWRlbz4KPHNjcmlw dD4KdmFyIHZpZGVvID0gZG9jdW1lbnQuZ2V0RWxlbWVudHNCeVRhZ05hbWUoInZpZGVvIilbMF07 CnZpZGVvLmFkZEV2ZW50TGlzdGVuZXIoImNhbnBsYXl0aHJvdWdoIiwgZnVuY3Rpb24oKSB7CiAg dmlkZW8uc3JjID0gIi9pbnZhbGlkLm1vdiI7Cn0pOwo8L3NjcmlwdD4K