[Webkit-unassigned] [Bug 65875] [Qt][WK2] MiniBrowser is firing twice the QDesktopWebView::mousePressEvent method

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=65875


Benjamin Poulain <benjamin at webkit.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #103419|review?, commit-queue?      |review-
               Flag|                            |




--- Comment #8 from Benjamin Poulain <benjamin at webkit.org>  2011-08-10 06:20:50 PST ---
(From update of attachment 103419)
View in context: https://bugs.webkit.org/attachment.cgi?id=103419&action=review

This has nothing to do with the ControlModifier. The problem is we still send the original mouse event after we send the touch events. QWidget also generate a fake mouse event for the touch event so we end up with 2 mouse events. The original event should be discarded when a touch event is generated.

> Tools/ChangeLog:9
> +        On some windowing systems, mouse events are also sent for the primary touch point. This
> +        means it is possible for your widget to receive both QTouchEvent and QMouseEvent for the
> +        same user interaction point. MiniBrowser when setting the Qt::TouchPointPressed for a touch

Qt is supposed to always give you a mouse event for the primary touch point.

> Tools/ChangeLog:12
> +        to be called twice for a single mouse click. Now the Qt::TouchPointPressed is used just
> +        when Ctrl modifier is pressed.

This is not at all the intended behavior of the code of touch mocking.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.