WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
173299
Include PID in some UIProcess-side release logging that relates to a particular WebProcess
https://bugs.webkit.org/show_bug.cgi?id=173299
Summary
Include PID in some UIProcess-side release logging that relates to a particul...
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+
Details
Formatted Diff
Diff
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.
Top of Page
Format For Printing
XML
Clone This Bug