1294680 - Prepare the search reset feature for a Shield study

Status: RESOLVED FIXED

(Firefox :: Search, defect, P1)

Description

[Florian Quèze [:florian]]

Product Management has decided that we need data about how the search reset feature performs on actual release users before deploying it to all our user base. I'll attach here patches to prepare the code for this. We'll need to pref the feature off for 49 so that the Shield study can re-enable it.

Comment 1

[Florian Quèze [:florian]]

Attachment: remove the engine drop down,

Comment 2

[Panos Astithas (he/him) [:past] (please ni?)]

Comment on attachment 8780518 [details] [diff] [review]
remove the engine drop down,

Review of attachment 8780518 [details] [diff] [review]:
-----------------------------------------------------------------

Is there another bug for adding telemetry to the "settings" link?

Comment 3

[Florian Quèze [:florian]]

Attachment: instrument the 'Settings page' button

Comment 4

[Florian Quèze [:florian]]

Attachment: pref off the search reset feature for 49

Note: removing the browser/app/profile/firefox.js entry makes us fallback to the default (false) from modules/libpref/init/all.js

Comment 5

[Panos Astithas (he/him) [:past] (please ni?)]

Comment on attachment 8780528 [details] [diff] [review]
instrument the 'Settings page' button

Review of attachment 8780528 [details] [diff] [review]:
-----------------------------------------------------------------

Ah, there it is!

Comment 6

[Florian Quèze [:florian]]

https://hg.mozilla.org/integration/fx-team/rev/d1e65973534f70fbced2f0f22746eca2f00b37aa
Bug 1294680 - remove the engine drop down, r=past.

https://hg.mozilla.org/integration/fx-team/rev/124669ed2f09f68f7f8dd38f4fee5f3084da0aff
Bug 1294680 - instrument the 'Settings page' button, r=past.

Comment 7

[Florian Quèze [:florian]]

Comment on attachment 8780518 [details] [diff] [review]
remove the engine drop down,

Approval Request Comment
[Feature/regressing bug #]: bug 1203168
[Describe test coverage new/current, TreeHerder]: The patch adapts the existing automated test coverage.
[Risks and why]: Low risk, this is mostly a UI simplification/removal, and the feature will be pref'ed off for 49.
[String/UUID change made/needed]: none.

Comment 8

[Florian Quèze [:florian]]

Comment on attachment 8780528 [details] [diff] [review]
instrument the 'Settings page' button

Approval Request Comment
[Feature/regressing bug #]: bug 1203168
[impact if declined]: Impossible for us to distinguish people clicking the Settings link from people who just closed the prompt when looking at Telemetry data.
[Describe test coverage new/current, TreeHerder]: This change is covered by the included change to our automated test.
[Risks and why]: Low, this is just improving our telemetry recording of how the page is used.
[String/UUID change made/needed]: none.

Comment 9

[Florian Quèze [:florian]]

Comment on attachment 8780529 [details] [diff] [review]
pref off the search reset feature for 49

Approval Request Comment
[Feature/regressing bug #]: bug 1203168
[User impact if declined]: the search reset feature would be exposed before we have data to prove that it actually has the desired effect.
[Describe test coverage new/current, TreeHerder]: N/A.
[Risks and why]: low, just pref'ing the feature off for one cycle for now.
[String/UUID change made/needed]: none.

Comment 10

[Liz Henry (:lizzard) (relman/hg->git project)]

Comment on attachment 8780518 [details] [diff] [review]
remove the engine drop down,

From discussion in email, please uplift to aurora and beta. This should make it into beta 4.

Comment 11

[Liz Henry (:lizzard) (relman/hg->git project)]

Comment on attachment 8780529 [details] [diff] [review]
pref off the search reset feature for 49

Please uplift this patch to beta only.

Comment 12

[Liz Henry (:lizzard) (relman/hg->git project)]

(Actually, hold off on uplift until this lands on m-c)

Comment 13

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/d1e65973534f
https://hg.mozilla.org/mozilla-central/rev/124669ed2f09

Comment 14

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/releases/mozilla-aurora/rev/a6c0964964be
https://hg.mozilla.org/releases/mozilla-aurora/rev/0d2e9c07cc9a

Comment 15

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/releases/mozilla-beta/rev/c250152accdc
https://hg.mozilla.org/releases/mozilla-beta/rev/1ab4e3c7085d
https://hg.mozilla.org/releases/mozilla-beta/rev/eb93bd0c50d6