diff --git a/src/core/browser_context_adapter_client.h b/src/core/browser_context_adapter_client.h index ff65f0c8f7f2167b0077fcaca8a64bb306ee96b7..a9e7eb726c82a884fde2cae640a2b42df738e3bb 100644 --- a/src/core/browser_context_adapter_client.h +++ b/src/core/browser_context_adapter_client.h @@ -64,7 +64,7 @@ public: const qint64 receivedBytes; QString path; - bool cancelled; + bool accepted; }; virtual ~BrowserContextAdapterClient() { } diff --git a/src/core/download_manager_delegate_qt.cpp b/src/core/download_manager_delegate_qt.cpp index 73a0976dae3ad8b0337d1130bbad46be20e311ad..3ef8cc9afe2317a1b71bb8f5916b95eeb2d3fb37 100644 --- a/src/core/download_manager_delegate_qt.cpp +++ b/src/core/download_manager_delegate_qt.cpp @@ -126,7 +126,6 @@ bool DownloadManagerDelegateQt::DetermineDownloadTarget(content::DownloadItem* i item->AddObserver(this); if (m_contextAdapter->client()) { - bool cancelled = false; BrowserContextAdapterClient::DownloadItemInfo info = { item->GetId(), toQt(item->GetURL()), @@ -134,18 +133,18 @@ bool DownloadManagerDelegateQt::DetermineDownloadTarget(content::DownloadItem* i item->GetTotalBytes(), item->GetReceivedBytes(), suggestedFilePath, - cancelled + false /* accepted */ }; m_contextAdapter->client()->downloadRequested(info); suggestedFile.setFile(info.path); - if (!info.cancelled && !suggestedFile.absoluteDir().mkpath(suggestedFile.absolutePath())) { + if (info.accepted && !suggestedFile.absoluteDir().mkpath(suggestedFile.absolutePath())) { qWarning("Creating download path failed, download cancelled: %s", suggestedFile.absolutePath().toUtf8().data()); - cancelled = true; + info.accepted = false; } - if (info.cancelled) { + if (!info.accepted) { cancelDownload(callback); return true; } @@ -180,7 +179,7 @@ void DownloadManagerDelegateQt::OnDownloadUpdated(content::DownloadItem *downloa download->GetTotalBytes(), download->GetReceivedBytes(), QString(), - false + true /* accepted */ }; m_contextAdapter->client()->downloadUpdated(info); } diff --git a/src/webengine/api/qquickwebengineprofile.cpp b/src/webengine/api/qquickwebengineprofile.cpp index cac309291ade47fd0980fa0dbec232d5ca1226ba..5905fb3d98b8bf27f2a53945df9cc289892cd1cf 100644 --- a/src/webengine/api/qquickwebengineprofile.cpp +++ b/src/webengine/api/qquickwebengineprofile.cpp @@ -100,8 +100,8 @@ void QQuickWebEngineProfilePrivate::downloadRequested(DownloadItemInfo &info) QQuickWebEngineDownloadItem::DownloadState state = download->state(); info.path = download->path(); - info.cancelled = state == QQuickWebEngineDownloadItem::DownloadCancelled - || state == QQuickWebEngineDownloadItem::DownloadRequested; + info.accepted = state != QQuickWebEngineDownloadItem::DownloadCancelled + && state != QQuickWebEngineDownloadItem::DownloadRequested; } void QQuickWebEngineProfilePrivate::downloadUpdated(const DownloadItemInfo &info) diff --git a/src/webenginewidgets/api/qwebengineprofile.cpp b/src/webenginewidgets/api/qwebengineprofile.cpp index 64876bf1abcfec22a41e954bb938e08903e982a5..afe0774c05335a7d273fef8e9c8a016db59307f7 100644 --- a/src/webenginewidgets/api/qwebengineprofile.cpp +++ b/src/webenginewidgets/api/qwebengineprofile.cpp @@ -152,11 +152,11 @@ void QWebEngineProfilePrivate::downloadRequested(DownloadItemInfo &info) QWebEngineDownloadItem::DownloadState state = download->state(); info.path = download->path(); - info.cancelled = state == QWebEngineDownloadItem::DownloadCancelled; + info.accepted = state != QWebEngineDownloadItem::DownloadCancelled; if (state == QWebEngineDownloadItem::DownloadRequested) { // Delete unaccepted downloads. - info.cancelled = true; + info.accepted = false; m_ongoingDownloads.remove(info.id); delete download; }