Our workaround for omitting duplicate GDK_SCROLL_SMOOTH events causes synthesized scroll events to be skipped when GDK_SCROLL_SMOOTH is used for scroll synthesis, making gfx/layers/apz/test/test_wheel_scroll.html fail on the tryserver. This is because synthesized events are all initialized to GDK_CURRENT_TIME. > #if GTK_CHECK_VERSION(3,4,0) > // check for duplicate legacy scroll event, see GNOME bug 726878 > if (mLastScrollEventTime == aEvent->time) > return; > #endif
Created attachment 8614688 [details] [diff] [review] Only omit legacy scroll events on GTK3. This patch modifies our detection of duplicate scroll events as documented in bug 958868 to only omit legacy events. Thanks!
Attachment #8614688 - Flags: review?(karlt)
Attachment #8614688 - Flags: review?(karlt) → review+
Status: NEW → RESOLVED
Last Resolved: 3 years ago
status-firefox41: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla41
You need to log in before you can comment on or make changes to this bug.