strict JS error in phishingDetector.js

RESOLVED FIXED

Status

--
trivial
RESOLVED FIXED
14 years ago
13 years ago

People

(Reporter: iann_bugzilla, Assigned: iann_bugzilla)

Tracking

({fixed-seamonkey1.1a, fixed1.8.1})

Trunk
fixed-seamonkey1.1a, fixed1.8.1

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

14 years ago
Line 108 of phishingDetector.js gives a strict JS error, hrefURL is not declared.
(Assignee)

Comment 1

14 years ago
Created attachment 194956 [details] [diff] [review]
Simple patch v0.1

This patch:
* Declares hrefURL
Assignee: mail → iann_bugzilla
Status: NEW → ASSIGNED
Attachment #194956 - Flags: review?(neil.parkwaycc.co.uk)
Comment on attachment 194956 [details] [diff] [review]
Simple patch v0.1

>-  hrefURL  = ioService.newURI(href, null, null);
>+  var hrefURL  = ioService.newURI(href, null, null);
Nit: fix the doubled space before = while you're at it.
Attachment #194956 - Flags: review?(neil.parkwaycc.co.uk) → review+
(Assignee)

Comment 3

14 years ago
Created attachment 194964 [details] [diff] [review]
Tweaked patch v0.1a (Checked in on trunk and MOZILLA_1_8_BRANCH)

Changes since v0.1:
* Removed extra space from before =

Carrying forward r= and requesting sr=
Attachment #194956 - Attachment is obsolete: true
Attachment #194964 - Flags: superreview?(bienvenu)
Attachment #194964 - Flags: review+

Updated

14 years ago
Attachment #194964 - Flags: superreview?(bienvenu) → superreview+
(Assignee)

Comment 4

14 years ago
Comment on attachment 194964 [details] [diff] [review]
Tweaked patch v0.1a (Checked in on trunk and MOZILLA_1_8_BRANCH)

Checking in
phishingDetector.js;
new revision: 1.3; previous revision: 1.2
done
Attachment #194964 - Attachment description: Tweaked patch v0.1a → Tweaked patch v0.1a (Checked in)
(Assignee)

Updated

14 years ago
Severity: normal → trivial
Status: ASSIGNED → RESOLVED
Last Resolved: 14 years ago
OS: Windows XP → All
Hardware: PC → All
Resolution: --- → FIXED

Comment 5

13 years ago
Comment on attachment 194964 [details] [diff] [review]
Tweaked patch v0.1a (Checked in on trunk and MOZILLA_1_8_BRANCH)

See bug 343059.
Attachment #194964 - Flags: approval-seamonkey1.1a?

Comment 6

13 years ago
Comment on attachment 194964 [details] [diff] [review]
Tweaked patch v0.1a (Checked in on trunk and MOZILLA_1_8_BRANCH)

a=me for 1.1
Attachment #194964 - Flags: approval-seamonkey1.1a? → approval-seamonkey1.1a+

Updated

13 years ago
Attachment #194964 - Attachment description: Tweaked patch v0.1a (Checked in) → Tweaked patch v0.1a (Checked in on trunk and MOZILLA_1_8_BRANCH)

Updated

13 years ago
Keywords: fixed-seamonkey1.1a, fixed1.8.1
Comment on attachment 194964 [details] [diff] [review]
Tweaked patch v0.1a (Checked in on trunk and MOZILLA_1_8_BRANCH)

Checkin: {
2006-07-12 13:05	mnyromyr%tprac.de 	mozilla/mailnews/base/resources/content/phishingDetector.js 	1.1.2.5 	MOZILLA_1_8_BRANCH
}

Comment 8

13 years ago
(In reply to comment #7)
> Checkin: {
...
Actually, I consider such comments bugspam, hence I don't do it.
I rather mark the *patch* as being checked in, so you can see its state by looking at the patch without grepping through gazillions of comments...
You need to log in before you can comment on or make changes to this bug.