Bug 488928 - statusText is set twice in Downloader::onStopRequest. r=dtownsend
authorRobert Strong <robert.bugzilla@gmail.com>
Mon, 20 Apr 2009 18:58:15 -0700
changeset 27560 176afb7235f8d8676cd6f0f7da8b7293dcc7f297
parent 27559 36f215de943333d59249cb9a9e41c2448ecb9690
child 27561 6f8a29abe2bf1074932e2768cfad2ecf4d415ecd
push id6614
push userrstrong@mozilla.com
push dateTue, 21 Apr 2009 01:59:14 +0000
treeherdermozilla-central@176afb7235f8 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersdtownsend
bugs488928
milestone1.9.2a1pre
Bug 488928 - statusText is set twice in Downloader::onStopRequest. r=dtownsend
toolkit/mozapps/update/src/nsUpdateService.js.in
--- a/toolkit/mozapps/update/src/nsUpdateService.js.in
+++ b/toolkit/mozapps/update/src/nsUpdateService.js.in
@@ -2675,20 +2675,16 @@ Downloader.prototype = {
         writeVersionFile(getUpdatesDir(), this._update.extensionVersion);
         this._update.installDate = (new Date()).getTime();
         this._update.statusText = updateStrings.GetStringFromName("installPending");
       }
       else {
         LOG("Downloader", "onStopRequest - download verification failed");
         state = STATE_DOWNLOAD_FAILED;
 
-        var brandStrings = sbs.createBundle(URI_BRAND_PROPERTIES);
-        var brandShortName = brandStrings.GetStringFromName("brandShortName");
-        this._update.statusText = updateStrings.formatStringFromName("verificationError",
-                                                                     [brandShortName], 1);
         // TODO: use more informative error code here
         status = Cr.NS_ERROR_UNEXPECTED;
 
         // Yes, this code is a string.
         const vfCode = "verification_failed";
         var message = getStatusTextFromCode(vfCode, vfCode);
         this._update.statusText = message;