Move decoratedClient from Client to AbstractClient

abstract_client.cpp

@@ -18,6 +18,7 @@ You should have received a copy of the GNU General Public License
 along with this program.  If not, see <http://www.gnu.org/licenses/>.
 *********************************************************************/
 #include "abstract_client.h"
+#include "decorations/decoratedclient.h"
 #include "decorations/decorationpalette.h"
 #include "cursor.h"
 #include "effects.h"
@@ -1470,4 +1471,14 @@ void AbstractClient::showContextHelp()
 {
 }
 
+QPointer<Decoration::DecoratedClientImpl> AbstractClient::decoratedClient() const
+{
+    return m_decoratedClient;
+}
+
+void AbstractClient::setDecoratedClient(QPointer< Decoration::DecoratedClientImpl > client)
+{
+    m_decoratedClient = client;
+}
+
 }

abstract_client.h

@@ -52,6 +52,7 @@ class TabBoxClientImpl;
 
 namespace Decoration
 {
+class DecoratedClientImpl;
 class DecorationPalette;
 }
 
@@ -509,6 +510,8 @@ public:
     bool isDecorated() const {
         return m_decoration != nullptr;
     }
+    QPointer<Decoration::DecoratedClientImpl> decoratedClient() const;
+    void setDecoratedClient(QPointer<Decoration::DecoratedClientImpl> client);
     bool decorationHasAlpha() const;
     void triggerDecorationRepaint();
     virtual void layoutDecorationRects(QRect &left, QRect &top, QRect &right, QRect &bottom) const;
@@ -910,6 +913,7 @@ private:
     } m_moveResize;
 
     KDecoration2::Decoration *m_decoration = nullptr;
+    QPointer<Decoration::DecoratedClientImpl> m_decoratedClient;
     QElapsedTimer m_decorationDoubleClickTimer;
 
 

client.cpp

@@ -2111,16 +2111,6 @@ void Client::showOnScreenEdge()
     xcb_delete_property(connection(), window(), atoms->kde_screen_edge_show);
 }
 
-QPointer<Decoration::DecoratedClientImpl> Client::decoratedClient() const
-{
-    return m_decoratedClient;
-}
-
-void Client::setDecoratedClient(QPointer< Decoration::DecoratedClientImpl > client)
-{
-    m_decoratedClient = client;
-}
-
 void Client::addDamage(const QRegion &damage)
 {
     if (!ready_for_painting) { // avoid "setReadyForPainting()" function calling overhead

client.h

@@ -48,11 +48,6 @@ struct xcb_sync_alarm_notify_event_t;
 namespace KWin
 {
 
-namespace Decoration
-{
-class DecoratedClientImpl;
-}
-
 
 /**
  * @brief Defines Predicates on how to search for a Client.
@@ -330,9 +325,6 @@ public:
     bool hasOffscreenXineramaStrut() const;
 
     // Decorations <-> Effects
-    QPointer<Decoration::DecoratedClientImpl> decoratedClient() const;
-    void setDecoratedClient(QPointer<Decoration::DecoratedClientImpl> client);
-
     QRect decorationRect() const;
 
     QRect transparentRect() const;
@@ -533,7 +525,6 @@ private:
     Xcb::Window m_client;
     Xcb::Window m_wrapper;
     Xcb::Window m_frame;
-    QPointer<Decoration::DecoratedClientImpl> m_decoratedClient;
     QStringList activityList;
     int m_activityUpdatesBlocked;
     bool m_blockedActivityUpdatesRequireTransients;