summaryrefslogtreecommitdiffstats
path: root/src/plugins/qnx/common
diff options
context:
space:
mode:
authorDan Cape <dcape@qnx.com>2016-02-29 15:26:12 -0500
committerDan Cape <dcape@qnx.com>2016-03-20 03:53:27 +0000
commitd48fbf286b6ae88218286d35493cd5416a7713a9 (patch)
treec2750df4807a8409f7eb21f0d917f5d49504180c /src/plugins/qnx/common
parentbf899dbf4d541bb85e292208a1e97ed3ae95d19e (diff)
QNX: Fixed issue with non Qt-related mm-renderer windows
Windowgrabber would catch events from windows that were not related to Qt Media Playback. Now there is a check to ensure the window id to compare against is set before assigning the window to the Windowgrabber. Change-Id: Ic85198de5fdb05e9fa740fc7b3b81f3bdffd631d Reviewed-by: Marc Mutz <marc.mutz@kdab.com> Reviewed-by: James McDonnell <jmcdonnell@qnx.com> Reviewed-by: Rafael Roquetto <rafael.roquetto@kdab.com>
Diffstat (limited to 'src/plugins/qnx/common')
-rw-r--r--src/plugins/qnx/common/windowgrabber.cpp3
1 files changed, 2 insertions, 1 deletions
diff --git a/src/plugins/qnx/common/windowgrabber.cpp b/src/plugins/qnx/common/windowgrabber.cpp
index 51ce0dbf4..c16e38206 100644
--- a/src/plugins/qnx/common/windowgrabber.cpp
+++ b/src/plugins/qnx/common/windowgrabber.cpp
@@ -276,7 +276,8 @@ bool WindowGrabber::handleScreenEvent(screen_event_t screen_event)
return false;
}
- if (m_windowId == idString) {
+ // Grab windows that have a non-empty ID string and a matching window id to grab
+ if (idString[0] != '\0' && m_windowId == idString) {
m_window = window;
start();
}