Title: [210156] releases/WebKitGTK/webkit-2.14/Source/WebCore
Revision
210156
Author
[email protected]
Date
2016-12-27 02:43:40 -0800 (Tue, 27 Dec 2016)

Log Message

Merge r207287 - [GTK] Video playback doesn't work properly with accelerated compositing disabled
https://bugs.webkit.org/show_bug.cgi?id=163386

Patch by Miguel Gomez <[email protected]> on 2016-10-13
Reviewed by Carlos Garcia Campos.

Trigger a repaint of the player when a new frame arrives and accelerated compositing is disabled.

Covered by existent tests.

* platform/graphics/gstreamer/MediaPlayerPrivateGStreamerBase.cpp:
(WebCore::MediaPlayerPrivateGStreamerBase::triggerRepaint):

Modified Paths

Diff

Modified: releases/WebKitGTK/webkit-2.14/Source/WebCore/ChangeLog (210155 => 210156)


--- releases/WebKitGTK/webkit-2.14/Source/WebCore/ChangeLog	2016-12-27 10:33:36 UTC (rev 210155)
+++ releases/WebKitGTK/webkit-2.14/Source/WebCore/ChangeLog	2016-12-27 10:43:40 UTC (rev 210156)
@@ -1,3 +1,17 @@
+2016-10-13  Miguel Gomez  <[email protected]>
+
+        [GTK] Video playback doesn't work properly with accelerated compositing disabled
+        https://bugs.webkit.org/show_bug.cgi?id=163386
+
+        Reviewed by Carlos Garcia Campos.
+
+        Trigger a repaint of the player when a new frame arrives and accelerated compositing is disabled.
+
+        Covered by existent tests.
+
+        * platform/graphics/gstreamer/MediaPlayerPrivateGStreamerBase.cpp:
+        (WebCore::MediaPlayerPrivateGStreamerBase::triggerRepaint):
+
 2016-09-07  Carlos Garcia Campos  <[email protected]>
 
         [GTK] Crash of WebProcess on the last WebView disconnect

Modified: releases/WebKitGTK/webkit-2.14/Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamerBase.cpp (210155 => 210156)


--- releases/WebKitGTK/webkit-2.14/Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamerBase.cpp	2016-12-27 10:33:36 UTC (rev 210155)
+++ releases/WebKitGTK/webkit-2.14/Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamerBase.cpp	2016-12-27 10:43:40 UTC (rev 210156)
@@ -555,14 +555,21 @@
 
 #if USE(COORDINATED_GRAPHICS_THREADED)
 #if USE(GSTREAMER_GL)
-    pushTextureToCompositor();
+    if (m_player->client().mediaPlayerRenderingCanBeAccelerated(m_player))
+        pushTextureToCompositor();
+    else {
+        LockHolder locker(m_drawMutex);
+        m_drawTimer.startOneShot(0);
+        m_drawCondition.wait(m_drawMutex);
+    }
 #else
-    {
+    if (m_player->client().mediaPlayerRenderingCanBeAccelerated(m_player)) {
         LockHolder lock(m_drawMutex);
         if (!m_platformLayerProxy->scheduleUpdateOnCompositorThread([this] { this->pushTextureToCompositor(); }))
             return;
         m_drawCondition.wait(m_drawMutex);
-    }
+    } else
+        repaint();
 #endif
     return;
 #else
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to