Attachment 371949 Details for Bug 198377 – Patch

[patch] Patch

bug-198377-20190612142924.patch (text/plain), 1.83 KB, created by Philippe Normand on 2019-06-12 06:29:25 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Philippe Normand

Created: 2019-06-12 06:29:25 PDT

Size: 1.83 KB

patch

obsolete

>Subversion Revision: 246349
>diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
>index 4a275d05a3013ee7d97a8957a7b907ca116c7d3f..7a3fed3c5b98238183d0ada2b8dbc26923d47271 100644
>--- a/Source/WebCore/ChangeLog
>+++ b/Source/WebCore/ChangeLog
>@@ -1,3 +1,14 @@
>+2019-06-12  Philippe Normand  <pnormand@igalia.com>
>+
>+        [GStreamer] HLS stream slow start
>+        https://bugs.webkit.org/show_bug.cgi?id=198377
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        * platform/graphics/gstreamer/WebKitWebSourceGStreamer.cpp:
>+        (webKitWebSrcCreate): Cut down the adapter condition to 200 ms to
>+        improve start-up times for HLS playback.
>+
> 2019-06-11  Wenson Hsieh  <wenson_hsieh@apple.com>
> 
>         [iOS] Idempotent text autosizing needs to react properly to viewport changes
>diff --git a/Source/WebCore/platform/graphics/gstreamer/WebKitWebSourceGStreamer.cpp b/Source/WebCore/platform/graphics/gstreamer/WebKitWebSourceGStreamer.cpp
>index 3f6c2f36e1aeaea8fdce69256deabebf650152ff..cff19ab942864f054117941c2e5cf731875f8d20 100644
>--- a/Source/WebCore/platform/graphics/gstreamer/WebKitWebSourceGStreamer.cpp
>+++ b/Source/WebCore/platform/graphics/gstreamer/WebKitWebSourceGStreamer.cpp
>@@ -389,7 +389,9 @@ static GstFlowReturn webKitWebSrcCreate(GstPushSrc* pushSrc, GstBuffer** buffer)
>         unsigned retries = 0;
>         size_t available = gst_adapter_available_fast(priv->adapter.get());
>         while (available < size && !isAdapterDrained) {
>-            priv->adapterCondition.waitFor(priv->adapterLock, Seconds(1));
>+            priv->adapterCondition.waitFor(priv->adapterLock, 200_ms, [&] {
>+                return gst_adapter_available_fast(priv->adapter.get()) >= size;
>+            });
>             retries++;
>             available = gst_adapter_available_fast(priv->adapter.get());
>             if (available && available < size)

Flags:

calvaris: review+

Actions: View | Formatted Diff | Diff

Attachments on bug 198377: 371949