1488694 - Dropdown menus can be displayed at the same time if opened while tabs are loading

Status: NEW

(Firefox :: Menus, defect, P3)

Description

[Cristian Fogel, QA [:cfogel]]

Attachment: tabs+hamMenu_dropdowns.png

[Affected versions]:
- 61.0.2, 62.0b14  63.0b3, 64.0a1, 60.1.0esr

[Affected platforms]:
- Win10x64, macOS 10.9, Ubuntu 16.04LTS

[Steps to reproduce]:
1. Launch Firefox;
2. Set up multiple pages to be opened when clicking the Home button(ideally 10+ );
3. Click on the home button a couple of times;
4. Click on the 3line menu button and on the List all tabs button;

[Expected result]:
- only the dropdown clicked last is displayed;

[Actual result]:
- both dropdowns are displayed;

[Regression range]:
- possible regression ... it might be inaccurate, since manually tracking it turned out to be a bit tricky;
- pushlog URL: 
https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=58753259bfeb3b818eac7870871b0aae1f8de64a&tochange=c40ca7a1bdd93632c6bdc5e23bd33d984d508b19
- ran the regression check only with list all tabs and 3line;

[Additional notes]:
- attached screenshot with the issue;
- button order is not important;
- 2nd button should be clicked before the 1st one has time to be displayed;
- can be reproduced with the bookmarks button displayed after the show sidebars enable; reproduced as well with the view history&more and the list all tabs btn;

Comment 1

[:Gijs (he/him)]

Paolo, is this something you can take a look at when you're back? Thank you!

Comment 2

[Mike Conley (:mconley) (:⚙️) (PTO: May 24 - June 2nd)]

Kinda glitchy looking, but given the steps required to get into this state, and how recoverable it is, I'm inclined to mark this a P3.

Comment 3

[:Paolo Amadini]

I don't think this is PanelMultiView code, but a general platform issue with popups. It's also not clear if this is a regression in all cases. Given the lower priority and the other things I'm following, I don't think I'll have time to work on this now.

Comment 4

[Liz Henry (:lizzard Please n-i to RyanVM, jcristau, or pascal)]

Marking fix-optional for 64. We could still take a patch for 65, and if it's verified and doesn't seem risky, could still take fixes for 64 as well.

Comment 5

[Pascal Chevrel:pascalc]

Updating flags as we get closer to the 64 release.