Closed Bug 1821112 Opened 1 year ago Closed 5 months ago

QuickSuggest may be bypassing maxRichResults?

Categories

(Firefox :: Address Bar, defect, P3)

defect

Tracking

()

VERIFIED FIXED
122 Branch
Tracking Status
firefox122 --- verified
firefox123 --- verified

People

(Reporter: mak, Assigned: adw)

References

Details

(Keywords: privacy, Whiteboard: [snt-scrubbed][search-privacy])

Attachments

(2 files)

We were debugging a case where an employee was getting no results from the urlbar today, and we discovered the problem was maxRichResults being set to 0.

Though one of the screenshots clearly showed a QuickSuggest Wikipedia result, that should not have been there.
about:studies didn't show anything active.

The problem is here, when the muxer adds group-relative suggested-index results, which quick suggest results are. I don't remember if it was intentional or not -- I don't think so. The muxer always adds these results even if availableSpan and/or maxResultCount aren't sufficient.

Whiteboard: [snt-scrubbed]
Whiteboard: [snt-scrubbed] → [snt-scrubbed][search-privacy]
Assignee: nobody → adw
Status: NEW → ASSIGNED
Pushed by dwillcoxon@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/106f09ba0e1b
Make the urlbar muxer take span and count limits into account when adding suggested-index results. r=mak
Status: ASSIGNED → RESOLVED
Closed: 5 months ago
Resolution: --- → FIXED
Target Milestone: --- → 122 Branch

Reproducible on a 2023-03-08 Nightly build on Windows 10.
Verified as fixed on Firefox 122.0 and Nightly 123.0a1 on Windows 10, Ubuntu 22, macOS 12.
With maxRichResults set to 0 and QuickSuggest prefs set to true, there are no suggestions in the address bar.

Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: