[Differential] [Request, 89 lines] D1331: Provide the platform() -> AbstractBackend* in KWin::Application
graesslin (Martin Gräßlin)
noreply at phabricator.kde.org
Wed Apr 6 15:36:26 UTC 2016
graesslin created this revision.
graesslin added reviewers: Plasma, sebas.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
REVISION SUMMARY
This is the first change in a refactoring series. The aim is to:
- rename AbstractBackend to Platform
- move backends/ to plugins/platforms/
- don't bind platforms to Wayland only
- provide a platform plugin for "normal" X11
- share more code between X11 and Wayland
This change moves the platform/backend from waylandServer to Application.
The init of the plugin happens directly in the Application from the
KPluginMetaData. There is no need to externally init it and set the
parent.
WaylandServer::backend() currently just delegates to
kwinApp()->platform(), the idea is to drop this method completely.
The test infrastructure is also adjusted to this change.
TEST PLAN
kwin_wayland still works, all tests pass
REPOSITORY
rKWIN KWin
BRANCH
platform-application
REVISION DETAIL
https://phabricator.kde.org/D1331
AFFECTED FILES
abstract_backend.cpp
autotests/wayland/kwin_wayland_test.cpp
main.cpp
main.h
main_wayland.cpp
wayland_server.cpp
wayland_server.h
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To: graesslin, Plasma, sebas
Cc: plasma-devel, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160406/91eadba6/attachment.html>
More information about the Plasma-devel
mailing list