Summary: | [Cocoa] Send sandbox extensions for Network Extension services in load parameters | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
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: | P2 | Keywords: | InRadar | ||||||||
Version: | WebKit Nightly Build | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
Attachments: |
|
Description
Per Arne Vollan
2021-02-22 12:28:38 PST
Created attachment 421226 [details]
Patch
Comment on attachment 421226 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=421226&action=review r=me > Source/WebKit/ChangeLog:10 > + policy decision is made, but that is not soon enough in all cases. They should also be sent as part of the load parameters. Should we remove the policy decision part, since we are doing it at load time now? Created attachment 421233 [details]
Patch
Created attachment 421234 [details]
Patch
(In reply to Brent Fulgham from comment #3) > Comment on attachment 421226 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=421226&action=review > > r=me > > > Source/WebKit/ChangeLog:10 > > + policy decision is made, but that is not soon enough in all cases. They should also be sent as part of the load parameters. > > Should we remove the policy decision part, since we are doing it at load > time now? I believe this is still needed, since the order of events can vary. Thanks for reviewing! Committed r273286: <https://commits.webkit.org/r273286> All reviewed patches have been landed. Closing bug and clearing flags on attachment 421234 [details]. |