Enable safebrowsing on b2g by default

RESOLVED FIXED

Status

Firefox OS
General
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: fabrice, Assigned: fabrice)

Tracking

unspecified
ARM
Gonk (Firefox OS)

Firefox Tracking Flags

(firefox45 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

2 years ago
We don't target low end devices anymore so there's no resource issues to worry about.
(Assignee)

Comment 1

2 years ago
Tested by opening http://www.itisatrap.org/firefox/its-an-attack.html
We get a nice error page explaining how bad this page is.
(Assignee)

Comment 2

2 years ago
Created attachment 8688097 [details] [diff] [review]
default-safebrowsing.patch
Assignee: nobody → fabrice
Attachment #8688097 - Flags: review?(anygregor)
Blocks: 1180503
What changed since bug 1180503?
(Assignee)

Comment 4

2 years ago
iirc François was saying that turning on safebrowsing could cost ~ 3MB, and at the time we still had to support 256MB configuration. This is not the case anymore so I think it's fine to turn on by default. It's easy enough to disable on a per-device basis if we need to.
Attachment #8688097 - Flags: review?(anygregor) → review+

Comment 5

2 years ago
https://hg.mozilla.org/integration/b2g-inbound/rev/7fed9cad6d94

Comment 6

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/7fed9cad6d94
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox45: --- → fixed
Resolution: --- → FIXED
This doesn't download stuff while on a mobile data connection?
You need to log in before you can comment on or make changes to this bug.