[Webkit-unassigned] [Bug 190469] [MSE][GStreamer] Missing opusparse gstreamer element crashes WebKitWebProcess

[bugzilla-daemon at webkit.org]

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

--- Comment #8 from Michael Catanzaro <mcatanzaro at igalia.com> ---
(In reply to Alicia Boya García from comment #7)
> In the current master this is a critical rather than a release assert, but
> still...

That's even worse, since criticals are programming errors, so if you don't want it to fail gracefully, I would change it back to a release assert (which can be used to indicate a packaging issue rather than a programming error).

> I'd rather fail fast on every case so that packagers actually notice the
> problem and install the missing dependencies.

I guess it's OK, but then we need to announce the new dependency. We should also fail the build if it's missing at build time: not strictly necessary, since it's a runtime dependency, but otherwise packagers are not going to notice. I would had assumed the desired behavior would have been to just not play media if required codecs are missing, but whatever.

-- 
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/20181013/e214231e/attachment.html>