Closed Bug 823042 Opened 7 years ago Closed 7 years ago

"variable status redeclares argument" from update.js in error console

Categories

(Toolkit :: Add-ons Manager, defect)

x86_64
Linux
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla20

People

(Reporter: jik, Assigned: jik)

Details

Attachments

(1 file)

Timestamp: 12/19/2012 09:00:12 AM
Warning: TypeError: variable status redeclares argument
Source File: chrome://mozapps/content/extensions/update.js
Line: 216, Column: 8
Source Code:
    var status = document.getElementById("versioninfo.status"); 

Timestamp: 12/19/2012 09:00:12 AM
Warning: TypeError: variable status redeclares argument
Source File: chrome://mozapps/content/extensions/update.js
Line: 290, Column: 8
Source Code:
    var status = document.getElementById("checking.status"); 

I will attach a proposed (trivial) patch.
Attachment #693853 - Flags: review?(joshmoz)
Assignee: nobody → jik
Status: NEW → ASSIGNED
This is in the browser module, although i'm not entirely sure which specific component in the Firefox product this should be in.
Component: Plug-ins → General
Product: Core → Firefox
(In reply to Georg Fritzsche [:gfritzsche] from comment #2)
> This is in the browser module, although i'm not entirely sure which specific
> component in the Firefox product this should be in.

It's used by Thunderbird (I saw the error in the error console in TB, not Firefox), so I don't think the bug belongs in the browser module.
Sorry, took a second look - the affected part is in toolkit, so moving it there.
Component: General → Add-ons Manager
Product: Firefox → Toolkit
Comment on attachment 693853 [details] [diff] [review]
rename function-local status variable

Clearing out the review request for Josh as he's not a peer there.
Attachment #693853 - Flags: review?(joshmoz)
Attachment #693853 - Flags: review?(dtownsend+bugmail)
Attachment #693853 - Flags: review?(dtownsend+bugmail) → review+
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/4f74d77d6d8b
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla20
You need to log in before you can comment on or make changes to this bug.