49687 2010-11-17 14:20:44 -0800 Web Inspector: WebKit2: crash in WebKit::WebInspectorClient::sendMessageToFrontend 2010-11-17 14:47:00 -0800 1 1 1 Unclassified WebKit Web Inspector (Deprecated) 528+ (Nightly build) All All RESOLVED FIXED InRadar P2 Normal --- 1 timothy timothy apavlov bweinstein joepeck keishi loislo pfeldman pmuellr rik timothy yurys oldest_to_newest 310526 0 timothy 2010-11-17 14:20:44 -0800 4 com.apple.WebKit2: WebKit::WebInspector::inspectorPage const + 12 <== 4 com.apple.WebKit2: WebKit::WebInspectorClient::sendMessageToFrontend + 40 4 com.apple.WebCore: WebCore::InspectorFrontend::frameDetachedFromParent + 713 4 com.apple.WebCore: WebCore::InspectorResourceAgent::frameDetachedFromParent + 47 4 com.apple.WebCore: WebCore::InspectorController::frameDetachedFromParent + 137 4 com.apple.WebCore: WebCore::FrameLoader::detachFromParent + 195 4 com.apple.WebKit2: WebKit::WebPage::close + 326 Steps to reproduce: 0) Use WK2 1) Open a page 2) Try to show the Web Inspector (currently nothing shows) 3) Close the page to see the crash 310527 1 timothy 2010-11-17 14:20:59 -0800 <rdar://problem/8672603> 310533 2 74157 timothy 2010-11-17 14:28:30 -0800 Created attachment 74157 Proposed Change 310542 3 timothy 2010-11-17 14:47:00 -0800 Landed in r72246. 74157 2010-11-17 14:28:30 -0800 2010-11-17 14:31:39 -0800 Proposed Change 0001-Fix-a-crash-when-closing-the-inspected-page-when-the.patch text/plain 2193 timothy RnJvbSAwNzg3NTJjZmIwOWRkYTIwMDk2YTc3Y2ZhNzY1ODEwMzk0NjY3MTZkIE1vbiBTZXAgMTcg MDA6MDA6MDAgMjAwMQpGcm9tOiBUaW1vdGh5IEhhdGNoZXIgPHRpbW90aHlAYXBwbGUuY29tPgpE YXRlOiBXZWQsIDE3IE5vdiAyMDEwIDE0OjIzOjM0IC0wODAwClN1YmplY3Q6IFtQQVRDSF0gRml4 IGEgY3Jhc2ggd2hlbiBjbG9zaW5nIHRoZSBpbnNwZWN0ZWQgcGFnZSB3aGVuIHRoZSBXZWIgSW5z cGVjdG9yCiBwYWdlIHdhcyBuZXZlciBjcmVhdGVkLgoKaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcv c2hvd19idWcuY2dpP2lkPTQ5Njg3CgpSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KCiogV2Vi UHJvY2Vzcy9XZWJDb3JlU3VwcG9ydC9XZWJJbnNwZWN0b3JDbGllbnQuY3BwOgooV2ViS2l0OjpX ZWJJbnNwZWN0b3JDbGllbnQ6OnNlbmRNZXNzYWdlVG9Gcm9udGVuZCk6IE51bGwgY2hlY2sgdGhl IEluc3BlY3RvciBXZWJQYWdlLgotLS0KIFdlYktpdDIvQ2hhbmdlTG9nICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgIHwgICAxMiArKysrKysrKysrKysKIC4uLi9XZWJDb3JlU3VwcG9y dC9XZWJJbnNwZWN0b3JDbGllbnQuY3BwICAgICAgICAgIHwgICAgNCArKystCiAyIGZpbGVzIGNo YW5nZWQsIDE1IGluc2VydGlvbnMoKyksIDEgZGVsZXRpb25zKC0pCgpkaWZmIC0tZ2l0IGEvV2Vi S2l0Mi9DaGFuZ2VMb2cgYi9XZWJLaXQyL0NoYW5nZUxvZwppbmRleCAyNGNkNTQzLi5kY2EyNTY5 IDEwMDY0NAotLS0gYS9XZWJLaXQyL0NoYW5nZUxvZworKysgYi9XZWJLaXQyL0NoYW5nZUxvZwpA QCAtMSw1ICsxLDE3IEBACiAyMDEwLTExLTE3ICBUaW1vdGh5IEhhdGNoZXIgIDx0aW1vdGh5QGFw cGxlLmNvbT4KIAorICAgICAgICBGaXggYSBjcmFzaCB3aGVuIGNsb3NpbmcgdGhlIGluc3BlY3Rl ZCBwYWdlIHdoZW4gdGhlIFdlYiBJbnNwZWN0b3IKKyAgICAgICAgcGFnZSB3YXMgbmV2ZXIgY3Jl YXRlZC4KKworICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9 NDk2ODcKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICAq IFdlYlByb2Nlc3MvV2ViQ29yZVN1cHBvcnQvV2ViSW5zcGVjdG9yQ2xpZW50LmNwcDoKKyAgICAg ICAgKFdlYktpdDo6V2ViSW5zcGVjdG9yQ2xpZW50OjpzZW5kTWVzc2FnZVRvRnJvbnRlbmQpOiBO dWxsIGNoZWNrIHRoZSBJbnNwZWN0b3IgV2ViUGFnZS4KKworMjAxMC0xMS0xNyAgVGltb3RoeSBI YXRjaGVyICA8dGltb3RoeUBhcHBsZS5jb20+CisKICAgICAgICAgUHJvdmlkZSBhIHdheSB0byBj cmVhdGUgYSBXS1ZpZXcgKFdlYlBhZ2UpIHRoYXQgaXMgaGlkZGVuIGZyb20KICAgICAgICAgdGhl IEluamVjdGVkQnVuZGxlLiBUaGlzIGlzIG5lZWRlZCBmb3IgdGhlIFdlYiBJbnNwZWN0b3IuCiAK ZGlmZiAtLWdpdCBhL1dlYktpdDIvV2ViUHJvY2Vzcy9XZWJDb3JlU3VwcG9ydC9XZWJJbnNwZWN0 b3JDbGllbnQuY3BwIGIvV2ViS2l0Mi9XZWJQcm9jZXNzL1dlYkNvcmVTdXBwb3J0L1dlYkluc3Bl Y3RvckNsaWVudC5jcHAKaW5kZXggYTZkMzg1NS4uZTI5NzIzZCAxMDA2NDQKLS0tIGEvV2ViS2l0 Mi9XZWJQcm9jZXNzL1dlYkNvcmVTdXBwb3J0L1dlYkluc3BlY3RvckNsaWVudC5jcHAKKysrIGIv V2ViS2l0Mi9XZWJQcm9jZXNzL1dlYkNvcmVTdXBwb3J0L1dlYkluc3BlY3RvckNsaWVudC5jcHAK QEAgLTc0LDcgKzc0LDkgQEAgdm9pZCBXZWJJbnNwZWN0b3JDbGllbnQ6OnN0b3JlU2V0dGluZyhj b25zdCBTdHJpbmcmLCBjb25zdCBTdHJpbmcmKQogCiBib29sIFdlYkluc3BlY3RvckNsaWVudDo6 c2VuZE1lc3NhZ2VUb0Zyb250ZW5kKGNvbnN0IFN0cmluZyYgbWVzc2FnZSkKIHsKLSAgICByZXR1 cm4gZG9EaXNwYXRjaE1lc3NhZ2VPbkZyb250ZW5kUGFnZShtX3BhZ2UtPmluc3BlY3RvcigpLT5p bnNwZWN0b3JQYWdlKCktPmNvcmVQYWdlKCksIG1lc3NhZ2UpOworICAgIGlmIChXZWJQYWdlKiBp bnNwZWN0b3JQYWdlID0gbV9wYWdlLT5pbnNwZWN0b3IoKS0+aW5zcGVjdG9yUGFnZSgpKQorICAg ICAgICByZXR1cm4gZG9EaXNwYXRjaE1lc3NhZ2VPbkZyb250ZW5kUGFnZShpbnNwZWN0b3JQYWdl LT5jb3JlUGFnZSgpLCBtZXNzYWdlKTsKKyAgICByZXR1cm4gZmFsc2U7CiB9CiAKIH0gLy8gbmFt ZXNwYWNlIFdlYktpdAotLSAKMS43LjAuMgoK