Bug 185420

Summary: [EME][GStreamer] Dispatch the protection event from AppendPipeline to MediaPlayer
Product: WebKit Reporter: Yacine Bandou <bandou.yacine>
Component: WebKitGTKAssignee: Nobody <webkit-unassigned>
Status: RESOLVED INVALID    
Severity: Normal CC: bugs-noreply, calvaris, olivier.blin
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Yacine Bandou
Reported 2018-05-08 06:15:17 PDT
After the protection event support will be added in the bug 185245 the fucntion MediaPlayerPrivateGStreamerBase::handleProtectionEvent will be reached, so we should remove the ASSERT_NOT_REACHED.
Attachments
Patch (1.74 KB, patch)
2018-05-08 07:06 PDT, Yacine Bandou 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Yacine Bandou
Comment 1 2018-05-08 07:06:49 PDT
Created attachment 339814 [details] Patch
Xabier Rodríguez Calvar
Comment 2 2018-05-09 00:30:29 PDT
I think I explained myself poorly when I asked you to have this part of the code in a different bug. I meant that you should add the code to dispatch the event in a different bug other than the one for bug 185245. Would you have time to do it here?
Yacine Bandou
Comment 3 2018-05-09 02:54:12 PDT
The bug is renamed from "Avoid a crashes after the protection event support is added in 185245" to "Dispatch the protection event from AppendPipeline to MediaPlayer" because I have removed the code which dispatches the Protection event from bug 185245 to this one.
Yacine Bandou
Comment 4 2018-05-10 18:26:40 PDT
For more detail see 185245
Note You need to log in before you can comment on or make changes to this bug.