Changeset 109415 in webkit

Timestamp:

Mar 1, 2012, 1:51:26 PM (15 years ago)

Author:

abecsi@webkit.org

Message:

[Qt][WK2] Make the interaction with the Flickable work on the N9
​https://bugs.webkit.org/show_bug.cgi?id=80029

Reviewed by Simon Hausmann.

Because the WebView item accepts all touch events it receives and sends
them to the web process before propagating them to the gesture recognizers,
which is correct behaviour, we can not rely on the touch->mouse conversion
of Qt5 when controlling Flickable. Hence we need to convert the received
touch events to mouse events in the QtFlickProvider.

• UIProcess/qt/QtFlickProvider.cpp:

(QtFlickProvider::handleTouchFlickEvent):
Do the touch to mouse event conversion for the Flickable.

• UIProcess/qt/QtPanGestureRecognizer.cpp:

(WebKit::QtPanGestureRecognizer::recognize):
A touch begin event should cancel the previous pan gesture
and stop the ongoing flick animation.

Location:

trunk/Source/WebKit2

Files:

• ChangeLog (modified) (1 diff)
• UIProcess/qt/QtFlickProvider.cpp (modified) (2 diffs)
• UIProcess/qt/QtPanGestureRecognizer.cpp (modified) (1 diff)

trunk/Source/WebKit2/ChangeLog

@@ +1 @@
+2012-03-01  Andras Becsi  <andras.becsi@nokia.com>
+
+        [Qt][WK2] Make the interaction with the Flickable work on the N9
+        https://bugs.webkit.org/show_bug.cgi?id=80029
+
+        Reviewed by Simon Hausmann.
+
+        Because the WebView item accepts all touch events it receives and sends
+        them to the web process before propagating them to the gesture recognizers,
+        which is correct behaviour, we can not rely on the touch->mouse conversion
+        of Qt5 when controlling Flickable. Hence we need to convert the received
+        touch events to mouse events in the QtFlickProvider.
+
+        * UIProcess/qt/QtFlickProvider.cpp:
+        (QtFlickProvider::handleTouchFlickEvent):
+        Do the touch to mouse event conversion for the Flickable.
+        * UIProcess/qt/QtPanGestureRecognizer.cpp:
+        (WebKit::QtPanGestureRecognizer::recognize):
+        A touch begin event should cancel the previous pan gesture
+        and stop the ongoing flick animation.
+
 2012-03-01  Anders Carlsson  <andersca@apple.com>
 

trunk/Source/WebKit2/UIProcess/qt/QtFlickProvider.cpp

@@ -27 +27 @@
 #include <QCoreApplication>
 #include <QDeclarativeEngine>
+#include <QMouseEvent>
 #include <QPointF>
+#include <QQuickCanvas>
 #include <QQuickItem>
 #include <QTouchEvent>
@@ -117 +119 @@
 }
 
-void QtFlickProvider::handleTouchFlickEvent(QTouchEvent* event)
-{
-    QCoreApplication::sendEvent(m_flickable, event);
+void QtFlickProvider::handleTouchFlickEvent(QTouchEvent* touchEvent)
+{
+    // Since the Flickable does not handle touch events directly the sent
+    // touch event would end up in the WebView again and would thus trigger
+    // an infinite loop.
+    // Hence do the touch to mouse event conversion for the Flickable here.
+    QEvent::Type mouseEventType = QEvent::None;
+    Qt::MouseButton mouseButton = Qt::NoButton;
+
+    switch (touchEvent->type()) {
+    case QEvent::TouchBegin:
+        mouseEventType = QEvent::MouseButtonPress;
+
+        // We need to set the mouse button so that the Flickable
+        // item receives the initial mouse press event.
+        mouseButton = Qt::LeftButton;
+        break;
+    case QEvent::TouchUpdate:
+        mouseEventType = QEvent::MouseMove;
+        break;
+    case QEvent::TouchEnd:
+        mouseEventType = QEvent::MouseButtonRelease;
+        break;
+    default:
+        ASSERT_NOT_REACHED();
+        break;
+    }
+
+    QPointF touchPosition = touchEvent->touchPoints().last().pos();
+    QMouseEvent mouseEvent(mouseEventType, touchPosition, mouseButton, mouseButton, Qt::NoModifier);
+
+    // Send the event to the canvas and let the canvas propagate it
+    // to the Flickable. This makes sure that the Flickable grabs
+    // the mouse so that it also receives the events of gestures
+    // which started inside the viewport but ended outside of it.
+    QCoreApplication::sendEvent(m_flickable->canvas(), &mouseEvent);
 }
 

trunk/Source/WebKit2/UIProcess/qt/QtPanGestureRecognizer.cpp

@@ -64 +64 @@
         m_firstPosition = touchPoint.screenPos();
         m_touchBegin.reset(new QTouchEvent(*event));
+        interactionEngine()->panGestureCancelled();
         return false;
     case QEvent::TouchUpdate: {