Bug 1038465: Fix bad merge to get rid of aSuggestedFilename (r=me)
authorMonica Chew <mmc@mozilla.com>
Wed, 16 Jul 2014 10:21:35 -0700
changeset 216342 80af4d90c3d570f428d842d8147adcdca91fa6eb
parent 216341 21974ff3c9d76b927d96b285403eeef783c838f7
child 216343 8e8f3ba646554c13de1f26140d8a954b79d7549b
child 216419 ed555132e89568a10b95d3bc030ea348f4dc73b9
push id515
push userraliiev@mozilla.com
push dateMon, 06 Oct 2014 12:51:51 +0000
treeherdermozilla-release@267c7a481bef [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersme
bugs1038465
milestone33.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1038465: Fix bad merge to get rid of aSuggestedFilename (r=me)
toolkit/components/jsdownloads/src/DownloadIntegration.jsm
--- a/toolkit/components/jsdownloads/src/DownloadIntegration.jsm
+++ b/toolkit/components/jsdownloads/src/DownloadIntegration.jsm
@@ -512,23 +512,22 @@ this.DownloadIntegration = {
     if (!hash || !sigInfo) {
       return Promise.resolve(false);
     }
     let deferred = Promise.defer();
     let aReferrer = null;
     if (aDownload.source.referrer) {
       aReferrer: NetUtil.newURI(aDownload.source.referrer);
     }
-    let aSuggestedFileName = OS.Path.basename(aDownload.target.path);
     gApplicationReputationService.queryReputation({
       sourceURI: NetUtil.newURI(aDownload.source.url),
       referrerURI: aReferrer,
       fileSize: aDownload.currentBytes,
       sha256Hash: hash,
-      suggestedFileName: aSuggestedFileName,
+      suggestedFileName: OS.Path.basename(aDownload.target.path),
       signatureInfo: sigInfo,
       redirects: channelRedirects },
       function onComplete(aShouldBlock, aRv) {
         deferred.resolve(aShouldBlock);
       });
     return deferred.promise;
   },