From 2602b4402f3c8d867d11ea62b613b03dacc8a232 Mon Sep 17 00:00:00 2001 From: Robin Burchell Date: Wed, 13 Nov 2013 12:41:19 +0100 Subject: Fix touch to mouse synthesis/propagation. Having mouse events synthesised from both QtGui and internally in QtQuick is not a great way togo about things, especially when QtGui doesn't have the same degree of knowledge as QtQuick about the items in the scene. Thus, we now accept all events inside QtQuick to block QtGui synthesis, which should fix a significant amount of edge-case touch breakage/bad behavior. Change-Id: I14e1c87761c8f43160049b5e6f9da15b4e5edbb7 Done-with: Martin Jones Reviewed-by: Shawn Rutledge --- tests/auto/quick/qquickitem/tst_qquickitem.cpp | 15 +++- tests/auto/quick/touchmouse/tst_touchmouse.cpp | 100 ++++++++++++++++++++++++- 2 files changed, 111 insertions(+), 4 deletions(-) (limited to 'tests') diff --git a/tests/auto/quick/qquickitem/tst_qquickitem.cpp b/tests/auto/quick/qquickitem/tst_qquickitem.cpp index 2c6dcd72ba..ad3c4fc208 100644 --- a/tests/auto/quick/qquickitem/tst_qquickitem.cpp +++ b/tests/auto/quick/qquickitem/tst_qquickitem.cpp @@ -1310,7 +1310,10 @@ void tst_qquickitem::touchEventAcceptIgnore() bool accepted = window.event(&event); QVERIFY(item->touchEventReached); - QCOMPARE(accepted && event.isAccepted(), itemSupportsTouch); + + // always true because QtQuick always eats touch events so as to not + // allow QtGui to synthesise them for us. + QCOMPARE(accepted && event.isAccepted(), true); } { QTouchEvent::TouchPoint point; @@ -1330,7 +1333,10 @@ void tst_qquickitem::touchEventAcceptIgnore() bool accepted = window.event(&event); QCOMPARE(item->touchEventReached, itemSupportsTouch); - QCOMPARE(accepted && event.isAccepted(), itemSupportsTouch); + + // always true because QtQuick always eats touch events so as to not + // allow QtGui to synthesise them for us. + QCOMPARE(accepted && event.isAccepted(), true); } { QTouchEvent::TouchPoint point; @@ -1350,7 +1356,10 @@ void tst_qquickitem::touchEventAcceptIgnore() bool accepted = window.event(&event); QCOMPARE(item->touchEventReached, itemSupportsTouch); - QCOMPARE(accepted && event.isAccepted(), itemSupportsTouch); + + // always true because QtQuick always eats touch events so as to not + // allow QtGui to synthesise them for us. + QCOMPARE(accepted && event.isAccepted(), true); } } diff --git a/tests/auto/quick/touchmouse/tst_touchmouse.cpp b/tests/auto/quick/touchmouse/tst_touchmouse.cpp index 6cf0aa4749..5b4ad0ffa3 100644 --- a/tests/auto/quick/touchmouse/tst_touchmouse.cpp +++ b/tests/auto/quick/touchmouse/tst_touchmouse.cpp @@ -154,6 +154,7 @@ private slots: void mouseOverTouch(); void buttonOnFlickable(); + void buttonOnDelayedPressFlickable(); void buttonOnTouch(); void pinchOnFlickable(); @@ -162,9 +163,22 @@ private slots: void tapOnDismissiveTopMouseAreaClicksBottomOne(); +protected: + bool eventFilter(QObject *, QEvent *event) + { + if (event->type() == QEvent::MouseButtonPress || + event->type() == QEvent::MouseMove || + event->type() == QEvent::MouseButtonRelease) { + QMouseEvent *me = static_cast(event); + filteredEventList.append(Event(me->type(), me->pos(), me->globalPos())); + } + return false; + } + private: QQuickView *createView(); QTouchDevice *device; + QList filteredEventList; }; QQuickView *tst_TouchMouse::createView() @@ -496,7 +510,7 @@ void tst_TouchMouse::buttonOnFlickable() QCOMPARE(eventItem1->eventList.size(), 0); QPoint p1 = QPoint(20, 130); QTest::touchEvent(window, device).press(0, p1, window); - QCOMPARE(eventItem1->eventList.size(), 2); + QTRY_COMPARE(eventItem1->eventList.size(), 2); QCOMPARE(eventItem1->eventList.at(0).type, QEvent::TouchBegin); QCOMPARE(eventItem1->eventList.at(1).type, QEvent::MouseButtonPress); QTest::touchEvent(window, device).release(0, p1, window); @@ -559,12 +573,96 @@ void tst_TouchMouse::buttonOnFlickable() QCOMPARE(eventItem1->eventList.at(3).type, QEvent::MouseMove); QCOMPARE(windowPriv->mouseGrabberItem, flickable); + QCOMPARE(windowPriv->touchMouseId, 0); + QCOMPARE(windowPriv->itemForTouchPointId[0], flickable); QVERIFY(flickable->isMovingVertically()); QTest::touchEvent(window, device).release(0, p3, window); delete window; } +void tst_TouchMouse::buttonOnDelayedPressFlickable() +{ + // flickable - height 500 / 1000 + // - eventItem1 y: 100, height 100 + // - eventItem2 y: 300, height 100 + + qApp->setAttribute(Qt::AA_SynthesizeMouseForUnhandledTouchEvents, true); + filteredEventList.clear(); + + QQuickView *window = createView(); + + window->setSource(testFileUrl("buttononflickable.qml")); + window->show(); + QVERIFY(QTest::qWaitForWindowExposed(window)); + window->requestActivate(); + QVERIFY(QTest::qWaitForWindowActive(window)); + QVERIFY(window->rootObject() != 0); + + QQuickFlickable *flickable = window->rootObject()->findChild("flickable"); + QVERIFY(flickable); + + window->installEventFilter(this); + + flickable->setPressDelay(60); + + // should a mouse area button be clickable on top of flickable? yes :) + EventItem *eventItem1 = window->rootObject()->findChild("eventItem1"); + QVERIFY(eventItem1); + eventItem1->setAcceptedMouseButtons(Qt::LeftButton); + eventItem1->acceptMouse = true; + + // should a touch button be touchable on top of flickable? yes :) + EventItem *eventItem2 = window->rootObject()->findChild("eventItem2"); + QVERIFY(eventItem2); + QCOMPARE(eventItem2->eventList.size(), 0); + eventItem2->acceptTouch = true; + + // wait to avoid getting a double click event + QTest::qWait(qApp->styleHints()->mouseDoubleClickInterval() + 10); + + // check that flickable moves - mouse button + QCOMPARE(eventItem1->eventList.size(), 0); + QPoint p1 = QPoint(10, 110); + QTest::touchEvent(window, device).press(0, p1, window); + // Flickable initially steals events + QCOMPARE(eventItem1->eventList.size(), 0); + // but we'll get the delayed mouse press after a delay + QTRY_COMPARE(eventItem1->eventList.size(), 1); + QCOMPARE(eventItem1->eventList.at(0).type, QEvent::MouseButtonPress); + + // eventItem1 should have the mouse grab, and have moved the itemForTouchPointId + // for the touchMouseId to the new grabber. + QQuickWindowPrivate *windowPriv = QQuickWindowPrivate::get(window); + QCOMPARE(windowPriv->touchMouseId, 0); + QCOMPARE(windowPriv->itemForTouchPointId[0], eventItem1); + QCOMPARE(windowPriv->mouseGrabberItem, eventItem1); + + p1 += QPoint(0, -10); + QPoint p2 = p1 + QPoint(0, -10); + QPoint p3 = p2 + QPoint(0, -10); + QTest::qWait(10); + QTest::touchEvent(window, device).move(0, p1, window); + QTest::qWait(10); + QTest::touchEvent(window, device).move(0, p2, window); + QTest::qWait(10); + QTest::touchEvent(window, device).move(0, p3, window); + QVERIFY(flickable->isMovingVertically()); + + // flickable should have the mouse grab, and have moved the itemForTouchPointId + // for the touchMouseId to the new grabber. + QCOMPARE(windowPriv->mouseGrabberItem, flickable); + QCOMPARE(windowPriv->touchMouseId, 0); + QCOMPARE(windowPriv->itemForTouchPointId[0], flickable); + + QTest::touchEvent(window, device).release(0, p3, window); + + // We should not have received any synthesised mouse events from Qt gui. + QCOMPARE(filteredEventList.count(), 0); + + delete window; +} + void tst_TouchMouse::buttonOnTouch() { // 400x800 -- cgit v1.2.3