Bug 241271
| Summary: | Enable HEIC decoding for all non-browser apps on macOS | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Per Arne Vollan <pvollan> |
| Component: | WebKit Misc. | Assignee: | Per Arne Vollan <pvollan> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | ap, webkit-bug-importer |
| Priority: | P2 | Keywords: | InRadar |
| Version: | Safari 15 | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Per Arne Vollan
Enable HEIC decoding for all non-browser apps on macOS, since this capability is also needed for other apps besides Mail. We do not want to enable it for Web browsers yet, since this currently requries an unconditional sandbox extension for trustd.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Per Arne Vollan
Pull request: https://github.com/WebKit/WebKit/pull/1276
Per Arne Vollan
<rdar://92983791>
EWS
Committed r295234 (251289@main): <https://commits.webkit.org/251289@main>
Reviewed commits have been landed. Closing PR #1276 and removing active labels.
Alexey Proskuryakov
This is probably OK, as the capability is so minor, and any UI process has it anyway. But it would be cleaner and generally safer to have the check in the XPC service before it enters the sandbox.
Per Arne Vollan
(In reply to Alexey Proskuryakov from comment #4)
> This is probably OK, as the capability is so minor, and any UI process has
> it anyway. But it would be cleaner and generally safer to have the check in
> the XPC service before it enters the sandbox.
Ah, yes, that is a good point! It would definitely be better to check this in the WebContent process itself.
Thanks for reviewing!