Skip to content

[GStreamer][MSE] HLS.JS improvements - #27

Merged
eocanha merged 1 commit into
WebPlatformForEmbedded:masterfrom
emutavchi:master
Apr 28, 2016
Merged

[GStreamer][MSE] HLS.JS improvements#27
eocanha merged 1 commit into
WebPlatformForEmbedded:masterfrom
emutavchi:master

Conversation

@emutavchi

Copy link
Copy Markdown
Collaborator

without this
hls.js doesn't work properly:
test url: http://test.borovkov.com/hlsjs/
(should show only 1 buffered range at all times)

@eocanha

Copy link
Copy Markdown
Member

This patch seems to break Youtube 2016 test "32. PlaybackState".

I would need more time to debug the issue in deep, but by now I've found this: The test expects the "loadedmetadata" event to be triggered at some point during the data append. In the unmodified code, it's triggered after the audio data has been appended and before feeding the video data, so the test already finds the right condition when everything has been appended. The event is originally triggered by a HTMLMediaElement.setReadyState(HaveEnoughData) as part of a MediaSource.monitorSourceBuffers() call. I can't see beyond in the backtrace with my current release build.

@emutavchi
emutavchiforce-pushed the master branch 4 times, most recently from 9a0f554 to 5034ef4CompareApril 27, 2016 15:32
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@emutavchi@eocanha