Bug 223436 – [GStreamer][EME] Remove DRM resources release

• Status: RESOLVED
• Resolution: FIXED
• Component: New Bugs

[Xabier Rodríguez Calvar]

[GStreamer][EME] Remove DRM resources release

[Xabier Rodríguez Calvar]

Created attachment 423582 [details]
Patch

[EWS]

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

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

[Radar WebKit Bug Importer]

<rdar://problem/75620889>

[Yacine Bandou]

I think we should let

[Yacine Bandou]

we should have left this "CDMInstanceProxy::releaseDecryptionResources" method, in order to be overloaded when we need release the resources.

In some proprietary embedded DRM implementation we need to release some resources  
 in order to be able change the DRM system (like passing from PlayReady to Widevine), and start a new playback, before the javascript GC is performed.

For example, in some chipsets, we cannot initialize the TEE for multiple DRM systems at the same time.

In this EME test page "https://ytlr-cert.appspot.com/2020/main.html?test_type=encryptedmedia-test" we change from Widevine to Playready."

[Xabier Rodríguez Calvar]

(In reply to Yacine Bandou from comment #5)
> we should have left this "CDMInstanceProxy::releaseDecryptionResources"
> method, in order to be overloaded when we need release the resources.
> 
> In some proprietary embedded DRM implementation we need to release some
> resources  
>  in order to be able change the DRM system (like passing from PlayReady to
> Widevine), and start a new playback, before the javascript GC is performed.
> 
> For example, in some chipsets, we cannot initialize the TEE for multiple DRM
> systems at the same time.
> 
> In this EME test page
> "https://ytlr-cert.appspot.com/2020/main.html?test_type=encryptedmedia-test"
> we change from Widevine to Playready."

This was used before giving a spin to all session management and it was creating crashes with certain assets. For now, we don't see that necessary. I think you can keep that downstream if you don't find any other way.