[webkit-reviews] review granted: [Bug 240334] Add logging when taking a process assertion synchronously : [Attachment 459214] Patch

[bugzilla-daemon at webkit.org]

Chris Dumez <cdumez at apple.com> has granted youenn fablet <youennf at gmail.com>'s
request for review:
Bug 240334: Add logging when taking a process assertion synchronously
https://bugs.webkit.org/show_bug.cgi?id=240334

Attachment 459214: Patch

https://bugs.webkit.org/attachment.cgi?id=459214&action=review




--- Comment #2 from Chris Dumez <cdumez at apple.com> ---
Comment on attachment 459214
  --> https://bugs.webkit.org/attachment.cgi?id=459214
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: