[PATCH xserver v2 5/5] xwayland: use _XWAYLAND_ALLOW_COMMITS property

[Daniel Stone]

Hi,

On 2 January 2017 at 22:17, Adam Jackson <ajax at nwnk.net> wrote:
> On Fri, 2016-12-09 at 14:24 +0200, Pekka Paalanen wrote:
>> The X11 window manager (XWM) of a Wayland compositor can use the
>> _XWAYLAND_ALLOW_COMMITS property to control when Xwayland sends
>> wl_surface.commit requests. If the property is not set, the behaviour
>> remains what it was.
>
> I'm still thinking over whether I like this or whether I'd rather have
> this keyed off the netwm sync props. (Did we think that was a thing
> that could work? Forgive me, I'm freshly back from holidays.) Still,
> some style notes below.

Hm. So, is what you're proposing that the server would intercept
_NET_WM_SYNC_REQUEST ClientMessages from WM -> client and not send any
further commits for that window's surface, until the last SYNC_REQUEST
sent and the SYNC_REQUEST_COUNTER become coherent? I think that'd
largely work; it does exclude NetWM clients, but eh.

Cheers,
Daniel