Title: [276257] trunk/Source/WebKit
Revision
276257
Author
[email protected]
Date
2021-04-19 09:46:44 -0700 (Mon, 19 Apr 2021)

Log Message

Add more GPUProcess release logging to facilitate debugging
https://bugs.webkit.org/show_bug.cgi?id=224761

Reviewed by Darin Adler.

* GPUProcess/GPUProcess.cpp:
(WebKit::GPUProcess::GPUProcess):
(WebKit::GPUProcess::createGPUConnectionToWebProcess):
(WebKit::GPUProcess::removeGPUConnectionToWebProcess):
(WebKit::GPUProcess::initializeGPUProcess):

Modified Paths

Diff

Modified: trunk/Source/WebKit/ChangeLog (276256 => 276257)


--- trunk/Source/WebKit/ChangeLog	2021-04-19 16:45:58 UTC (rev 276256)
+++ trunk/Source/WebKit/ChangeLog	2021-04-19 16:46:44 UTC (rev 276257)
@@ -1,3 +1,16 @@
+2021-04-19  Chris Dumez  <[email protected]>
+
+        Add more GPUProcess release logging to facilitate debugging
+        https://bugs.webkit.org/show_bug.cgi?id=224761
+
+        Reviewed by Darin Adler.
+
+        * GPUProcess/GPUProcess.cpp:
+        (WebKit::GPUProcess::GPUProcess):
+        (WebKit::GPUProcess::createGPUConnectionToWebProcess):
+        (WebKit::GPUProcess::removeGPUConnectionToWebProcess):
+        (WebKit::GPUProcess::initializeGPUProcess):
+
 2021-04-19  Kimmo Kinnunen  <[email protected]>
 
         Enable -Wthread-safety, add attributes to custom lock classes, and provide macros to declare guards

Modified: trunk/Source/WebKit/GPUProcess/GPUProcess.cpp (276256 => 276257)


--- trunk/Source/WebKit/GPUProcess/GPUProcess.cpp	2021-04-19 16:45:58 UTC (rev 276256)
+++ trunk/Source/WebKit/GPUProcess/GPUProcess.cpp	2021-04-19 16:46:44 UTC (rev 276257)
@@ -78,6 +78,7 @@
 GPUProcess::GPUProcess(AuxiliaryProcessInitializationParameters&& parameters)
 {
     initialize(WTFMove(parameters));
+    RELEASE_LOG(Process, "%p - GPUProcess::GPUProcess:", this);
 }
 
 GPUProcess::~GPUProcess()
@@ -86,6 +87,7 @@
 
 void GPUProcess::createGPUConnectionToWebProcess(ProcessIdentifier identifier, PAL::SessionID sessionID, GPUProcessConnectionParameters&& parameters, CompletionHandler<void(Optional<IPC::Attachment>&&)>&& completionHandler)
 {
+    RELEASE_LOG(Process, "%p - GPUProcess::createGPUConnectionToWebProcess: processIdentifier=%" PRIu64, this, identifier.toUInt64());
     auto ipcConnection = createIPCConnectionPair();
     if (!ipcConnection) {
         completionHandler({ });
@@ -109,6 +111,7 @@
 
 void GPUProcess::removeGPUConnectionToWebProcess(GPUConnectionToWebProcess& connection)
 {
+    RELEASE_LOG(Process, "%p - GPUProcess::removeGPUConnectionToWebProcess: processIdentifier=%" PRIu64, this, connection.webProcessIdentifier().toUInt64());
     ASSERT(m_webProcessConnections.contains(connection.webProcessIdentifier()));
     m_webProcessConnections.remove(connection.webProcessIdentifier());
     tryExitIfUnusedAndUnderMemoryPressure();
@@ -162,6 +165,7 @@
 
 void GPUProcess::initializeGPUProcess(GPUProcessCreationParameters&& parameters)
 {
+    RELEASE_LOG(Process, "%p - GPUProcess::initializeGPUProcess:", this);
     WTF::Thread::setCurrentThreadIsUserInitiated();
     AtomString::init();
 
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to