Closed Bug 335387 Opened 19 years ago Closed 18 years ago

remove remnants of content packs

Categories

(Core :: Internationalization: Localization, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: kairo, Assigned: kairo)

References

Details

Attachments

(1 file)

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.
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
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 ;-)
This should remove everything still relevant that I know of where region/content packs are still mentioned.
Attachment #265292 - Flags: review?(benjamin)
Attachment #265292 - Flags: review?(benjamin) → review+
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.

Attachment

General

Creator:
Created:
Updated:
Size: