Bug 173299

Summary: Include PID in some UIProcess-side release logging that relates to a particular WebProcess
Product: WebKit Reporter: Chris Dumez <cdumez>
Component: WebKit2Assignee: Chris Dumez <cdumez>
Status: RESOLVED FIXED    
Severity: Normal CC: achristensen, beidson, ggaren, kling, koivisto, rniwa, sam
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch rniwa: review+

Chris Dumez
Reported 2017-06-12 19:32:31 PDT
Include PID in some UIProcess-side release logging that relates to a particular WebProcess.
Attachments
Patch (5.64 KB, patch)
2017-06-12 19:33 PDT, Chris Dumez 		rniwa: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Chris Dumez
Comment 1 2017-06-12 19:33:01 PDT
Created attachment 312742 [details] Patch
Ryosuke Niwa
Comment 2 2017-06-13 13:55:22 PDT
Comment on attachment 312742 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=312742&action=review > Source/WebKit2/UIProcess/WebProcessProxy.cpp:1217 > + RELEASE_LOG_ERROR(PerformanceLogging, "%p - WebProcessProxy::didExceedActiveMemoryLimit() Terminating WebProcess with pid %d that has exceeded the active memory limit", this, processIdentifier()); Can we be consist and use (pid: %d)?
Chris Dumez
Comment 3 2017-06-13 13:58:20 PDT
Committed r218201: <http://trac.webkit.org/changeset/218201>
Note You need to log in before you can comment on or make changes to this bug.