Closed
Bug 1259288
Opened 8 years ago
Closed 8 years ago
Safe Browsing should be fully disabled in our test suites
Categories
(Toolkit :: Safe Browsing, defect)
Toolkit
Safe Browsing
Tracking
()
RESOLVED
FIXED
mozilla48
Tracking | Status | |
---|---|---|
firefox48 | --- | fixed |
People
(Reporter: francois, Assigned: francois)
Details
Attachments
(1 file)
I found that while the basic Safe Browsing functionality is disabled in these places, some of the new additions (tracking protection, forbidden sites) are missing: https://hg.mozilla.org/mozilla-central/file/6606ab35633b/testing/talos/talos/config.py#l110 https://dxr.mozilla.org/mozilla-central/rev/6202ade0e6d688ffb67932398e56cfc6fa04ceb3/layout/tools/reftest/reftest-preferences.js#83 https://dxr.mozilla.org/mozilla-central/rev/6202ade0e6d688ffb67932398e56cfc6fa04ceb3/js/src/tests/user.js#26-27 https://dxr.mozilla.org/mozilla-central/source/testing/marionette/client/marionette_driver/geckoinstance.py#205 This means that list updates could actually happen during theses tests.
Assignee | ||
Comment 1•8 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/42133/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/42133/
Attachment #8734201 -
Flags: review?(gpascutto)
Comment 2•8 years ago
|
||
Comment on attachment 8734201 [details] MozReview Request: Bug 1259288 - Safe Browsing should be fully disabled in our test suites. r?gcp https://reviewboard.mozilla.org/r/42133/#review38703 I suppose some of those may be disabled anyway due to not setting the proper URLs.
Attachment #8734201 -
Flags: review?(gpascutto) → review+
Comment 4•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/9edb10202fd2
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-firefox48:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
You need to log in
before you can comment on or make changes to this bug.
Description
•