RESOLVED INVALID 37855
[Qt] cleaning up QNetworkReply 
https://bugs.webkit.org/show_bug.cgi?id=37855
Summary [Qt] cleaning up QNetworkReply
Johannes Zellner
Reported 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*)
Attachments
Add attachment proposed patch, testcase, etc.
Jesus Sanchez-Palencia
Comment 1 2010-05-12 08:08:35 PDT
Can you please provide a simple test case for this?
Piotr Dobrogost
Comment 2 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?
Johannes Zellner
Comment 3 2010-06-14 01:01:03 PDT
sorry you are right, I meant the void QNetworkAccessManager::finished ( QNetworkReply * reply ) [signal]
Jocelyn Turcotte
Comment 4 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.
Note You need to log in before you can comment on or make changes to this bug.