1743709 - Duplicated Switch to Tab. There should not be duplications.

Status: VERIFIED FIXED

(Firefox :: Address Bar, defect, P3)

Description

[Mandy Cheang [:mcheang][she/her]]

Attachment: Screen Shot 2021-11-30 at 5.01.01 PM.png

Steps to reproduce

1. Visit http://example.com.
2. Open a new tab.
3. Type the string example and in the drop down, place your mouse over the switch-to-tab of Example domain.
4. Hold down the shift key to change the switch-to-tab option to url, and click it. This will open http://exmaple.com in the new tab.
5. Close the new tab with the http://exmaple.com page.
6. Open a new tab and click on the url bar and type the string example.
7. In the drop down, there is now two http://example.com switch-to-tab. This is a duplication. It should only show one.

Expected:

There should only be one Example Domain Switch to Tab suggestion.

Current:

There are two Example Domain Switch to Tab suggestions. This is a duplication because on the browser, there's only one tab with example.com loaded.

Potential fix:

Investigate how to de-duplicate the switch-to-tab in the muxer.
https://searchfox.org/mozilla-central/rev/70b32246fce5ca1f53af573a21c1939df58cb969/browser/components/urlbar/UrlbarMuxerUnifiedComplete.jsm#761-767

Comment 1

[Jeronimo Torti (Please send NI request to Brindusa Tot)]

Hi,
this was revised and Verified in Win10, using Nightly 97.0a1 (2022-01-05) (64-bit) and Release 95.0.2 (64-bit). Defect is no longer occuring.
In any case this needs to be revised again, please advise!.

Thanks.

Comment 2

[Mandy Cheang [:mcheang][she/her]]

Attachment: Bug 1743709 - Dedupe switchTab by checking if tab is already added to results. r?harry,adw

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/400d19ddbff7
Dedupe switchTab by checking if tab is already added to results. r=harry,adw

Comment 4

[Noemi Erli[:noemi_erli]]

https://hg.mozilla.org/mozilla-central/rev/400d19ddbff7

Comment 5

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

Since the status are different for nightly and release, what's the status for beta?
For more information, please visit auto_nag documentation.

Comment 6

[Drew Willcoxon :adw]

Mandy, are this bug's Version and tracking fields correct? Does it go back all the way to 96? If so, this seems like something we should uplift to 97.

Also, what happened with the bug status here... It looks like Jeronimo mistakenly resolved and verified it? The fix has not been verified by QA AFAICT.

Comment 7

[Mandy Cheang [:mcheang][she/her]]

Comment on attachment 9257739 [details]
Bug 1743709 - Dedupe switchTab by checking if tab is already added to results. r?harry,adw

Beta/Release Uplift Approval Request

• User impact if declined: User will see duplicated switch-to-tabs in the address bar view suggestion in certain situations where the url is in another tab and the url is also in adaptive history (the user's typed string that matches a url).
• Is this code covered by automated tests?: Yes
• Has the fix been verified in Nightly?: No
• Needs manual test from QE?: Yes
• If yes, steps to reproduce: 1. Go to http://www.example.com/

2. Open a new blank tab, and close the tab that has http://www.example.com/
3. Type string example in the urlbar of the open tab and select http://www.example.com/ in the suggestion drop down
4. Open a second new tab and type the string "example" in the urlbar
5. There should be a duplicated switch-to-tab options for http://www.example.com/

• List of other uplifts needed: None
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): This patch is not risky because it does not update critical components that would break search. The patch de-duplicates switch-to-tab in the results suggestions and does not affect any other search results that are being shown to the user.
• String changes made/needed:

Comment 8

[Ryan VanderMeulen [:RyanVM]]

Comment on attachment 9257739 [details]
Bug 1743709 - Dedupe switchTab by checking if tab is already added to results. r?harry,adw

Approved for 97.0b6. Thanks for including new tests for this.

Comment 9

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-beta/rev/977f35a7a6fd

Comment 10

[Peter Magyari (Desktop QA)]

I've reproduced the issue using Firefox 97.0b5 and verified the fix using Beta 97.0b6 and Nightly 98.0a1 on Windows 10, MacOS 11.6 and Ubuntu 20.04.