Bug 93237 - [Qt] When deleting a webview that has an open javascript alert will cause a crash
Summary: [Qt] When deleting a webview that has an open javascript alert will cause a c...
Status: RESOLVED INVALID
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P3 Normal
Assignee: Nobody
URL:
Keywords: Qt
Depends on:
Blocks:
 
Reported: 2012-08-06 02:19 PDT by andy.shaw
Modified: 2014-02-03 03:22 PST (History)
1 user (show)

See Also:

Attachments
Example to reproduce the issue (2.82 KB, application/x-zip-compressed)
2012-08-06 02:20 PDT, andy.shaw 		no flags Details
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 andy.shaw 2012-08-06 02:19:44 PDT 
[Qt] When deleting a webview that has an open javascript alert will cause a crash.  To workaround this then delay the deletion of the webview until after the alert is closed.  The attached example reproduces the problem and uncommenting the code in MainApp::deleteWebView() will give a workaround.
Comment 1 andy.shaw 2012-08-06 02:20:09 PDT 
Created attachment 156630 [details]
Example to reproduce the issue
Comment 2 Matt Horan 2012-08-29 20:37:43 PDT 
We're seeing a similar issue in capybara-webkit.  See the discussion here: http://git.io/jKcw_g.

Basically, if deleteLater causes a WebPage to be deleted while evaluateJavaScript is still running, the resultant message is delivered by WebCore/JSC internals to an invalid object.  This results in a segfault.

Although the errors in capybara-webkit require redesign to work properly, given that loadFinished does not signify that javaScriptWindowObjectCleared has been fired, it is still concerning that using deleteLater still causes a segfault, even though it is supposed to wait for all signals to be delivered.
Comment 3 Jocelyn Turcotte 2014-02-03 03:22:03 PST 
=== Bulk closing of Qt bugs ===

If you believe that this bug report is still relevant for a non-Qt port of webkit.org, please re-open it and remove [Qt] from the summary.

If you believe that this is still an important QtWebKit bug, please fill a new report at https://bugreports.qt-project.org and add a link to this issue. See http://qt-project.org/wiki/ReportingBugsInQt for additional guidelines.