1879642 - Update Yelp suggestion desktop integration for changes in Rust prefix matching

Status: RESOLVED FIXED

(Firefox :: Address Bar, task, P1)

Description

[Drew Willcoxon :adw]

We'll need to update the Yelp JS to account for changes in the Rust component in bug 1879640 and maybe make some other changes too to make sure the desired prefix matching behavior is implemented correctly.

Comment 1

[Drew Willcoxon :adw]

Attachment: Bug 1879642 - Update Yelp suggestion desktop integration for changes in Rust prefix matching.

The main change is to makeResult(). I talked with Loren and we came up with
these rules for prefix matching:

• The entire search string has to be at least 5 characters long, and
• At least 2 characters of the subject have to be typed

This patch enforces the 5 char overall threshold using the existing Nimbus
variable. If a subject is less than 5 chars, then we'll show it if there's an
exact match.

Other changes:

• Add yelp.minKeywordLength and yelp.showLessFrequentlyCap fallback prefs to
  make it easier to write tests and demo this feature
• Rename the quicksuggest.yelpPriority pref so it's consistent with others
• Add yelp.featureGate to firefox.js so it shows up in about:config so it's
  easier to try out this feature
• Alphabetize the prefs in UrlbarPrefs. Some newer ones are out of order.

Depends on D201300

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/f72266ed1a1f
Update Yelp suggestion desktop integration for changes in Rust prefix matching. r=daisuke

Comment 3

[Natalia Csoregi [:nataliaCs]]

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