NEW219592
[macOS] Create sandbox extension to HI service when UniversalAccess zoom is enabled 
https://bugs.webkit.org/show_bug.cgi?id=219592
Summary [macOS] Create sandbox extension to HI service when UniversalAccess zoom is e...
Per Arne Vollan
Reported 2020-12-07 02:04:46 PST
When UniversalAccess zoom is enabled on macOS, access to the service 'com.apple.hiservices-xpcservice' is needed in the WebContent process, and a sandbox extension should be created.
Attachments
Patch (5.36 KB, patch)
2020-12-07 02:09 PST, Per Arne Vollan 		no flags
DetailsFormatted DiffDiff
Patch (5.34 KB, patch)
2020-12-07 04:05 PST, Per Arne Vollan 		no flags
DetailsFormatted DiffDiff
Patch (5.36 KB, patch)
2020-12-07 07:50 PST, Per Arne Vollan 		pvollan: review?
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Per Arne Vollan
Comment 1 2020-12-07 02:09:25 PST
Created attachment 415540 [details] Patch
Per Arne Vollan
Comment 2 2020-12-07 04:05:54 PST
Created attachment 415549 [details] Patch
Per Arne Vollan
Comment 3 2020-12-07 07:50:04 PST
Created attachment 415555 [details] Patch
Radar WebKit Bug Importer
Comment 4 2020-12-07 07:50:55 PST
<rdar://problem/72047078>
Sam Weinig
Comment 5 2020-12-07 08:44:30 PST
Comment on attachment 415555 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=415555&action=review > Source/WebKit/ChangeLog:9 > + When UniversalAccess zoom is enabled on macOS, access to the service 'com.apple.hiservices-xpcservice' is needed in the WebContent process, > + and a sandbox extension should be created. What in the WebContent process requires this? Is it something that could be proxied to the UIProcess instead?
Per Arne Vollan
Comment 6 2020-12-08 03:48:21 PST
(In reply to Sam Weinig from comment #5) > Comment on attachment 415555 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=415555&action=review > > > Source/WebKit/ChangeLog:9 > > + When UniversalAccess zoom is enabled on macOS, access to the service 'com.apple.hiservices-xpcservice' is needed in the WebContent process, > > + and a sandbox extension should be created. > > What in the WebContent process requires this? Is it something that could be > proxied to the UIProcess instead? That is a good point. This is required when calling UAZoomChangeFocus in FrameSelection::notifyAccessibilityForSelectionChange. I have uploaded a patch where this is proxied to the UI process in https://bugs.webkit.org/show_bug.cgi?id=219635. Thanks for reviewing!
Note You need to log in before you can comment on or make changes to this bug.