diff options
author | Kari Oikarinen <kari.oikarinen@qt.io> | 2018-02-14 16:59:52 +0200 |
---|---|---|
committer | Kari Oikarinen <kari.oikarinen@qt.io> | 2018-03-02 07:20:32 +0000 |
commit | d42a9d19ff1ed83bdb31a62e28a5757384e05367 (patch) | |
tree | df8a76ff351af5d1d9ab45d8574ac4e20080a7f7 /tests/auto/widgets/kernel/qapplication | |
parent | 810bc3fb1942fa241c7ca263aec6eb53085003bf (diff) |
Remove tst_QGLThreads::textureUploadInThread
The test has been crashing flakily recently.
1) It is creating a QGLWidget
2) It is stealing the QGLContext of that widget and moves it into a separate
thread.
3) In that secondary thread it makes the context current.
4) Meanwhile the QGLWidget itself may receive for example a resizeEvent or
other events and - since it assumes that it owns the context - attempts to
make it current.
5) Attempting to call makeCurrent() on a QGLContext that is in a different
thread than the current thread (via QObject thread affinity) will result
in a call to qFatal() and consequently the test aborts.
The conclusion from Simon Hausmann is that this test is testing a pattern from
Qt4 times that may or may not have worked back then. Nowadays with the Qt5
QOpenGL* API we do support this properly and there appears little sense testing
this.
Therefore remove the test altogether.
Task-number: QTBUG-66411
Task-number: QTBUG-66216
Change-Id: Ie2d66705bc7c3914ace6abcba9557c7c67ad4db3
Reviewed-by: Laszlo Agocs <laszlo.agocs@qt.io>
Diffstat (limited to 'tests/auto/widgets/kernel/qapplication')
0 files changed, 0 insertions, 0 deletions