-
Notifications
You must be signed in to change notification settings - Fork 32
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add multi-DRM use-case #62
Comments
From the December 1st IITF meeting, Widevine + native player is not defined. MSE+EME is preferred/required for Widevine. What would be a pass / success? In principle, anything other than a crash would be a pass. Fail with 'DRM not supported' would be a pass. Successful playback would be a pass. Fail with 'no license' might be a pass. |
Normal manifest with multiple drm signals
Use @jpiesing Something else in mind for manifest structure? One |
The spec says until ... the application stops for any reason.
I'll send you an example MPD. |
This is what I had in mind, .... A new tab for "multi-DRM, PlayReady and Widevine" with perhaps the following. 7.1 PlayReady license requested, setActiveDRM not used |
@jpiesing ok thanks, saying
Then it's up to a device decision with an optional help of This set of tests should not use DRMSystemId values Currently most drm test cases we already have do signal multiple drm elements (playready, widevine, marlin) but no Device is instructed to use an appropriate laUrl by invoking |
See http://refapp.hbbtv.org/staging/catalogue/ 7.1-7.3 |
Staging app has 7.1 - 7.6 test cases for
|
I believe this is ready for first feedback @jpiesing @bobcampbell-eurofins |
We'll leave this open so people are aware its available and can try it out |
Staging has an updated 7.1 - 7.6 MultiDRM Manifests don't have an embedded LaUrl anymore, manifest contain just
|
Now there is one Widevine use-case, we should consider adding the following;
MPD with both PlayReady and Widevine
Both MSE/EME and native player
With and without calling the setActiveDRM method (A.2.27)
The text was updated successfully, but these errors were encountered: