<?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>153945</bug_id>
          
          <creation_ts>2016-02-06 09:35:07 -0800</creation_ts>
          <short_desc>REGRESSION(r196183): [GTK] Broke TestInspector</short_desc>
          <delta_ts>2016-02-09 22:54:06 -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>Other</version>
          <rep_platform>PC</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          <see_also>https://bugs.webkit.org/show_bug.cgi?id=152867</see_also>
          <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="Michael Catanzaro">mcatanzaro</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>bugs-noreply</cc>
    
    <cc>cgarcia</cc>
    
    <cc>mcatanzaro</cc>
    
    <cc>timothy</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1162624</commentid>
    <comment_count>0</comment_count>
    <who name="Michael Catanzaro">mcatanzaro</who>
    <bug_when>2016-02-06 09:35:07 -0800</bug_when>
    <thetext>r196183 broke TestInspector:

TEST: ./Tools/gtk/../../WebKitBuild/Release/bin/TestWebKitAPI/WebKit2Gtk/TestInspector... (pid=6821)

  /webkit2/WebKitWebInspector/default:                                 OK

  /webkit2/WebKitWebInspector/manual-attach-detach:                    

(./Tools/gtk/../../WebKitBuild/Release/bin/TestWebKitAPI/WebKit2Gtk/TestInspector:6821): Gtk-CRITICAL **: gtk_container_remove: assertion &apos;gtk_widget_get_parent (widget) == GTK_WIDGET (container) || GTK_IS_ASSISTANT (container) || GTK_IS_ACTION_BAR (container) || GTK_IS_POPOVER_MENU (container)&apos; failed

FAIL</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1163165</commentid>
    <comment_count>1</comment_count>
      <attachid>270920</attachid>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2016-02-09 03:18:48 -0800</bug_when>
    <thetext>Created attachment 270920
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1163411</commentid>
    <comment_count>2</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2016-02-09 22:54:06 -0800</bug_when>
    <thetext>Committed r196363: &lt;http://trac.webkit.org/changeset/196363&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>270920</attachid>
            <date>2016-02-09 03:18:48 -0800</date>
            <delta_ts>2016-02-09 06:50:10 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>wk2-inspector-detach-on-close.diff</filename>
            <type>text/plain</type>
            <size>2550</size>
            <attacher name="Carlos Garcia Campos">cgarcia</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQyL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJLaXQyL0No
YW5nZUxvZwppbmRleCA4Nzk0MzVhLi40NWMxNzkxIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViS2l0
Mi9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYktpdDIvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMjYg
QEAKKzIwMTYtMDItMDkgIENhcmxvcyBHYXJjaWEgQ2FtcG9zICA8Y2dhcmNpYUBpZ2FsaWEuY29t
PgorCisgICAgICAgIFJFR1JFU1NJT04ocjE5NjE4Myk6IFtHVEtdIEJyb2tlIFRlc3RJbnNwZWN0
b3IKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTE1Mzk0
NQorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIFdoZW4g
dGhlIGluc3BlY3RvciB2aWV3IGlzIGRldGFjaGVkIGFuZCBub3QgYWRkZWQgdG8gYSB3aW5kb3cs
CisgICAgICAgIHdoaWNoIGhhcHBlbnMgd2hlbiBpdCdzIGNsb3NlZCwgdGhlIGluc3BlY3RvciB2
aWV3IGlzIGRlc3Ryb3llZAorICAgICAgICBiZWNhdXNlIGl0IGRvZXNuJ3QgaGF2ZSBhIHBhcmVu
dCBhbnltb3JlLiBXaGVuIHRoZSBpbnNwZWN0b3IgdmlldworICAgICAgICBpcyBkZXN0cm95ZWQg
d2Ugbm90aWZ5IHRoZSB3ZWIgcHJvY2VzcyB0aGF0IHRoZSBpbnNwZWN0b3Igd2FzCisgICAgICAg
IGNsb3NlZC4gQmVmb3JlIHIxOTYxODMgdGhpcyB3YXMgbm90IGEgcHJvYmxlbSwgYmVjYXVzZSB0
aGUgY2FsbCB0bworICAgICAgICBXZWJJbnNwZWN0b3JQcm94eTo6ZGlkQ2xvc2UoKSBmcm9tIHBs
YXRmb3JtRGV0YWNoKCkgcmV0dXJuZWQgZWFybHkKKyAgICAgICAgYmVjYXVzZSBXZWJJbnNwZWN0
b3JQcm94eTo6ZGlkQ2xvc2UoKSBoYWQgYWxyZWFkeSBzZXQKKyAgICAgICAgbV9pbnNwZWN0b3JQ
YWdlIHRvIG51bGxwdHIuIEluIHIxOTYxODMgbV9pbnNwZWN0b3JQYWdlIGlzIHNldCB0bworICAg
ICAgICBudWxscHRyIGFmdGVyIHBsYXRmb3JtRGV0YWNoKCksIHNvIHdlIGVuZCB1cCB0cnlpbmcg
dG8gZGV0YWNoIHRoZQorICAgICAgICBpbnBlY3RvciB2aWV3IGFnYWluLiBUbyBwcmV2ZW50IHRo
aXMgY3ljbGUsIHdlIHNob3VsZCBkaXNjb25uZWN0CisgICAgICAgIHRoZSBkZXN0cm95ZWQgc2ln
bmFsIGhhbmRsZXIgZnJvbSB0aGUgaW5zcGVjdG9yIHZpZXcgd2hlbgorICAgICAgICBwbGF0Zm9y
bURldGFjaCgpIGlzIGNhbGxlZCBmcm9tIFdlYkluc3BlY3RvclByb3h5OjpkaWRDbG9zZSgpLgor
CisgICAgICAgICogVUlQcm9jZXNzL2d0ay9XZWJJbnNwZWN0b3JQcm94eUd0ay5jcHA6CisgICAg
ICAgIChXZWJLaXQ6OldlYkluc3BlY3RvclByb3h5OjpwbGF0Zm9ybURldGFjaCk6CisKIDIwMTYt
MDItMDkgIEZ1amlpIEhpcm9ub3JpICA8SGlyb25vcmkuRnVqaWlAanAuc29ueS5jb20+CiAKICAg
ICAgICAgcG9zc2libGUgYnVmZmVyIG92ZXJydW4gaW4gQ29ubmVjdGlvbjo6cHJvY2Vzc01lc3Nh
Z2Ugb2YgU291cmNlL1dlYktpdDIvUGxhdGZvcm0vSVBDL3VuaXgvQ29ubmVjdGlvblVuaXguY3Bw
CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0Mi9VSVByb2Nlc3MvZ3RrL1dlYkluc3BlY3RvclBy
b3h5R3RrLmNwcCBiL1NvdXJjZS9XZWJLaXQyL1VJUHJvY2Vzcy9ndGsvV2ViSW5zcGVjdG9yUHJv
eHlHdGsuY3BwCmluZGV4IDA3ZTEzZmQuLjJjMGUxNDUgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJL
aXQyL1VJUHJvY2Vzcy9ndGsvV2ViSW5zcGVjdG9yUHJveHlHdGsuY3BwCisrKyBiL1NvdXJjZS9X
ZWJLaXQyL1VJUHJvY2Vzcy9ndGsvV2ViSW5zcGVjdG9yUHJveHlHdGsuY3BwCkBAIC0zMTksOCAr
MzE5LDE1IEBAIHZvaWQgV2ViSW5zcGVjdG9yUHJveHk6OnBsYXRmb3JtRGV0YWNoKCkKICAgICAg
ICAgICAgIGd0a19jb250YWluZXJfcmVtb3ZlKEdUS19DT05UQUlORVIocGFyZW50KSwgbV9pbnNw
ZWN0b3JWaWV3KTsKICAgICB9CiAKLSAgICBpZiAoIW1faXNWaXNpYmxlKQorICAgIC8vIFJldHVy
biBlYXJseSBpZiB3ZSBhcmUgbm90IHZpc2libGUuIFRoaXMgbWVhbnMgdGhlIGluc3BlY3RvciB3
YXMgY2xvc2VkIHdoaWxlIGF0dGFjaGVkCisgICAgLy8gYW5kIHdlIHNob3VsZCBub3QgY3JlYXRl
IGFuZCBzaG93IHRoZSBpbnNwZWN0b3Igd2luZG93LgorICAgIGlmICghbV9pc1Zpc2libGUpIHsK
KyAgICAgICAgLy8gVGhlIGluc3BlY3RvciB2aWV3IHdpbGwgYmUgZGVzdHJveWVkLCBidXQgd2Ug
ZG9uJ3QgbmVlZCB0byBub3RpZnkgdGhlIHdlYiBwcm9jZXNzIHRvIGNsb3NlIHRoZQorICAgICAg
ICAvLyBpbnNwZWN0b3IgaW4gdGhpcyBjYXNlLCBzaW5jZSBpdCdzIGFscmVhZHkgY2xvc2VkLgor
ICAgICAgICBnX3NpZ25hbF9oYW5kbGVyc19kaXNjb25uZWN0X2J5X2Z1bmMobV9pbnNwZWN0b3JW
aWV3LCByZWludGVycHJldF9jYXN0PHZvaWQqPihpbnNwZWN0b3JWaWV3RGVzdHJveWVkKSwgdGhp
cyk7CisgICAgICAgIG1faW5zcGVjdG9yVmlldyA9IG51bGxwdHI7CiAgICAgICAgIHJldHVybjsK
KyAgICB9CiAKICAgICBjcmVhdGVJbnNwZWN0b3JXaW5kb3coKTsKIH0K
</data>
<flag name="review"
          id="295764"
          type_id="1"
          status="+"
          setter="mcatanzaro"
    />
          </attachment>
      

    </bug>

</bugzilla>