Closed Bug 1053716 Opened 10 years ago Closed 10 years ago

[e10s] Scrolling by dragging the scroll handle breaks if the mouse leaves the window

Categories

(Core :: DOM: UI Events & Focus Handling, defect)

x86
Windows 8.1
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla35
Tracking Status
e10s + ---

People

(Reporter: cwiiis, Assigned: Felipe)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

If you scroll by dragging the scroll handle, scrolling will stop if the mouse cursor leaves the window. If you let go of the mouse button while the cursor is outside of the window, re-entering the window will cause scrolling to get 'stuck' to the mouse until you click.
Felipe, working theory is you have a dupe of this assigned to you. Assigning this to you so you can handle it appropriately.
Assignee: nobody → felipc
Blocks: old-e10s-m2
tracking-e10s: --- → +
This bug has similarities with bug 1003943 and bug 1003934. It's not only related to the scrollbar dragging, but any click-and-hold operation (like selecting text). When there's a mousedown, we have to start capturing the mouse at the OS level, so that we're informed of all the mouse actions even outside our window. But this doesn't seem to happen with e10s content.
Component: General → Event Handling
Product: Firefox → Core
Attached patch Capture events at remote target (deleted) — Splinter Review
Are there other kinds of events, other than mouse events, that would require the same capturing treatment?
Attachment #8481619 - Flags: review?(bugs)
Comment on attachment 8481619 [details] [diff] [review] Capture events at remote target Would be a bit safer if you would check first that nothing is capturing the event already. So if (dispatchedToContentProcess) { -> if (dispatchedToContentProcess && !nsIPresShell::GetCapturingContent()) {
Attachment #8481619 - Flags: review?(bugs) → review+
(In reply to Brad Lassey [:blassey] (use needinfo?) from comment #1) > Felipe, working theory is you have a dupe of this assigned to you. Assigning > this to you so you can handle it appropriately. Yeah, the dupes were bug 992423 and bug 947911. I'll dupe them forward to this one as I ended up working on the patch/review here.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla35
Depends on: 1251634
Component: Event Handling → User events and focus handling
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: