1851012 - Align Search Engine Suggestion Provider behaviour with Desktop

Status: VERIFIED FIXED

(Fenix :: Search, enhancement, P3)

Description

[Gabriel Luong [:gl] (ΦωΦ)]

For the Search Engine Suggestion Provider, we are currently displaying a search engine suggestion by simply typing a single character as a result of the DEFAULT_CHARACTERS_THRESHOLD = 1.

STR:

1. Type "m" in the search bar.
2. See "Search Amazon.com" in the suggestions.

Expected:
We should not see this suggestion.

Testing on Desktop - it appears their threshold is set to at least 2 characters and it performs a startsWith(). This means if we typed "ma" we would not see the "Search Amazon.com" in the suggestions, and it needs to starts with "am" in order see the Amazon search engine suggestion.

I believe we should align the behaviour with desktop. We should do the following:

1. Increase DEFAULT_CHARACTERS_THRESHOLD to 2
2. Perform a startsWith instead of contains https://searchfox.org/mozilla-mobile/source/firefox-android/android-components/components/feature/awesomebar/src/main/java/mozilla/components/feature/awesomebar/provider/SearchEngineSuggestionProvider.kt#52.

Comment 1

[Verdi [:verdi] Best to slack me]

That makes sense. I agree we should only show the suggestion when we match the first 2+ characters. So to be clear, if I typed "am" the amazon suggestion would show and if I continued and typed the full "amazon.com" the suggestion would still be there. Of course if I hit enter at that point I'd navigate to amazon.com.

Comment 2

[BMO Github Automation]

Attachment: [mozilla-mobile/firefox-android] Bug 1851012 – aligned Search Engine Suggestion Provider behaviour with Desktop (#3455)

Comment 3

[BMO Github Automation]

Authored by mike a
https://github.com/mozilla-mobile/firefox-android/commit/5389b39020f8308483111d0591395a35cd66fdde
[main] Bug 1851012 – aligned Search Engine Suggestion Provider behaviour with Desktop

Comment 4

[Gabriel Luong [:gl] (ΦωΦ)]

For QA, we are blocked on Bug 1851006 landing before this can be tested.

Comment 5

[Ryan VanderMeulen [:RyanVM]]

Is this ready for a v118 backport PR and Beta approval request?

Comment 6

[Gabriel Luong [:gl] (ΦωΦ)]

Let's get a backport PR up, but let's wait until QA has verified this before uplifting?

Comment 7

[BMO Github Automation]

Attachment: [mozilla-mobile/firefox-android] Bug 1851012 – aligned Search Engine Suggestion Provider behaviour with Desktop (backport #3455) (#3587)

Comment 8

[Mike a [:mavduevskiy]]

waiting for the QA approval to open a request

Comment 9

[Adina Petridean QA [:apetridean]]

Verified as fixed on the Fenix Nightly 119.0a1 from 9/11 with Google Pixel 7 Pro ( Android 14) and Motorola Moto G9 plus (Android 11).

Comment 10

[Adina Petridean QA [:apetridean]]

Attachment: search.JPEG

Comment 11

[Mike a [:mavduevskiy]]

Comment on attachment 9352525 [details] [review]
[mozilla-mobile/firefox-android] Bug 1851012 – aligned Search Engine Suggestion Provider behaviour with Desktop (backport #3455) (#3587)

Beta/Release Uplift Approval Request

• User impact if declined: It's a small QoL improvement for search, users will get less noise when they start entering the search keyword – search engine suggestions will be shown if 2+ charachters match. Current rules is 1, so any single character in a word bing should add that suggestion to the first search result list.
• Is this code covered by automated tests?: Yes
• Has the fix been verified in Nightly?: Yes
• Needs manual test from QE?: No
• If yes, steps to reproduce:
• List of other uplifts needed: None
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): it's a change for the default value, with to edge cases
• String changes made/needed:
• Is Android affected?: Yes

Comment 12

[Donal Meehan [:dmeehan]]

Comment on attachment 9352525 [details] [review]
[mozilla-mobile/firefox-android] Bug 1851012 – aligned Search Engine Suggestion Provider behaviour with Desktop (backport #3455) (#3587)

Approved for Fenix 118.0b8

Comment 13

[BMO Github Automation]

Authored by https://github.com/mergify[bot]
https://github.com/mozilla-mobile/firefox-android/commit/796d6baa50525ba3f3e779c51ce335a81b773626
[releases_v118] Bug 1851012 – aligned Search Engine Suggestion Provider behaviour with Desktop (backport #3455) (#3587)

Comment 14

[Lorand Janos]

Verified as fixed on the latest Beta build (118.0b8).

Device used: Redmi 9C NFC (Android 10).

Marking the ticket as verified for 118 as well.