Title: [294222] trunk/Source/WebKit
Revision
294222
Author
you...@apple.com
Date
2022-05-16 00:14:59 -0700 (Mon, 16 May 2022)

Log Message

Add logging when taking a process assertion synchronously
https://bugs.webkit.org/show_bug.cgi?id=240334

Reviewed by Chris Dumez.

No observable change of behavior.

* UIProcess/ios/ProcessAssertionIOS.mm:
(WebKit::ProcessAssertion::acquireSync):

Modified Paths

Diff

Modified: trunk/Source/WebKit/ChangeLog (294221 => 294222)


--- trunk/Source/WebKit/ChangeLog	2022-05-16 06:37:21 UTC (rev 294221)
+++ trunk/Source/WebKit/ChangeLog	2022-05-16 07:14:59 UTC (rev 294222)
@@ -1,3 +1,15 @@
+2022-05-16  Youenn Fablet  <you...@apple.com>
+
+        Add logging when taking a process assertion synchronously
+        https://bugs.webkit.org/show_bug.cgi?id=240334
+
+        Reviewed by Chris Dumez.
+
+        No observable change of behavior.
+
+        * UIProcess/ios/ProcessAssertionIOS.mm:
+        (WebKit::ProcessAssertion::acquireSync):
+
 2022-05-10  Yusuke Suzuki  <ysuz...@apple.com>
 
         Rename EventTrackingRegions::Event to EventTrackingRegions::EventType

Modified: trunk/Source/WebKit/UIProcess/ios/ProcessAssertionIOS.mm (294221 => 294222)


--- trunk/Source/WebKit/UIProcess/ios/ProcessAssertionIOS.mm	2022-05-16 06:37:21 UTC (rev 294221)
+++ trunk/Source/WebKit/UIProcess/ios/ProcessAssertionIOS.mm	2022-05-16 07:14:59 UTC (rev 294222)
@@ -367,15 +367,17 @@
 
 void ProcessAssertion::acquireSync()
 {
+    RELEASE_LOG(ProcessSuspension, "%p - ProcessAssertion::acquireSync Trying to take RBS assertion '%{public}s' for process with PID=%d", this, m_reason.utf8().data(), m_pid);
+
     NSError *acquisitionError = nil;
     if (![m_rbsAssertion acquireWithError:&acquisitionError]) {
-        RELEASE_LOG_ERROR(ProcessSuspension, "%p - ProcessAssertion: Failed to acquire RBS assertion '%{public}s' for process with PID=%d, error: %{public}@", this, m_reason.utf8().data(), m_pid, acquisitionError);
+        RELEASE_LOG_ERROR(ProcessSuspension, "%p - ProcessAssertion::acquireSync Failed to acquire RBS assertion '%{public}s' for process with PID=%d, error: %{public}@", this, m_reason.utf8().data(), m_pid, acquisitionError);
         RunLoop::main().dispatch([weakThis = WeakPtr { *this }] {
             if (weakThis)
                 weakThis->processAssertionWasInvalidated();
         });
     } else
-        RELEASE_LOG(ProcessSuspension, "%p - ProcessAssertion: Successfully took RBS assertion '%{public}s' for process with PID=%d", this, m_reason.utf8().data(), m_pid);
+        RELEASE_LOG(ProcessSuspension, "%p - ProcessAssertion::acquireSync Successfully took RBS assertion '%{public}s' for process with PID=%d", this, m_reason.utf8().data(), m_pid);
 }
 
 ProcessAssertion::~ProcessAssertion()
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to