Since the WebView accepts all the received touch events, the event to control the Flickable does not get translated to a mouse event on the N9. Do the touch->mouse conversion in QtFlickProvider instead of relying on Qt to translate the unhandled touch events.
Created attachment 129706 [details] proposed patch
Comment on attachment 129706 [details] proposed patch View in context: https://bugs.webkit.org/attachment.cgi?id=129706&action=review > Source/WebKit2/UIProcess/qt/QtFlickProvider.cpp:156 > + QCoreApplication::sendEvent(m_flickable->canvas(), &mouseEvent); This is a bit unfortunately, it sounds like we need a function in QQuickCanvas to send an event to a specific item, in order to ensure delivery. Couldn't theoretically the event end up in a different item if it happens to be "on top"? (of course we shouldn't have received the touch event in the first place then...) Anyway, this is an improvement and clearly makes the code more robust (less relying on magic event conversion elsewhere).
(In reply to comment #2) Thanks for the review, Simon. > (From update of attachment 129706 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=129706&action=review > > > Source/WebKit2/UIProcess/qt/QtFlickProvider.cpp:156 > > + QCoreApplication::sendEvent(m_flickable->canvas(), &mouseEvent); > > This is a bit unfortunately, it sounds like we need a function in QQuickCanvas to send an event to a specific item, in order to ensure delivery. Couldn't theoretically the event end up in a different item if it happens to be "on top"? (of course we shouldn't have received the touch event in the first place then...) > We could theoretically send the event directly to the Flickable with QQuickCanvas::sendEvent, but since this code path goes to Flickable::handleMouseFooEvent the item does not grab the mouse and this results in strange behavior for gestures which move out of the viewport, which is of course only an issue on desktop. I hope we can leave behind these issues with the touch related changes in Qt5 suggested by Shawn.
Comment on attachment 129706 [details] proposed patch Rejecting attachment 129706 [details] from commit-queue. Failed to run "['/mnt/git/webkit-commit-queue/Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '-..." exit_code: 1 ERROR: /mnt/git/webkit-commit-queue/ChangeLog neither lists a valid reviewer nor contains the string "Unreviewed" or "Rubber stamp" (case insensitive). Full output: http://queues.webkit.org/results/11776526
Comment on attachment 129706 [details] proposed patch Clearing flags on attachment: 129706 Committed r109415: <http://trac.webkit.org/changeset/109415>
All reviewed patches have been landed. Closing bug.