Bug 54892 - [Qt] [WK2] QtWebProcess is leaked if it's not terminated.
Summary: [Qt] [WK2] QtWebProcess is leaked if it's not terminated.
Status: RESOLVED INVALID
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit2 (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC Linux
: P2 Normal
Assignee: Kimmo Kinnunen
URL:
Keywords: Qt
Depends on:
Blocks:
 
Reported: 2011-02-21 10:44 PST by Kimmo Kinnunen
Modified: 2014-02-03 03:17 PST (History)
4 users (show)

See Also:


Attachments
fix (6.82 KB, patch)
2011-02-21 13:05 PST, Kimmo Kinnunen
kenneth: review+
commit-queue: commit-queue-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Kimmo Kinnunen 2011-02-21 10:44:02 PST
In ProcessLauncher, QtWebProcess is instantiated in work queue thread but deleted from ui thread. this causes segfaults.
Comment 1 Kimmo Kinnunen 2011-02-21 10:46:31 PST
Changing the summary. Actually the web process instance is leaked if the process is not terminated.
Comment 2 Kimmo Kinnunen 2011-02-21 10:46:55 PST
(and also it's is accessed from the wrong thread)
Comment 3 Kimmo Kinnunen 2011-02-21 13:05:42 PST
Created attachment 83209 [details]
fix
Comment 4 WebKit Commit Bot 2011-05-16 14:58:09 PDT
Comment on attachment 83209 [details]
fix

Rejecting attachment 83209 [details] from commit-queue.

Failed to run "['./Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '--bot-id=eseidel-cq-sf', 'ap..." exit_code: 2

Last 500 characters of output:
 1 hunk ignored
patching file Source/WebKit2/Platform/WorkQueue.h
Hunk #1 succeeded at 86 (offset 2 lines).
Hunk #2 succeeded at 165 (offset 3 lines).
patching file Source/WebKit2/Platform/qt/WorkQueueQt.cpp
patching file Source/WebKit2/UIProcess/Launcher/qt/ProcessLauncherQt.cpp
Hunk #1 succeeded at 144 (offset 14 lines).
Hunk #2 succeeded at 155 (offset 14 lines).

Failed to run "[u'/Projects/CommitQueue/Tools/Scripts/svn-apply', u'--reviewer', u'Kenneth Rohde Christiansen', ..." exit_code: 1

Full output: http://queues.webkit.org/results/8707072
Comment 5 Jocelyn Turcotte 2014-02-03 03:17:19 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.