Provide the platform() -> AbstractBackend* in KWin::Application
ClosedPublic

Authored by graesslin on Apr 6 2016, 3:36 PM.

Details

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

Diff Detail

Repository
R108 KWin
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
graesslin updated this revision to Diff 3165.Apr 6 2016, 3:36 PM
graesslin retitled this revision from to Provide the platform() -> AbstractBackend* in KWin::Application.
graesslin updated this object.
graesslin edited the test plan for this revision. (Show Details)
graesslin added reviewers: Plasma, sebas.
Restricted Application added a project: Plasma. · View Herald TranscriptApr 6 2016, 3:36 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
sebas accepted this revision.Apr 6 2016, 5:24 PM
sebas edited edge metadata.
This revision is now accepted and ready to land.Apr 6 2016, 5:24 PM
This revision was automatically updated to reflect the committed changes.