Bug 120445 – <video> elements with HLS stream sources do not fire 'progress' events.

• Component: New Bugs
• Status: NEW

[Jer Noble]

<video> elements with HLS stream sources do not fire 'progress' events.

[Jer Noble]

Created attachment 209936 [details]
Patch

[Build Bot]

Comment on attachment 209936 [details]
Patch

Attachment 209936 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.appspot.com/results/1623433

New failing tests:
http/tests/media/hls-play-progress.html

[Build Bot]

Created attachment 209947 [details]
Archive of layout-test-results from webkit-ews-08 for mac-mountainlion

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: webkit-ews-08  Port: mac-mountainlion  Platform: Mac OS X 10.8.4

[Eric Carlson]

Comment on attachment 209936 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=209936&action=review

> LayoutTests/ChangeLog:17
> +        * http/tests/media/resources/test.hls/iframe_index.m3u8: Added.
> +        * http/tests/media/resources/test.hls/index.m3u8: Added.
> +        * http/tests/media/resources/test.hls/prog_index.m3u8: Added.
> +        * http/tests/media/resources/test.hls/test.mp4.0.ts: Added.
> +        * http/tests/media/resources/test.hls/test.mp4.1.ts: Added.
> +        * http/tests/media/resources/test.hls/test.mp4.2.ts: Added.

Having a dot in the folder name is odd, why not just "hls"?

> LayoutTests/http/tests/media/hls-play-progress.html:8
> +              var progressCount = 0;

This is unused.

> LayoutTests/http/tests/media/hls-play-progress.html:16
> +              video.src = "http://127.0.0.1:8000/media/resources/test.hls/index.m3u8";
> +              run("video.play()");

You might want to add an 'error' event handler that prints the error code to help diagnose future (or present ;-) ) failures.

[Build Bot]

Comment on attachment 209936 [details]
Patch

Attachment 209936 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.appspot.com/results/1639096

New failing tests:
http/tests/media/hls-play-progress.html

[Build Bot]

Created attachment 209952 [details]
Archive of layout-test-results from webkit-ews-10 for mac-mountainlion-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: webkit-ews-10  Port: mac-mountainlion-wk2  Platform: Mac OS X 10.8.4

[Ahmad Saleem]

I noticed that this r+ patch has not landed by checking bug ID in Webkit GitHub and also I was not able to find the failing test "hls-play-progress" as well.

Is this needed anymore? Thanks!