<?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>204068</bug_id>
          
          <creation_ts>2019-11-11 07:10:22 -0800</creation_ts>
          <short_desc>REGRESSION(r250707): [GTK] UIClient::setWindowFrame only works the first time</short_desc>
          <delta_ts>2019-11-12 00:08:01 -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>Gtk, Regression</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Carlos Garcia Campos">cgarcia</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>aperez</cc>
    
    <cc>berto</cc>
    
    <cc>bugs-noreply</cc>
    
    <cc>ews-watchlist</cc>
    
    <cc>gustavo</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1589182</commentid>
    <comment_count>0</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2019-11-11 07:10:22 -0800</bug_when>
    <thetext>This is because we are connecting to configure-event of parent window, but not returning FALSE from the callback to propagate the event.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1589183</commentid>
    <comment_count>1</comment_count>
      <attachid>383267</attachid>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2019-11-11 07:11:26 -0800</bug_when>
    <thetext>Created attachment 383267
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1589185</commentid>
    <comment_count>2</comment_count>
    <who name="EWS Watchlist">ews-watchlist</who>
    <bug_when>2019-11-11 07:12:22 -0800</bug_when>
    <thetext>Thanks for the patch. If this patch contains new public API please make sure it follows the guidelines for new WebKit2 GTK+ API. See http://trac.webkit.org/wiki/WebKitGTK/AddingNewWebKit2API</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1589467</commentid>
    <comment_count>3</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2019-11-12 00:08:01 -0800</bug_when>
    <thetext>Committed r252355: &lt;https://trac.webkit.org/changeset/252355&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>383267</attachid>
            <date>2019-11-11 07:11:26 -0800</date>
            <delta_ts>2019-11-11 13:16:08 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>wd-set-window-rect.diff</filename>
            <type>text/plain</type>
            <size>1870</size>
            <attacher name="Carlos Garcia Campos">cgarcia</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQvQ2hhbmdlTG9nIGIvU291cmNlL1dlYktpdC9DaGFu
Z2VMb2cKaW5kZXggNjI2MjFlNmI1ZjIuLjRiMTNjYjFkOTcxIDEwMDY0NAotLS0gYS9Tb3VyY2Uv
V2ViS2l0L0NoYW5nZUxvZworKysgYi9Tb3VyY2UvV2ViS2l0L0NoYW5nZUxvZwpAQCAtMSwzICsx
LDE2IEBACisyMDE5LTExLTExICBDYXJsb3MgR2FyY2lhIENhbXBvcyAgPGNnYXJjaWFAaWdhbGlh
LmNvbT4KKworICAgICAgICBSRUdSRVNTSU9OKHIyNTA3MDcpOiBbR1RLXSBVSUNsaWVudDo6c2V0
V2luZG93RnJhbWUgb25seSB3b3JrcyB0aGUgZmlyc3QgdGltZQorICAgICAgICBodHRwczovL2J1
Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MjA0MDY4CisKKyAgICAgICAgUmV2aWV3ZWQg
YnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgVGhpcyBpcyBiZWNhdXNlIHdlIGFyZSBjb25u
ZWN0aW5nIHRvIGNvbmZpZ3VyZS1ldmVudCBvZiBwYXJlbnQgd2luZG93LCBidXQgbm90IHJldHVy
bmluZyBGQUxTRSBmcm9tIHRoZSBjYWxsYmFjaworICAgICAgICB0byBwcm9wYWdhdGUgdGhlIGV2
ZW50LgorCisgICAgICAgICogVUlQcm9jZXNzL0FQSS9nbGliL1dlYktpdFVJQ2xpZW50LmNwcDoK
KyAgICAgICAgKFVJQ2xpZW50Ojp3aW5kb3dDb25maWd1cmVFdmVudENhbGxiYWNrKTogTWFrZSBp
dCBib29sZWFuIGFuZCByZXR1cm4gRkFMU0UuCisKIDIwMTktMTEtMDggIENhcmxvcyBHYXJjaWEg
Q2FtcG9zICA8Y2dhcmNpYUBpZ2FsaWEuY29tPgogCiAgICAgICAgIFtHVEtdIFdlYkRyaXZlcjog
aW1wbGVtZW50IG5ldyB3aW5kb3cgY29tbWFuZApkaWZmIC0tZ2l0IGEvU291cmNlL1dlYktpdC9V
SVByb2Nlc3MvQVBJL2dsaWIvV2ViS2l0VUlDbGllbnQuY3BwIGIvU291cmNlL1dlYktpdC9VSVBy
b2Nlc3MvQVBJL2dsaWIvV2ViS2l0VUlDbGllbnQuY3BwCmluZGV4IDk3MDYwYTc2OWYyLi5iZjdm
OTVkN2Y2ZCAxMDA2NDQKLS0tIGEvU291cmNlL1dlYktpdC9VSVByb2Nlc3MvQVBJL2dsaWIvV2Vi
S2l0VUlDbGllbnQuY3BwCisrKyBiL1NvdXJjZS9XZWJLaXQvVUlQcm9jZXNzL0FQSS9nbGliL1dl
YktpdFVJQ2xpZW50LmNwcApAQCAtMTM5LDEzICsxMzksMTUgQEAgcHJpdmF0ZToKICAgICB9CiAK
ICNpZiBQTEFURk9STShHVEspCi0gICAgc3RhdGljIHZvaWQgd2luZG93Q29uZmlndXJlRXZlbnRD
YWxsYmFjayhHdGtXaW5kb3cqIHdpbmRvdywgR2RrRXZlbnRDb25maWd1cmUqLCBHZGtSZWN0YW5n
bGUqIHRhcmdldEdlb21ldHJ5KQorICAgIHN0YXRpYyBnYm9vbGVhbiB3aW5kb3dDb25maWd1cmVF
dmVudENhbGxiYWNrKEd0a1dpbmRvdyogd2luZG93LCBHZGtFdmVudENvbmZpZ3VyZSosIEdka1Jl
Y3RhbmdsZSogdGFyZ2V0R2VvbWV0cnkpCiAgICAgewogICAgICAgICBHZGtSZWN0YW5nbGUgZ2Vv
bWV0cnkgPSB7IDAsIDAsIDAsIDAgfTsKICAgICAgICAgZ3RrX3dpbmRvd19nZXRfcG9zaXRpb24o
d2luZG93LCAmZ2VvbWV0cnkueCwgJmdlb21ldHJ5LnkpOwogICAgICAgICBndGtfd2luZG93X2dl
dF9zaXplKHdpbmRvdywgJmdlb21ldHJ5LndpZHRoLCAmZ2VvbWV0cnkuaGVpZ2h0KTsKICAgICAg
ICAgaWYgKGdlb21ldHJ5LnggPT0gdGFyZ2V0R2VvbWV0cnktPnggJiYgZ2VvbWV0cnkueSA9PSB0
YXJnZXRHZW9tZXRyeS0+eSAmJiBnZW9tZXRyeS53aWR0aCA9PSB0YXJnZXRHZW9tZXRyeS0+d2lk
dGggJiYgZ2VvbWV0cnkuaGVpZ2h0ID09IHRhcmdldEdlb21ldHJ5LT5oZWlnaHQpCiAgICAgICAg
ICAgICBSdW5Mb29wOjpjdXJyZW50KCkuc3RvcCgpOworCisgICAgICAgIHJldHVybiBGQUxTRTsK
ICAgICB9CiAKICAgICB2b2lkIHNldFdpbmRvd0ZyYW1lVGltZXJGaXJlZCgpCg==
</data>
<flag name="review"
          id="399016"
          type_id="1"
          status="+"
          setter="aperez"
    />
          </attachment>
      

    </bug>

</bugzilla>