Title: [293822] trunk/Source/WebKit
Revision
293822
Author
[email protected]
Date
2022-05-04 23:36:37 -0700 (Wed, 04 May 2022)

Log Message

Empty remote backing store collection prepare starts up GPU process
https://bugs.webkit.org/show_bug.cgi?id=240067

Patch by Kimmo Kinnunen <[email protected]> on 2022-05-04
Reviewed by Simon Fraser.

Avoid starting up GPU process by using an early return if
prepareBuffersForDisplay input array is empty.

* WebProcess/GPU/graphics/RemoteRenderingBackendProxy.cpp:
(WebKit::RemoteRenderingBackendProxy::prepareBuffersForDisplay):

Modified Paths

Diff

Modified: trunk/Source/WebKit/ChangeLog (293821 => 293822)


--- trunk/Source/WebKit/ChangeLog	2022-05-05 05:48:33 UTC (rev 293821)
+++ trunk/Source/WebKit/ChangeLog	2022-05-05 06:36:37 UTC (rev 293822)
@@ -1,3 +1,16 @@
+2022-05-04  Kimmo Kinnunen  <[email protected]>
+
+        Empty remote backing store collection prepare starts up GPU process
+        https://bugs.webkit.org/show_bug.cgi?id=240067
+
+        Reviewed by Simon Fraser.
+
+        Avoid starting up GPU process by using an early return if
+        prepareBuffersForDisplay input array is empty.
+
+        * WebProcess/GPU/graphics/RemoteRenderingBackendProxy.cpp:
+        (WebKit::RemoteRenderingBackendProxy::prepareBuffersForDisplay):
+
 2022-05-04  Tim Horton  <[email protected]>
 
         REGRESSION (r293716): macCatalyst WebKit build fails; overlapping content at /System/Library/FeatureFlags/Domain/WebKit.plist

Modified: trunk/Source/WebKit/WebProcess/GPU/graphics/RemoteRenderingBackendProxy.cpp (293821 => 293822)


--- trunk/Source/WebKit/WebProcess/GPU/graphics/RemoteRenderingBackendProxy.cpp	2022-05-05 05:48:33 UTC (rev 293821)
+++ trunk/Source/WebKit/WebProcess/GPU/graphics/RemoteRenderingBackendProxy.cpp	2022-05-05 06:36:37 UTC (rev 293822)
@@ -273,6 +273,9 @@
 
 auto RemoteRenderingBackendProxy::prepareBuffersForDisplay(const Vector<LayerPrepareBuffersData>& prepareBuffersInput) -> Vector<SwapBuffersResult>
 {
+    if (prepareBuffersInput.isEmpty())
+        return { };
+
     auto bufferIdentifier = [](ImageBuffer* buffer) -> std::optional<RenderingResourceIdentifier> {
         if (!buffer)
             return std::nullopt;
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to