<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>241271</bug_id>
          
          <creation_ts>2022-06-03 09:20:04 -0700</creation_ts>
          <short_desc>Enable HEIC decoding for all non-browser apps on macOS</short_desc>
          <delta_ts>2022-06-06 07:21:15 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>WebKit Misc.</component>
          <version>Safari 15</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>InRadar</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Per Arne Vollan">pvollan</reporter>
          <assigned_to name="Per Arne Vollan">pvollan</assigned_to>
          <cc>ap</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1873939</commentid>
    <comment_count>0</comment_count>
    <who name="Per Arne Vollan">pvollan</who>
    <bug_when>2022-06-03 09:20:04 -0700</bug_when>
    <thetext>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.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1873940</commentid>
    <comment_count>1</comment_count>
    <who name="Per Arne Vollan">pvollan</who>
    <bug_when>2022-06-03 09:23:43 -0700</bug_when>
    <thetext>Pull request: https://github.com/WebKit/WebKit/pull/1276</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1873941</commentid>
    <comment_count>2</comment_count>
    <who name="Per Arne Vollan">pvollan</who>
    <bug_when>2022-06-03 09:25:39 -0700</bug_when>
    <thetext>&lt;rdar://92983791&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1874029</commentid>
    <comment_count>3</comment_count>
    <who name="EWS">ews-feeder</who>
    <bug_when>2022-06-03 14:22:04 -0700</bug_when>
    <thetext>Committed r295234 (251289@main): &lt;https://commits.webkit.org/251289@main&gt;

Reviewed commits have been landed. Closing PR #1276 and removing active labels.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1874198</commentid>
    <comment_count>4</comment_count>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2022-06-05 13:52:16 -0700</bug_when>
    <thetext>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.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1874300</commentid>
    <comment_count>5</comment_count>
    <who name="Per Arne Vollan">pvollan</who>
    <bug_when>2022-06-06 07:21:15 -0700</bug_when>
    <thetext>(In reply to Alexey Proskuryakov from comment #4)
&gt; This is probably OK, as the capability is so minor, and any UI process has
&gt; it anyway. But it would be cleaner and generally safer to have the check in
&gt; 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!</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>