<?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>275134</bug_id>
          
          <creation_ts>2024-06-04 16:26:24 -0700</creation_ts>
          <short_desc>Enable GraphicsContext filters for Canvas filter</short_desc>
          <delta_ts>2024-06-20 02:37:40 -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>Canvas</component>
          <version>WebKit Nightly Build</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="Said Abou-Hallawa">sabouhallawa</reporter>
          <assigned_to name="Said Abou-Hallawa">sabouhallawa</assigned_to>
          <cc>bfulgham</cc>
    
    <cc>sabouhallawa</cc>
    
    <cc>simon.fraser</cc>
    
    <cc>webkit-bug-importer</cc>
    
    <cc>zalan</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>2039770</commentid>
    <comment_count>0</comment_count>
    <who name="Said Abou-Hallawa">sabouhallawa</who>
    <bug_when>2024-06-04 16:26:24 -0700</bug_when>
    <thetext>GraphicsContext filters were disabled for the canvas filter in 278000@main. The canvas drop-shadow filter was drawn upside down. So software filters were used instead to make the layout test pass.

It turned out all we need is to flip the sign of the vertical offset of the drop-shadow. This is what CanvasRenderingContext2DBase::applyShadow() does when it sets GraphicsContext drop-shadow by calling:

    c-&gt;setDropShadow({ { width, -height }, state().shadowBlur, state().shadowColor, ShadowRadiusMode::Legacy });</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2039771</commentid>
    <comment_count>1</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2024-06-04 16:26:51 -0700</bug_when>
    <thetext>&lt;rdar://problem/129249264&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2039773</commentid>
    <comment_count>2</comment_count>
    <who name="Said Abou-Hallawa">sabouhallawa</who>
    <bug_when>2024-06-04 16:32:59 -0700</bug_when>
    <thetext>Pull request: https://github.com/WebKit/WebKit/pull/29522</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2042184</commentid>
    <comment_count>3</comment_count>
    <who name="EWS">ews-feeder</who>
    <bug_when>2024-06-20 02:37:38 -0700</bug_when>
    <thetext>Committed 280195@main (8e66e149a07d): &lt;https://commits.webkit.org/280195@main&gt;

Reviewed commits have been landed. Closing PR #29522 and removing active labels.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>