822548 - fix a bunch of JS warnings on startup

Status: RESOLVED FIXED

(Firefox :: General, defect)

Description

[Cameron McCormack (:heycam)]

Attachment: patch

There are a "did you mean = instead of ==" and "argument/variable name redeclaration" warnings at startup.

Comment 1

[Dão Gottwald [:dao]]

Comment on attachment 693237 [details] [diff] [review]
patch

>--- a/browser/components/sessionstore/src/SessionStore.jsm
>+++ b/browser/components/sessionstore/src/SessionStore.jsm
>@@ -1274,7 +1274,7 @@ let SessionStoreInternal = {
>     // we need to reset that state. If the tab was restoring, we will attempt to
>     // restore the next tab.
>     let previousState;
>-    if (previousState = browser.__SS_restoreState) {
>+    if ((previousState = browser.__SS_restoreState)) {
>       this._resetTabRestoringState(aTab);
>       if (previousState == TAB_STATE_RESTORING)
>         this.restoreNextTab();

write this instead:

let previousState = browser.__SS_restoreState;
if (previousState) {

>--- a/toolkit/mozapps/update/nsUpdateService.js
>+++ b/toolkit/mozapps/update/nsUpdateService.js
>@@ -3622,9 +3622,9 @@ Downloader.prototype = {
>         LOG("Downloader:onStopRequest - verification of patch failed, " +
>             "downloading complete update patch");
>         this._update.isCompleteUpdate = true;
>-        var status = this.downloadUpdate(this._update);
>+        var updateStatus = this.downloadUpdate(this._update);
> 
>-        if (status == STATE_NONE) {
>+        if (updateStatus == STATE_NONE) {
>           cleanupActiveUpdate();
>         } else {
>           allFailed = false;

use 'let' rather than 'var'

Comment 2

[Cameron McCormack (:heycam)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/18e8cb967f00

Comment 3

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/18e8cb967f00