Closed Bug 557750 Opened 10 years ago Closed 10 years ago

Remove some old code

Categories

(Toolkit :: Safe Browsing, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
Firefox 3.7a5

People

(Reporter: glandium, Assigned: glandium)

Details

Attachments

(1 file)

While discussing with people @google about getting safebrowsing to work with Iceweasel, it appeared there is at least one variable that is (rightfully) not being used anymore, namely browser.safebrowsing.provider.0.lookupURL. It is apparently remaining from the old protocol. There is a getter for it in browser/components/safebrowsing/content/globalstore.js, which is thus pointless.

I'm wondering if there aren't more of these now unused variables.
OS: Linux → All
Hardware: x86 → All
This already takes care of lookupURL
Assignee: nobody → mh+mozilla
Attachment #437511 - Flags: review?(vladimir)
Keywords: checkin-needed
Status: NEW → ASSIGNED
http://hg.mozilla.org/mozilla-central/rev/8c3222567a05
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3.7a5
Product: Firefox → Toolkit
You need to log in before you can comment on or make changes to this bug.