Bug 148597

Summary:

[Mac] http/tests/media/video-play-stall.html is flakey

Product:

WebKit

Reporter:

Andy Estes <aestes>

Component:

Media

Assignee:

Eric Carlson <eric.carlson>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

bfulgham, eric.carlson, ews-watchlist, glenn, jer.noble, philipj, rene.warnsloh, sergio, webkit-bug-importer

Priority:

Keywords:

InRadar

Version:

WebKit Nightly Build

Hardware:

Mac

OS:

All

See Also:

https://bugs.webkit.org/show_bug.cgi?id=221694

Attachments:

Description	Flags
Patch	none

Description Andy Estes 2015-08-28 15:53:42 PDT

http/tests/media/video-play-stall.html occasionally fails with the following diff on Mavericks (WK1 and WK2):

--- /Volumes/Data/slave/mavericks-release-tests-wk2/build/layout-test-results/http/tests/media/video-play-stall-expected.txt
+++ /Volumes/Data/slave/mavericks-release-tests-wk2/build/layout-test-results/http/tests/media/video-play-stall-actual.txt
@@ -4,11 +4,9 @@
 EVENT(durationchange)
 EVENT(loadedmetadata)
 EVENT(loadeddata)
-EVENT(canplay)
-RUN(video.play())
 EVENT(stalled)
-EXPECTED (video.currentTime != '0') OK
+EXPECTED (video.currentTime != '0'), OBSERVED '0' FAIL
 EXPECTED (video.playbackRate === '1') OK
-EXPECTED (video.paused === 'false') OK
+EXPECTED (video.paused === 'false'), OBSERVED 'true' FAIL
 END OF TEST
 
See https://build.webkit.org/results/Apple%20Mavericks%20Release%20WK2%20(Tests)/r189124%20(16768)/results.html for an example of this.

Comment 1 Radar WebKit Bug Importer 2015-08-28 15:54:50 PDT

<rdar://problem/22484300>

Comment 2 Andy Estes 2015-08-28 15:55:21 PDT

Oh, it's flakey on Yosemite too.

Comment 3 Andy Estes 2015-08-28 16:01:30 PDT

Marked as flakey in r189131: http://trac.webkit.org/changeset/189131

Comment 4 Alexey Proskuryakov 2015-08-28 23:41:07 PDT

A Yosemite failure:

@@ -1,14 +1,9 @@
 
 Test that a stalled event is sent when media loading stalls.
 
-EVENT(durationchange)
-EVENT(loadedmetadata)
-EVENT(loadeddata)
-EVENT(canplay)
-RUN(video.play())
 EVENT(stalled)
-EXPECTED (video.currentTime != '0') OK
+EXPECTED (video.currentTime != '0'), OBSERVED '0' FAIL
 EXPECTED (video.playbackRate === '1') OK
-EXPECTED (video.paused === 'false') OK
+EXPECTED (video.paused === 'false'), OBSERVED 'true' FAIL
 END OF TEST

Comment 5 Eric Carlson 2021-03-11 11:13:30 PST

Created attachment 422948 [details]
Patch

Comment 6 EWS 2021-03-11 12:17:15 PST

Committed r274292: <https://commits.webkit.org/r274292>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 422948 [details].