From 790ddc0909b22785726b7fcb63b829592859f3ef Mon Sep 17 00:00:00 2001
From: Vlad Zahorodnii <vlad.zahorodnii@kde.org>
Date: Fri, 4 Sep 2020 09:13:26 +0300
Subject: [PATCH] Remove WaylandClient::isInputMethod()

We assume that the input method server has at most one surface at a time,
which corresponds to the virtual keyboard. But it's not guaranteed to be
always true.

For example, an xdg_toplevel surface can be created for a configuration
dialog, in which case isInputMethod() should not return true.
---
 waylandclient.cpp | 5 -----
 waylandclient.h   | 1 -
 2 files changed, 6 deletions(-)

diff --git a/waylandclient.cpp b/waylandclient.cpp
index fc4701530..b134e0732 100644
--- a/waylandclient.cpp
+++ b/waylandclient.cpp
@@ -109,11 +109,6 @@ bool WaylandClient::isLockScreen() const
     return surface()->client() == waylandServer()->screenLockerClientConnection();
 }
 
-bool WaylandClient::isInputMethod() const
-{
-    return surface()->client() == waylandServer()->inputMethodConnection();
-}
-
 bool WaylandClient::isLocalhost() const
 {
     return true;
diff --git a/waylandclient.h b/waylandclient.h
index 159dee42e..eb566af41 100644
--- a/waylandclient.h
+++ b/waylandclient.h
@@ -34,7 +34,6 @@ public:
     quint32 windowId() const override;
     pid_t pid() const override;
     bool isLockScreen() const override;
-    bool isInputMethod() const override;
     bool isLocalhost() const override;
     double opacity() const override;
     void setOpacity(double opacity) override;