From 120aa8a81e3b3aceb68cca0b998154ccddf89d6c Mon Sep 17 00:00:00 2001 From: NTD Date: Thu, 31 Jan 2019 07:35:32 -0500 Subject: [PATCH] Only state the stop notification for JSDownloads in nsWebBrowserPersist::EndDownload --- .../components/webbrowserpersist/nsWebBrowserPersist.cpp | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/embedding/components/webbrowserpersist/nsWebBrowserPersist.cpp b/embedding/components/webbrowserpersist/nsWebBrowserPersist.cpp index 437d21997a..a31414e7fa 100644 --- a/embedding/components/webbrowserpersist/nsWebBrowserPersist.cpp +++ b/embedding/components/webbrowserpersist/nsWebBrowserPersist.cpp @@ -2351,16 +2351,18 @@ 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))