Summary: | [Cocoa] Delay issuing ManagedSession & Network Extension sandbox extensions until a load is actually issued | ||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Chris Dumez <cdumez> | ||||||||||||||
Component: | WebKit2 | Assignee: | Chris Dumez <cdumez> | ||||||||||||||
Status: | RESOLVED FIXED | ||||||||||||||||
Severity: | Normal | CC: | bfulgham, ggaren, pvollan, webkit-bug-importer | ||||||||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||||||||
Version: | WebKit Nightly Build | ||||||||||||||||
Hardware: | Unspecified | ||||||||||||||||
OS: | Unspecified | ||||||||||||||||
Attachments: |
|
Description
Chris Dumez
2020-06-19 16:22:22 PDT
Created attachment 402349 [details]
Patch
Comment on attachment 402349 [details]
Patch
R=me, after fixing compile issue.
Created attachment 402353 [details]
Patch
Comment on attachment 402353 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=402353&action=review > Source/WebKit/UIProcess/WebProcessProxy.h:588 > + bool m_hasNetworkExtensionSandboxAccess { false }; Should this be inside #if PLATFORM(COCOA)? (In reply to Per Arne Vollan from comment #5) > Comment on attachment 402353 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=402353&action=review > > > Source/WebKit/UIProcess/WebProcessProxy.h:588 > > + bool m_hasNetworkExtensionSandboxAccess { false }; > > Should this be inside #if PLATFORM(COCOA)? Oh right. I will add it. Created attachment 402356 [details]
Patch
Created attachment 402362 [details]
Patch
Tools/Scripts/svn-apply failed to apply attachment 402362 [details] to trunk.
Please resolve the conflicts and upload a new patch.
Created attachment 402368 [details]
Patch
Patch 402368 does not build Created attachment 402372 [details]
Patch
Committed r263313: <https://trac.webkit.org/changeset/263313> All reviewed patches have been landed. Closing bug and clearing flags on attachment 402372 [details]. |