-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weβll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Disable telemetry for known notifications #28677
Disable telemetry for known notifications #28677
Conversation
EWS run on previous version of this PR (hash 576baac) |
plistbuddy Add :com.apple.private.darwin-notification.introspect:17 string com.apple.system.lowpowermode | ||
plistbuddy Add :com.apple.private.darwin-notification.introspect:18 string com.apple.system.timezone | ||
plistbuddy Add :com.apple.private.darwin-notification.introspect:19 string com.apple.zoomwindow | ||
plistbuddy Add :com.apple.private.darwin-notification.introspect:8 string com.apple.accessibility.cache.guided.access |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This seems not in const Vector<ASCIILiteral> notificationMessages
, is it forwarded?
@pvollan Any update on this? |
Ah, sorry for the delayed response. This is actually a notification that is used only in-process, so I don't believe we need to forward this one. And the sandbox change should not be required either, then. I will update the patch. Thanks for reviewing, all! |
576baac
to
8cd5ddf
Compare
EWS run on previous version of this PR (hash 8cd5ddf) |
8cd5ddf
to
632a68e
Compare
EWS run on current version of this PR (hash 632a68e) |
https://bugs.webkit.org/show_bug.cgi?id=274279 rdar://128213400 Reviewed by Sihui Liu. Disable telemetry for notifications that we know are being observed in the WebContent process. For in-process notifications, we do not need to forward them from the UI process. * Source/WebKit/Resources/SandboxProfiles/ios/com.apple.WebKit.WebContent.sb.in: * Source/WebKit/Scripts/process-entitlements.sh: * Source/WebKit/UIProcess/Cocoa/WebProcessPoolCocoa.mm: (WebKit::WebProcessPool::registerNotificationObservers): * Source/WebKit/WebProcess/com.apple.WebProcess.sb.in: Canonical link: https://commits.webkit.org/279065@main
632a68e
to
00575a1
Compare
Committed 279065@main (00575a1): https://commits.webkit.org/279065@main Reviewed commits have been landed. Closing PR #28677 and removing active labels. |
00575a1
632a68e