1berry/qtbase
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Client: Don't assume windows have surfaces in QWaylandInputContext::update

Browse Source 
Fixes: QTBUG-72751
Change-Id: I6018a34d4a4cfcbdef5d6cd05d2d4ef12846efea
Reviewed-by: Paul Olav Tvete <paul.tvete@qt.io>
This commit is contained in:
Johan Klokkhammer Helsing 2019-01-02 12:33:03 +01:00 committed by Johan Helsing
parent c09a3a27b6
commit d812e2ea35
1 changed files with 16 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
src/plugins/platforms/wayland/qwaylandinputcontext.cpp
View File

@ -417,6 +417,15 @@ void QWaylandInputContext::commit()
textInput()->commit();
}
static ::wl_surface *surfaceForWindow(QWindow *window)
{
if (!window || !window->handle())
return nullptr;
auto *waylandWindow = static_cast<QWaylandWindow *>(window->handle());
return waylandWindow->wl_surface::object();
}
void QWaylandInputContext::update(Qt::InputMethodQueries queries)
{
qCDebug(qLcQpaInputMethods) << Q_FUNC_INFO << queries;
@ -424,15 +433,15 @@ void QWaylandInputContext::update(Qt::InputMethodQueries queries)
if (!QGuiApplication::focusObject() || !textInput())
return;
if (mCurrentWindow && mCurrentWindow->handle() && !inputMethodAccepted()) {
struct ::wl_surface *surface = static_cast<QWaylandWindow *>(mCurrentWindow->handle())->object();
textInput()->disable(surface);
auto *currentSurface = surfaceForWindow(mCurrentWindow);
if (currentSurface && !inputMethodAccepted()) {
textInput()->disable(currentSurface);
mCurrentWindow.clear();
} else if (!mCurrentWindow && inputMethodAccepted()) {
} else if (!currentSurface && inputMethodAccepted()) {
QWindow *window = QGuiApplication::focusWindow();
if (window && window->handle()) {
struct ::wl_surface *surface = static_cast<QWaylandWindow *>(window->handle())->object();
textInput()->enable(surface);
if (auto *focusSurface = surfaceForWindow(window)) {
textInput()->enable(focusSurface);
mCurrentWindow = window;
}
}