[PATCH 1/9] shell: Position DRIVER fullscreen surfaces at origin

Kristian Høgsberg hoegsberg at gmail.com
Wed May 29 07:19:04 PDT 2013


On Wed, May 29, 2013 at 09:20:25AM +0200, Alexander Larsson wrote:
> On tis, 2013-05-28 at 16:10 -0400, Kristian Høgsberg wrote:
> > On Tue, May 28, 2013 at 04:23:32PM +0200, alexl at redhat.com wrote:
> > > From: Alexander Larsson <alexl at redhat.com>
> > > 
> > > When a window is fullscreened with DRIVER method and we succeeded
> > > in changing mode we need to actually move the surface to the
> > > origin of the output, or it won't be used for scanout.
> > > ---
> > >  src/shell.c | 3 +++
> > >  1 file changed, 3 insertions(+)
> > > 
> > > diff --git a/src/shell.c b/src/shell.c
> > > index 1443328..07be564 100644
> > > --- a/src/shell.c
> > > +++ b/src/shell.c
> > > @@ -1802,6 +1802,9 @@ shell_configure_fullscreen(struct shell_surface *shsurf)
> > >                               shsurf->fullscreen.framerate};
> > >  
> > >                       if (weston_output_switch_mode(output, &mode) == 0) {
> > > +                             weston_surface_set_position(surface,
> > > +                                                         output->x - surf_x,
> > > +                                                         output->y - surf_y);
> > >                               weston_surface_configure(shsurf->fullscreen.black_surface,
> > >                                                        output->x - surf_x,
> > >                                                        output->y - surf_y,
> > 
> > Is that not done by this weston_surface_configure() call?
> 
> No, they apply to different surfaces, one the client surface, the other
> the black surface.

Right, thanks, I knew I was missing something.  Committed.

Kristian


More information about the wayland-devel mailing list