1953129 - browser.search.totalSearches capped at 101 and not 100

Status: UNCONFIRMED

(Firefox :: Messaging System, defect, P1)

Description

[Sertonix]

Steps to reproduce:

Go to about:config and set browser.search.totalSearches to 100. Then search in a new tab.

Actual results:

browser.search.totalSearches increased from 100 to 101

Expected results:

browser.search.totalSearches shouldn't increase anymore when it reached 100

Comment 1

[Sertonix]

I think the fix would simply be replacing totalSearches <= totalSearchesCap with totalSearches < totalSearchesCap.

Comment 2

[BugBot [:suhaib / :marco/ :calixte]]

The Bugbug bot thinks this bug should belong to the 'Fenix::General' component, and is moving the bug to that component. Please correct in case you think the bot is wrong.

Comment 3

[BugBot [:suhaib / :marco/ :calixte]]

:jprickett, since you are the author of the regressor, bug 1912351, could you take a look? Also, could you set the severity field?

For more information, please visit BugBot documentation.

Comment 4

[Vasilica Tamas QA [:vtamas]]

There is no browser.search.totalSearches in about:config on Fenix. Taking into account the regressor, I'm moving this bug to Firefox product.

Comment 5

[Jason Prickett]

Attachment: Bug 1953129 - Cap totalSearches pref at 100 r?#omc-reviewers