Add mouse motion and Gesture magnify events compress for Qt Port

RESOLVED FIXED

Status

Core Graveyard
Widget: Qt
RESOLVED FIXED
7 years ago
a year ago

People

(Reporter: romaxa, Assigned: romaxa)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 2 obsolete attachments)

(Assignee)

Description

7 years ago
this is available in Gtk part, and need to be done for Qt port, improve event handling and safe CPU.

Similar to this
http://mxr.mozilla.org/mozilla-central/source/widget/src/gtk2/nsWindow.cpp#2530
(Assignee)

Comment 1

7 years ago
Also it would be useful for Meego netbooks devices with Gtk environment...
(Assignee)

Comment 2

7 years ago
Created attachment 500905 [details] [diff] [review]
Simple compress with 0 timer

This is the only way I found how to compress Qt motion and pinch events...
GTK compress way does not work on Qt.

This help us to avoid too many motion events processing.
Attachment #500905 - Flags: feedback?(roc)
Looks reasonable, but instead of a zero-length timer, it might be simpler and work just as well to use a simple XPCOM event instead ... NS_DispatchToMainThread etc.
(Assignee)

Comment 4

7 years ago
Created attachment 503464 [details] [diff] [review]
Updated, filter motion and pinch events

1) Use DispatchtoMainThread for filtering.
2) Minor code cleanup.
3) calculate distance touchPoint and center point.
Attachment #500905 - Attachment is obsolete: true
Attachment #503464 - Flags: review?(doug.turner)
Attachment #500905 - Flags: feedback?(roc)
(Assignee)

Comment 5

7 years ago
Created attachment 503465 [details] [diff] [review]
Updated, filter motion and pinch events

Ups, previous patch  contained wrong changes
Assignee: nobody → romaxa
Attachment #503464 - Attachment is obsolete: true
Status: NEW → ASSIGNED
Attachment #503465 - Flags: review?(doug.turner)
Attachment #503464 - Flags: review?(doug.turner)
(Assignee)

Comment 6

7 years ago
Comment on attachment 503465 [details] [diff] [review]
Updated, filter motion and pinch events

Wolfgang could you check quickly this patch?
Attachment #503465 - Flags: review?(doug.turner) → review?(mozilla)
Comment on attachment 503465 [details] [diff] [review]
Updated, filter motion and pinch events

looks good to me
Attachment #503465 - Flags: review?(mozilla) → review+
(Assignee)

Comment 8

7 years ago
http://hg.mozilla.org/mozilla-central/rev/8c57ddbf4ae2
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.