384683 - autoscroll should reset if the pointer is outside the content

Status: NEW

(Firefox :: General, defect)

Description

[Peter van der Woude [:Peter6]]

Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a6pre) Gecko/20070615 Minefield/3.0a6pre ID:2007061522

repro:
Open a page that has scrollbars
middleclick to set autoscroll mode on
move the pointer outside the page (anywhere in chrome)

result:
the autoscroll image doesn't go away.

side effect, for any action in chrome (e.g. try to open bookmarks menu) you need 2 clicks, one to remove the autoscroll, two to do what you wanted in the fist place.

Comment 1

[Dão Gottwald [:dao]]

That's mostly expected behaviour. Only the "side effect" is a regression from bug 242621.

Comment 2

[Chris Thomas (CTho) [formerly [e-mail] [e-mail]]]

I consider that pretty much by design... the cursor is tracked over chrome, so if you happen to middle-click at the very top of the content area you can still scroll up.  This should be exactly what we've had in SeaMonkey for at least 2 releases.

Comment 3

[Chris Thomas (CTho) [formerly [e-mail] [e-mail]]]

After using old Firefox for a bit, that behavior is actually decent... I'll look at what it'd take to get it working.

Comment 4

[Chris Thomas (CTho) [formerly [e-mail] [e-mail]]]

Neil, I tried playing with captureRollupEvent, but couldn't get the event to go through.  Could you help me here?

Comment 5

[Neil Deakin]

You called popupBoxObject.setConsumeRollupEvent with ROLLUP_NO_CONSUME just before opening the popup?

Comment 6

[Chris Thomas (CTho) [formerly [e-mail] [e-mail]]]

Yup:

            this._autoScrollPopup.popupBoxObject.setConsumeRollupEvent(2);
            this._autoScrollPopup.showPopup(document.documentElement,
                                            event.screenX,
                                            event.screenY,
                                            "popup", null, null);

Maybe I'm misunderstanding what that does...

Comment 7

[Neil Deakin]

Looks like the flag only has any effect on Windows. But it isn't too hard to fix on other platforms. On Linux, the function nsWindow::check_for_rollup just has to return false if the flag was set after calling Rollup. On Mac, don't return early in nsChildView::mouseDown.