Uploaded image for project: 'Qt'
  1. Qt
  2. QTBUG-129943

tst_PointHandler::tabletStylus is flaky

    XMLWordPrintable

Details

    • ee3bfd7a3 (dev), abf6dfa99 (6.8)

    Description

      https://testresults.qt.io/grafana/d/3dhio4K7k/fastcheck-ci-test-info?orgId=1&var-testCase=tst_qquickpointhandler&var-testFunction=tabletStylus&var-branch=dev

      It became obvious when https://codereview.qt-project.org/c/qt/qtdeclarative/+/461683 was auto-picked to 6.8 and failed the first time.

      For one thing, if I manually move the mouse over the window while it's running, at least this can fail:

          QWindowSystemInterface::handleTabletEvent(window, point, window->mapToGlobal(point),
              int(QInputDevice::DeviceType::Stylus), int(QPointingDevice::PointerType::Pen), Qt::LeftButton, 0.5, 25, 35, 0.6, 12.3, 3, stylusId, Qt::NoModifier);
          QTRY_COMPARE(handler->active(), true);
      

      Another failure I saw on 6.8:

          QPoint delta(10, 10);
          QPoint deltaLocalDPI = QHighDpi::fromNativeLocalPosition(delta, window);
          point += delta;
          QWindowSystemInterface::handleTabletEvent(window, point, window->mapToGlobal(point),
              int(QInputDevice::DeviceType::Stylus), int(QPointingDevice::PointerType::Pen), Qt::LeftButton, 0.45, 23, 33, 0.57, 15.6, 3, stylusId, Qt::NoModifier);
          QTRY_COMPARE(pointSpy.size(), guiSynthMouse ? 5 : 2);
      ...
          QCOMPARE_GT(handler->point().velocity().x(), 0);
      

      If velocity == 0, it's not greater. So the movement was not seen...

      Attachments

        Issue Links

          No reviews matched the request. Check your Options in the drop-down menu of this sections header.

          Activity

            People

              srutledg Shawn Rutledge
              srutledg Shawn Rutledge
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Gerrit Reviews

                  There is 1 open Gerrit change