Closed Bug 320361 Opened 20 years ago Closed 19 years ago

Phishing Detector - unneeded code.

Categories

(MailNews Core :: Backend, defect)

Product:
MailNews Core
Component:
Backend
Type:
defect
Priority:
Not set
Severity:
trivial

Tracking

(Not tracked)

Status:
RESOLVED INVALID

People

(Reporter: ray, Assigned: ray)

Details

I know this is a nit, but is this line of code needed: http://lxr.mozilla.org/seamonkey/source/mail/base/content/phishingDetector.js#144 All calls to this function have already checked if the URL is http or https so why another if statement? Opening as UNCON as I would like this confirmed by someone else.
I'm not sure that's the case - it looks to me like some code is checking the link text, and some is checking the url.
Yeah, you're right. Closing.
Status: UNCONFIRMED → RESOLVED
Closed: 19 years ago
Resolution: --- → INVALID
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.