Bug 54892

Summary: [Qt] [WK2] QtWebProcess is leaked if it's not terminated.
Product: WebKit Reporter: Kimmo Kinnunen <kimmo.t.kinnunen@nokia.com>
Component: WebKit2Assignee: Kimmo Kinnunen <kimmo.t.kinnunen@nokia.com>
Status: RESOLVED INVALID    
Severity: Normal CC: abecsi@webkit.org, commit-queue@webkit.org, kbalazs@webkit.org, ossy@webkit.org
Priority: P2 Keywords: Qt
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Linux   
Attachments:
Description Flags
fix kenneth: review+, commit-queue: commit‑queue-

Description From 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 From 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 From 2011-02-21 10:46:55 PST -------
(and also it's is accessed from the wrong thread)
------- Comment #3 From 2011-02-21 13:05:42 PST -------
Created an attachment (id=83209) [details]
fix
------- Comment #4 From 2011-05-16 14:58:09 PST -------
(From update of attachment 83209 [details])
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 From 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.