<?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>250276</bug_id>
          
          <creation_ts>2023-01-07 10:09:56 -0800</creation_ts>
          <short_desc>Adjust error-handling for invalid filter primitive references</short_desc>
          <delta_ts>2024-05-28 19:38:19 -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>SVG</component>
          <version>Safari Technology Preview</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, LayerBasedSVGEngine</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Ahmad Saleem">ahmad.saleem792</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>sabouhallawa</cc>
    
    <cc>simon.fraser</cc>
    
    <cc>webkit-bug-importer</cc>
    
    <cc>zimmermann</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1923825</commentid>
    <comment_count>0</comment_count>
    <who name="Ahmad Saleem">ahmad.saleem792</who>
    <bug_when>2023-01-07 10:09:56 -0800</bug_when>
    <thetext>Hi Team,

While going through Blink&apos;s commit, I came across another failing test case:

Test Case - https://jsfiddle.net/xothm1vc/show

^ Safari 16.2 &amp; STP160 only show two green rectangles. While Firefox Nightly 110 and Chrome Canary 111 show full green bar rectangle, not just two slots.

Blink Commit - https://chromium.googlesource.com/chromium/blink/+/ffa39ae78904542a2864e2cb7d1c16c24ee2324e

I haven&apos;t checked whether it is 1-1 merge or not but just wanted to raise for tracking.

Thanks!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1925583</commentid>
    <comment_count>1</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2023-01-14 10:10:15 -0800</bug_when>
    <thetext>&lt;rdar://problem/104262208&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1953502</commentid>
    <comment_count>2</comment_count>
    <who name="Ahmad Saleem">ahmad.saleem792</who>
    <bug_when>2023-05-07 04:17:26 -0700</bug_when>
    <thetext>WebKit ToT (263769@main) get full bar but half is &apos;red&apos; while STP169 still shows only two rectangles.

Just wanted to highlight small-progression.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1953503</commentid>
    <comment_count>3</comment_count>
    <who name="Ahmad Saleem">ahmad.saleem792</who>
    <bug_when>2023-05-07 04:19:39 -0700</bug_when>
    <thetext>This commit removed &apos;SVGFilterBuilder.cpp&apos; in WebKit - https://commits.webkit.org/251522@main</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1953618</commentid>
    <comment_count>4</comment_count>
    <who name="Ahmad Saleem">ahmad.saleem792</who>
    <bug_when>2023-05-08 02:29:16 -0700</bug_when>
    <thetext>(In reply to Ahmad Saleem from comment #2)
&gt; WebKit ToT (263769@main) get full bar but half is &apos;red&apos; while STP169 still
&gt; shows only two rectangles.
&gt; 
&gt; Just wanted to highlight small-progression.

This is with LBSE turned on.

LBSE off is still same.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1991316</commentid>
    <comment_count>5</comment_count>
    <who name="Ahmad Saleem">ahmad.saleem792</who>
    <bug_when>2023-11-09 15:26:19 -0800</bug_when>
    <thetext>Something to do here ?: https://searchfox.org/wubkat/source/Source/WebCore/svg/graphics/filters/SVGFilterGraph.h#64</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2034152</commentid>
    <comment_count>6</comment_count>
    <who name="Ahmad Saleem">ahmad.saleem792</who>
    <bug_when>2024-05-08 16:38:21 -0700</bug_when>
    <thetext>This compiles and fixes this test case:

    RefPtr&lt;NodeType&gt; getNamedNode(const AtomString&amp; id) const
    {
        if (!id.isEmpty()) {
            if (m_sourceNodes.contains(id))
                return m_sourceNodes.get(id);
            
            if (m_namedNodes.contains(id))
                return m_namedNodes.get(id);
        }

        if (m_lastNode)
            return m_lastNode;

        return sourceGraphic();
    }

in https://searchfox.org/wubkat/rev/7bf250e4832adfb28c2bf2bc518ad59958f2ae1b/Source/WebCore/svg/graphics/filters/SVGFilterGraph.h#78

For LBSE - there isn&apos;t any improvement or regression.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2038352</commentid>
    <comment_count>7</comment_count>
    <who name="EWS">ews-feeder</who>
    <bug_when>2024-05-28 19:38:17 -0700</bug_when>
    <thetext>Committed 279421@main (8e028375b9c5): &lt;https://commits.webkit.org/279421@main&gt;

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

    </bug>

</bugzilla>