[webkit-reviews] review granted: [Bug 192670] [macOS] Block coreservicesd in sandbox. : [Attachment 359280] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Jan 16 11:55:53 PST 2019


Geoffrey Garen <ggaren at apple.com> has granted Per Arne Vollan
<pvollan at apple.com>'s request for review:
Bug 192670: [macOS] Block coreservicesd in sandbox.
https://bugs.webkit.org/show_bug.cgi?id=192670

Attachment 359280: Patch

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




--- Comment #9 from Geoffrey Garen <ggaren at apple.com> ---
Comment on attachment 359280
  --> https://bugs.webkit.org/attachment.cgi?id=359280
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=359280&action=review

r=me

> Source/WebKit/Shared/ChildProcess.cpp:77
> +    initializeProcessName(parameters);

Can you manually verify that WebContent process names still show up in Activity
Monitor correctly? They should appear either as "http://example.com" for one
page per process, "http://example.com, ..." for multiple pages per process, or
"Safari WebContent (Prewarmed)" for the prewarmed PSON process.


More information about the webkit-reviews mailing list