D16783: Initialize Wayland output in AbstractOutput

davidedmundson accepted this revision.
davidedmundson added a comment.
This revision is now accepted and ready to land.


  > Note, that there are still some properties in DrmOutput for which we still need initDrmWaylandOutput. Hopefully goes away in a few patches though.
  
  Ack, it's in a bit of a messy state. But no worse than the current one.

INLINE COMMENTS

> abstract_output.cpp:140
> +    m_waylandOutput = waylandServer()->display()->createOutput();
> +    setWaylandOutput(m_waylandOutput.data());
> +    createXdgOutput();

this does nothing, I think setWaylandOutput can be killed completely

> abstract_output.cpp:146
> +    m_waylandOutput->setPhysicalSize(rawPhysicalSize());
> +
> +

2 newlines

REPOSITORY
  R108 KWin

BRANCH
  abstractWaylandOutputInit

REVISION DETAIL
  https://phabricator.kde.org/D16783

To: romangg, #kwin, davidedmundson
Cc: davidedmundson, kwin, mkulinski, ragreen, jackyalcine, Pitel, iodelay, bwowk, ZrenBot, ngraham, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart