Closed
Bug 310374
Opened 19 years ago
Closed 19 years ago
strict warning in updates.js
Categories
(Toolkit :: Application Update, defect)
Toolkit
Application Update
Tracking
()
VERIFIED
FIXED
mozilla1.8.1
People
(Reporter: Peter6, Assigned: whimboo)
References
Details
Attachments
(1 file)
|
2.15 KB,
patch
|
benjamin
:
review+
|
Details | Diff | Splinter Review |
Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.8b5) Gecko/20050928
Firefox/1.4 ID:2005092815
Check for updates
JSC:
Warning: trailing comma is not legal in ECMA-262 object initializers
Source file: chrome://mozapps/content/update/updates.js
Line: 1482, Column: 0
Source code:
};
1481 gUpdates.wiz.getButton("finish").focus();
1482 },
1483 };
should be ;
|
Assignee | |
Comment 2•19 years ago
|
||
I'll take a look at these warnings. Btw. js strict warnings wont be fixed for 1.8. So this should happen on trunk.
Status: NEW → ASSIGNED
OS: Windows 2000 → All
Hardware: PC → All
Version: 1.5 Branch → Trunk
|
|
Assignee | |
Comment 3•19 years ago
|
||
Assignee: nobody → hskupin
Attachment #197878 -
Flags: review?(benjamin)
|
||
Updated•19 years ago
|
Attachment #197878 -
Flags: review?(benjamin) → review+
|
|
Assignee | |
Updated•19 years ago
|
Whiteboard: [checkin needed]
|
||
Comment 4•19 years ago
|
||
Trunk: mozilla/toolkit/mozapps/update/content/updates.js; new revision: 1.51;
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Whiteboard: [checkin needed]
Target Milestone: --- → Firefox1.6-
|
|
Assignee | |
Updated•19 years ago
|
Status: RESOLVED → VERIFIED
|
||
Updated•16 years ago
|
Product: Firefox → Toolkit
You need to log in
before you can comment on or make changes to this bug.
Description
•