1566205 - Consider moving the discovery stream pref into firefox.js

Status: NEW

(Firefox :: Pocket, enhancement, P2)

Description

[Scott [:thecount] Downe]

So I think the issues as to what we get from this are clear, rollouts and default state changes not un enrolling people from experiments.

Comment 1

[Ed Lee :Mardak]

To prevent unenrolling a "user" pref experiment, we still need to make sure the firefox.js pref does not change. Although moving the pref to firefox.js does allow "default" pref experiments as well as rollouts. However, discoverystream.config is a dynamic pref because it depends on dynamic state of the browser -- namely search geo/region.

Comment 2

[Ed Lee :Mardak]

The likely fix is to split discoverystream.config into individual prefs instead of a json pref. Where some can be moved to firefox.js

Comment 3

[Scott [:thecount] Downe]

This has issues around locale. I'm not seeing a direct path forward at the time being, that doesn't cause a pref change, making it something that we should visit in 71 at the earliest.