D29250: [wayland] Enable synchronized resizing for Xwayland clients

[Méven Car]

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


  Did you test with XWayland 1.21 ?
  
  All in all, I think this is the proper way to do it : leveraging the improvements from upstream.

INLINE COMMENTS

> platform.cpp:521
> +    case Application::OperationModeX11:
> +        kwinApp()->setX11Time(QX11Info::getTimestamp(), Application::TimestampUpdate::Always);
> +        break;

I was thinking we could remove the argument Application::TimestampUpdate::Always by simply doing an overflow check in setX11Time.

REPOSITORY
  R108 KWin

BRANCH
  xwayland-synchronized-resize

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

To: zzag, #kwin, meven
Cc: ahiemstra, kwin, Orage, cacarry, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, mkulinski, ragreen, jackyalcine, iodelay, crozbo, bwowk, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, romangg, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwin/attachments/20200429/fd618567/attachment.html>