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

Properly clean up screens during reconnect

Browse Source 
Otherwise the old screens are left dangling. Also create a placeholder
screen so the windows get assigned a valid screen upon reconnect.

Change-Id: Iea5d1da6f32be5e87464412447ae1449d91d8e75
Reviewed-by: David Edmundson <davidedmundson@kde.org>
This commit is contained in:
David Redondo 2023-04-27 15:22:26 +02:00
parent f2344f7d71
commit 117f8dedcc
2 changed files with 31 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
src/plugins/platforms/wayland/qwaylanddisplay.cpp
View File

@ -420,6 +420,12 @@ void QWaylandDisplay::reconnect()
qDeleteAll(mWaitingScreens);
mWaitingScreens.clear();
const auto screens = std::exchange(mScreens, {});
ensureScreen();
for (QWaylandScreen *screen : screens) {
QWindowSystemInterface::handleScreenRemoved(screen);
}
// mCompositor
mShm.reset();
mCursorThemes.clear();
@ -449,9 +455,6 @@ void QWaylandDisplay::reconnect()
qDeleteAll(std::exchange(mInputDevices, {}));
mLastInputDevice = nullptr;
auto screens = mScreens;
mScreens.clear();
for (const RegistryGlobal &global : mGlobals) {
emit globalRemoved(global);
}

25
tests/auto/wayland/reconnect/tst_reconnect.cpp
View File

@ -101,6 +101,7 @@ private Q_SLOTS:
//core
void cleanup() { QTRY_VERIFY2(m_comp->isClean(), qPrintable(m_comp->dirtyMessage())); }
void basicWindow();
void screens();
//input
void keyFocus();
@ -141,6 +142,30 @@ void tst_WaylandReconnect::basicWindow()
QCOMPOSITOR_TRY_VERIFY(m_comp->xdgToplevel());
}
void tst_WaylandReconnect::screens()
{
QRasterWindow window;
window.resize(64, 48);
window.show();
QCOMPOSITOR_TRY_VERIFY(m_comp->xdgToplevel());
auto originalScreens = QGuiApplication::screens();
QSignalSpy screenRemovedSpy(qGuiApp, &QGuiApplication::screenRemoved);
QVERIFY(screenRemovedSpy.isValid());
triggerReconnect();
// All screens plus temporary placeholder screen removed
QCOMPARE(screenRemovedSpy.count(), originalScreens.count() + 1);
for (const auto &screen : std::as_const(screenRemovedSpy)) {
originalScreens.removeOne(screen[0].value<QScreen *>());
}
QVERIFY(originalScreens.isEmpty());
QVERIFY(window.screen());
QVERIFY(QGuiApplication::screens().contains(window.screen()));
}
void tst_WaylandReconnect::keyFocus()
{
TestWindow window;