Title: [295186] branches/safari-613-branch/Source/WebKit
Revision
295186
Author
[email protected]
Date
2022-06-02 23:45:35 -0700 (Thu, 02 Jun 2022)

Log Message

Cherry-pick 9a7e68adec7a. rdar://problem/93329781

    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):

    Canonical link: https://commits.webkit.org/250580@main
    git-svn-id: https://svn.webkit.org/repository/webkit/trunk@294222 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Modified Paths

Diff

Modified: branches/safari-613-branch/Source/WebKit/ChangeLog (295185 => 295186)


--- branches/safari-613-branch/Source/WebKit/ChangeLog	2022-06-03 06:45:32 UTC (rev 295185)
+++ branches/safari-613-branch/Source/WebKit/ChangeLog	2022-06-03 06:45:35 UTC (rev 295186)
@@ -1,3 +1,15 @@
+2022-05-16  Youenn Fablet  <[email protected]>
+
+        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-04-14  Wenson Hsieh  <[email protected]>
 
         [iOS] [WK2] Managed pasteboard should function for all managed domains

Modified: branches/safari-613-branch/Source/WebKit/UIProcess/ios/ProcessAssertionIOS.mm (295185 => 295186)


--- branches/safari-613-branch/Source/WebKit/UIProcess/ios/ProcessAssertionIOS.mm	2022-06-03 06:45:32 UTC (rev 295185)
+++ branches/safari-613-branch/Source/WebKit/UIProcess/ios/ProcessAssertionIOS.mm	2022-06-03 06:45:35 UTC (rev 295186)
@@ -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
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to