RESOLVED FIXED 162914
[GStreamer][EME] Utility function to create decryptor
https://bugs.webkit.org/show_bug.cgi?id=162914
Summary [GStreamer][EME] Utility function to create decryptor
Enrique Ocaña
Reported 2016-10-04 10:12:38 PDT
Add a utility function to find a suitable GStreamer decryptor element.
Attachments
Patch (4.94 KB, patch)
2016-10-04 10:15 PDT, Enrique Ocaña
no flags
Patch (5.21 KB, patch)
2016-10-16 12:09 PDT, Enrique Ocaña
no flags
Patch (5.21 KB, patch)
2016-10-26 01:23 PDT, Enrique Ocaña
no flags
Enrique Ocaña
Comment 1 2016-10-04 10:15:38 PDT
Xabier Rodríguez Calvar
Comment 2 2016-10-08 07:49:34 PDT
Comment on attachment 290610 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=290610&action=review > Source/WebCore/platform/graphics/gstreamer/GStreamerUtilities.cpp:200 > +#if GST_CHECK_VERSION(1, 5, 3) Let's flag this also with ENCRYPTED_MEDIA. Btw, add a comment about moving the version check from the code to the CMake part. EME needs that GStreamer version otherwise it won't function. But of course, we can do this in a follow up. You promise to do it or the guilt will chase until you die and even afterwards. > Source/WebCore/platform/graphics/gstreamer/GStreamerUtilities.cpp:206 > + GST_DEBUG("looking for decryptor for %s", protectionSystem); GST_TRACE > Source/WebCore/platform/graphics/gstreamer/GStreamerUtilities.cpp:215 > + GstStaticPadTemplate* templ = static_cast<GstStaticPadTemplate*>(current->data); Full name > Source/WebCore/platform/graphics/gstreamer/GStreamerUtilities.cpp:216 > + GstCaps* caps = gst_static_pad_template_get_caps(templ); GRefPtr > Source/WebCore/platform/graphics/gstreamer/GStreamerUtilities.cpp:217 > + guint leng = gst_caps_get_size(caps); Full name, type unsigned > Source/WebCore/platform/graphics/gstreamer/GStreamerUtilities.cpp:220 > + for (guint i = 0; !decryptor && i < leng; ++i) { type unsigned > Source/WebCore/platform/graphics/gstreamer/GStreamerUtilities.cpp:221 > + GstStructure* st = gst_caps_get_structure(caps, i); Full name
Enrique Ocaña
Comment 3 2016-10-16 12:09:52 PDT
Enrique Ocaña
Comment 4 2016-10-26 01:23:01 PDT
Enrique Ocaña
Comment 5 2016-10-26 01:48:09 PDT
Comment on attachment 292899 [details] Patch Clearing flags on attachment: 292899 Committed r207886: <http://trac.webkit.org/changeset/207886>
Enrique Ocaña
Comment 6 2016-10-26 01:48:16 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.