Closed
Bug 335387
Opened 19 years ago
Closed 18 years ago
remove remnants of content packs
Categories
(Core :: Internationalization: Localization, defect)
Core
Internationalization: Localization
Tracking
()
RESOLVED
FIXED
People
(Reporter: kairo, Assigned: kairo)
References
Details
Attachments
(1 file)
2.29 KB,
patch
|
benjamin
:
review+
|
Details | Diff | Splinter Review |
After content packs have been killed in toolkit a long time ago, and bug 325473 also removed those separate packs from suite, we should wipe the remnants of content pack handling from our code as well, like contentlocale prefs etc.
Those lxr searches turn up places where we still refer to content packs:
http://lxr.mozilla.org/mozilla/search?string=contentlocale
http://lxr.mozilla.org/mozilla/search?string=content+pack
http://lxr.mozilla.org/mozilla/search?string=content-pack
http://lxr.mozilla.org/mozilla/search?string=localeType
http://lxr.mozilla.org/seamonkey/search?string=regus
Note that help knows the concept of "help content packs", which are something
different than what we are handling here, but turn up in some searches.
![]() |
Assignee | |
Comment 1•18 years ago
|
||
OK, going through the occurrences we have left there, I'm doing some analysis:
http://lxr.mozilla.org/mozilla/search?string=content-pack
http://lxr.mozilla.org/seamonkey/search?string=regus
no (relevant) results.
http://lxr.mozilla.org/mozilla/search?string=localeType
3 results:
1+2 are filtering out obsolete region packs from selections,
3 is in old, obsolete chrome registry (/rdf/chrome/)
http://lxr.mozilla.org/mozilla/search?string=content+pack
all results talking about help content packs, which are something different, or content packages, which are different as well, so:
no relevant results.
http://lxr.mozilla.org/mozilla/search?string=contentlocale
left over startup switch, prefs, etc. - this really needs to be cleanup up
![]() |
Assignee | |
Comment 2•18 years ago
|
||
bsmedberg says everything in that last result except the toolkit nsAppRunner and all.js is dead or obsolete, so I need to clean up only those two ;-)
![]() |
Assignee | |
Comment 3•18 years ago
|
||
This should remove everything still relevant that I know of where region/content packs are still mentioned.
Attachment #265292 -
Flags: review?(benjamin)
Updated•18 years ago
|
Attachment #265292 -
Flags: review?(benjamin) → review+
![]() |
Assignee | |
Comment 4•18 years ago
|
||
checked in, fixed.
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•