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

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!