Move Window::geometryShapeChanged to X11Window

src/internalwindow.cpp

@@ -310,8 +310,6 @@ void InternalWindow::createDecoration(const QRectF &oldGeometry)
 {
     setDecoration(std::shared_ptr<KDecoration2::Decoration>(Workspace::self()->decorationBridge()->createDecoration(this)));
     moveResize(QRectF(oldGeometry.topLeft(), clientSizeToFrameSize(clientSize())));
-
-    Q_EMIT geometryShapeChanged(oldGeometry);
 }
 
 void InternalWindow::destroyDecoration()
@@ -508,7 +506,6 @@ void InternalWindow::commitGeometry(const QRectF &rect)
     if (oldOutput != m_output) {
         Q_EMIT outputChanged();
     }
-    Q_EMIT geometryShapeChanged(oldFrameGeometry);
 }
 
 void InternalWindow::setCaption(const QString &caption)

src/scene/surfaceitem_wayland.cpp

@@ -207,7 +207,7 @@ SurfaceItemXwayland::SurfaceItemXwayland(X11Window *window, Scene *scene, Item *
     : SurfaceItemWayland(window->surface(), scene, parent)
     , m_window(window)
 {
-    connect(window, &Window::geometryShapeChanged, this, &SurfaceItemXwayland::discardQuads);
+    connect(window, &X11Window::geometryShapeChanged, this, &SurfaceItemXwayland::discardQuads);
 }
 
 QVector<QRectF> SurfaceItemXwayland::shape() const

src/scene/surfaceitem_x11.cpp

@@ -19,7 +19,7 @@ SurfaceItemX11::SurfaceItemX11(X11Window *window, Scene *scene, Item *parent)
 {
     connect(window, &Window::bufferGeometryChanged,
             this, &SurfaceItemX11::handleBufferGeometryChanged);
-    connect(window, &Window::geometryShapeChanged,
+    connect(window, &X11Window::geometryShapeChanged,
             this, &SurfaceItemX11::handleGeometryShapeChanged);
 
     m_damageHandle = xcb_generate_id(kwinApp()->x11Connection());

src/waylandwindow.cpp

@@ -322,7 +322,6 @@ void WaylandWindow::updateGeometry(const QRectF &rect)
     if (oldOutput != m_output) {
         Q_EMIT outputChanged();
     }
-    Q_EMIT geometryShapeChanged(oldFrameGeometry);
 }
 
 void WaylandWindow::markAsMapped()

src/window.cpp

@@ -2642,7 +2642,6 @@ void Window::setDecoration(std::shared_ptr<KDecoration2::Decoration> decoration)
             if (!isShade()) {
                 checkWorkspacePosition(oldGeometry);
             }
-            Q_EMIT geometryShapeChanged(oldGeometry);
         });
         connect(decoratedClient()->decoratedClient(), &KDecoration2::DecoratedClient::sizeChanged,
                 this, &Window::updateDecorationInputShape);

src/window.h

@@ -1324,7 +1324,6 @@ Q_SIGNALS:
     void opacityChanged(KWin::Window *window, qreal oldOpacity);
     void damaged(KWin::Window *window);
     void inputTransformationChanged();
-    void geometryShapeChanged(const QRectF &old);
     void closed();
     void windowShown(KWin::Window *window);
     void windowHidden(KWin::Window *window);

src/x11window.cpp

@@ -326,7 +326,7 @@ X11Window::X11Window()
     connect(clientMachine(), &ClientMachine::localhostChanged, this, &X11Window::updateCaption);
     connect(options, &Options::configChanged, this, &X11Window::updateMouseGrab);
     connect(options, &Options::condensedTitleChanged, this, &X11Window::updateCaption);
-    connect(this, &Window::geometryShapeChanged, this, &X11Window::discardShapeRegion);
+    connect(this, &X11Window::geometryShapeChanged, this, &X11Window::discardShapeRegion);
 
     if (kwinApp()->operationMode() == Application::OperationModeX11) {
         connect(this, &X11Window::moveResizeCursorChanged, this, [this](CursorShape cursor) {

src/x11window.h

@@ -358,6 +358,7 @@ Q_SIGNALS:
      * Emitted whenever the Client's block compositing state changes.
      */
     void blockingCompositingChanged(KWin::X11Window *client);
+    void geometryShapeChanged(const QRectF &old);
 
 private:
     void exportMappingState(int s); // ICCCM 4.1.3.1, 4.1.4, NETWM 2.5.1