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)
Tracking
()
RESOLVED
FIXED
mozilla35
Tracking | Status | |
---|---|---|
e10s | + | --- |
People
(Reporter: cwiiis, Assigned: Felipe)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
(deleted),
patch
|
smaug
:
review+
|
Details | Diff | Splinter Review |
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.
Comment 1•10 years ago
|
||
Felipe, working theory is you have a dupe of this assigned to you. Assigning this to you so you can handle it appropriately.
Assignee | ||
Comment 2•10 years ago
|
||
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.
Assignee | ||
Updated•10 years ago
|
Component: General → Event Handling
Product: Firefox → Core
Assignee | ||
Comment 3•10 years ago
|
||
Are there other kinds of events, other than mouse events, that would require the same capturing treatment?
Attachment #8481619 -
Flags: review?(bugs)
Comment 4•10 years ago
|
||
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+
Assignee | ||
Comment 5•10 years ago
|
||
(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.
Assignee | ||
Comment 7•10 years ago
|
||
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla35
Updated•6 years ago
|
Component: Event Handling → User events and focus handling
You need to log in
before you can comment on or make changes to this bug.
Description
•