52599 2011-01-17 15:07:37 -0800 UIProcess crash in WebPageProxy::reattachToWebProcess when web process crashes with an empty, new window. 2011-01-18 17:47:08 -0800 1 1 1 Unclassified WebKit WebKit2 528+ (Nightly build) All All RESOLVED FIXED InRadar P2 Normal --- 1 beidson beidson darin oldest_to_newest 335545 0 beidson 2011-01-17 15:07:37 -0800 UIProcess crash in WebPageProxy::reattachToWebProcess when web process crashes with an empty, new window. In radar as <rdar://problem/8860833> Windows backtrace (but can repro on Mac as well) > WebKit.dll!WebKit::WebPageProxy::reattachToWebProcessWithItem(WebKit::WebBackForwardListItem * item=0x00000000) Line 226 C++ WebKit.dll!WebKit::WebPageProxy::reload(bool reloadFromOrigin=false) Line 372 C++ WebKit.dll!WKPageReload(const OpaqueWKPage * pageRef=0x7f84db80) Line 88 C++ Safari.dll!Safari::BrowserContentViewController::reload() Line 2307 + 0x19 bytes C++ Safari.dll!Safari::BrowserPageLoaderClient::processDidExit(const Safari::WK::Page & page=0x7f84db80) Line 431 + 0x8 bytes C++ Safari.dll!Safari::WK::processDidExit(const OpaqueWKPage * pageRef=0x7f84db80, const void * clientInfo=0x7f848cf0) Line 206 C++ WebKit.dll!WebKit::WebPageProxy::processDidCrash() Line 2201 + 0xe bytes C++ WebKit.dll!WebKit::WebProcessProxy::didClose(CoreIPC::Connection * __formal=0x7ff39200) Line 303 + 0x8 bytes C++ WebKit.dll!CoreIPC::Connection::dispatchConnectionDidClose() Line 357 336252 1 beidson 2011-01-18 17:42:11 -0800 Happens on Mac, too. And happens anytime the WebProcess crashes before it's notified the UIProcess of the current item in a new frame. Null-check should suffice. 336254 2 79370 beidson 2011-01-18 17:43:57 -0800 Created attachment 79370 Patch v1 336255 3 beidson 2011-01-18 17:47:08 -0800 r76089 79370 2011-01-18 17:43:57 -0800 2011-01-18 17:45:09 -0800 Patch v1 patch.txt text/plain 1355 beidson SW5kZXg6IFNvdXJjZS9XZWJLaXQyL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi S2l0Mi9DaGFuZ2VMb2cJKHJldmlzaW9uIDc2MDg3KQorKysgU291cmNlL1dlYktpdDIvQ2hhbmdl TG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTMgQEAKKzIwMTEtMDEtMTggIEJyYWR5IEVp ZHNvbiAgPGJlaWRzb25AYXBwbGUuY29tPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAo T09QUyEpLgorCisgICAgICAgIDxyZGFyOi8vcHJvYmxlbS84ODYwODMzPiBhbmQgaHR0cHM6Ly9i dWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTUyNTk5CisgICAgICAgIFVJUHJvY2VzcyBj cmFzaCBpbiBXZWJQYWdlUHJveHk6OnJlYXR0YWNoVG9XZWJQcm9jZXNzIHdoZW4gd2ViIHByb2Nl c3MgY3Jhc2hlcyB3aXRoIGEgbmV3IHRhYi93aW5kb3cuCisKKyAgICAgICAgKiBVSVByb2Nlc3Mv V2ViUGFnZVByb3h5LmNwcDoKKyAgICAgICAgKFdlYktpdDo6V2ViUGFnZVByb3h5OjpyZWF0dGFj aFRvV2ViUHJvY2Vzc1dpdGhJdGVtKTogTnVsbCBjaGVjayBpdGVtICpib3RoKiBwbGFjZXMgaXQg aXMgdXNlZC4KKwogMjAxMS0wMS0xOCAgQnJhZHkgRWlkc29uICA8YmVpZHNvbkBhcHBsZS5jb20+ CiAKICAgICAgICAgUmV2aWV3ZWQgYnkgTWFjaWVqIFN0YWNob3dpYWsuIApJbmRleDogU291cmNl L1dlYktpdDIvVUlQcm9jZXNzL1dlYlBhZ2VQcm94eS5jcHAKPT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291cmNl L1dlYktpdDIvVUlQcm9jZXNzL1dlYlBhZ2VQcm94eS5jcHAJKHJldmlzaW9uIDc2MDg3KQorKysg U291cmNlL1dlYktpdDIvVUlQcm9jZXNzL1dlYlBhZ2VQcm94eS5jcHAJKHdvcmtpbmcgY29weSkK QEAgLTIyNyw3ICsyMjcsOSBAQCB2b2lkIFdlYlBhZ2VQcm94eTo6cmVhdHRhY2hUb1dlYlByb2Nl c3NXCiAgICAgICAgIG1fYmFja0ZvcndhcmRMaXN0LT5nb1RvSXRlbShpdGVtKTsKICAgICAKICAg ICByZWF0dGFjaFRvV2ViUHJvY2VzcygpOwotICAgIHByb2Nlc3MoKS0+c2VuZChNZXNzYWdlczo6 V2ViUGFnZTo6R29Ub0JhY2tGb3J3YXJkSXRlbShpdGVtLT5pdGVtSUQoKSksIG1fcGFnZUlEKTsK KyAgICAKKyAgICBpZiAoaXRlbSkKKyAgICAgICAgcHJvY2VzcygpLT5zZW5kKE1lc3NhZ2VzOjpX ZWJQYWdlOjpHb1RvQmFja0ZvcndhcmRJdGVtKGl0ZW0tPml0ZW1JRCgpKSwgbV9wYWdlSUQpOwog fQogCiB2b2lkIFdlYlBhZ2VQcm94eTo6aW5pdGlhbGl6ZVdlYlBhZ2UoKQo=