1660270 - Revert Prefs for A/B test survey analysis

Status: RESOLVED FIXED

(Toolkit :: Form Autofill, task, P3)

Description

[Zibi Braniecki [:zbraniecki][:gandalf]]

Prefs added in Bug #1654388 should be reverted after the A/B test is completed

Comment 1

[Zibi Braniecki [:zbraniecki][:gandalf]]

Attachment: Bug 1660270 - Revert Part 1: Track doorhanger interaction. r?mattn

Comment 2

[Zibi Braniecki [:zbraniecki][:gandalf]]

Attachment: Bug 1660270 - Revert Part 2: Record address and password usage. r?mattn

Depends on D87769

Comment 3

[Zibi Braniecki [:zbraniecki][:gandalf]]

@Mattn - I'm requesting r? now, of straight backouts, but will not land them until after A/B test release (so, likely it'll land in the Nightly of 83).

I'd like to have this patchset ready for landing (with your r+) ready so that landing is just a formality and not a process that someone will have to get back to in several months :)

Comment 4

[BugBot [:suhaib / :marco/ :calixte]]

There are some r+ patches which didn't land and no activity in this bug for 2 weeks.
:zbraniecki, could you have a look please?
For more information, please visit auto_nag documentation.

Comment 5

[Zibi Braniecki [:zbraniecki][:gandalf]]

Has the A/B study been conducted and completed? Can we back out the A/B study prefs once 83 opens?

Comment 6

[Jim Mathies [:jimm]]

(In reply to Zibi Braniecki [:zbraniecki][:gandalf] from comment #5)

Has the A/B study been conducted and completed? Can we back out the A/B study prefs once 83 opens?

Yes, it's complete.

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/593d42151377
Revert Part 1: Track doorhanger interaction. r=MattN
https://hg.mozilla.org/integration/autoland/rev/db7b1c4a37b1
Revert Part 2: Record address and password usage. r=MattN

Comment 8

[Narcis Beleuzu [:NarcisB]]

Backed out for lint failure on test_basic_creditcard_autocomplete_form.html

Backout link: https://hg.mozilla.org/integration/autoland/rev/8c98a135a710784a4569e97e4eb9cec00386ce16
Log link: https://treeherder.mozilla.org/logviewer?job_id=321260894&repo=autoland&lineNumber=129

Comment 9

[Narcis Beleuzu [:NarcisB]]

Please also check mochitest failures on test_basic_creditcard_autocomplete_form.html -> https://treeherder.mozilla.org/logviewer?job_id=321264097&repo=autoland&lineNumber=2271

Comment 10

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/f2c6b44f3e7e
Revert Part 1: Track doorhanger interaction. r=MattN
https://hg.mozilla.org/integration/autoland/rev/67281a48cc78
Revert Part 2: Record address and password usage. r=MattN

Comment 11

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

https://hg.mozilla.org/mozilla-central/rev/f2c6b44f3e7e
https://hg.mozilla.org/mozilla-central/rev/67281a48cc78