ReferenceApplication icon indicating copy to clipboard operation
ReferenceApplication copied to clipboard

incorrect assumption about HTMLMediaElement decoder resource allocation

Open jpiesing opened this issue 3 years ago • 5 comments

From https://redmine.hbbtv.org/issues/12215 image

jpiesing avatar Feb 23 '22 19:02 jpiesing

@ping?

jpiesing avatar May 24 '22 08:05 jpiesing

Feedback in redmine ...

The code to stop the broadcast is actually already in https://github.com/HbbTV-Association/ReferenceApplication/blob/master/src/videoplayer/videoplayer_html5.js, around line 729, but it is apparently never called.

jpiesing avatar May 24 '22 14:05 jpiesing

@jpiesing It's actually called on DRM content but for some reason not on nonDrm playback. After following the redmine #12215 chat and meetings we agreed on the following(?).

  • always call objBroadcast.bindToChannel() + objBroadcast.stop() before starting a dash playback.
  • call objBroadcast.bindToChannel() after stopping a dash playback, if object was added to DOM context.
  • make this change on all three players (msememe, html5, oipf).

https://github.com/HbbTV-Association/ReferenceApplication/blob/9df685d8f63405c40def34862c3bf119ba7da39b/src/videoplayer/videoplayer_html5.js#L729

_mseeme.js=MSE (video tag mse + emeDrm)
_html5.js=Hbbtv2.x (video tag + oipfDrm)
_oipf.js=Hbbtv1.5(AvVideo obj + oipfDrm)

Murmur avatar May 25 '22 08:05 Murmur

You might not need to change the A/V control version - the HbbTV specification says that if there is background broadcast playback then attempting to use an A/V control object will stop the broadcast playback. When the A/V control object is stopped then the background broadcast playback is restarted.

I've got a potential fix for the HTML video element case, see pull request #52. I've not made any attempt to restart playback afterwards.

aoakley-roku avatar May 30 '22 09:05 aoakley-roku

@aoakley-roku Thank you, PR is ok, changes are implemented and published to a staging release. I made similar change on hbbtv1.5(_oipf.js) and hbbtv2(_html5.js) players, broacast channel is resumed on clearVideo() function after a dash playback is stopped.

Github source repository is not updated yet as we are restructuring the integration to an internal development git repository.

Murmur avatar Jun 02 '22 08:06 Murmur

github updated, PR merged (via the big sync push), ticket can be closed? @Murmur

juhajoki avatar Nov 17 '22 09:11 juhajoki

issue was resolved

Murmur avatar Nov 17 '22 09:11 Murmur