150319 2015-10-19 02:25:03 -0700 [GTK] ASSERTION FAILED: parent in WebInspectorProxy::platformDetach() 2015-10-19 02:49:59 -0700 1 1 1 Unclassified WebKit WebKitGTK WebKit Local Build Unspecified Unspecified RESOLVED FIXED Gtk, LayoutTestFailure P2 Normal --- 1 cgarcia webkit-unassigned bugs-noreply gustavo zan oldest_to_newest 1134515 0 cgarcia 2015-10-19 02:25:03 -0700 This happens when the inspector is shown/closed very quickly. Since we start the inspector attached by default, when the web inspector is created m_isAttached is set to true, so that when opened, it's attached to the view. If the inspector is closed before it has been actually attached, the inspector view doesn't have a parent, even if the m_isAttached is true. So, we should handle this case instead of asserting. This is what happens in test http/tests/xmlhttprequest/access-control-repeated-failed-preflight-crash.html 20:06:18.123 27815 ASSERTION FAILED: parent 20:06:18.123 27815 ../../Source/WebKit2/UIProcess/gtk/WebInspectorProxyGtk.cpp(315) : void WebKit::WebInspectorProxy::platformDetach() 20:06:18.123 27815 1 0x2b9f2c454d43 /home/slave/webkitgtk/gtk-linux-64-debug/build/WebKitBuild/Debug/lib/libjavascriptcoregtk-4.0.so.18(WTFCrash+0x1e) [0x2b9f2c454d43] 20:06:18.123 27815 2 0x2b9f31e25f25 /home/slave/webkitgtk/gtk-linux-64-debug/build/WebKitBuild/Debug/lib/libwebkit2gtk-4.0.so.37(_ZN6WebKit17WebInspectorProxy14platformDetachEv+0xa7) [0x2b9f31e25f25] 20:06:18.123 27815 3 0x2b9f31a3c151 /home/slave/webkitgtk/gtk-linux-64-debug/build/WebKitBuild/Debug/lib/libwebkit2gtk-4.0.so.37(_ZN6WebKit17WebInspectorProxy8didCloseEv+0xb7) [0x2b9f31a3c151] 20:06:18.123 27815 4 0x2b9f31e8487f /home/slave/webkitgtk/gtk-linux-64-debug/build/WebKitBuild/Debug/lib/libwebkit2gtk-4.0.so.37(_ZN3IPC22callMemberFunctionImplIN6WebKit17WebInspectorProxyEMS2_FvvESt5tupleIIEEIEEEvPT_T0_OT1_St14index_sequenceIIXspT2_EEE+0x65) [0x2b9f31e8487f] 20:06:18.123 27815 5 0x2b9f31e84393 /home/slave/webkitgtk/gtk-linux-64-debug/build/WebKitBuild/Debug/lib/libwebkit2gtk-4.0.so.37(_ZN3IPC18callMemberFunctionIN6WebKit17WebInspectorProxyEMS2_FvvESt5tupleIIEESt19make_index_sequenceILm0EEEEvOT1_PT_T0_+0x41) [0x2b9f31e84393] 20:06:18.123 27815 6 0x2b9f31e838a8 /home/slave/webkitgtk/gtk-linux-64-debug/build/WebKitBuild/Debug/lib/libwebkit2gtk-4.0.so.37(_ZN3IPC13handleMessageIN8Messages17WebInspectorProxy8DidCloseEN6WebKit17WebInspectorProxyEMS5_FvvEEEvRNS_14MessageDecoderEPT0_T1_+0x8f) [0x2b9f31e838a8] 20:06:18.123 27815 7 0x2b9f31e82ea9 /home/slave/webkitgtk/gtk-linux-64-debug/build/WebKitBuild/Debug/lib/libwebkit2gtk-4.0.so.37(_ZN6WebKit17WebInspectorProxy17didReceiveMessageERN3IPC10ConnectionERNS1_14MessageDecoderE+0x11f) [0x2b9f31e82ea9] 20:06:18.123 27815 8 0x2b9f3194bb02 /home/slave/webkitgtk/gtk-linux-64-debug/build/WebKitBuild/Debug/lib/libwebkit2gtk-4.0.so.37(_ZN3IPC18MessageReceiverMap15dispatchMessageERNS_10ConnectionERNS_14MessageDecoderE+0x120) [0x2b9f3194bb02] 20:06:18.123 27815 9 0x2b9f31967d09 /home/slave/webkitgtk/gtk-linux-64-debug/build/WebKitBuild/Debug/lib/libwebkit2gtk-4.0.so.37(_ZN6WebKit17ChildProcessProxy15dispatchMessageERN3IPC10ConnectionERNS1_14MessageDecoderE+0x2f) [0x2b9f31967d09] 20:06:18.123 27815 10 0x2b9f31ac8fdb /home/slave/webkitgtk/gtk-linux-64-debug/build/WebKitBuild/Debug/lib/libwebkit2gtk-4.0.so.37(_ZN6WebKit15WebProcessProxy17didReceiveMessageERN3IPC10ConnectionERNS1_14MessageDecoderE+0x2b) [0x2b9f31ac8fdb] 20:06:18.123 27815 11 0x2b9f319391f0 /home/slave/webkitgtk/gtk-linux-64-debug/build/WebKitBuild/Debug/lib/libwebkit2gtk-4.0.so.37(_ZN3IPC10Connection15dispatchMessageERNS_14MessageDecoderE+0x3a) [0x2b9f319391f0] 20:06:18.123 27815 12 0x2b9f31939353 /home/slave/webkitgtk/gtk-linux-64-debug/build/WebKitBuild/Debug/lib/libwebkit2gtk-4.0.so.37(_ZN3IPC10Connection15dispatchMessageESt10unique_ptrINS_14MessageDecoderESt14default_deleteIS2_EE+0x161) [0x2b9f31939353] 20:06:18.123 27815 13 0x2b9f3193953a /home/slave/webkitgtk/gtk-linux-64-debug/build/WebKitBuild/Debug/lib/libwebkit2gtk-4.0.so.37(_ZN3IPC10Connection18dispatchOneMessageEv+0xc8) [0x2b9f3193953a] 20:06:18.123 27815 14 0x2b9f3193903e /home/slave/webkitgtk/gtk-linux-64-debug/build/WebKitBuild/Debug/lib/libwebkit2gtk-4.0.so.37(+0x488903e) [0x2b9f3193903e] 20:06:18.123 27815 15 0x2b9f3193a7f9 /home/slave/webkitgtk/gtk-linux-64-debug/build/WebKitBuild/Debug/lib/libwebkit2gtk-4.0.so.37(+0x488a7f9) [0x2b9f3193a7f9] 20:06:18.123 27815 16 0x46cdae /home/slave/webkitgtk/gtk-linux-64-debug-tests/build/WebKitBuild/Debug/bin/WebKitTestRunner(_ZNKSt8functionIFvvEEclEv+0x32) [0x46cdae] 20:06:18.123 27815 17 0x2b9f2c46e72c /home/slave/webkitgtk/gtk-linux-64-debug/build/WebKitBuild/Debug/lib/libjavascriptcoregtk-4.0.so.18(_ZN3WTF7RunLoop11performWorkEv+0x1b8) [0x2b9f2c46e72c] 20:06:18.123 27815 18 0x2b9f2c4aa992 /home/slave/webkitgtk/gtk-linux-64-debug/build/WebKitBuild/Debug/lib/libjavascriptcoregtk-4.0.so.18(+0x1830992) [0x2b9f2c4aa992] 20:06:18.123 27815 19 0x2b9f2c4aaf1a /home/slave/webkitgtk/gtk-linux-64-debug/build/WebKitBuild/Debug/lib/libjavascriptcoregtk-4.0.so.18(+0x1830f1a) [0x2b9f2c4aaf1a] 20:06:18.123 27815 20 0x46cdae /home/slave/webkitgtk/gtk-linux-64-debug-tests/build/WebKitBuild/Debug/bin/WebKitTestRunner(_ZNKSt8functionIFvvEEclEv+0x32) [0x46cdae] 20:06:18.123 27815 21 0x2b9f2c4a890d /home/slave/webkitgtk/gtk-linux-64-debug/build/WebKitBuild/Debug/lib/libjavascriptcoregtk-4.0.so.18(_ZN3WTF15GMainLoopSource12voidCallbackEv+0x6d) [0x2b9f2c4a890d] 20:06:18.123 27815 22 0x2b9f2c4a9011 /home/slave/webkitgtk/gtk-linux-64-debug/build/WebKitBuild/Debug/lib/libjavascriptcoregtk-4.0.so.18(_ZN3WTF15GMainLoopSource18voidSourceCallbackEPS0_+0x23) [0x2b9f2c4a9011] 20:06:18.123 27815 23 0x2b9f385d13a5 /home/slave/webkitgtk/gtk-linux-64-debug/build/WebKitBuild/DependenciesGTK/Root/lib64/libglib-2.0.so.0(+0x563a5) [0x2b9f385d13a5] 20:06:18.123 27815 24 0x2b9f385cea26 /home/slave/webkitgtk/gtk-linux-64-debug/build/WebKitBuild/DependenciesGTK/Root/lib64/libglib-2.0.so.0(+0x53a26) [0x2b9f385cea26] 20:06:18.123 27815 25 0x2b9f385cf854 /home/slave/webkitgtk/gtk-linux-64-debug/build/WebKitBuild/DependenciesGTK/Root/lib64/libglib-2.0.so.0(g_main_context_dispatch+0x33) [0x2b9f385cf854] 20:06:18.123 27815 26 0x2b9f385cfa39 /home/slave/webkitgtk/gtk-linux-64-debug/build/WebKitBuild/DependenciesGTK/Root/lib64/libglib-2.0.so.0(+0x54a39) [0x2b9f385cfa39] 20:06:18.123 27815 27 0x2b9f385cfe60 /home/slave/webkitgtk/gtk-linux-64-debug/build/WebKitBuild/DependenciesGTK/Root/lib64/libglib-2.0.so.0(g_main_loop_run+0x1d7) [0x2b9f385cfe60] 20:06:18.123 27815 28 0x2b9f2c4aa7f6 /home/slave/webkitgtk/gtk-linux-64-debug/build/WebKitBuild/Debug/lib/libjavascriptcoregtk-4.0.so.18(_ZN3WTF7RunLoop3runEv+0xac) [0x2b9f2c4aa7f6] 20:06:18.123 27815 29 0x48c499 /home/slave/webkitgtk/gtk-linux-64-debug-tests/build/WebKitBuild/Debug/bin/WebKitTestRunner(_ZN3WTR14TestController16platformRunUntilERbd+0xdb) [0x48c499] 20:06:18.123 27815 30 0x46533a /home/slave/webkitgtk/gtk-linux-64-debug-tests/build/WebKitBuild/Debug/bin/WebKitTestRunner(_ZN3WTR14TestController8runUntilERbd+0x52) [0x46533a] 20:06:18.123 27815 31 0x472c77 /home/slave/webkitgtk/gtk-linux-64-debug-tests/build/WebKitBuild/Debug/bin/WebKitTestRunner(_ZN3WTR14TestInvocation6invokeEv+0x49f) [0x472c77] 1134516 1 263459 cgarcia 2015-10-19 02:28:23 -0700 Created attachment 263459 Patch 1134523 2 cgarcia 2015-10-19 02:49:59 -0700 Committed r191284: <http://trac.webkit.org/changeset/191284> 263459 2015-10-19 02:28:23 -0700 2015-10-19 02:42:49 -0700 Patch wk2-inspector-assert.diff text/plain 2358 cgarcia ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQyL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJLaXQyL0No YW5nZUxvZwppbmRleCBkMmIxZWI2Li5iY2Y5OTU1IDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViS2l0 Mi9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYktpdDIvQ2hhbmdlTG9nCkBAIC0xLDUgKzEsMjUg QEAKIDIwMTUtMTAtMTkgIENhcmxvcyBHYXJjaWEgQ2FtcG9zICA8Y2dhcmNpYUBpZ2FsaWEuY29t PgogCisgICAgICAgIFtHVEtdIEFTU0VSVElPTiBGQUlMRUQ6IHBhcmVudCBpbiBXZWJJbnNwZWN0 b3JQcm94eTo6cGxhdGZvcm1EZXRhY2goKQorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9y Zy9zaG93X2J1Zy5jZ2k/aWQ9MTUwMzE5CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChP T1BTISkuCisKKyAgICAgICAgVGhpcyBoYXBwZW5zIHdoZW4gdGhlIGluc3BlY3RvciBpcyBzaG93 bi9jbG9zZWQgdmVyeQorICAgICAgICBxdWlja2x5LiBTaW5jZSB3ZSBzdGFydCB0aGUgaW5zcGVj dG9yIGF0dGFjaGVkIGJ5IGRlZmF1bHQsIHdoZW4KKyAgICAgICAgdGhlIHdlYiBpbnNwZWN0b3Ig aXMgY3JlYXRlZCBtX2lzQXR0YWNoZWQgaXMgc2V0IHRvIHRydWUsIHNvIHRoYXQKKyAgICAgICAg d2hlbiBvcGVuZWQsIGl0J3MgYXR0YWNoZWQgdG8gdGhlIHZpZXcuIElmIHRoZSBpbnNwZWN0b3Ig aXMgY2xvc2VkCisgICAgICAgIGJlZm9yZSBpdCBoYXMgYmVlbiBhY3R1YWxseSBhdHRhY2hlZCwg dGhlIGluc3BlY3RvciB2aWV3IGRvZXNuJ3QKKyAgICAgICAgaGF2ZSBhIHBhcmVudCwgZXZlbiBp ZiB0aGUgbV9pc0F0dGFjaGVkIGlzIHRydWUuIFNvLCB3ZSBzaG91bGQKKyAgICAgICAgaGFuZGxl IHRoaXMgY2FzZSBpbnN0ZWFkIG9mIGFzc2VydGluZy4gVGhpcyBpcyB3aGF0IGhhcHBlbnMgaW4K KyAgICAgICAgdGVzdCBodHRwL3Rlc3RzL3htbGh0dHByZXF1ZXN0L2FjY2Vzcy1jb250cm9sLXJl cGVhdGVkLWZhaWxlZC1wcmVmbGlnaHQtY3Jhc2guaHRtbC4KKworICAgICAgICAqIFVJUHJvY2Vz cy9ndGsvV2ViSW5zcGVjdG9yUHJveHlHdGsuY3BwOgorICAgICAgICAoV2ViS2l0OjpXZWJJbnNw ZWN0b3JQcm94eTo6cGxhdGZvcm1EZXRhY2gpOiBSZW1vdmUgdGhlIHZpZXcgZnJvbQorICAgICAg ICBpdHMgcGFyZW50IGNvbnRhaW5lciBpZiBpdCB3YXMgYWN0dWFsbHkgYXR0YWNoZWQuCisKKzIw MTUtMTAtMTkgIENhcmxvcyBHYXJjaWEgQ2FtcG9zICA8Y2dhcmNpYUBpZ2FsaWEuY29tPgorCiAg ICAgICAgIE5ldHdvcmtQcm9jZXNzOiBETlMgcHJlZmV0Y2ggaGFwcGVucyBpbiB0aGUgV2ViIFBy b2Nlc3MKICAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTE0 NzgyNAogCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0Mi9VSVByb2Nlc3MvZ3RrL1dlYkluc3Bl Y3RvclByb3h5R3RrLmNwcCBiL1NvdXJjZS9XZWJLaXQyL1VJUHJvY2Vzcy9ndGsvV2ViSW5zcGVj dG9yUHJveHlHdGsuY3BwCmluZGV4IDRmOWU1OTYuLjRkNmE3MjMgMTAwNjQ0Ci0tLSBhL1NvdXJj ZS9XZWJLaXQyL1VJUHJvY2Vzcy9ndGsvV2ViSW5zcGVjdG9yUHJveHlHdGsuY3BwCisrKyBiL1Nv dXJjZS9XZWJLaXQyL1VJUHJvY2Vzcy9ndGsvV2ViSW5zcGVjdG9yUHJveHlHdGsuY3BwCkBAIC0z MTEsOSArMzExLDExIEBAIHZvaWQgV2ViSW5zcGVjdG9yUHJveHk6OnBsYXRmb3JtRGV0YWNoKCkK IAogICAgIEdSZWZQdHI8R3RrV2lkZ2V0PiBpbnNwZWN0b3JWaWV3ID0gbV9pbnNwZWN0b3JWaWV3 OwogICAgIGlmICghbV9jbGllbnQuZGV0YWNoKHRoaXMpKSB7Ci0gICAgICAgIEd0a1dpZGdldCog cGFyZW50ID0gZ3RrX3dpZGdldF9nZXRfcGFyZW50KG1faW5zcGVjdG9yVmlldyk7Ci0gICAgICAg IEFTU0VSVChwYXJlbnQpOwotICAgICAgICBndGtfY29udGFpbmVyX3JlbW92ZShHVEtfQ09OVEFJ TkVSKHBhcmVudCksIG1faW5zcGVjdG9yVmlldyk7CisgICAgICAgIC8vIERldGFjaCBpcyBjYWxs ZWQgd2hlbiBtX2lzQXR0YWNoZWQgaXMgdHJ1ZSwgYnV0IGl0IGNvdWxkIGNhbGxlZCBiZWZvcmUK KyAgICAgICAgLy8gdGhlIGluc3BlY3RvciBpcyBvcGVuZWQgaWYgdGhlIGluc3BlY3RvciBpcyBz aG93bi9jbG9zZWQgcXVpY2tseS4gU28sCisgICAgICAgIC8vIHdlIG1pZ2h0IG5vdCBoYXZlIGEg cGFyZW50IHlldC4KKyAgICAgICAgaWYgKEd0a1dpZGdldCogcGFyZW50ID0gZ3RrX3dpZGdldF9n ZXRfcGFyZW50KG1faW5zcGVjdG9yVmlldykpCisgICAgICAgICAgICBndGtfY29udGFpbmVyX3Jl bW92ZShHVEtfQ09OVEFJTkVSKHBhcmVudCksIG1faW5zcGVjdG9yVmlldyk7CiAgICAgfQogCiAg ICAgaWYgKCFtX2lzVmlzaWJsZSkK