On Fri, Feb 19, 2016 at 07:21:51PM -0800, Bryce Harrington wrote: > On Wed, Dec 02, 2015 at 08:06:46PM -0500, Mike Blumenkrantz wrote: > > this change ensures that the client will set its initial state > > before performing any drawing, ensuring that there is no mismatch > > when creating a surface with a non-default state > > (eg. maximize, fullscreen, ...) > > > > looking at the following event flows: > > 1) wl_surface.attach, wl_surface.commit, xdg_shell.get_xdg_surface > > > > 2) wl_surface.attach, xdg_shell.get_xdg_surface, wl_surface.commit > > > > 3) xdg_shell.get_xdg_surface, wl_surface.commit, xdg_surface.configure, > > wl_surface.attach, wl_surface.commit > > > > only 3) is now valid, while 1) and 2) will trigger errors as a result > > of handling buffers prior to creating the xdg surface > > > > Reviewed-by: Jasper St. Pierre <[email protected]> > > Signed-off-by: Mike Blumenkrantz <[email protected]> > > Signed-off-by: Jonas Ådahl <[email protected]> > > Reviewed-by: Bryce Harrington <[email protected]>
Pushed: To ssh://git.freedesktop.org/git/wayland/wayland-protocols 22a9cd8..275fd34 master -> master Although, the filename in git is -v5 not -v6. Bryce > > --- > > unstable/xdg-shell/xdg-shell-unstable-v6.xml | 14 ++++++-------- > > 1 file changed, 6 insertions(+), 8 deletions(-) > > > > diff --git a/unstable/xdg-shell/xdg-shell-unstable-v6.xml > > b/unstable/xdg-shell/xdg-shell-unstable-v6.xml > > index 196c332..a03a615 100644 > > --- a/unstable/xdg-shell/xdg-shell-unstable-v6.xml > > +++ b/unstable/xdg-shell/xdg-shell-unstable-v6.xml > > @@ -147,14 +147,12 @@ > > them, and associate metadata like title and app id. > > > > The client must call wl_surface.commit on the corresponding > > wl_surface > > - for the xdg_surface state to take effect. Prior to committing the new > > - state, it can set up initial configuration, such as maximizing or > > setting > > - a window geometry. > > - > > - Even without attaching a buffer the compositor must respond to > > initial > > - committed configuration, for instance sending a configure event with > > - expected window geometry if the client maximized its surface during > > - initialization. > > + for the xdg_surface state to take effect. > > + > > + Creating an xdg_surface from a wl_surface which has a buffer > > attached or > > + committed is a client error, and any attempts by a client to attach > > or > > + manipulate a buffer prior to the first xdg_surface.configure call > > must > > + also be treated as errors. > > > > For a surface to be mapped by the compositor the client must have > > committed both an xdg_surface state and a buffer. > > -- > > 2.4.3 > > > > _______________________________________________ > > wayland-devel mailing list > > [email protected] > > http://lists.freedesktop.org/mailman/listinfo/wayland-devel > _______________________________________________ > wayland-devel mailing list > [email protected] > https://lists.freedesktop.org/mailman/listinfo/wayland-devel _______________________________________________ wayland-devel mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/wayland-devel
