Bug 37855 - [Qt] cleaning up QNetworkReply
Summary: [Qt] cleaning up QNetworkReply
Status: RESOLVED INVALID
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Qt (show other bugs)
Version: 418.x
Hardware: PC Linux
: P3 Normal
Assignee: Nobody
URL:
Keywords: EasyFix, Qt, QtTriaged
Depends on:
Blocks:
 
Reported: 2010-04-20 06:59 PDT by Johannes Zellner
Modified: 2014-02-03 03:13 PST (History)
4 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Johannes Zellner 2010-04-20 06:59:26 PDT
Regarding the documentation it is not quite clear who is when responsible to cleanup the QNetworkReply used in the signals QWebPage::unsupportedContent(QNetworkReply *reply); and QWebPage::finished(QNetworkReply*)

Using "deleteLater()" on the QNetworkReply, as stated in the docs, is ok in the SLOT connected to QWebPage::unsupportedContent(QNetworkReply *reply), but leads to segfault if used in the SLOT connected to QWebPage::finished(QNetworkReply*)
Comment 1 Jesus Sanchez-Palencia 2010-05-12 08:08:35 PDT
Can you please provide a simple test case for this?
Comment 2 Piotr Dobrogost 2010-06-13 02:14:41 PDT
There's no
QWebPage::finished(QNetworkReply*) [signal]
present in the API. There are
void QWebPage::loadFinished ( bool ok ) [signal]
and 
void QNetworkAccessManager::finished ( QNetworkReply * reply ) [signal]

Did you mean the second one?
Comment 3 Johannes Zellner 2010-06-14 01:01:03 PDT
sorry you are right, I meant the 

void QNetworkAccessManager::finished ( QNetworkReply * reply ) [signal]
Comment 4 Jocelyn Turcotte 2014-02-03 03:13:29 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.