Bug 181619 - [iOS] Remove unneeded accessibility-related sandbox rules
Summary: [iOS] Remove unneeded accessibility-related sandbox rules
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit2 (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Brent Fulgham
URL:
Keywords: InRadar
Depends on:
Blocks: 190019
  Show dependency tree / graph
 
Reported: 2018-01-12 15:49 PST by Brent Fulgham
Modified: 2018-09-26 17:17 PDT (History)
3 users (show)

See Also:

Attachments
Patch (3.13 KB, patch)
2018-01-12 15:53 PST, Brent Fulgham 		eric.carlson: review+
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Brent Fulgham 2018-01-12 15:49:35 PST 
A number of accessibility features are enabled by the WebContent process sandbox. These aren't needed, because Safari is the application that actually handles the accessibility user interface.

After reviewing with the AX team, we have identified a number of symbols and endpoints we can remove.
Comment 1 Radar WebKit Bug Importer 2018-01-12 15:50:19 PST 
<rdar://problem/36485356>
Comment 2 Brent Fulgham 2018-01-12 15:53:27 PST 
Created attachment 331247 [details]
Patch
Comment 3 Brent Fulgham 2018-01-12 16:24:07 PST 
I've run this revised profile locally and confirmed VoiceOver works properly in WebContent.
Comment 4 Eric Carlson 2018-01-12 16:25:48 PST 
Comment on attachment 331247 [details]
Patch

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

> Source/WebKit/ChangeLog:10
> +        not needed in the WebContent process, since 

Nit: since what?
Comment 5 Brent Fulgham 2018-01-12 16:26:26 PST 
Comment on attachment 331247 [details]
Patch

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

>> Source/WebKit/ChangeLog:10
>> +        not needed in the WebContent process, since 
> 
> Nit: since what?

Ack!
Comment 6 Brent Fulgham 2018-01-12 16:28:14 PST 
Committed r226926: <https://trac.webkit.org/changeset/226926>