<?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>218854</bug_id>
          
          <creation_ts>2020-11-12 09:51:45 -0800</creation_ts>
          <short_desc>[GTK][WPE] FilterInfo must cleared when reusing BitmapTextureGL</short_desc>
          <delta_ts>2020-11-13 00:22:53 -0800</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>WebKitGTK</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></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Miguel Gomez">magomez</reporter>
          <assigned_to name="Miguel Gomez">magomez</assigned_to>
          <cc>bugs-noreply</cc>
    
    <cc>cgarcia</cc>
    
    <cc>cmarcelo</cc>
    
    <cc>ews-watchlist</cc>
    
    <cc>fujii</cc>
    
    <cc>kondapallykalyan</cc>
    
    <cc>luiz</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1706897</commentid>
    <comment_count>0</comment_count>
    <who name="Miguel Gomez">magomez</who>
    <bug_when>2020-11-12 09:51:45 -0800</bug_when>
    <thetext>When a BitmapTextureGL is reused, its internal values are reset (flags, internal texture, and content). But the FilterInfo variable is not, so the every time that BitmapTextureGL is painted it will use the the old filter, altering the result.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1706901</commentid>
    <comment_count>1</comment_count>
      <attachid>413946</attachid>
    <who name="Miguel Gomez">magomez</who>
    <bug_when>2020-11-12 09:55:02 -0800</bug_when>
    <thetext>Created attachment 413946
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1706999</commentid>
    <comment_count>2</comment_count>
    <who name="Fujii Hironori">fujii</who>
    <bug_when>2020-11-12 13:33:09 -0800</bug_when>
    <thetext>Good catch.
Are there any existing tests failing? If so, please add a comment about it. If not, can you add a new test?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1707130</commentid>
    <comment_count>3</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2020-11-12 23:46:40 -0800</bug_when>
    <thetext>I think this will fix tests but on top pach attached to bug #215445. So, it will be covered by backdrop filter tests.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1707135</commentid>
    <comment_count>4</comment_count>
    <who name="Miguel Gomez">magomez</who>
    <bug_when>2020-11-13 00:05:49 -0800</bug_when>
    <thetext>(In reply to Carlos Garcia Campos from comment #3)
&gt; I think this will fix tests but on top pach attached to bug #215445. So, it
&gt; will be covered by backdrop filter tests.

Yes, this together with the fix to bug #215445 will fix several backdrop filter related bugs.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1707137</commentid>
    <comment_count>5</comment_count>
    <who name="EWS">ews-feeder</who>
    <bug_when>2020-11-13 00:22:51 -0800</bug_when>
    <thetext>Committed r269765: &lt;https://trac.webkit.org/changeset/269765&gt;

All reviewed patches have been landed. Closing bug and clearing flags on attachment 413946.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>413946</attachid>
            <date>2020-11-12 09:55:02 -0800</date>
            <delta_ts>2020-11-13 00:22:52 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-218854-20201112185500.patch</filename>
            <type>text/plain</type>
            <size>1382</size>
            <attacher name="Miguel Gomez">magomez</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjY5NzMxCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggMDEyZjFhOGY3ODZmNWRm
MGM4NWI5ZjlhNGUyZWFkZTc3ZjNlOWE4ZC4uMWFjZjg2OTQzNjhmOGVlMTczZWFhMzQxODJiMTBm
ZTk3ZTNhZDY1MSAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE1IEBACisyMDIwLTExLTEyICBNaWd1
ZWwgR29tZXogIDxtYWdvbWV6QGlnYWxpYS5jb20+CisKKyAgICAgICAgW0dUS11bV1BFXSBGaWx0
ZXJJbmZvIG11c3QgY2xlYXJlZCB3aGVuIHJldXNpbmcgQml0bWFwVGV4dHVyZUdMCisgICAgICAg
IGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0yMTg4NTQKKworICAgICAg
ICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBDbGVhciB0aGUgRmlsdGVy
SW5mbyBhdHRyaWJ1dGUgd2hlbiByZXVzaW5nIGEgQml0bWFwVGV4dHVyZUdMLgorCisgICAgICAg
ICogcGxhdGZvcm0vZ3JhcGhpY3MvdGV4bWFwL0JpdG1hcFRleHR1cmVHTC5jcHA6CisgICAgICAg
IChXZWJDb3JlOjpCaXRtYXBUZXh0dXJlR0w6OmRpZFJlc2V0KToKKwogMjAyMC0xMS0xMiAgU2Vy
Z2lvIFZpbGxhciBTZW5pbiAgPHN2aWxsYXJAaWdhbGlhLmNvbT4KIAogICAgICAgICBDcmFzaCBp
biBSZW5kZXJCb3g6Om92ZXJyaWRlQ29udGFpbmluZ0Jsb2NrQ29udGVudEhlaWdodCgpCmRpZmYg
LS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9ncmFwaGljcy90ZXhtYXAvQml0bWFwVGV4
dHVyZUdMLmNwcCBiL1NvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL2dyYXBoaWNzL3RleG1hcC9CaXRt
YXBUZXh0dXJlR0wuY3BwCmluZGV4IGE2MTEwOTJlYzkxNDcwMDkyMjYwZDhiMDRiN2NhMzUxYzc2
ZjQ2ZmYuLjM2MmI2NDJlNTA4YjgwNzkwMTNlZmRhZmUwZDRhODFlYWFhMTdmMWYgMTAwNjQ0Ci0t
LSBhL1NvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL2dyYXBoaWNzL3RleG1hcC9CaXRtYXBUZXh0dXJl
R0wuY3BwCisrKyBiL1NvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL2dyYXBoaWNzL3RleG1hcC9CaXRt
YXBUZXh0dXJlR0wuY3BwCkBAIC04MCw2ICs4MCw3IEBAIHZvaWQgQml0bWFwVGV4dHVyZUdMOjpk
aWRSZXNldCgpCiAKICAgICBtX3Nob3VsZENsZWFyID0gdHJ1ZTsKICAgICBtX2NvbG9yQ29udmVy
dEZsYWdzID0gVGV4dHVyZU1hcHBlckdMOjpOb0ZsYWc7CisgICAgbV9maWx0ZXJJbmZvID0gRmls
dGVySW5mbygpOwogICAgIGlmIChtX3RleHR1cmVTaXplID09IGNvbnRlbnRTaXplKCkpCiAgICAg
ICAgIHJldHVybjsKIAo=
</data>

          </attachment>
      

    </bug>

</bugzilla>