Closed Bug 1301784 Opened 3 years ago Closed 3 years ago

Pref-off Search Reset UI for 50 beta

Categories

(Firefox :: Search, defect, P1)

50 Branch
defect

Tracking

()

VERIFIED FIXED
Firefox 50
Tracking Status
firefox49 --- disabled
firefox50 blocking verified

People

(Reporter: javaun, Assigned: florian)

References

Details

(Whiteboard: [fxsearch])

Attachments

(1 file)

Let's talk about this one next week. We were deciding whether to pref this UI off in 50 before Beta, since we're going to run Shield Studies in 49 to see how users respond to it.
Priority: -- → P1
Whiteboard: [fxsearch]
Attached patch PatchSplinter Review
Already r+'ed by Panos as attachment 8780529 [details] [diff] [review] in bug 1294680.
Assignee: nobody → florian
Status: NEW → ASSIGNED
Comment on attachment 8792819 [details] [diff] [review]
Patch

Similar to bug 1294680 comment 9. We still don't have results from the Shield study (that's planned for the 50 cycle if I understood correctly). This patch should land on mozilla-beta after the merge before beta1 is built.

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.
Attachment #8792819 - Flags: review+
Attachment #8792819 - Flags: approval-mozilla-beta?
[Tracking Requested - why for this release]: see comment 2.
Comment on attachment 8792819 [details] [diff] [review]
Patch

Ok, taking it on the behalf of Ritu to make sure it lands before we build beta 1.
Thanks
Attachment #8792819 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Flagging this for verification, just to make sure it's now default-off on Fx50.
Flags: qe-verify+
I believe this is now done.
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 50
Verified that the preference "browser.search.reset.enabled" is set to false by default on Firefox 50 beta 9 under Win 10 64-bit and Ubuntu 14.04 LTS 64-bit.
Status: RESOLVED → VERIFIED
Flags: qe-verify+
Blocks: 1328913
You need to log in before you can comment on or make changes to this bug.