Title: [217936] trunk/Source/WebCore
Revision
217936
Author
jer.no...@apple.com
Date
2017-06-08 10:57:08 -0700 (Thu, 08 Jun 2017)

Log Message

YouTube audio stutters when page changes visibility.
https://bugs.webkit.org/show_bug.cgi?id=173102

Reviewed by Eric Carlson.

Don't change renderers when the visibility changes; only use the decompression session
when we were explicitly asked to paint into an accelerated surface.

* platform/graphics/avfoundation/objc/MediaPlayerPrivateMediaSourceAVFObjC.mm:
(WebCore::MediaPlayerPrivateMediaSourceAVFObjC::acceleratedRenderingStateChanged):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (217935 => 217936)


--- trunk/Source/WebCore/ChangeLog	2017-06-08 17:51:47 UTC (rev 217935)
+++ trunk/Source/WebCore/ChangeLog	2017-06-08 17:57:08 UTC (rev 217936)
@@ -1,5 +1,18 @@
 2017-06-08  Jer Noble  <jer.no...@apple.com>
 
+        YouTube audio stutters when page changes visibility.
+        https://bugs.webkit.org/show_bug.cgi?id=173102
+
+        Reviewed by Eric Carlson.
+
+        Don't change renderers when the visibility changes; only use the decompression session
+        when we were explicitly asked to paint into an accelerated surface.
+
+        * platform/graphics/avfoundation/objc/MediaPlayerPrivateMediaSourceAVFObjC.mm:
+        (WebCore::MediaPlayerPrivateMediaSourceAVFObjC::acceleratedRenderingStateChanged):
+
+2017-06-08  Jer Noble  <jer.no...@apple.com>
+
         Clients of the WK2 C-API don't have their mediaContentTypesRequiringHardwareSupport setting initialized correctly.
         https://bugs.webkit.org/show_bug.cgi?id=173091
 

Modified: trunk/Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateMediaSourceAVFObjC.mm (217935 => 217936)


--- trunk/Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateMediaSourceAVFObjC.mm	2017-06-08 17:51:47 UTC (rev 217935)
+++ trunk/Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateMediaSourceAVFObjC.mm	2017-06-08 17:57:08 UTC (rev 217936)
@@ -635,7 +635,7 @@
 
 void MediaPlayerPrivateMediaSourceAVFObjC::acceleratedRenderingStateChanged()
 {
-    if (!m_hasBeenAskedToPaintGL && m_player->visible() && m_player->client().mediaPlayerRenderingCanBeAccelerated(m_player)) {
+    if (!m_hasBeenAskedToPaintGL && m_player->client().mediaPlayerRenderingCanBeAccelerated(m_player)) {
         destroyDecompressionSession();
         ensureLayer();
     } else {
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to