Closed
Bug 557750
Opened 14 years ago
Closed 14 years ago
Remove some old code
Categories
(Toolkit :: Safe Browsing, defect)
Toolkit
Safe Browsing
Tracking
()
RESOLVED
FIXED
Firefox 3.7a5
People
(Reporter: glandium, Assigned: glandium)
Details
Attachments
(1 file)
|
2.89 KB,
patch
|
vlad
:
review+
|
Details | Diff | Splinter Review |
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.
|
Assignee | |
Updated•14 years ago
|
OS: Linux → All
Hardware: x86 → All
|
|
Assignee | |
Comment 1•14 years ago
|
||
This already takes care of lookupURL
Assignee: nobody → mh+mozilla
Attachment #437511 -
Flags: review?(vladimir)
Attachment #437511 -
Flags: review?(vladimir) → review+
|
|
Assignee | |
Updated•14 years ago
|
Keywords: checkin-needed
|
|
Assignee | |
Updated•14 years ago
|
Status: NEW → ASSIGNED
http://hg.mozilla.org/mozilla-central/rev/8c3222567a05
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3.7a5
|
||
Updated•10 years ago
|
Product: Firefox → Toolkit
You need to log in
before you can comment on or make changes to this bug.
Description
•