Bug 241513 - [GStreamer][EME] Restore the "CDMInstanceProxy::releaseDecryptionResources" virtual method
Summary: [GStreamer][EME] Restore the "CDMInstanceProxy::releaseDecryptionResources" v...
Status: RESOLVED WORKSFORME
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-06-10 09:24 PDT by Yacine Bandou
Modified: 2022-06-12 23:55 PDT (History)
3 users (show)

See Also:


Attachments
Patch (2.80 KB, patch)
2022-06-10 09:45 PDT, Yacine Bandou
bandou.yacine: review?
bandou.yacine: commit-queue?
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Yacine Bandou 2022-06-10 09:24:20 PDT
This method is removed since this commit "https://github.com/WebKit/WebKit/commit/7b806050f2bbebb".
    
We should restore this method, in order to be overloaded when we need release the DRM 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,
without the javascript GC is performed.
    
For example, in some chipsets, we can not 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 DRM
system from Widevine to Playready.
Comment 1 Yacine Bandou 2022-06-10 09:45:35 PDT
Created attachment 460160 [details]
Patch
Comment 2 Xabier Rodríguez Calvar 2022-06-12 23:55:22 PDT
As I commented on https://bugs.webkit.org/show_bug.cgi?id=223436#c6 . I think you can keep this downstream. It does not belong here anymore.