From 7c52cc8e7f358c155b93a106bda75d926b46aa42 Mon Sep 17 00:00:00 2001 From: Fedor Date: Thu, 7 May 2020 14:44:52 +0300 Subject: [PATCH] Revert "Only state the stop notification for JSDownloads in nsWebBrowserPersist::EndDownload" --- .../components/webbrowserpersist/nsWebBrowserPersist.cpp | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/embedding/components/webbrowserpersist/nsWebBrowserPersist.cpp b/embedding/components/webbrowserpersist/nsWebBrowserPersist.cpp index a31414e7f..437d21997 100644 --- a/embedding/components/webbrowserpersist/nsWebBrowserPersist.cpp +++ b/embedding/components/webbrowserpersist/nsWebBrowserPersist.cpp @@ -2351,18 +2351,16 @@ nsWebBrowserPersist::EndDownload(nsresult aResult) { mPersistResult = aResult; } + // mCompleted needs to be set before issuing the stop notification. // (Bug 1224437) mCompleted = true; - -#ifdef MOZ_JSDOWNLOADS // State stop notification if (mProgressListener) { mProgressListener->OnStateChange(nullptr, nullptr, nsIWebProgressListener::STATE_STOP | nsIWebProgressListener::STATE_IS_NETWORK, mPersistResult); } -#endif // Do file cleanup if required if (NS_FAILED(aResult) && (mPersistFlags & PERSIST_FLAGS_CLEANUP_ON_FAILURE))