100330 2012-10-24 22:44:04 -0700 [BlackBerry] WebView process crashed at AuthenticationChallengeManager::pageVisibilityChanged() 2012-10-25 19:38:26 -0700 1 1 1 Unclassified WebKit WebKit BlackBerry 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED P2 Normal --- 0 xuewen.ok webkit-unassigned charles.wei mifenton rwlbuis tonikitoo webkit.review.bot oldest_to_newest 750379 0 xuewen.ok 2012-10-24 22:44:04 -0700 DESCRIPTION: When closing current tab, tabs view process assert failed at AuthenticationChallengeManager::pageVisibilityChanged(). When web page exit, it delete it self from AuthenticationChallengeManager, and then set it self invisible. Since setting invisible depends on AuthenticationChallengeManager, so it assert failed. HOW TO REPRODUCE: Build debug of webkit repo, then open two tabs, then close the current tab. EXPECTED: The current tab is closed and returns to the tab bar. ACTUAL: Browser crashed at the following assertion failure. Thread [3] (Suspended: Signal 'SIGSEGV' received. Description: Segmentation fault.) 26 WebCore::AuthenticationChallengeManager::pageVisibilityChanged() AuthenticationChallengeManager.cpp:135 0x7e00496c 25 BlackBerry::WebKit::WebPage::setVisible() WebPage.cpp:3241 0x79111e7c 24 BlackBerry::WebKit::WebPagePrivate::~WebPagePrivate() WebPage.cpp:462 0x791059a8 23 BlackBerry::WebKit::WebPagePrivate::~WebPagePrivate() WebPage.cpp:501 0x79105e34 22 BlackBerry::Platform::deleteGuardedObject() BlackBerryPlatformGuardedPointer.cpp:290 0x78bd7ce8 21 BlackBerry::WebKit::WebPage::~WebPage() WebPage.cpp:505 0x79105ea4 20 BlackBerry::WebKit::WebPage::~WebPage() WebPage.cpp:507 0x79105f08 19 BlackBerry::Platform::deleteGuardedObject() BlackBerryPlatformGuardedPointer.cpp:290 0x78bd7ce8 18 BlackBerry::WebKit::WebPage::destroy() WebPage.cpp:3121 0x79111924 750402 1 170556 xuewen.ok 2012-10-24 23:11:00 -0700 Created attachment 170556 patch 750580 2 170556 charles.wei 2012-10-25 02:21:51 -0700 Comment on attachment 170556 patch Looks good to me. Need a reviewer to approve it . 750841 3 170556 rwlbuis 2012-10-25 08:05:10 -0700 Comment on attachment 170556 patch LGTM. 751434 4 170556 charles.wei 2012-10-25 18:55:28 -0700 Comment on attachment 170556 patch commit 751458 5 170556 webkit.review.bot 2012-10-25 19:38:23 -0700 Comment on attachment 170556 patch Clearing flags on attachment: 170556 Committed r132561: <http://trac.webkit.org/changeset/132561> 751459 6 webkit.review.bot 2012-10-25 19:38:26 -0700 All reviewed patches have been landed. Closing bug. 170556 2012-10-24 23:11:00 -0700 2012-10-25 19:38:22 -0700 patch 0001-BlackBerry-WebView-process-crashed-at-Authentication.patch text/plain 2882 xuewen.ok RnJvbSA1MDM1ZjlmYTA5YTI3ZGNkMWFiMzhkMWY5YTZjMWIxYjA0ZTAwYTFiIE1vbiBTZXAgMTcg MDA6MDA6MDAgMjAwMQpGcm9tOiBTZWFuIFdhbmcgPFh1ZXdlbi5XYW5nQHRvcmNobW9iaWxlLmNv bS5jbj4KRGF0ZTogVGh1LCAyNSBPY3QgMjAxMiAxMzo0OToyNSArMDgwMApTdWJqZWN0OiBbUEFU Q0hdIFtCbGFja0JlcnJ5XSBXZWJWaWV3IHByb2Nlc3MgY3Jhc2hlZCBhdAogQXV0aGVudGljYXRp b25DaGFsbGVuZ2VNYW5hZ2VyOjpwYWdlVmlzaWJpbGl0eUNoYW5nZWQoKQogaHR0cHM6Ly9idWdz LndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTEwMDMzMAoKUmV2aWV3ZWQgYnkgTk9CT0RZIChP T1BTISkuCgpSZXZpZXdlZCBpbnRlcm5hbGx5IGJ5IENoYXJsZXMgV2VpLgpQUiAjMjMxMjMyCgpT aW5jZSBBdXRoZW50aWNhdGlvbkNoYWxsZW5nZU1hbmFnZXIga2VlcHMgYSBtYXAgb2YgV2ViUGFn ZXMsIFdlYlBhZ2Ugc2V0dGluZwp2aXNpYmxlIG5lZWRzIEF1dGhlbnRpY2F0aW9uQ2hhbGxlbmdl TWFuYWdlciB0byB1cGRhdGUgaXRzIHJlY29yZHMsIGRlbGV0aW5nCndlYnBhZ2UgZnJvbSBBdXRo ZW50aWNhdGlvbkNoYWxsZW5nZU1hbmFnZXIncyBtYXAgc2hvdWxkIGJlIGRvbmUgYWZ0ZXIgc2V0 dGluZwp3ZWJwYWdlIHZpc2liaWxpdHkuCgoqIEFwaS9XZWJQYWdlLmNwcDoKKEJsYWNrQmVycnk6 OldlYktpdDo6V2ViUGFnZVByaXZhdGU6On5XZWJQYWdlUHJpdmF0ZSk6Ci0tLQogU291cmNlL1dl YktpdC9ibGFja2JlcnJ5L0FwaS9XZWJQYWdlLmNwcCB8ICAgIDMgKystCiBTb3VyY2UvV2ViS2l0 L2JsYWNrYmVycnkvQ2hhbmdlTG9nICAgICAgIHwgICAxOCArKysrKysrKysrKysrKysrKysKIDIg ZmlsZXMgY2hhbmdlZCwgMjAgaW5zZXJ0aW9ucygrKSwgMSBkZWxldGlvbigtKQoKZGlmZiAtLWdp dCBhL1NvdXJjZS9XZWJLaXQvYmxhY2tiZXJyeS9BcGkvV2ViUGFnZS5jcHAgYi9Tb3VyY2UvV2Vi S2l0L2JsYWNrYmVycnkvQXBpL1dlYlBhZ2UuY3BwCmluZGV4IGE1ZGZiMjguLjdkNTliMzcgMTAw NjQ0Ci0tLSBhL1NvdXJjZS9XZWJLaXQvYmxhY2tiZXJyeS9BcGkvV2ViUGFnZS5jcHAKKysrIGIv U291cmNlL1dlYktpdC9ibGFja2JlcnJ5L0FwaS9XZWJQYWdlLmNwcApAQCAtNDUzLDcgKzQ1Myw2 IEBAIFdlYlBhZ2U6OldlYlBhZ2UoV2ViUGFnZUNsaWVudCogY2xpZW50LCBjb25zdCBCbGFja0Jl cnJ5OjpQbGF0Zm9ybTo6U3RyaW5nJiBwYWdlCiAKIFdlYlBhZ2VQcml2YXRlOjp+V2ViUGFnZVBy aXZhdGUoKQogewotICAgIEF1dGhlbnRpY2F0aW9uQ2hhbGxlbmdlTWFuYWdlcjo6aW5zdGFuY2Uo KS0+cGFnZURlbGV0ZWQodGhpcyk7CiAgICAgLy8gSGFuZCB0aGUgYmFja2luZ3N0b3JlIGJhY2sg dG8gYW5vdGhlciBvd25lciBpZiBuZWNlc3NhcnkuCiAgICAgbV93ZWJQYWdlLT5zZXRWaXNpYmxl KGZhbHNlKTsKICAgICBpZiAoQmFja2luZ1N0b3JlUHJpdmF0ZTo6Y3VycmVudEJhY2tpbmdTdG9y ZU93bmVyKCkgPT0gbV93ZWJQYWdlKQpAQCAtNDk0LDYgKzQ5Myw4IEBAIFdlYlBhZ2VQcml2YXRl Ojp+V2ViUGFnZVByaXZhdGUoKQogICAgIGRlbGV0ZSBtX2R1bXBSZW5kZXJUcmVlOwogICAgIG1f ZHVtcFJlbmRlclRyZWUgPSAwOwogI2VuZGlmCisKKyAgICBBdXRoZW50aWNhdGlvbkNoYWxsZW5n ZU1hbmFnZXI6Omluc3RhbmNlKCktPnBhZ2VEZWxldGVkKHRoaXMpOwogfQogCiBXZWJQYWdlOjp+ V2ViUGFnZSgpCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0L2JsYWNrYmVycnkvQ2hhbmdlTG9n IGIvU291cmNlL1dlYktpdC9ibGFja2JlcnJ5L0NoYW5nZUxvZwppbmRleCBkMTNlOGJhLi5iNzc4 NWJjIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViS2l0L2JsYWNrYmVycnkvQ2hhbmdlTG9nCisrKyBi L1NvdXJjZS9XZWJLaXQvYmxhY2tiZXJyeS9DaGFuZ2VMb2cKQEAgLTEsNSArMSwyMyBAQAogMjAx Mi0xMC0yNCAgU2VhbiBXYW5nICA8WHVld2VuLldhbmdAdG9yY2htb2JpbGUuY29tLmNuPgogCisg ICAgICAgIFtCbGFja0JlcnJ5XSBXZWJWaWV3IHByb2Nlc3MgY3Jhc2hlZCBhdCBBdXRoZW50aWNh dGlvbkNoYWxsZW5nZU1hbmFnZXI6OnBhZ2VWaXNpYmlsaXR5Q2hhbmdlZCgpCisgICAgICAgIGh0 dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xMDAzMzAKKworICAgICAgICBS ZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBSZXZpZXdlZCBpbnRlcm5hbGx5 IGJ5IENoYXJsZXMgV2VpLgorICAgICAgICBQUiAjMjMxMjMyCisKKyAgICAgICAgU2luY2UgQXV0 aGVudGljYXRpb25DaGFsbGVuZ2VNYW5hZ2VyIGtlZXBzIGEgbWFwIG9mIFdlYlBhZ2VzLCBXZWJQ YWdlIHNldHRpbmcKKyAgICAgICAgdmlzaWJsZSBuZWVkcyBBdXRoZW50aWNhdGlvbkNoYWxsZW5n ZU1hbmFnZXIgdG8gdXBkYXRlIGl0cyByZWNvcmRzLCBkZWxldGluZworICAgICAgICB3ZWJwYWdl IGZyb20gQXV0aGVudGljYXRpb25DaGFsbGVuZ2VNYW5hZ2VyJ3MgbWFwIHNob3VsZCBiZSBkb25l IGFmdGVyIHNldHRpbmcKKyAgICAgICAgd2VicGFnZSB2aXNpYmlsaXR5LgorCisgICAgICAgICog QXBpL1dlYlBhZ2UuY3BwOgorICAgICAgICAoQmxhY2tCZXJyeTo6V2ViS2l0OjpXZWJQYWdlUHJp dmF0ZTo6fldlYlBhZ2VQcml2YXRlKToKKworMjAxMi0xMC0yNCAgU2VhbiBXYW5nICA8WHVld2Vu LldhbmdAdG9yY2htb2JpbGUuY29tLmNuPgorCiAgICAgICAgIFtCbGFja0JlcnJ5XSBBU1NFUlQo IW5lZWRzT25lU2hvdERyYXdpbmdTeW5jaHJvbml6YXRpb24oKSkgZmFpbGVkIGluIGNvbW1pdFJv b3RMYXllcklmTmVlZGVkKCkKICAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19i dWcuY2dpP2lkPTEwMDIxNAogCi0tIAoxLjcuOS41Cgo=