autotests: remove qWait(1) in touch sequences
After d04982dc84d66bec92b4b3767538676cf925ef17 in qtbase, QTest::touchEvent() and QTouchEventSequence::commit() slow down every sequence of touch events, so we don't need to do it in individual tests. Change-Id: Id8133b100797d4bd2d7282ee874dbb81ed2cab47 Reviewed-by: Liang Qi <liang.qi@qt.io>
This commit is contained in:
parent
d54d28981c
commit
16834ed8cb
|
@ -1461,7 +1461,6 @@ void tst_qquickflickable::flickWithTouch(QQuickWindow *window, QTouchDevice *tou
|
|||
for (int i = 1; i <= 8; ++i) {
|
||||
QTest::touchEvent(window, touchDevice).move(0, from + i*diff/8, window);
|
||||
QQuickTouchUtils::flush(window);
|
||||
QTest::qWait(1); // because Flickable pays attention to velocity, we need some time between movements
|
||||
}
|
||||
QTest::touchEvent(window, touchDevice).release(0, to, window);
|
||||
QQuickTouchUtils::flush(window);
|
||||
|
|
|
@ -596,22 +596,18 @@ void tst_QQuickMultiPointTouchArea::inFlickable()
|
|||
QQuickTouchUtils::flush(window.data());
|
||||
|
||||
p1 += QPoint(0,15);
|
||||
QTest::qWait(1); // because Flickable pays attention to velocity, we need some time between movements
|
||||
QTest::touchEvent(window.data(), device).move(0, p1);
|
||||
QQuickTouchUtils::flush(window.data());
|
||||
|
||||
p1 += QPoint(0,15);
|
||||
QTest::qWait(1);
|
||||
QTest::touchEvent(window.data(), device).move(0, p1);
|
||||
QQuickTouchUtils::flush(window.data());
|
||||
|
||||
p1 += QPoint(0,15);
|
||||
QTest::qWait(1);
|
||||
QTest::touchEvent(window.data(), device).move(0, p1);
|
||||
QQuickTouchUtils::flush(window.data());
|
||||
|
||||
p1 += QPoint(0,15);
|
||||
QTest::qWait(1);
|
||||
QTest::touchEvent(window.data(), device).move(0, p1);
|
||||
QQuickTouchUtils::flush(window.data());
|
||||
|
||||
|
@ -787,22 +783,18 @@ void tst_QQuickMultiPointTouchArea::inFlickable2()
|
|||
QCOMPARE(point11->pressed(), true);
|
||||
|
||||
p1 += QPoint(0,15);
|
||||
QTest::qWait(1);
|
||||
QTest::touchEvent(window.data(), device).move(0, p1);
|
||||
QQuickTouchUtils::flush(window.data());
|
||||
|
||||
p1 += QPoint(0,15);
|
||||
QTest::qWait(1);
|
||||
QTest::touchEvent(window.data(), device).move(0, p1);
|
||||
QQuickTouchUtils::flush(window.data());
|
||||
|
||||
p1 += QPoint(0,15);
|
||||
QTest::qWait(1);
|
||||
QTest::touchEvent(window.data(), device).move(0, p1);
|
||||
QQuickTouchUtils::flush(window.data());
|
||||
|
||||
p1 += QPoint(0,15);
|
||||
QTest::qWait(1);
|
||||
QTest::touchEvent(window.data(), device).move(0, p1);
|
||||
QQuickTouchUtils::flush(window.data());
|
||||
|
||||
|
|
|
@ -586,13 +586,10 @@ void tst_TouchMouse::buttonOnFlickable()
|
|||
QPoint p2 = p1 + QPoint(0, -10);
|
||||
QPoint p3 = p2 + QPoint(0, -10);
|
||||
QQuickTouchUtils::flush(window);
|
||||
QTest::qWait(1); // because Flickable pays attention to velocity, we need some time between movements
|
||||
QTest::touchEvent(window, device).move(0, p1, window);
|
||||
QQuickTouchUtils::flush(window);
|
||||
QTest::qWait(1);
|
||||
QTest::touchEvent(window, device).move(0, p2, window);
|
||||
QQuickTouchUtils::flush(window);
|
||||
QTest::qWait(1);
|
||||
QTest::touchEvent(window, device).move(0, p3, window);
|
||||
QQuickTouchUtils::flush(window);
|
||||
|
||||
|
@ -674,13 +671,10 @@ void tst_TouchMouse::buttonOnDelayedPressFlickable()
|
|||
QPoint p2 = p1 + QPoint(0, -10);
|
||||
QPoint p3 = p2 + QPoint(0, -10);
|
||||
QQuickTouchUtils::flush(window);
|
||||
QTest::qWait(1);
|
||||
QTest::touchEvent(window, device).move(0, p1, window);
|
||||
QQuickTouchUtils::flush(window);
|
||||
QTest::qWait(1);
|
||||
QTest::touchEvent(window, device).move(0, p2, window);
|
||||
QQuickTouchUtils::flush(window);
|
||||
QTest::qWait(1);
|
||||
QTest::touchEvent(window, device).move(0, p3, window);
|
||||
QQuickTouchUtils::flush(window);
|
||||
QVERIFY(flickable->isMovingVertically());
|
||||
|
@ -872,19 +866,15 @@ void tst_TouchMouse::pinchOnFlickable()
|
|||
QQuickTouchUtils::flush(window);
|
||||
QCOMPARE(rect->position(), QPointF(200.0, 200.0));
|
||||
p -= QPoint(10, 0);
|
||||
QTest::qWait(1);
|
||||
QTest::touchEvent(window, device).move(0, p, window);
|
||||
QQuickTouchUtils::flush(window);
|
||||
p -= QPoint(10, 0);
|
||||
QTest::qWait(1);
|
||||
QTest::touchEvent(window, device).move(0, p, window);
|
||||
QQuickTouchUtils::flush(window);
|
||||
p -= QPoint(10, 0);
|
||||
QTest::qWait(1);
|
||||
QTest::touchEvent(window, device).move(0, p, window);
|
||||
QQuickTouchUtils::flush(window);
|
||||
p -= QPoint(10, 0);
|
||||
QTest::qWait(1);
|
||||
QTest::touchEvent(window, device).move(0, p, window);
|
||||
QQuickTouchUtils::flush(window);
|
||||
QTest::touchEvent(window, device).release(0, p, window);
|
||||
|
@ -1097,16 +1087,13 @@ void tst_TouchMouse::mouseOnFlickableOnPinch()
|
|||
QQuickTouchUtils::flush(window);
|
||||
QCOMPARE(rect->position(), QPointF(200.0, 200.0));
|
||||
p -= QPoint(10, 0);
|
||||
QTest::qWait(1);
|
||||
pinchSequence.move(0, p, window).commit();
|
||||
QQuickTouchUtils::flush(window);
|
||||
p -= QPoint(10, 0);
|
||||
QTest::qWait(1);
|
||||
pinchSequence.move(0, p, window).commit();
|
||||
QQuickTouchUtils::flush(window);
|
||||
QGuiApplication::processEvents();
|
||||
p -= QPoint(10, 0);
|
||||
QTest::qWait(1);
|
||||
pinchSequence.move(0, p, window).commit();
|
||||
QQuickTouchUtils::flush(window);
|
||||
|
||||
|
|
Loading…
Reference in New Issue