All users were logged out of Bugzilla on October 13th, 2018

Moving mouse inside desktop window adds too many events to kinetic queue

RESOLVED WONTFIX

Status

RESOLVED WONTFIX
10 years ago
9 years ago

People

(Reporter: bcombee, Assigned: bcombee)

Tracking

Trunk
x86
Windows Vista
Bug Flags:
wanted-fennec1.0 -

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

10 years ago
05:51 <@mfinkle> bcombee: I pulled the latest source and panning seems broken
06:07 < bcombee> mfinkle: how?  what platform?
06:08 < bcombee> I don't see much kinetic on desktop, but it's working on my n810
06:18 < bcombee> I know that panning locking is broken now with stuart's recent
                 changes, working on getting that back in
06:23 < bcombee> hmmm, new bug I'm noticing
06:23 < bcombee> open a page, scroll down a ways, pan to left to bring up nav
                 buttons
06:24 < bcombee> at this time, you see the title bar
06:24 < bcombee> hit the star button to bookmark it
06:24 < bcombee> title bar goes away, leaving up the nav buttons and leaving a
                 checkerboard in the corner
06:28 <@mfinkle> yes, I have seen that one too
06:29 <@mfinkle> in my build (windows desktop), I get weird jumps whenever I pan
06:30 <@mfinkle> it's like the onMouseDown is moving the canvas somewhere and then
                 panning starts
06:31 < bcombee> yeah, so on the n810, we actually get mousemove events sent before
                 the mouse down
06:31 < bcombee> stuart was trying to optimize for that
06:31 < bcombee> but on desktop, those moves might be irrelevant
06:31 < bcombee> we may be buffering too many of them too, I'll double check
                 stuart's change
06:33 <@mfinkle> maybe
Flags: wanted-fennec1.0?
(Assignee)

Comment 1

10 years ago
Created attachment 371861 [details] [diff] [review]
Remove old events in momentum buffer in startKinetic

This should help a lot on desktop builds where mouseMoves can really accumulate, unlike the touchscreen devices where only a few will show up before the mouseDown.
Attachment #371861 - Flags: superreview?(pavlov)
Attachment #371861 - Flags: review?(mark.finkle)
Comment on attachment 371861 [details] [diff] [review]
Remove old events in momentum buffer in startKinetic

Problem is not fixed. I'm not convinced this is a kinetic problem. My initial mousedown seems to cause the jump.

I'll put some logging in to get some data.
Attachment #371861 - Flags: review?(mark.finkle) → review-
(Assignee)

Comment 3

10 years ago
OK, I think there still may be an issue with keeping too many mousemove events on desktop.  I'm going to alter this patch to work in addData instead of startKinetic to reduce memory pressure.
(Assignee)

Updated

10 years ago
Summary: Desktop panning acts funny → Moving mouse inside desktop window adds too many events to kinetic queue
(Assignee)

Comment 4

10 years ago
Created attachment 371969 [details] [diff] [review]
Try #2 - move filter to addData instead of startKinetic

Not all the way there, but a starting point
Attachment #371861 - Attachment is obsolete: true
Attachment #371969 - Flags: review?(pavlov)
Attachment #371861 - Flags: superreview?(pavlov)

Comment 5

10 years ago
Comment on attachment 371969 [details] [diff] [review]
Try #2 - move filter to addData instead of startKinetic

this doesn't really help
Attachment #371969 - Flags: review?(pavlov) → review-
we shouldn't block on desktop only bugs
Flags: wanted-fennec1.0? → wanted-fennec1.0-
(Assignee)

Comment 7

9 years ago
I don't think this is needed anymore, we've got workaround on desktop using pref to disable those early moves
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.