[Webkit-unassigned] [Bug 185244] [EME][GStreamer] Crash when the mediaKeys are created before loading the media in debug conf

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=185244

--- Comment #10 from Xabier Rodríguez Calvar <calvaris at igalia.com> ---
(In reply to Yacine Bandou from comment #9)
> I added it because I had a crash in some test.
> Otherwise, If we remove the assert from attach, I don't see the
> inconvenience of removing it from dettach.

It had the same idea but different consequences. In the case of attach, we do get it called multiple times and nothing happens if it is the same CDM instance. Actually, we might want to add an assert to check if we are assigned a different instance.

In the case of detach, we should still assert because we are getting a request to detach an instance we don't know.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20180507/1f0aa61c/attachment-0001.html>