RESOLVED FIXED 240334
Add logging when taking a process assertion synchronously 
https://bugs.webkit.org/show_bug.cgi?id=240334
Summary Add logging when taking a process assertion synchronously
youenn fablet
Reported 2022-05-12 01:55:48 PDT
Add logging when taking a process assertion synchronously
Attachments
Patch (1.69 KB, patch)
2022-05-12 02:23 PDT, youenn fablet 		cdumez: review+
DetailsFormatted DiffDiff
Patch for landing (3.02 KB, patch)
2022-05-15 23:20 PDT, youenn fablet 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
youenn fablet
Comment 1 2022-05-12 02:23:41 PDT
Created attachment 459214 [details] Patch
Chris Dumez
Comment 2 2022-05-13 07:25:28 PDT
Comment on attachment 459214 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=459214&action=review > Source/WebKit/UIProcess/ios/ProcessAssertionIOS.mm:370 > + RELEASE_LOG(ProcessSuspension, "%p - ProcessAssertion: Trying to take RBS assertion '%{public}s' for process with PID=%d", this, m_reason.utf8().data(), m_pid); Please put method name in the logging: ProcessAssertion::acquireSync: not ProcessAssertion:
youenn fablet
Comment 3 2022-05-15 23:20:07 PDT
Created attachment 459402 [details] Patch for landing
EWS
Comment 4 2022-05-16 00:16:09 PDT
Committed r294222 (250580@main): <https://commits.webkit.org/250580@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 459402 [details].
Radar WebKit Bug Importer
Comment 5 2022-05-16 00:17:13 PDT
<rdar://problem/93329781>
Note You need to log in before you can comment on or make changes to this bug.