[Webkit-unassigned] [Bug 167120] [GTK][GStreamer][MSE] Crash on youtube when MSE is enabled but gstreamer cant find the decoder element.

[bugzilla-daemon at webkit.org]

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

--- Comment #4 from Enrique Ocaña <eocanha at igalia.com> ---
Comment on attachment 304937
  --> https://bugs.webkit.org/attachment.cgi?id=304937
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=304937&action=review

> Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp:113
> +bool doInitializeGStreamerAndRegisterWebKitElements()

I'd thank feedback from some reviewer about if this is the best way to expose the original initializeGStreamerAndRegisterWebKitElements() internal function to be called from another internal function in MediaPlayerPrivateGStreamer.cpp.

If resorting to declare a friend function is too cumbersome, maybe a more direct approach would be to just declare the original initializeGStreamerAndRegisterWebKitElements() publicly in the header file. I'm still not sure.

> Source/WebCore/platform/graphics/gstreamer/mse/MediaPlayerPrivateGStreamerMSE.cpp:776
> +        Vector<GstCapsWebKitMapping> mapping = {

It's great that you have removed the old elements not mapping to any webkitCodecs.

Now the AudioDecoder elementType category has no elements. Still, I think keeping this AudioDecoder/VideoDecoder distinction is useful in case we add support for more mime types in MSE in the future.

> Source/WebCore/platform/graphics/gstreamer/mse/MediaPlayerPrivateGStreamerMSE.cpp:837
> +        for (String pattern : codecSet()) {

If you use "const auto&" here instead of String, you're avoiding String copies and increasing the performance.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.webkit.org/pipermail/webkit-unassigned/attachments/20170320/6522e742/attachment.html>