1390698 - Make an add-on for the opt-out study of Safe Browsing V2/V4

Status: RESOLVED FIXED

(Shield :: Shield Study, enhancement, P1)

Description

[François Marier [:francois]]

We need a bootstrap add-on that flips these prefs in the experiment group:

pref("urlclassifier.malwareTable", "goog-malware-shavar,goog-unwanted-shavar,test-malware-simple,test-unwanted-simple");
pref("urlclassifier.phishTable", "goog-phish-shavar,test-phish-simple");
pref("urlclassifier.downloadAllowTable", "goog-downloadwhite-digest256");
pref("urlclassifier.downloadBlockTable", "goog-badbinurl-shavar");

and then restores them to the default values when it's removed.

The add-on does not need to send any extra telemetry.

Comment 1

[François Marier [:francois]]

Kamyar, are you able to get this done by next Monday so that we can test it next week and confirm that this approach will work?

Comment 2

[François Marier [:francois]]

Attachment: SBv4-crash-rate-test.xpi

Here is the bootstrap add-on for the study.

Mike, is that all you need to get the add-on signed and deployed on Normandy once the study has been approved in bug 1377267?

Comment 3

[Michael Cooper [:mythmon]]

I'm going on pto soon, so I'm redirecting this to Osmose.

Comment 4

[François Marier [:francois]]

Attachment: SBv4-crash-rate-test.xpi

Kamyar added a small cleanup: https://github.com/raymak/sbv4-crash-shield-study/commit/f1ff8d44232ccde1d605f618806fe7cc84ef36bb#diff-4b80223300be1233ce41b6b9d0c08761R70

Comment 5

[Osmose [:osmose, :mkelly]]

As per the study guidelines[1], we need two things before signing an opt-out study:

- A description of the population we'll be sending the study to, particularly which channels.
- Sign-off that the functionality of the add-on has been tested on the platforms that we'll be sending it to. If the study is going to pre-release channels only, this can be provided by the developer directly.

Include both of those in this bug, and I can sign the add-on for ya'll. :D

[1] https://docs.google.com/document/d/1hOMjZ7l1K0HL8DUp7HCr8BeRO7NGw0SwYjK2nfwtRJQ/edit#heading=h.q8cndfdy29ua

Comment 6

[François Marier [:francois]]

(In reply to Michael Kelly [:mkelly,:Osmose] from comment #5)
> - A description of the population we'll be sending the study to,
> particularly which channels.

100% of Beta 56 population.

> - Sign-off that the functionality of the add-on has been tested on the
> platforms that we'll be sending it to. If the study is going to pre-release
> channels only, this can be provided by the developer directly.

I tested it on Linux, Win32 and Win64. Kamyar tested it on Mac.

Comment 7

[François Marier [:francois]]

Michael, anything else you need before you can sign this add-on?

Comment 8

[Osmose [:osmose, :mkelly]]

I've signed and uploaded the study to Normandy under the name "Safe Browsing v4 (Crash Rate Test)".