Bug 241271 - Enable HEIC decoding for all non-browser apps on macOS
Summary: Enable HEIC decoding for all non-browser apps on macOS
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Misc. (show other bugs)
Version: Safari 15
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Per Arne Vollan
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2022-06-03 09:20 PDT by Per Arne Vollan
Modified: 2022-06-06 07:21 PDT (History)
2 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Per Arne Vollan 2022-06-03 09:20:04 PDT 
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.
Comment 1 Per Arne Vollan 2022-06-03 09:23:43 PDT 
Pull request: https://github.com/WebKit/WebKit/pull/1276
Comment 2 Per Arne Vollan 2022-06-03 09:25:39 PDT 
<rdar://92983791>
Comment 3 EWS 2022-06-03 14:22:04 PDT 
Committed r295234 (251289@main): <https://commits.webkit.org/251289@main>

Reviewed commits have been landed. Closing PR #1276 and removing active labels.
Comment 4 Alexey Proskuryakov 2022-06-05 13:52:16 PDT 
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.
Comment 5 Per Arne Vollan 2022-06-06 07:21:15 PDT 
(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!