D17498: Xdg Decoration Support
This revision was automatically updated to reflect the committed changes. Closed by commit R127:46390781daad: Xdg Decoration Support (authored by davidedmundson). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D17498?vs=47468&id=47955#toc REPOSITORY R127 KWayland CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D17498?vs=47468&id=47955 REVISION DETAIL https://phabricator.kde.org/D17498 AFFECTED FILES autotests/client/CMakeLists.txt autotests/client/test_xdg_decoration.cpp src/client/CMakeLists.txt src/client/protocols/xdg-decoration-unstable-v1.xml src/client/registry.cpp src/client/registry.h src/client/xdgdecoration.cpp src/client/xdgdecoration.h src/server/CMakeLists.txt src/server/display.cpp src/server/display.h src/server/xdgdecoration_interface.cpp src/server/xdgdecoration_interface.h src/server/xdgshell_interface.h src/tools/mapping.txt To: davidedmundson, #kwin, zzag Cc: zzag, kde-frameworks-devel, michaelh, ngraham, bruns
D17498: Xdg Decoration Support
zzag accepted this revision. This revision is now accepted and ready to land. REPOSITORY R127 KWayland BRANCH master REVISION DETAIL https://phabricator.kde.org/D17498 To: davidedmundson, #kwin, zzag Cc: zzag, kde-frameworks-devel, michaelh, ngraham, bruns
D17498: Xdg Decoration Support
davidedmundson updated this revision to Diff 47468. davidedmundson added a comment. remove unused REPOSITORY R127 KWayland CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D17498?vs=47339&id=47468 BRANCH master REVISION DETAIL https://phabricator.kde.org/D17498 AFFECTED FILES autotests/client/CMakeLists.txt autotests/client/test_xdg_decoration.cpp src/client/CMakeLists.txt src/client/protocols/xdg-decoration-unstable-v1.xml src/client/registry.cpp src/client/registry.h src/client/xdgdecoration.cpp src/client/xdgdecoration.h src/server/CMakeLists.txt src/server/display.cpp src/server/display.h src/server/xdgdecoration_interface.cpp src/server/xdgdecoration_interface.h src/server/xdgshell_interface.h src/tools/mapping.txt To: davidedmundson, #kwin Cc: zzag, kde-frameworks-devel, michaelh, ngraham, bruns
D17498: Xdg Decoration Support
zzag added inline comments. INLINE COMMENTS > test_xdg_decoration.cpp:104 > +QSignalSpy compositorSpy(m_registry, &Registry::compositorAnnounced); > +QSignalSpy v6xdgShellSpy(m_registry, > &Registry::xdgShellUnstableV6Announced); > +QSignalSpy xdgShellSpy(m_registry, &Registry::xdgShellStableAnnounced); Do we need this spy? REPOSITORY R127 KWayland REVISION DETAIL https://phabricator.kde.org/D17498 To: davidedmundson, #kwin Cc: zzag, kde-frameworks-devel, michaelh, ngraham, bruns
D17498: Xdg Decoration Support
davidedmundson marked 5 inline comments as done. REPOSITORY R127 KWayland REVISION DETAIL https://phabricator.kde.org/D17498 To: davidedmundson, #kwin Cc: zzag, kde-frameworks-devel, michaelh, ngraham, bruns
D17498: Xdg Decoration Support
davidedmundson updated this revision to Diff 47339. davidedmundson added a comment. zzag's comments REPOSITORY R127 KWayland CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D17498?vs=47336&id=47339 BRANCH xdg_deco REVISION DETAIL https://phabricator.kde.org/D17498 AFFECTED FILES autotests/client/CMakeLists.txt autotests/client/test_xdg_decoration.cpp src/client/CMakeLists.txt src/client/protocols/xdg-decoration-unstable-v1.xml src/client/registry.cpp src/client/registry.h src/client/xdgdecoration.cpp src/client/xdgdecoration.h src/server/CMakeLists.txt src/server/display.cpp src/server/display.h src/server/xdgdecoration_interface.cpp src/server/xdgdecoration_interface.h src/server/xdgshell_interface.h src/tools/mapping.txt To: davidedmundson, #kwin Cc: zzag, kde-frameworks-devel, michaelh, ngraham, bruns
D17498: Xdg Decoration Support
davidedmundson updated this revision to Diff 47336. davidedmundson added a comment. misc cleanups REPOSITORY R127 KWayland CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D17498?vs=47331&id=47336 BRANCH xdg_deco REVISION DETAIL https://phabricator.kde.org/D17498 AFFECTED FILES autotests/client/CMakeLists.txt autotests/client/test_xdg_decoration.cpp src/client/CMakeLists.txt src/client/protocols/xdg-decoration-unstable-v1.xml src/client/registry.cpp src/client/registry.h src/client/xdgdecoration.cpp src/client/xdgdecoration.h src/server/CMakeLists.txt src/server/display.cpp src/server/display.h src/server/xdgdecoration_interface.cpp src/server/xdgdecoration_interface.h src/server/xdgshell_interface.h src/tools/mapping.txt To: davidedmundson, #kwin Cc: zzag, kde-frameworks-devel, michaelh, ngraham, bruns
D17498: Xdg Decoration Support
zzag added inline comments. INLINE COMMENTS > test_xdg_decoration.cpp:178-181 > + > QTest::addColumn("configuredMode"); > + > QTest::addColumn("configuredModeExp"); > +QTest::addColumn("setMode"); > + > QTest::addColumn("setModeExp"); Do we actually use it? > test_xdg_decoration.cpp:219 > +QVERIFY(shellSurfaceIface); > +qDebug() << decorationIface->surface(); > +QCOMPARE(decorationIface->surface(), shellSurfaceIface); Please delete it or provide useful output. > test_xdg_decoration.cpp:237 > +QVERIFY(modeRequestedSpy.wait()); > +qDebug() << modeRequestedSpy.count(); > +qDebug() << modeRequestedSpy.first().first(); Same here. > registry.h:665 > + * @see createXdgDecorationManager > + * @since 5.47 > + **/ since 5.54? > registry.h:1724 > +/** > + * Emitted whenever a zxdg_decoration_manger_v1 gets removed. > + * @param name The name of the removed interface Typo: manger REPOSITORY R127 KWayland REVISION DETAIL https://phabricator.kde.org/D17498 To: davidedmundson, #kwin Cc: zzag, kde-frameworks-devel, michaelh, ngraham, bruns
D17498: Xdg Decoration Support
davidedmundson created this revision. davidedmundson added a reviewer: KWin. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. davidedmundson requested review of this revision. REVISION SUMMARY Usual massive boilerplate. Should allow us to remove the injection in our QPT, as Qt 5.12 implements this protocol directly. Will probably be relevant for other toolkits as it's more standardised than our custom one. Client is mostly 1:1 with the protocol, server side has some tweaking to fit with kwayland turning an event based API into a state based one. TEST PLAN Relevant unit test Tried in modified kwin with our QPT disabled REPOSITORY R127 KWayland BRANCH xdg_deco REVISION DETAIL https://phabricator.kde.org/D17498 AFFECTED FILES autotests/client/CMakeLists.txt autotests/client/test_xdg_decoration.cpp src/client/CMakeLists.txt src/client/protocols/xdg-decoration-unstable-v1.xml src/client/registry.cpp src/client/registry.h src/client/xdgdecoration.cpp src/client/xdgdecoration.h src/server/CMakeLists.txt src/server/display.cpp src/server/display.h src/server/xdgdecoration_interface.cpp src/server/xdgdecoration_interface.h src/server/xdgshell_interface.h src/tools/mapping.txt To: davidedmundson, #kwin Cc: kde-frameworks-devel, michaelh, ngraham, bruns