Open JackPu opened 2 weeks ago
The compatibility section of the README states that HLS.js is supported on Chrome 39 and up.
https://github.com/video-dev/hls.js?tab=readme-ov-file#compatibility
Would you like to submit a PR to avoid this escape, assuming that optional chaining resolves the exception described above media?.setMediaKeys(null)?.catch((error) => {
?
Was media detached intentionally or is the exception described a result of tear-down after a larger compatibility related issue?
Sorry about ignoring the compatibility document about the hls.js usage.
And I would like to submit a PR to avoid the special case from your suggestion.
What version of Hls.js are you using?
1.5.14
What browser (including version) are you using?
Chrome38.x
What OS (including version) are you using?
Mac OS
Test stream
https://storage.googleapis.com/shaka-demo-assets/angel-one-widevine-hls/hls.m3u8
Configuration
Additional player setup steps
No response
Checklist
Steps to reproduce
https://hlsjs.video-dev.org/demo/?src=https%3A%2F%2Fstorage.googleapis.com%2Fshaka-demo-assets%2Fangel-one-widevine-hls%2Fhls.m3u8&demoConfig=eyJlbmFibGVTdHJlYW1pbmciOnRydWUsImF1dG9SZWNvdmVyRXJyb3IiOnRydWUsInN0b3BPblN0YWxsIjpmYWxzZSwiZHVtcGZNUDQiOmZhbHNlLCJsZXZlbENhcHBpbmciOi0xLCJsaW1pdE1ldHJpY3MiOi0xfQ==
in the special chrome versiondetach media
buttonExpected behaviour
No Error thrown
What actually happened?
Throw an error
catch in undefined
I guess it could be the line:
https://github.com/video-dev/hls.js/blob/01f50bbbccdf6dea92460b003c78d37a5ecf904e/src/controller/eme-controller.ts#L1256-L1259
media?.setMediaKeys(null)
returnundefined
here. We may remove thecatch
.Console output
Chrome media internals output
No response