375243 - Remove some unused browser strings

Status: RESOLVED FIXED

(Firefox :: General, defect)

Description

[Phil Ringnalda (:philor)]

Attachment: Fix v.1

I bet it adds a lot to the fun of doing a new locale, or QAing an old one, having to guess what strings are unused, or which are duplicates of others elsewhere that are actually used, or in what context an unused string might hypothetically be used in some unimagined future.

I'd like to take some of that fun away.

Comment 1

[Axel Hecht [:Pike]]

Comment on attachment 259549 [details] [diff] [review]
Fix v.1

I didn't check the strings itself, but this is a really good idea, thanks for this. Scary how much you found, too.

One question on the patch, you changed the position of historyShowSidebarCmd.label to be below the
<!-- XXX remove above if places sticks -->
comment, was that intentional?

Comment 2

[Phil Ringnalda (:philor)]

It was: both places and placeless are using that same entity now, so "if places sticks" resolves down to nothing but a different entity name for the string "Home".

Comment 3

[Asaf Romano (gone)]

Comment on attachment 259549 [details] [diff] [review]
Fix v.1

r=mano.

Comment 4

[Phil Ringnalda (:philor)]

browser/locales/en-US/chrome/browser/aboutDialog.dtd: 1.8
browser/locales/en-US/chrome/browser/browser.dtd: 1.60
browser/locales/en-US/chrome/browser/browser.properties: 1.35
browser/locales/en-US/chrome/browser/openLocation.dtd: 1.3
browser/locales/en-US/chrome/browser/page-drawer.dtd: delete
browser/locales/en-US/chrome/browser/pageInfo.dtd: 1.9
browser/locales/jar.mn: 1.64