Port |Bug 572463 - Prototype urlclassifier/safe browsing client code in JS| to SeaMonkey

VERIFIED FIXED in seamonkey2.2

Status

VERIFIED FIXED
8 years ago
6 years ago

People

(Reporter: sgautherie, Assigned: sgautherie)

Tracking

(Depends on: 1 bug)

Trunk
seamonkey2.2
Dependency tree / graph
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

Comment hidden (empty)
(Assignee)

Comment 1

8 years ago
Created attachment 528584 [details] [diff] [review]
(Av1) Package nsUrlClassifierHashCompleter.js, except on MOZILLA_5_0_BRANCH
[Checked in: Comment 2]
Attachment #528584 - Flags: review?(bugspam.Callek)

Updated

8 years ago
Attachment #528584 - Flags: review?(bugspam.Callek) → review+
(Assignee)

Comment 2

8 years ago
Comment on attachment 528584 [details] [diff] [review]
(Av1) Package nsUrlClassifierHashCompleter.js, except on MOZILLA_5_0_BRANCH
[Checked in: Comment 2]

http://hg.mozilla.org/comm-central/rev/7684d92c8e42
Attachment #528584 - Attachment description: (Av1) Package nsUrlClassifierHashCompleter.js, except on MOZILLA_5_0_BRANCH → (Av1) Package nsUrlClassifierHashCompleter.js, except on MOZILLA_5_0_BRANCH [Checked in: Comment 2]
(Assignee)

Updated

8 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
(Assignee)

Comment 3

8 years ago
Arf, I assumed SM needed this (as is) as FF and TB include this file, but I was wrong :-(

nsUrlClassifierHashCompleter.js is in (m-c)/toolkit/components/url-classifier/,
which is built with MOZ_URL_CLASSIFIER only,
which is usually (automatically) enabled with MOZ_SAFE_BROWSING.

http://mxr.mozilla.org/comm-central/search?string=MOZ_SAFE_BROWSING%3D&case=1&find=%2Fconfvars%5C.sh%24
FF and TB enable the latter, but not SM (yet).

Should I file a bug to investigate enabling Safe Browsing? (Or what's the plan/status?)
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(Assignee)

Comment 4

8 years ago
Created attachment 528835 [details] [diff] [review]
(Bv1) SeaMonkey needs |ifdef MOZ_URL_CLASSIFIER| (ftb)
[Checked in: Comment 7]

Untested, but should be it.
Attachment #528835 - Flags: review?(bugspam.Callek)

Comment 5

8 years ago
(In reply to comment #3)
> Should I file a bug to investigate enabling Safe Browsing? (Or what's the
> plan/status?)

I think a bug is filed, just nobody working on it. And some people think the feature itself is controversial - I was always for including it if someone has the time, but some people don't want us to block anything, if even it's known malware-installing sites.
(Assignee)

Updated

8 years ago
Depends on: 653605
(Assignee)

Comment 6

8 years ago
(In reply to comment #5)
> I think a bug is filed

I filed bug 653605, to be sure.
Blocks: 653605
No longer depends on: 653605

Updated

8 years ago
No longer blocks: 653605

Updated

8 years ago
Blocks: 477718

Updated

8 years ago
Attachment #528835 - Flags: review?(bugspam.Callek) → review+
(Assignee)

Comment 7

8 years ago
Comment on attachment 528835 [details] [diff] [review]
(Bv1) SeaMonkey needs |ifdef MOZ_URL_CLASSIFIER| (ftb)
[Checked in: Comment 7]

http://hg.mozilla.org/comm-central/rev/10c820cd498b
Attachment #528835 - Attachment description: (Bv1) SeaMonkey needs |ifdef MOZ_URL_CLASSIFIER| (ftb) → (Bv1) SeaMonkey needs |ifdef MOZ_URL_CLASSIFIER| (ftb) [Checked in: Comment 7]
(Assignee)

Updated

8 years ago
Status: REOPENED → RESOLVED
Last Resolved: 8 years ago8 years ago
Resolution: --- → FIXED
(Assignee)

Comment 8

8 years ago
http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1305438151.1305441945.18430.gz&fulltext=1
Linux comm-central-trunk build on 2011/05/14 22:42:31
{
Warning: package error or possible missing or unnecessary file: bin/components/nsUrlClassifierHashCompleter.js (package-manifest, 294).
}

http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1305448015.1305451047.25033.gz&fulltext=1
Linux comm-central-trunk build on 2011/05/15 01:26:55

V.Fixed.
Status: RESOLVED → VERIFIED
(Assignee)

Updated

8 years ago
Depends on: 526333
(Assignee)

Updated

7 years ago
Blocks: 738883
(Assignee)

Updated

7 years ago
No longer blocks: 477718

Updated

6 years ago
Blocks: 477718
You need to log in before you can comment on or make changes to this bug.