2064790 - Set browser.smartwindow.searchTheWebFast default to true

Status: RESOLVED FIXED

(Core :: Machine Learning: Models, task)

Comment 1

[Bryan]

Attachment: Bug 2064790 - Set browser.smartwindow.searchTheWebFast default to true r?#ai-models-reviewers

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/c256f3937637
https://hg.mozilla.org/integration/autoland/rev/dfac628ec79b
Set browser.smartwindow.searchTheWebFast default to true r=vazish

Comment 3

[Norisz Fay [:noriszfay]]

https://hg.mozilla.org/mozilla-central/rev/dfac628ec79b

Comment 4

[Phabricator Automation]

firefox-release Uplift Approval Request

• User impact if declined/Reason for urgency: The approach decreases the time searchTheWeb takes from 10 seconds down to 2 seconds, greatly improving user experience
• Code covered by automated testing?: yes
• Fix verified in Nightly?: yes
• Needs manual QE testing?: yes
• Steps to reproduce for manual QE testing: - verify browser.smartwindow.searchTheWebFast pref is set to "true" (should be the default)
• ask SW a question requiring web search to answer. the model should think for 1-2s and then a "searching the web with Exa" sign should appear - quickly followed by an answer
• Risk associated with taking this patch: low
• Explanation of risk level: simplifies the pipeline that previously required several model calls and handoffs
• String changes made/needed?: no
• Is Android affected?: yes

Comment 5

[Bryan]

Attachment: Bug 2064790 - Set browser.smartwindow.searchTheWebFast default to true

Original Revision: https://phabricator.services.mozilla.com/D319871

Comment 6

[Ciprian Georgiu, Desktop QA]

Verified as fixed on latest Nightly 156.0a1 under Win 11, macOS 27 and Ubuntu 26.

Comment 7

[Phabricator Automation]

firefox-beta Uplift Approval Request

• User impact if declined/Reason for urgency: Greatly improves user experience with search the web (5x increase in response time).
• Code covered by automated testing?: yes
• Fix verified in Nightly?: yes
• Needs manual QE testing?: no
• Steps to reproduce for manual QE testing: QA approved
• Risk associated with taking this patch: low
• Explanation of risk level: Low - just changes default value of a pref
• String changes made/needed?: no
• Is Android affected?: no

Comment 8

[Bryan]

Attachment: Bug 2064790 - Set browser.smartwindow.searchTheWebFast default to true

Original Revision: https://phabricator.services.mozilla.com/D319871

Comment 9

[Ryan VanderMeulen [:RyanVM]]

Why can't this pref flip be rolled out to 155 via Nimbus instead of an in-tree uplift?

Comment 10

[Ed Lee :Mardak]

we don't have a normal nimbus feature manifest entry for this, so would it be preferable to use the nimbus emergency pref flip to enable this? the main functionality did land 155 with bug 2062711, so an off-train rollout should work

https://experimenter.info/platform-guides/desktop/incident-response/

Comment 11

[Ryan VanderMeulen [:RyanVM]]

No, I don't necessarily think it's something we want to (ab)use the emergency pref flip mechanism for. Just something for us to consider going forward is that post-Metronome, the goal is to utilize remote feature rollouts more heavily instead of relying on landing pref flips during the Beta cycle. It's less uplift churn and easier to control if anything goes sideways.

Comment 12

[Pulsebot]

https://github.com/mozilla-firefox/firefox/commit/00efaea9cc25
https://hg.mozilla.org/releases/mozilla-beta/rev/081774e43489