Kristian Høgsberg [Wed, 1 Aug 2012 02:18:28 +0000 (22:18 -0400)]
compositor-drm: Handle incomplete [output] sections without crashing
Kristian Høgsberg [Tue, 31 Jul 2012 21:29:30 +0000 (17:29 -0400)]
shell: Give keyboard focus to top surface when current keyboard focus goes away
Kristian Høgsberg [Tue, 31 Jul 2012 20:36:34 +0000 (16:36 -0400)]
shell: Always track focus state per workspace
It's a little simpler than always creating and destroying it when we
switch workspace, and we'll need the focus_state soon.
Kristian Høgsberg [Tue, 31 Jul 2012 18:54:48 +0000 (14:54 -0400)]
compositor: Set WAYLAND_DISPLAY if we're given a --socket argument
This lets us run multiple compositors (nested or on different VTs).
https://bugs.freedesktop.org/show_bug.cgi?id=46629
Kristian Høgsberg [Tue, 31 Jul 2012 18:32:01 +0000 (14:32 -0400)]
compositor: Use a minimal restore handler for crash clean up
When we hit a segv, it's often the case that we might crash again in
the attempt to clean up. Instead we introduce a minimal restore callback
in the backend abstraction, that shuts down as simply as possible. Then
we can call that from the segv handler, and then to aid debugging, we
raise SIGTRAP in the segv handler. This lets us run gdb on weston from
a different vt, and if we tell gdb
(gdb) handle SIGSEGV nostop
gdb won't stop when the segv happens but let weston clean up and switch vt,
and then stop when SIGTRAP is raised.
It's also possible to just let gdb catch the segv, and then use sysrq+k
followed by manual vt switch to get back.
Kristian Høgsberg [Tue, 31 Jul 2012 15:59:12 +0000 (11:59 -0400)]
window.c: Set input region after (potentially) adjusting the surface size
Regressed in
dd263e546bc20a2a7f42, spotted by Scott Moreau <oreaus@gmail.com>
Kristian Høgsberg [Tue, 31 Jul 2012 14:51:22 +0000 (10:51 -0400)]
shell: Remove unused shsurf->focus_state
Kristian Høgsberg [Tue, 31 Jul 2012 14:30:26 +0000 (10:30 -0400)]
shell: Use the right destroy listener in focus_state_surface_destroy()
We were dereferencing the seat destroy listener.
Scott Moreau [Tue, 31 Jul 2012 01:51:08 +0000 (19:51 -0600)]
Allow output configuration from config file.
Parse the config file for [output] sections and check for 'name'
and 'mode' keys. The key strings are compared to what is reported
by weston log. The 'mode' key string can be one of the following:
1) WIDTHxHEIGHT - one that is reported by weston log
2) off - Disables the output
3) preferred - Uses the preferred mode
4) current - Uses the mode currently driving the crtc
Wang Quanxian [Mon, 30 Jul 2012 22:09:46 +0000 (18:09 -0400)]
compositor-drm: Handle unconnected connectors better
If a connector is not attached to a crtc, weston fails to bring it up.
Typically kms fbdev drives all crtc and connectors, but if kms hasn't been
initialzed and weston is the first to set a mode of if a monitor is
hotplugged, we just fail to bring it up.
(krh: reformatted and edited a bit from original patch)
Kristian Høgsberg [Mon, 30 Jul 2012 19:41:14 +0000 (15:41 -0400)]
compositor: Add --help option
Kristian Høgsberg [Mon, 30 Jul 2012 19:40:57 +0000 (15:40 -0400)]
gears: Exit on ESC
Pekka Paalanen [Mon, 30 Jul 2012 13:56:57 +0000 (16:56 +0300)]
compositor: do clean-up on init failure
Simply exit(1)'ing the program will leave the VT unusable, since
DRM backend's clean-up does not run.
After a backend has been initialised, prefer jumping to clean-up instead
of directly exiting.
This fixes the case where 'weston-launch -- -i 5' would leave the
console unusable.
Signed-off-by: Pekka Paalanen <ppaalanen@gmail.com>
Ander Conselvan de Oliveira [Fri, 27 Jul 2012 14:18:13 +0000 (17:18 +0300)]
simple-egl: Add keyboard fullscreen toggle
Make simple-egl toggle the fullscreen state whenever the key F11 is
pressed. A sync callback is used to stop drawing while the surface has
not received the configure event, to prevent a buffer with the windowed
size to be attached to the surface after the set_fullscreen request.
Kristian Høgsberg [Fri, 27 Jul 2012 03:53:05 +0000 (23:53 -0400)]
gears: Keep gears inside screen in fullscreen mode
Kristian Høgsberg [Fri, 27 Jul 2012 03:03:57 +0000 (23:03 -0400)]
compositor-drm: Don't leak output->name
Kristian Høgsberg [Fri, 27 Jul 2012 02:44:55 +0000 (22:44 -0400)]
compositor-drm: Look through all crtc/encoder combinations
We used to only check the first possible encoder, now we go through all
possible encoders and look through all possible crtcs for each encoder.
Olivier Blin [Thu, 26 Jul 2012 16:55:53 +0000 (18:55 +0200)]
gears: add a warning about refresh rate limitation
Olivier Blin [Wed, 25 Jul 2012 13:19:23 +0000 (15:19 +0200)]
gears: show FPS count (as in glxgears from mesa/demos)
Kristian Høgsberg [Thu, 26 Jul 2012 21:57:15 +0000 (17:57 -0400)]
compositor-drm: Assign unique output names like xrandr does
Kristian Høgsberg [Thu, 26 Jul 2012 15:39:26 +0000 (11:39 -0400)]
compositor: Load modules with RTLD_NOW so we get missing symbols up front
Otherwise we have no way to clean up when we hit something undefined.
Kristian Høgsberg [Wed, 25 Jul 2012 20:35:28 +0000 (16:35 -0400)]
window: Handle Alt-F4 to close windows
Andre Heider [Wed, 25 Jul 2012 17:10:33 +0000 (19:10 +0200)]
window.c: Remove xkb leftovers
Olivier Blin [Wed, 25 Jul 2012 12:33:42 +0000 (14:33 +0200)]
gears: add fullscreen support
Damien Lespiau [Wed, 25 Jul 2012 13:51:09 +0000 (14:51 +0100)]
tests: Use pkg-config to find setbacklight CFLAGS and libs
setbacklight depends on udev and drm, one cannot just add the -l flags
if you want to support unusual install paths.
Kristian Høgsberg [Tue, 24 Jul 2012 20:01:29 +0000 (16:01 -0400)]
Bump version to 0.95.0
Kristian Høgsberg [Tue, 24 Jul 2012 02:59:33 +0000 (22:59 -0400)]
desktop-shell: Break command lines into env vars, executable and arguments
We now support specifying environment variables and arguments in launchers
by saying
path=GDK_BACKEND=wayland gnome-terminal --full-screen
for example.
https://bugs.freedesktop.org/show_bug.cgi?id=47920
Kristian Høgsberg [Tue, 24 Jul 2012 02:00:21 +0000 (22:00 -0400)]
image: Add fullscreen support
Kristian Høgsberg [Tue, 24 Jul 2012 01:56:31 +0000 (21:56 -0400)]
window.c: Add fullscreen handler to keep fullscreen state consistent
Kristian Høgsberg [Tue, 24 Jul 2012 01:32:46 +0000 (21:32 -0400)]
dnd: Clip flowers to window content area
https://bugs.freedesktop.org/show_bug.cgi?id=52420
Kristian Høgsberg [Tue, 24 Jul 2012 00:54:42 +0000 (20:54 -0400)]
Update src/weston-egl-ext.h
Daniel Stone [Mon, 23 Jul 2012 18:54:59 +0000 (19:54 +0100)]
evdev: Release weston_seat with underlying evdev device
Signed-off-by: Daniel Stone <daniel@fooishbar.org>
Daniel Stone [Mon, 23 Jul 2012 18:54:58 +0000 (19:54 +0100)]
Fix memory leak on compositor exit
Signed-off-by: Daniel Stone <daniel@fooishbar.org>
Daniel Stone [Mon, 23 Jul 2012 18:54:57 +0000 (19:54 +0100)]
evdev: Don't ignore multitouch touchscreens
Most touchscreen drivers provide ABS_X and BTN_TOUCH for legacy
single-touch emulation modes, but this isn't mandatory. Make sure we
don't ignore touchscreens with provide multitouch events with the new
API only.
Signed-off-by: Daniel Stone <daniel@fooishbar.org>
Scott Moreau [Mon, 23 Jul 2012 17:53:18 +0000 (11:53 -0600)]
wcap: Fix typo in usage output.
Kristian Høgsberg [Mon, 23 Jul 2012 15:10:20 +0000 (11:10 -0400)]
wcap: Clarify help message, don't dump yuv4mpeg2 to terminal
Scott Moreau [Mon, 23 Jul 2012 00:23:52 +0000 (18:23 -0600)]
desktop-shell: Declare grab_cursor as enum cursor_type.
Kristian Høgsberg [Mon, 23 Jul 2012 14:47:34 +0000 (10:47 -0400)]
wcap: Add GCC_CFLAGS and fix more compiler warnings
Scott Moreau [Mon, 23 Jul 2012 00:23:51 +0000 (18:23 -0600)]
wcap: Declare variable with same sign as convert_to_yv12() expects.
Kristian Høgsberg [Sun, 22 Jul 2012 19:49:57 +0000 (15:49 -0400)]
Handle new transform argument in wl_output.geometry event
Kristian Høgsberg [Sun, 22 Jul 2012 19:32:45 +0000 (15:32 -0400)]
xwm: Make override-redirect windows opaque
Tiago Vignatti [Fri, 20 Jul 2012 20:09:55 +0000 (23:09 +0300)]
xwm: use last focused window for guessing transient parent
On X the global absolute coordinates are sent in ConfigureNotify and transient
windows are mapped exactly on that position. On Wayland we don't have the
concept of global coordinates, and that's a problem for transient surfaces
without transient_for set because they rely on such hint for setting their
positioning.
So this solution is a workaround. It guesses a parent based on the last
focused window to determine the relative position of the transient surface.
This put transient windows of Chrome browser back to work.
Signed-off-by: Tiago Vignatti <tiago.vignatti@intel.com>
Tiago Vignatti [Fri, 20 Jul 2012 20:09:54 +0000 (23:09 +0300)]
xwm: Fix transient positioning
Commit
eaee7841 took out the configure positioning of windows. This patch
brings it back and addresses also logic for resizing and sub-menus, that was
not covered on that commit.
Signed-off-by: Tiago Vignatti <tiago.vignatti@intel.com>
Tiago Vignatti [Fri, 20 Jul 2012 20:09:53 +0000 (23:09 +0300)]
xwm: Initialize window decoration always as !override
Signed-off-by: Tiago Vignatti <tiago.vignatti@intel.com>
Scott Moreau [Thu, 19 Jul 2012 06:49:06 +0000 (00:49 -0600)]
weston.ini: Move binding-modifier to shell section.
The variable has no effect as part of the screensaver section.
Philipp Brüschweiler [Wed, 11 Jul 2012 20:25:31 +0000 (22:25 +0200)]
Extract the text_model_manager interface from input_method
This is necessary because all clients need a way to create
text_models, but only one client at a time can be bound to
the input_method global (else we don't know to whom we are
supposed to send events).
Kristian Høgsberg [Sun, 22 Jul 2012 15:51:42 +0000 (11:51 -0400)]
text-backend.c: Whitespace fixes
Use a temp variable to break a couple of long lines, break a few long
lines.
Philipp Brüschweiler [Wed, 11 Jul 2012 20:25:30 +0000 (22:25 +0200)]
text-backend: minor fixes
* make sure deactivation is done consistently when a text_model
is destroyed
* zero out allocated objects
Philipp Brüschweiler [Wed, 11 Jul 2012 20:25:29 +0000 (22:25 +0200)]
editor: deactivate old text model before activating the new one
Kristian Høgsberg [Sun, 22 Jul 2012 15:33:14 +0000 (11:33 -0400)]
compositor: Destroy pending surface callbacks when destroying surface
Reported by Dima Ryazanov <dima@gmail.com> in
<
1341937691-26234-1-git-send-email-dima@gmail.com> but we also need to
destroy callbacks when the surface is destroyed normally.
Kristian Høgsberg [Fri, 20 Jul 2012 22:05:10 +0000 (18:05 -0400)]
configure.ac: Don't require vpx now that we don't build the webm encoder
Tiago Vignatti [Fri, 20 Jul 2012 16:41:13 +0000 (19:41 +0300)]
shell: Remove unused pointer_end_grab
shell_grab_end already does that for us.
Signed-off-by: Tiago Vignatti <tiago.vignatti@intel.com>
Tiago Vignatti [Fri, 20 Jul 2012 16:41:12 +0000 (19:41 +0300)]
xwm: Don't try to change frame cursor when there's no decoration
It was causing some artifacts on chrome.
Signed-off-by: Tiago Vignatti <tiago.vignatti@intel.com>
Kristian Høgsberg [Fri, 20 Jul 2012 19:30:36 +0000 (15:30 -0400)]
shell: Center fullscreen surface whenever new buffer is attached
Kristian Høgsberg [Fri, 20 Jul 2012 16:26:23 +0000 (12:26 -0400)]
Flesh out README a bit, link to building instructions
Kristian Høgsberg [Fri, 20 Jul 2012 15:32:51 +0000 (11:32 -0400)]
clients: Implement minimum size for toy toolkit clients
We default to setting the minimum size to the initial size. To set a
different minimum size than the initial size, set the minimum size first
then then initial size. Good enough for a toy toolkit.
https://bugs.freedesktop.org/show_bug.cgi?id=50263
Kristian Høgsberg [Thu, 19 Jul 2012 18:20:20 +0000 (14:20 -0400)]
compositor: Update to use new tokens for EGL_WL_bind_wayland_display
Kristian Høgsberg [Thu, 19 Jul 2012 18:13:06 +0000 (14:13 -0400)]
Never install weston setuid
We don't support this anymore. weston requires a setuid helper (such as
weston-launch) to run under kms, and should never run as root itself.
Disabe the setuid warning in configure.ac since we now only install the
minimal weston-launch as setuid.
Kristian Høgsberg [Thu, 19 Jul 2012 18:02:00 +0000 (14:02 -0400)]
shell: Set shell_interface.shell at initialization
Kristian Høgsberg [Wed, 18 Jul 2012 21:23:41 +0000 (17:23 -0400)]
compositor-drm: Remove cursor debug message
Kristian Høgsberg [Wed, 18 Jul 2012 16:09:58 +0000 (12:09 -0400)]
compositor-drm: Only update hw cursor contents when it changes
We avoid the memcpy, gbm_bo_write and the drmModeSetCursor ioctl per frame.
Kristian Høgsberg [Wed, 18 Jul 2012 16:08:37 +0000 (12:08 -0400)]
compositor-drm: Only call drmModeMoveCursor() if hw cursor position changed
Saves an ioctl per frame.
Kristian Høgsberg [Wed, 18 Jul 2012 16:02:51 +0000 (12:02 -0400)]
compositor: Don't record surface damage when we move or transform surfaces
surface->damage is for when the contents of the surface changes. Instead,
use weston_surface_damage_below() to repaint the damaged area. We avoid
unecessary uploading shm and hw cursor contents this way.
Kristian Høgsberg [Wed, 18 Jul 2012 15:39:05 +0000 (11:39 -0400)]
wcap: Just make wcap-decode dump YUV4MPEG2
Instead of having a custom fork of the vpxenc tool in weston, we can
just dump raw YUV data in the YUV4MPEG2 format and feed that into the
upstream vpxenc. This also works with theora_encoder and probably many
other encoders.
Kristian Høgsberg [Wed, 18 Jul 2012 13:54:04 +0000 (09:54 -0400)]
shell: Default to maximize/fullscreen on current output
Kristian Høgsberg [Tue, 17 Jul 2012 18:21:25 +0000 (14:21 -0400)]
compositor: Free the right EGLImage
We forgot to index the image array and freed image[0] multiple times
Kristian Høgsberg [Sat, 14 Jul 2012 05:24:58 +0000 (01:24 -0400)]
compositor-drm: Allocate hw cursor like other planes
We no longer reserve the hw cursor for just the pointer sprite. Any
surface that satisfies the requirements for the hw cursor can be used.
Of course that's more or less always the pointer sprite, but at least now
we don't hard-code the references to compositor->seat anymore.
Kristian Høgsberg [Sat, 14 Jul 2012 05:06:28 +0000 (01:06 -0400)]
compositor-drm: Combine drm_output_set_cursor and weston_output_set_cursor
weston_output_set_cursor was the left-over of a core compositor cursor
helper that we moved into compositor-drm.c.
Kristian Høgsberg [Sat, 14 Jul 2012 04:31:32 +0000 (00:31 -0400)]
compositor-drm: Assign primary framebuffer in assign_planes callout
Determining whether it was possible to use a surface buffer as the
primary framebuffer was always a bit of a hack. Just before rendering,
we look at the second top-most surface to see if it's the right size and
position and then use it if it is. If we have a hw cursor and a drm
plane on top, it's no longer the second top-most and it wouldn't be picked
even if it could work.
With this change, we inspect surfaces in drm_assign_planes and try to find
a candidate for the primary fb there. We track the overlap like for
overlays and if there is a surface that's not overlapped and fills the
output we pick it.
This commit also consolidate the logic to track damage as we move a surface
in and out of the primary plane, which was missing in the drm plane case.
Kristian Høgsberg [Fri, 13 Jul 2012 19:25:07 +0000 (15:25 -0400)]
compositor-drm: Support hardware scaling with drm planes
The kms planes support scaling, so try to detect transformations that
are just translations + scaling and program the kms plane accordingly.
In particular, this lets us fullscreen a yuv surfaces with the scale method
and have the compositor use a kms plane for scaling and color conversion.
Kristian Høgsberg [Mon, 16 Jul 2012 20:44:16 +0000 (16:44 -0400)]
compositor-drm: Follow gbm_bo_get_pitch rename to gbm_bo_get_stride
It was always a confusing mis-nomer, fix it while we can.
Kristian Høgsberg [Sat, 14 Jul 2012 02:54:43 +0000 (22:54 -0400)]
compositor-drm: Use gbm_bo_import instead of gbm_bo_create_from_egl_surface
This lets us create the gbm bo from the wl_buffer instead of the
EGLImage. With planar buffer, the EGLImage corresponds to a plane
and not the entire planar buffer.
More importantly, with some YUV formats (packed YUYV and similar) we use
several EGLImages to sample the different components, but KMS only
want one plane in that case.
Tiago Vignatti [Mon, 16 Jul 2012 16:15:37 +0000 (12:15 -0400)]
xwm: set the root window pointer to left_ptr
More pretty and also consistent with toytoolkit.
Signed-off-by: Tiago Vignatti <tiago.vignatti@intel.com>
Tiago Vignatti [Mon, 16 Jul 2012 16:09:19 +0000 (12:09 -0400)]
xwm: set appropriate cursors for the frame window
Signed-off-by: Tiago Vignatti <tiago.vignatti@intel.com>
Tiago Vignatti [Mon, 16 Jul 2012 16:02:08 +0000 (12:02 -0400)]
xwayland: add simple xcb-cursor hook based on libXcursor
It's in fact based on the core of libXcursor, which doesn't bring any Xlib
dependency.
Signed-off-by: Tiago Vignatti <tiago.vignatti@intel.com>
Kristian Høgsberg [Mon, 16 Jul 2012 15:57:08 +0000 (11:57 -0400)]
xwayland: Look up 32 depth rgba render format as well
Kristian Høgsberg [Mon, 16 Jul 2012 15:52:40 +0000 (11:52 -0400)]
xwayland: Use the right length for iterating through the formats array
Spotted by Tiago Vignatti, minimal fix to just use the right length.
Ander Conselvan de Oliveira [Mon, 16 Jul 2012 11:15:50 +0000 (14:15 +0300)]
desktop-shell: Don't get a shell surface for the grab surface
The grab surface does not depend on any of the shell surface
functionality, so don't allocated one.
Ander Conselvan de Oliveira [Mon, 16 Jul 2012 11:15:49 +0000 (14:15 +0300)]
window: Workaround a stuck frame callback on the cursor surface
It is possible that a client loses the focus between receiving a
pointer.enter event and sending a pointer.set_cursor request. In that
case, the cursor surface might not be mapped and the frame callback
requested on it will never trigger.
Work around this by trying to remap the cursor surface whenever there
is a frame callback and the serial for the enter event is higher than
the cursor serial.
Ander Conselvan de Oliveira [Mon, 16 Jul 2012 11:15:48 +0000 (14:15 +0300)]
shell: Don't ping the grab surface
If the grab surface happens to be unresponsive, the busy cursor grab
will be started and that will cause the grab surface to receive focus,
which in turn leads to it being pingged again. Break the cycle by not
sending pings to it. If the shell is unresponsive it won't be able to
set the busy cursor anyway.
Kristian Høgsberg [Thu, 12 Jul 2012 16:34:43 +0000 (12:34 -0400)]
compositor: Don't send wl_surface.enter/leave if we didn't find a resource
Kristian Høgsberg [Thu, 12 Jul 2012 16:32:31 +0000 (12:32 -0400)]
shell: Scale fullscreen surface without cropping them, center on output
Scott Moreau [Thu, 12 Jul 2012 02:57:15 +0000 (20:57 -0600)]
toytoolkit: Remove obsolete function window_set_custom().
Scott Moreau [Thu, 12 Jul 2012 02:57:14 +0000 (20:57 -0600)]
wscreensaver: Fix --demo mode.
Tiago Vignatti [Wed, 11 Jul 2012 21:46:10 +0000 (00:46 +0300)]
compositor: remove useless weston_compositor_repick call
compositor->focus was just unset, and weston_compositor_repick() has no
effect.
Signed-off-by: Tiago Vignatti <tiago.vignatti@intel.com>
Philipp Brüschweiler [Wed, 11 Jul 2012 13:19:23 +0000 (15:19 +0200)]
Remove dead code
window->title is always NULL inside window_create_internal
Kristian Høgsberg [Wed, 11 Jul 2012 19:45:59 +0000 (15:45 -0400)]
image-loader: Use pixman_image_t, not struct pixman_image_t
Rafal Mielniczuk [Wed, 11 Jul 2012 16:48:25 +0000 (18:48 +0200)]
image-loader: don't leak pixman_image bits data
Set the pixman_image destroy function to free bits data
Rafal Mielniczuk [Wed, 11 Jul 2012 16:48:12 +0000 (18:48 +0200)]
clipboard: add seat destroy signal handler
Gwenole Beauchesne [Fri, 20 Apr 2012 09:44:06 +0000 (11:44 +0200)]
compositor: handle YUV buffers.
Signed-off-by: Gwenole Beauchesne <gwenole.beauchesne@intel.com>
Kristian Høgsberg [Tue, 10 Jul 2012 19:51:13 +0000 (15:51 -0400)]
Update weston-egl-ext.h with EGL_WL_bind_wayland_display changes
Gwenole Beauchesne [Fri, 20 Apr 2012 09:22:16 +0000 (11:22 +0200)]
compositor: add YUV shaders.
Add shaders for NV12 (2 planes), YUV (3 planes) and YUYV (sampled as 2 planes)
Signed-off-by: Gwenole Beauchesne <gwenole.beauchesne@intel.com>
Gwenole Beauchesne [Fri, 20 Apr 2012 09:15:51 +0000 (11:15 +0200)]
compositor: factor out RGB shader.
Simplify RGB shader code and split off common code that could be reused.
This is preparatory work for YUV shaders.
Signed-off-by: Gwenole Beauchesne <gwenole.beauchesne@intel.com>
Gwenole Beauchesne [Fri, 20 Apr 2012 09:07:06 +0000 (11:07 +0200)]
compositor: prepare for multi-planar surfaces.
Make weston_surface::texture and ::surface an array, while keeping
[0] for RGB surfaces.
Signed-off-by: Gwenole Beauchesne <gwenole.beauchesne@intel.com>
Juan Zhao [Tue, 10 Jul 2012 05:24:09 +0000 (22:24 -0700)]
shell: Use the correct the member name when iterating panel_list
It does not get the correct panel height for now. The memeber of list
to read should be layer_link not link now.
Signed-off-by: Juan Zhao <juan.j.zhao@linux.intel.com>
Kristian Høgsberg [Tue, 10 Jul 2012 02:22:37 +0000 (22:22 -0400)]
window.c: Only set input region when we have a decoration
https://bugs.freedesktop.org/show_bug.cgi?id=50490
Kristian Høgsberg [Tue, 10 Jul 2012 02:17:03 +0000 (22:17 -0400)]
tests: Handle modifier event in test-client.c
We need the handle the newly added modifiers event.
https://bugs.freedesktop.org/show_bug.cgi?id=51567
Kristian Høgsberg [Tue, 10 Jul 2012 01:43:22 +0000 (21:43 -0400)]
Only position fullscreen surface in surface->configure callback
We end up calling shell_configure_fullscreen() from activate(), where
surface->buffer may be NULL. Instead, call it on initial map and
first buffer attach after surface type change. Both code path
come from the surface->configure callback where surface->buffer is know
to be non-NULL.
https://bugs.freedesktop.org/show_bug.cgi?id=51678
Jan Engelhardt [Mon, 9 Jul 2012 15:18:10 +0000 (17:18 +0200)]
wcap-snapshot: add return value
Distro build with rpmlint aborts due to missing exit status.
Daniel Stone [Mon, 9 Jul 2012 11:27:28 +0000 (12:27 +0100)]
toytoolkit: Remove redundant XKB free
We already unref these in fini_xkb().
Signed-off-by: Daniel Stone <daniel@fooishbar.org>