Bug 100330 - [BlackBerry] WebView process crashed at AuthenticationChallengeManager::pageVisibilityChanged()
Summary: [BlackBerry] WebView process crashed at AuthenticationChallengeManager::pageV...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit BlackBerry (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-10-24 22:44 PDT by Sean Wang
Modified: 2012-10-25 19:38 PDT (History)
5 users (show)

See Also:

Attachments
patch (2.81 KB, patch)
2012-10-24 23:11 PDT, Sean Wang 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Sean Wang 2012-10-24 22:44:04 PDT 
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
Comment 1 Sean Wang 2012-10-24 23:11:00 PDT 
Created attachment 170556 [details]
patch
Comment 2 Charles Wei 2012-10-25 02:21:51 PDT 
Comment on attachment 170556 [details]
patch

Looks good to me. Need a reviewer to approve it .
Comment 3 Rob Buis 2012-10-25 08:05:10 PDT 
Comment on attachment 170556 [details]
patch

LGTM.
Comment 4 Charles Wei 2012-10-25 18:55:28 PDT 
Comment on attachment 170556 [details]
patch

commit
Comment 5 WebKit Review Bot 2012-10-25 19:38:23 PDT 
Comment on attachment 170556 [details]
patch

Clearing flags on attachment: 170556

Committed r132561: <http://trac.webkit.org/changeset/132561>
Comment 6 WebKit Review Bot 2012-10-25 19:38:26 PDT 
All reviewed patches have been landed.  Closing bug.