[Webkit-unassigned] [Bug 185787] [GTK][WPE] Start implementing MediaStream API

[bugzilla-daemon at webkit.org]

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

Philippe Normand <pnormand at igalia.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #342085|review?, commit-queue?      |review-, commit-queue-
              Flags|                            |

--- Comment #5 from Philippe Normand <pnormand at igalia.com> ---
Comment on attachment 342085
  --> https://bugs.webkit.org/attachment.cgi?id=342085
[GTK][WPE]: Implement MediaStream API

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

Ok I think this is almost ready to land. Is this patch supposed to apply on trunk? The EWS fails.

> Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp:55
> +#include "gstreamer/GStreamerMediaStreamSource.h"

No prefix please ;)

> Source/WebCore/platform/mediastream/gstreamer/GStreamerCapturer.cpp:177
> +    GST_INFO_OBJECT((gpointer) pipeline(), "Going to PLAYING!");

This cast is really needed?

> Source/WebCore/platform/mediastream/gstreamer/GStreamerCapturer.cpp:186
> +    GST_INFO_OBJECT((gpointer) pipeline(), "Tearing down!");

Ditto

> Source/WebCore/platform/mediastream/gstreamer/GStreamerMediaStreamSource.cpp:302
> +    GstElementClass* gstelement_klass = (GstElementClass*)klass;

Use GST_ELEMENT_CLASS(klass)

> Source/WebCore/platform/mediastream/gstreamer/GStreamerMediaStreamSource.cpp:356
> +        (GstPadChainFunction)webkitMediaStreamSrcChain);

You'll think I'm obsessed with casts ;)

> Source/WebCore/platform/mediastream/gstreamer/GStreamerMediaStreamSource.cpp:358
> +    g_assert(gst_element_add_pad(GST_ELEMENT(self), (GstPad*)ghostpad));

GST_PAD_CAST() :)
Also please use ASSERT instead of g_assert

-- 
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/20180607/2cc3f927/attachment-0001.html>