Bug 206021

Summary: Remove com.apple.locationd.registration from the WebContent sandbox
Product: WebKit Reporter: Brent Fulgham <bfulgham>
Component: WebKit2Assignee: Brent Fulgham <bfulgham>
Status: RESOLVED FIXED    
Severity: Normal CC: bfulgham, pvollan, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch pvollan: review+

Brent Fulgham
Reported 2020-01-09 11:46:50 PST
Now that we generate a dynamic extension for 'com.apple.locationd.registration', we should remove the blanket allow rule from the sandbox.
Attachments
Patch (2.45 KB, patch)
2020-01-09 12:16 PST, Brent Fulgham 		no flags
DetailsFormatted DiffDiff
Patch (5.69 KB, patch)
2020-01-09 17:26 PST, Brent Fulgham 		pvollan: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2020-01-09 11:47:10 PST
<rdar://problem/58451384>
Brent Fulgham
Comment 2 2020-01-09 12:16:52 PST
Created attachment 387253 [details] Patch
Per Arne Vollan
Comment 3 2020-01-09 12:56:50 PST
Should we also add a test?
Brent Fulgham
Comment 4 2020-01-09 14:23:27 PST
(In reply to Per Arne Vollan from comment #3) > Should we also add a test? Good point!
Brent Fulgham
Comment 5 2020-01-09 17:26:41 PST
Created attachment 387295 [details] Patch
Per Arne Vollan
Comment 6 2020-01-09 17:38:38 PST
Comment on attachment 387295 [details] Patch Nice! R=me.
Brent Fulgham
Comment 7 2020-01-10 12:39:38 PST
Committed r254363: <https://trac.webkit.org/changeset/254363>
Brent Fulgham
Comment 8 2020-01-10 12:40:44 PST
Note: The test failures are due to bot issues: api-mac: The change to the iOS sandbox does not impact this system. Mac-debug-wk1: The change to the iOS sandbox does not impact this system. api-ios: API tests are run on simulator, which does not use the iOS sandbox.
Note You need to log in before you can comment on or make changes to this bug.