Summary: | [WPE][GTK][STABLE] Remove sandbox APIs from 2.24 branch | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Michael Catanzaro <mcatanzaro> | ||||||
Component: | WebKitGTK | Assignee: | Michael Catanzaro <mcatanzaro> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | bugs-noreply, cgarcia, mcatanzaro, pgriffis | ||||||
Priority: | P2 | ||||||||
Version: | WebKit Nightly Build | ||||||||
Hardware: | PC | ||||||||
OS: | Linux | ||||||||
See Also: | https://bugs.webkit.org/show_bug.cgi?id=194274 | ||||||||
Attachments: |
|
Description
Michael Catanzaro
2019-02-12 12:54:06 PST
Created attachment 361824 [details]
Patch
Comment on attachment 361824 [details]
Patch
Not only the APIs, we should remove the implementation as well, we don't want dead code in the stable branch
I think all the code is guarded by ENABLE(BUBBLEWRAP_SANDBOX), so it should not be built. I can remove it all if you want, but there's probably no benefit, and there's a small cost in making backports harder. (In reply to Michael Catanzaro from comment #3) > I think all the code is guarded by ENABLE(BUBBLEWRAP_SANDBOX), so it should > not be built. I can remove it all if you want, but there's probably no > benefit, and there's a small cost in making backports harder. We are not going to backport any sandbox fix anyway. Created attachment 361925 [details]
Patch
Committed r241540: <https://trac.webkit.org/changeset/241540> |