Bug 211238

Summary:

[iOS] Every running WebContent process should be granted access to frontboard services when Accessibility is enabled

Product:

WebKit

Reporter:

Per Arne Vollan <pvollan>

Component:

WebKit Misc.

Assignee:

Per Arne Vollan <pvollan>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

bfulgham, webkit-bug-importer

Priority:

Keywords:

InRadar

Version:

WebKit Nightly Build

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Patch	none

Per Arne Vollan

Reported 2020-04-30 09:35:35 PDT

Currently, every WebContent process is granted access to frontboard services if Accessibility is enabled at the time of startup. However, WebContent processes running at the time when Accessibility is enabled are not granted access, which is a bug.

Attachments
Patch (7.58 KB, patch) 2020-04-30 10:06 PDT, Per Arne Vollan	no flags	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Radar WebKit Bug Importer

Comment 1 2020-04-30 09:35:57 PDT

<rdar://problem/62656173>

Per Arne Vollan

Comment 2 2020-04-30 10:06:18 PDT

Created attachment 398059 [details] Patch

Daniel Bates

Comment 3 2020-05-01 13:02:51 PDT

Comment on attachment 398059 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=398059&action=review Patch looks good. > Tools/TestWebKitAPI/Tests/WebKit/EnableAccessibility.mm:87 > + ASSERT_TRUE(sandboxAccess()); Ok as is. No change needed. Optimal patch uses EXPECT_TRUE() to future proof this code should any more asserts be added.

Per Arne Vollan

Comment 4 2020-05-01 13:06:27 PDT

Comment on attachment 398059 [details] Patch Thanks for reviewing!

EWS

Comment 5 2020-05-01 13:30:25 PDT

Committed r261016: <https://trac.webkit.org/changeset/261016> All reviewed patches have been landed. Closing bug and clearing flags on attachment 398059 [details].

Note You need to log in before you can comment on or make changes to this bug.