Closed Bug 375249 Opened 17 years ago Closed 17 years ago

Remove unused strings in safebrowsing

Categories

(Toolkit :: Safe Browsing, defect)

Product:
Toolkit
Component:
Safe Browsing
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
Firefox 3 alpha4

People

(Reporter: philor, Assigned: philor)

Details

Attachments

(1 file)

Fix v.1
8.28 KB, patch
Gavin
: review+
Details | Diff | Splinter Review
Attached patch Fix v.1 — — Splinter Review 
safeb.palm.toolmenu.* was from when the extension put things in the Tools menu. safeb.palm.addbutton.* I don't remember, and CVS archeology is failing me, but it's unused. safeb.palm.provider.logo.tooltip is only used inside a comment, and while it's a feather in the cap of the eight or ten locales that didn't translate it, without anyone planning on using it it's cruft. Then since I was in the neighborhood, I touched some whitespace.
Attachment #259560 - Flags: review?(gavin.sharp)
Attachment #259560 - Flags: review?(gavin.sharp) → review+
(Oops, meant to cc Axel and his watchers first thing, not last thing.)

browser/locales/en-US/chrome/browser/safebrowsing/phishing-afterload-warning-message.dtd: 1.9
browser/components/safebrowsing/content/warning-overlay.xul: 1.12
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Product: Firefox → Toolkit
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: