diff options
author | Romain Pokrzywka <romain.pokrzywka@kdab.com> | 2012-07-19 19:22:30 (GMT) |
---|---|---|
committer | Qt by Nokia <qt-info@nokia.com> | 2012-07-20 01:13:18 (GMT) |
commit | dc3ceed88dc12b71d7ff6b45a7374257800351ed (patch) | |
tree | 418a87cdae083ad238820d9e2860cdf568bb78c5 | |
parent | a9c50bccd055567340dd6925c92d1dd62b73e143 (diff) | |
download | Qt-dc3ceed88dc12b71d7ff6b45a7374257800351ed.zip Qt-dc3ceed88dc12b71d7ff6b45a7374257800351ed.tar.gz Qt-dc3ceed88dc12b71d7ff6b45a7374257800351ed.tar.bz2 |
Don't discard QPA mouse events without an associated window or widget
The corresponding window and widget are found by the function later on
if not provided by the plugin, so having a null widget pointer is valid.
This check is unnecessary and breaks integration of QPA plugins not
widget-aware, such as the LinuxInput and TsLib ones.
The bug has already been fixed in Qt5.
Change-Id: I650308ce14e84d42b0c6d64f4f0ae1720728054c
Reviewed-by: Girish Ramakrishnan <girish.1.ramakrishnan@nokia.com>
-rw-r--r-- | src/gui/kernel/qapplication_qpa.cpp | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/src/gui/kernel/qapplication_qpa.cpp b/src/gui/kernel/qapplication_qpa.cpp index 5e55cbe..fe7505f 100644 --- a/src/gui/kernel/qapplication_qpa.cpp +++ b/src/gui/kernel/qapplication_qpa.cpp @@ -626,8 +626,6 @@ void QApplication::setMainWidget(QWidget *mainWidget) void QApplicationPrivate::processMouseEvent(QWindowSystemInterfacePrivate::MouseEvent *e) { - if (!e->widget) - return; // qDebug() << "handleMouseEvent" << tlw << ev.pos() << ev.globalPos() << hex << ev.buttons(); static QWeakPointer<QWidget> implicit_mouse_grabber; |