74f98d4c12
Summary: If the ShellSurface gets destroyed while a decoration repaint is pending, it is possible that on tear down the decoration calls into ShellClient::caption. This used to call into ShellSurfaceInterface::title which accesses a d-ptr which is already destroyed at that point. This change caches the caption instead of delegating to ShellSurface, just like almost everything else caches. Thus the tear down cannot access invalid memory. Once we can depend on Frameworks 5.24, we should also make sure to connect to the new Resource::unbound signal to be able to handle tear down prior to the object being completely deleted. Test Plan: Unfortunately no test case as this depends on phase-of-moon aligning of destruction of the interfaces Reviewers: #kwin, #plasma_on_wayland Subscribers: plasma-devel, kwin Tags: #plasma_on_wayland, #kwin Differential Revision: https://phabricator.kde.org/D1913
205 lines
7.4 KiB
C++
205 lines
7.4 KiB
C++
/********************************************************************
|
|
KWin - the KDE window manager
|
|
This file is part of the KDE project.
|
|
|
|
Copyright (C) 2015 Martin Gräßlin <mgraesslin@kde.org>
|
|
|
|
This program is free software; you can redistribute it and/or modify
|
|
it under the terms of the GNU General Public License as published by
|
|
the Free Software Foundation; either version 2 of the License, or
|
|
(at your option) any later version.
|
|
|
|
This program is distributed in the hope that it will be useful,
|
|
but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
|
GNU General Public License for more details.
|
|
|
|
You should have received a copy of the GNU General Public License
|
|
along with this program. If not, see <http://www.gnu.org/licenses/>.
|
|
*********************************************************************/
|
|
#ifndef KWIN_SHELL_CLIENT_H
|
|
#define KWIN_SHELL_CLIENT_H
|
|
|
|
#include "abstract_client.h"
|
|
|
|
namespace KWayland
|
|
{
|
|
namespace Server
|
|
{
|
|
class ShellSurfaceInterface;
|
|
class ServerSideDecorationInterface;
|
|
class PlasmaShellSurfaceInterface;
|
|
class QtExtendedSurfaceInterface;
|
|
}
|
|
}
|
|
|
|
namespace KWin
|
|
{
|
|
|
|
class KWIN_EXPORT ShellClient : public AbstractClient
|
|
{
|
|
Q_OBJECT
|
|
public:
|
|
ShellClient(KWayland::Server::ShellSurfaceInterface *surface);
|
|
virtual ~ShellClient();
|
|
|
|
QStringList activities() const override;
|
|
QPoint clientContentPos() const override;
|
|
QSize clientSize() const override;
|
|
QRect transparentRect() const override;
|
|
bool shouldUnredirect() const override;
|
|
NET::WindowType windowType(bool direct = false, int supported_types = 0) const override;
|
|
void debug(QDebug &stream) const override;
|
|
double opacity() const override;
|
|
void setOpacity(double opacity) override;
|
|
QByteArray windowRole() const override;
|
|
|
|
KWayland::Server::ShellSurfaceInterface *shellSurface() const {
|
|
return m_shellSurface;
|
|
}
|
|
|
|
void blockActivityUpdates(bool b = true) override;
|
|
QString caption(bool full = true, bool stripped = false) const override;
|
|
void closeWindow() override;
|
|
AbstractClient *findModal(bool allow_itself = false) override;
|
|
bool isCloseable() const override;
|
|
bool isFullScreenable() const override;
|
|
bool isFullScreen() const override;
|
|
bool isMaximizable() const override;
|
|
bool isMinimizable() const override;
|
|
QRect iconGeometry() const override;
|
|
bool isMovable() const override;
|
|
bool isMovableAcrossScreens() const override;
|
|
bool isResizable() const override;
|
|
bool isShown(bool shaded_is_shown) const override;
|
|
void hideClient(bool hide) override;
|
|
MaximizeMode maximizeMode() const override;
|
|
QRect geometryRestore() const override {
|
|
return m_geomMaximizeRestore;
|
|
}
|
|
bool noBorder() const override;
|
|
const WindowRules *rules() const override;
|
|
void setFullScreen(bool set, bool user = true) override;
|
|
void setNoBorder(bool set) override;
|
|
void updateDecoration(bool check_workspace_pos, bool force = false) override;
|
|
void setOnAllActivities(bool set) override;
|
|
void setShortcut(const QString &cut) override;
|
|
const QKeySequence &shortcut() const override;
|
|
void takeFocus() override;
|
|
void updateWindowRules(Rules::Types selection) override;
|
|
bool userCanSetFullScreen() const override;
|
|
bool userCanSetNoBorder() const override;
|
|
bool wantsInput() const override;
|
|
using AbstractClient::resizeWithChecks;
|
|
void resizeWithChecks(int w, int h, ForceGeometry_t force = NormalGeometrySet) override;
|
|
using AbstractClient::setGeometry;
|
|
void setGeometry(int x, int y, int w, int h, ForceGeometry_t force = NormalGeometrySet) override;
|
|
bool hasStrut() const override;
|
|
|
|
void setInternalFramebufferObject(const QSharedPointer<QOpenGLFramebufferObject> &fbo) override;
|
|
|
|
quint32 windowId() const override {
|
|
return m_windowId;
|
|
}
|
|
bool isInternal() const;
|
|
bool isLockScreen() const override;
|
|
bool isInputMethod() const override;
|
|
QWindow *internalWindow() const {
|
|
return m_internalWindow;
|
|
}
|
|
|
|
void installPlasmaShellSurface(KWayland::Server::PlasmaShellSurfaceInterface *surface);
|
|
void installQtExtendedSurface(KWayland::Server::QtExtendedSurfaceInterface *surface);
|
|
void installServerSideDecoration(KWayland::Server::ServerSideDecorationInterface *decoration);
|
|
|
|
bool isInitialPositionSet() const;
|
|
|
|
bool isTransient() const override;
|
|
bool hasTransientPlacementHint() const override;
|
|
QPoint transientPlacementHint() const override;
|
|
|
|
QMatrix4x4 inputTransformation() const override;
|
|
|
|
protected:
|
|
void addDamage(const QRegion &damage) override;
|
|
bool belongsToSameApplication(const AbstractClient *other, bool active_hack) const override;
|
|
void doSetActive() override;
|
|
Layer layerForDock() const override;
|
|
void changeMaximize(bool horizontal, bool vertical, bool adjust) override;
|
|
void setGeometryRestore(const QRect &geo) override {
|
|
m_geomMaximizeRestore = geo;
|
|
}
|
|
void doResizeSync() override;
|
|
bool isWaitingForMoveResizeSync() const override;
|
|
bool acceptsFocus() const override;
|
|
|
|
private Q_SLOTS:
|
|
void clientFullScreenChanged(bool fullScreen);
|
|
|
|
private:
|
|
void init();
|
|
void requestGeometry(const QRect &rect);
|
|
void doSetGeometry(const QRect &rect);
|
|
void createDecoration(const QRect &oldgeom);
|
|
void destroyClient();
|
|
void unmap();
|
|
void createWindowId();
|
|
void findInternalWindow();
|
|
void updateInternalWindowGeometry();
|
|
void updateIcon();
|
|
void markAsMapped();
|
|
void setTransient();
|
|
bool shouldExposeToWindowManagement();
|
|
static void deleteClient(ShellClient *c);
|
|
|
|
KWayland::Server::ShellSurfaceInterface *m_shellSurface;
|
|
QSize m_clientSize;
|
|
|
|
ClearablePoint m_positionAfterResize; // co-ordinates saved from a requestGeometry call, real geometry will be updated after the next damage event when the client has resized
|
|
QRect m_geomFsRestore; //size and position of the window before it was set to fullscreen
|
|
bool m_closing = false;
|
|
quint32 m_windowId = 0;
|
|
QWindow *m_internalWindow = nullptr;
|
|
bool m_unmapped = true;
|
|
MaximizeMode m_maximizeMode = MaximizeRestore;
|
|
QRect m_geomMaximizeRestore; // size and position of the window before it was set to maximize
|
|
NET::WindowType m_windowType = NET::Normal;
|
|
QPointer<KWayland::Server::PlasmaShellSurfaceInterface> m_plasmaShellSurface;
|
|
QPointer<KWayland::Server::QtExtendedSurfaceInterface> m_qtExtendedSurface;
|
|
KWayland::Server::ServerSideDecorationInterface *m_serverDecoration = nullptr;
|
|
bool m_userNoBorder = false;
|
|
bool m_fullScreen = false;
|
|
bool m_transient = false;
|
|
bool m_internal;
|
|
qreal m_opacity = 1.0;
|
|
|
|
class RequestGeometryBlocker {
|
|
public:
|
|
RequestGeometryBlocker(ShellClient *client)
|
|
: m_client(client)
|
|
{
|
|
m_client->m_requestGeometryBlockCounter++;
|
|
}
|
|
~RequestGeometryBlocker()
|
|
{
|
|
m_client->m_requestGeometryBlockCounter--;
|
|
if (m_client->m_requestGeometryBlockCounter == 0) {
|
|
if (m_client->m_blockedRequestGeometry.isValid()) {
|
|
m_client->requestGeometry(m_client->m_blockedRequestGeometry);
|
|
}
|
|
}
|
|
}
|
|
private:
|
|
ShellClient *m_client;
|
|
};
|
|
friend class RequestGeometryBlocker;
|
|
int m_requestGeometryBlockCounter = 0;
|
|
QRect m_blockedRequestGeometry;
|
|
QString m_caption;
|
|
};
|
|
|
|
}
|
|
|
|
Q_DECLARE_METATYPE(KWin::ShellClient*)
|
|
|
|
#endif
|