Attachment 339498 Details for Bug 185242 – Patch

[patch] Patch

bug-185242-20180504030055.patch (text/plain), 2.84 KB, created by Yacine Bandou on 2018-05-03 18:00:56 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Yacine Bandou

Created: 2018-05-03 18:00:56 PDT

Size: 2.84 KB

patch

obsolete

>Subversion Revision: 231044
>diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
>index e182b0a8cb8d7a86ed5f59a819f45fc55e10fd7e..ff279fafc2e6a05b1cd5cc56da27268dae7fb80b 100644
>--- a/Source/WebCore/ChangeLog
>+++ b/Source/WebCore/ChangeLog
>@@ -1,3 +1,19 @@
>+2018-05-03  Yacine Bandou  <yacine.bandou_ext@softathome.com>
>+
>+        [MSE][GStreamer] Delete properly the stream from the WebKitMediaSource
>+        https://bugs.webkit.org/show_bug.cgi?id=185242
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        When the sourceBuffer is removed from mediasource, the appropriate stream is not
>+        properly deleted from WebKitMediaSource.
>+        The Appsrc and the Parser didn't removed from the WebKitMediaSource bin element.
>+
>+        This patch avoid the regression of r231089, see https://bugs.webkit.org/show_bug.cgi?id=185071
>+
>+        * platform/graphics/gstreamer/mse/WebKitMediaSourceGStreamer.cpp:
>+        (webKitMediaSrcFreeStream):
>+
> 2018-01-24  Yacine Bandou  <yacine.bandou_ext@softathome.com>
> 
>         [EME][GStreamer] Move the decryptor from AppendPipeline to PlaybackPipeline.
>diff --git a/Source/WebCore/platform/graphics/gstreamer/mse/WebKitMediaSourceGStreamer.cpp b/Source/WebCore/platform/graphics/gstreamer/mse/WebKitMediaSourceGStreamer.cpp
>index d32fba3ecde0eb19c76acd9fed5e5cdd4fdd926a..995a81b48a98936cdbd0abba8d65945d5690c3b9 100644
>--- a/Source/WebCore/platform/graphics/gstreamer/mse/WebKitMediaSourceGStreamer.cpp
>+++ b/Source/WebCore/platform/graphics/gstreamer/mse/WebKitMediaSourceGStreamer.cpp
>@@ -513,6 +513,15 @@ void webKitMediaSrcFreeStream(WebKitMediaSrc* source, Stream* stream)
>         // Don't trigger callbacks from this appsrc to avoid using the stream anymore.
>         gst_app_src_set_callbacks(GST_APP_SRC(stream->appsrc), &disabledAppsrcCallbacks, nullptr, nullptr);
>         gst_app_src_end_of_stream(GST_APP_SRC(stream->appsrc));
>+        gst_element_set_state(stream->appsrc, GST_STATE_NULL);
>+        gst_bin_remove(GST_BIN(source), stream->appsrc);
>+        stream->appsrc = nullptr;
>+    }
>+
>+    if (stream->parser) {
>+        gst_element_set_state(stream->parser, GST_STATE_NULL);
>+        gst_bin_remove(GST_BIN(source), stream->parser);
>+        stream->parser = nullptr;
>     }
> 
>     if (stream->type != WebCore::Invalid) {
>@@ -531,12 +540,15 @@ void webKitMediaSrcFreeStream(WebKitMediaSrc* source, Stream* stream)
>         int signal = -1;
>         switch (stream->type) {
>         case WebCore::Audio:
>+            source->priv->numberOfAudioStreams--;
>             signal = SIGNAL_AUDIO_CHANGED;
>             break;
>         case WebCore::Video:
>+            source->priv->numberOfVideoStreams--;
>             signal = SIGNAL_VIDEO_CHANGED;
>             break;
>         case WebCore::Text:
>+            source->priv->numberOfTextStreams--;
>             signal = SIGNAL_TEXT_CHANGED;
>             break;
>         default:

Actions: View | Formatted Diff | Diff

Attachments on bug 185242: 339498 | 339534