Comment 238 for bug 187313

Revision history for this message
Alexander Sack (asac) wrote : Re: [Bug 187313] Re: [MASTER] right click (with button release) might activate random popup-menu-item

On Fri, Jan 30, 2009 at 01:06:00PM -0000, 4711 wrote:
> 1.)
> The Mouse Gestures extension only changes the context menu behaviour from mousedown to mouseup
> for context menus inside the "content" where Mouse Gestures is working. In all other places the menu
> will be shown on mousedown! (Like the context menu for tabs or toolbars)
>
> 2.)
> I don't think the problem is related to Accessibility. At least on my system.
>
> 3.)
> With Fx3/Gecko 1.9 a lot of changes to the popup code where introduced.
> After some investigation with eventListeners (added by an extension) I have found one (big?) problem:
>
> -Sometimes mouseup events are fired to the popup although the popup is not visible yet.
> (the popup.state is "showing" not "open")
> -this mouseup will call the command of the menuitem it happened over and close the popup
> (I think positioning and showing the popup is VERY slow)

Right ... thats the current understanding. Also I somehow have the
feeling that this happens because the focus gets auto moved to the
none-existing popup. The randomness could be explained by a overflow
given that the actual popup size is still zero.

>
> For me the "ghost commands" where 100% reproducible when a mouseup was fired to the popup
> while its state was "showing" (about to become visible)

I tried to fix the state so its not in state open until the frames
have been created asychronously. However, this didnt help. OTOH, there
was a second bug with the accessibility code that could have made this
fix unvoid. I will try it again now that the accessibility fix has
landed.

>
> The problem happens more often if the popup has to be moved to the top of the cursor. (More calculations?)
> And I can reproduce it quite reliable if the window the popup
> belongs to has to be brought to the front first.

Thanks. I will check this one. Please post that info to the upstream
bug too.

 - Alexander