Closed
Bug 158072
Opened 23 years ago
Closed 23 years ago
remove dead function selectLocale from strres.js
Categories
(SeaMonkey :: UI Design, defect)
SeaMonkey
UI Design
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla1.1beta
People
(Reporter: jrgmorrison, Assigned: jrgmorrison)
References
()
Details
(Keywords: perf)
Attachments
(1 file)
|
1.72 KB,
patch
|
timeless
:
review+
jag+mozilla
:
superreview+
asa
:
approval+
|
Details | Diff | Splinter Review |
The function selectLocale() in strres.js
http://lxr.mozilla.org/seamonkey/source/xpfe/global/resources/content/strres.js#26
is dead code:
http://lxr.mozilla.org/seamonkey/search?string=selectLocale
However, it's loaded into 64 different xul windows and dialogs.
selectLocale must die (leaving strres.js as a curiously skinny <script> file)
|
Assignee | |
Comment 1•23 years ago
|
||
Let's kill this for 1.1final
Keywords: perf
Target Milestone: --- → mozilla1.1beta
|
|
Assignee | |
Comment 2•23 years ago
|
||
p.s. it's dead in the commercial tree too according to lxr.
|
|
Assignee | |
Comment 3•23 years ago
|
||
Attachment #91772 -
Flags: review+
|
||
Comment 4•23 years ago
|
||
Comment on attachment 91772 [details] [diff] [review]
remove js function selectLocale
DEATH!
sr=jag
Attachment #91772 -
Flags: superreview+
|
||
Comment 6•23 years ago
|
||
Comment on attachment 91772 [details] [diff] [review]
remove js function selectLocale
a=asa (on behalf of drivers) for checkin to 1.1
Attachment #91772 -
Flags: approval+
|
|
Assignee | |
Comment 7•23 years ago
|
||
Fixed. checked in, trunk 17:38 07/24
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
|
||
Updated•21 years ago
|
Product: Core → Mozilla Application Suite
You need to log in
before you can comment on or make changes to this bug.
Description
•