666719 - Locationbar result list closes immadiatly when using trackpad or magic mouse

Status: RESOLVED FIXED

(Core :: Widget: Cocoa, defect)

Description

[Mehmet]

User-Agent:       Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:7.0a1) Gecko/20110623 Firefox/7.0a1
Build Identifier: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:7.0a1) Gecko/20110623 Firefox/7.0a1

OS: Mac OS 10.6.7
Nightly Build 7.0a1 (2011-06-23)

***please see the steps to reproduce***

Thanks
Mehmet

Reproducible: Always

Steps to Reproduce:
1. enable momentum scrolling under Mac OS or use the magic mouse
2. type something into the Locationbar so the the result list is to see
3. move the mousecursor over the result list
4. now make a trackpad or mouse swipe down and move immadiatly the cursor out of the result list

Actual Results:  
The Result list closes

Expected Results:  
It should not close

This also happens to select-field-menus :-(

Comment 1

[Mehmet]

I have to correct me: This happens only with the magic mouse. The Trackpad is *NOT* affected.

Comment 2

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Mehmet, is that a regression for you? I can't reproduce it on my own box. Are you having add-ons installed?

Comment 3

[Markus Stange [:mstange]]

Here's the call that closes popups on scroll events:
http://hg.mozilla.org/mozilla-central/annotate/cfd2af15e1c3/widget/src/cocoa/nsChildView.mm#l3936

We can add a check for momentum scroll there.

(In reply to comment #1)
> I have to correct me: This happens only with the magic mouse. The Trackpad
> is *NOT* affected.

The difference is that the trackpad stops sending momentum scroll events as soon as the mouse is moved a certain amount of pixels. The magic mouse doesn't.

Comment 4

[Mehmet]

Attachment: Screencast

Hi. I think this is not a regression. It also happens in FF5. Not sure, but I think it was also an issue in FF4.

***Please see the enclosed screencast.***

Thanks and regards
Mehmet

Comment 5

[Markus Stange [:mstange]]

The patch in bug 677279 fixes this.

Comment 6

[Mounir Lamouri (:mounir)]

Markus, isn't it fixed now?

Comment 7

[Markus Stange [:mstange]]

Oh, right.