Title: [271576] trunk/Source/WebCore
Revision
271576
Author
[email protected]
Date
2021-01-18 02:24:25 -0800 (Mon, 18 Jan 2021)

Log Message

[WPE][GTK] Mark that the AminatedBackingStoreClient has changed only when layer requires it
https://bugs.webkit.org/show_bug.cgi?id=220590

Reviewed by Philippe Normand.

Mark AnimatedBackingStoreClient instance as changed when it really has changed. When the layer
doesn't require a client, make it nullptr and don't modify it anymore.

* platform/graphics/texmap/coordinated/CoordinatedGraphicsLayer.cpp:
(WebCore::CoordinatedGraphicsLayer::flushCompositingStateForThisLayerOnly):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (271575 => 271576)


--- trunk/Source/WebCore/ChangeLog	2021-01-18 08:40:55 UTC (rev 271575)
+++ trunk/Source/WebCore/ChangeLog	2021-01-18 10:24:25 UTC (rev 271576)
@@ -1,3 +1,16 @@
+2021-01-18  Miguel Gomez  <[email protected]>
+
+        [WPE][GTK] Mark that the AminatedBackingStoreClient has changed only when layer requires it
+        https://bugs.webkit.org/show_bug.cgi?id=220590
+
+        Reviewed by Philippe Normand.
+
+        Mark AnimatedBackingStoreClient instance as changed when it really has changed. When the layer
+        doesn't require a client, make it nullptr and don't modify it anymore.
+
+        * platform/graphics/texmap/coordinated/CoordinatedGraphicsLayer.cpp:
+        (WebCore::CoordinatedGraphicsLayer::flushCompositingStateForThisLayerOnly):
+
 2021-01-18  Youenn Fablet  <[email protected]>
 
         WebRTC live Opus audio stream stutters

Modified: trunk/Source/WebCore/platform/graphics/texmap/coordinated/CoordinatedGraphicsLayer.cpp (271575 => 271576)


--- trunk/Source/WebCore/platform/graphics/texmap/coordinated/CoordinatedGraphicsLayer.cpp	2021-01-18 08:40:55 UTC (rev 271575)
+++ trunk/Source/WebCore/platform/graphics/texmap/coordinated/CoordinatedGraphicsLayer.cpp	2021-01-18 10:24:25 UTC (rev 271576)
@@ -853,9 +853,13 @@
         if (!m_animatedBackingStoreHost)
             m_animatedBackingStoreHost = AnimatedBackingStoreHost::create(*this);
         m_nicosia.animatedBackingStoreClient = CoordinatedAnimatedBackingStoreClient::create(m_animatedBackingStoreHost.copyRef(), m_coordinator->visibleContentsRect(), { }, m_size, effectiveContentsScale());
+        m_nicosia.delta.animatedBackingStoreClientChanged = true;
+    } else  {
+        if (m_nicosia.animatedBackingStoreClient) {
+            m_nicosia.animatedBackingStoreClient = nullptr;
+            m_nicosia.delta.animatedBackingStoreClientChanged = true;
+        }
     }
-    // Each layer flush changes the AnimatedBackingStoreClient, being it null or a real one.
-    m_nicosia.delta.animatedBackingStoreClientChanged = true;
 
     // Determine image backing presence according to the composited image source.
     if (m_compositedNativeImage) {
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to