Convert wl_input_device to wl_seat (and friends)

wl_input_device has been both renamed and split.  wl_seat is now a
virtual object representing a group of logically related input devices
with related focus.

It now only generates one event: to let clients know that it has new
capabilities.  It takes requests which hand back objects for the
wl_pointer, wl_keyboard and wl_touch interfaces it exposes which all
provide the old input interface, just under different names.

This commit tracks these changes in weston and the clients, as well as
similar renames (e.g. weston_input_device -> weston_seat).  Some other
changes were necessary, e.g. renaming the name for the visible mouse
sprite from 'pointer' to 'cursor' so as to not conflict.

For simplicity, every seat is always exposed with all three interfaces,
although this will change as time goes on.

Signed-off-by: Daniel Stone <daniel@fooishbar.org>

Convert wl_fixed_t to int in weston_output_update_zoom().

This way, we don't have to use wl_fixed_to_int() for each
call to weston_output_update_zoom(). It accepts wl_fixed_t
types and converts internally.

xwm: memset struct to 0

We rely on focus_window being initialize to NULL.

shell: Remove lock/unlock listeners on shutdown

xwm: Paint different decoration for active surface

src: Don't add COMPOSITOR_CFLAGS to AM_CPPFLAGS

It's a CFLAGS variable, not CPPFLAGS and we don't want to add it to every
target.  In particular we don't want anything leaking into weston-launch.

configure: Add pixman-1 to the compositor required packages

We used to get it through the IMAGE_LIBS/CFLAGS but we use it directly
in the compositor to add it as a direct dependency.

Rename libconfig-parser to libshared

Still a bad name.

Move cairo-util to shared/

window: Move frame rendering to cairo-util

window: Move theme rendering code to cairo-util

window: Move all frame theme related state into new struct theme

xwm: Don't repaint after destroying window

window: Compute title vertical position based on font extents

We just fudged it before.

window: Store the outputs that the window is on

Using the surface enter/leave events track which outputs the window is on and
store those in a "window_output_list" on the window.

To create this list we define a struct window_output that is the list
relationship between the window and the output.

compositor: Fix cursor positioning right after input_device.attach

The type of fields x and y in wl_input_device was changed to wl_fixed_t
but input_device_attach() was still using it as if it were integer.
This bug caused the pointer sprite to be configured in the wrong place
on the screen (usually outside the visible area) but it would soon be
corrected in notify_motion() making it hard to notice and usually only
causing a quick flicker.

tests: Fix event-test

notify_motion() now receives coordinates in wl_fixed_t but the test was
still passing integers.

xwm: Update window title when window property change

xwm: Put a title bar on it

xwm: Reparent X windows, paint a red decoration frame

It's red, and that's about it.

compositor: Only call weston_surface_update_transform from repaint

compositor: Remove a few leftover screenshooter declarations

toytoolkit: Recalculate input region if client has modified it's size.

If a client changes it's width/height values in it's widget resize handler,
the input region will be wrong because of the assumptions toytoolkit makes
in frame_resize_handler(). So far, gears is the only client that does this.

window: fix missed xkb API adaptation

Oddly enough, this failed to build on Android, but not otherwise.

Signed-off-by: Pekka Paalanen <ppaalanen@gmail.com>

simple-shm: Handle ping event.

Since simple-shm does not use toytoolkit, it needs to respond to ping events so it is not deemed unresponsive.

clients: Remove superfluous #includes

In particular window.c and many clients were including glib.h without
using it and without the right cflags.

Update to new libxkbcommon API

We no longer depend on xproto, we use xkbcommon keycodes now. Yay!

resizor: More resizing

Interacts better with interactive resizing, left/right keys now
changes window width.

simple-egl: Add option -o to make surface opaque

clients: Add motion event trails to clickdot

Useful for testing pointer device.

Signed-off-by: Jonas Ådahl <jadahl@gmail.com>

evdev: Use wl_fixed_t for relative motions

Signed-off-by: Jonas Ådahl <jadahl@gmail.com>

window.c: frame_button: Maximize, minimize, close, icon buttons in window frame

window.c: Just use float instead of GLfloat, remove GLES2.h include

Fix a few -pedantic warnings

We're not enabling -pedantic by default, but a quick test brought up a few
issues that we should fix.

compositor: Report refresh rates in mHz

We compute the higher resolution from the mode timing parameters.

compositor: Simplify clip_pointer_motion() and make it handle dead areas

We were testing if the pointer were outside any output and doing a lot of
work to compute the bounding box of all output and then clip against that.
Just clip against previous (valid) output and don't bother with the
bounding box.

compositor: When clipping pointer motions, don't loose precision

Signed-off-by: Jonas Ådahl <jadahl@gmail.com>

compositor: Use wl_fixed_t for incoming input events

This changes notify_motion, notify_pointer_focus and notify_touch to take
wl_fixed_t types for input coordinates.

compositor: Move pointer motion clipping to its own function

evdev: Convert wl_fixed_t to int before using internally

Signed-off-by: Jonas Ådahl <jadahl@gmail.com>

window: Don't crash if a cursor image failed to load.

Signed-off-by: Dima Ryazanov <dima@gmail.com>

window: fix check of cursor image load result

Load an XKB keymap and state in the compositor

Not used yet, but will be in future commits.

Signed-off-by: Daniel Stone <daniel@fooishbar.org>

Link compositors to xkbcommon as well

Signed-off-by: Daniel Stone <daniel@fooishbar.org>

Add xkb_names member plus config + X11 support

Add an xkb_names member to the base compositor info which contains the
RMLVO to use when building an XKB keymap.  Add support for filling this
from the config file or from the underlying X11 server, with the usual
defaults.

Signed-off-by: Daniel Stone <daniel@fooishbar.org>

Port Wayland clients to new xkbcommon API

A little different from Daniels initial patch.  We look up the common
modifiers at xkb init time and convert the xkb serialized modifier mask
to our own modifier bitmask.

Convert wire input co-ordinates to fixed-point

To add greater precision when working with transformed surfaces and/or
high-resolution input devices.

Signed-off-by: Daniel Stone <daniel@fooishbar.org>

Convert internal input co-ordinates to GLfloat

Change all client motion handlers to take GLfloat for co-ordinates,
rather than int32_t.

Signed-off-by: Daniel Stone <daniel@fooishbar.org>

Add fixed-point versions of weston_surface_{to, from}_global

To be used by input code, paralleling the existing integer versions.

Enlarge the surface_{to,from}_global_float input types to GLfloat to
avoid losing precision.

Signed-off-by: Daniel Stone <daniel@fooishbar.org>

build: Tighten mtdev dependency to reflect requirement

The mtdev_new_open symbol was added in 1.1.0.

shell: use own struct for transient surfaces

No functional changes; it's only opening space for modifications coming along
on the next commits.

Signed-off-by: Tiago Vignatti <tiago.vignatti@intel.com>

shell: fix uninitialized variable warning

and shut-up valgrind:
 Conditional jump or move depends on uninitialised value(s)
    at 0xB5AFB05: shell_surface_configure (shell.c:2162)
    by 0x407B0C: surface_attach (compositor.c:1225)
    by 0x621FA13: ffi_call_unix64
    by 0x621F434: ffi_call
    by 0x4E3D3F5: wl_closure_invoke (connection.c:758)
    by 0x4E3786C: wl_client_connection_data (wayland-server.c:255)
    by 0x4E3AA19: wl_event_source_fd_dispatch (event-loop.c:78)
    by 0x4E3B533: wl_event_loop_dispatch (event-loop.c:460)
    by 0x4E38D2C: wl_display_run (wayland-server.c:907)
    by 0x40B5DD: main (compositor.c:2748)

Signed-off-by: Tiago Vignatti <tiago.vignatti@intel.com>

window: use only one shm pool for all the cursor images

Cursor images are fairly small and having one pool for each image adds
a lot of unnecessary overhead. Instead, create one large pool and
allocated all cursor images from that.

In order to do that, however, the code that creates shm surface needed
some refactoring. This patch adds a new struct shm_pool that is used
by the cursor and also changes struct window to use it.

Rename 'state' in axis (or ambiguous) bindings to 'value'

Since it's usually an axis value rather than a boolean, be slightly more
explicit about it.

Signed-off-by: Daniel Stone <daniel@fooishbar.org>

Change button from int to uint32_t

Since you can't really have a negative button number.

Signed-off-by: Daniel Stone <daniel@fooishbar.org>

Change key/button grab bindings to take unsigned state

'state' here meaning 'is it up or down?', obviously.

Signed-off-by: Daniel Stone <daniel@fooishbar.org>

test-client: Include poll.h to silence warning

Signed-off-by: Daniel Stone <daniel@fooishbar.org>

drm: Make KMS hw cursors work again

window: use libXcursor for loading pointer images

simple-egl: add option for running fullscreen

compositor-drm: alow scanning out of buffers with XRGB8888 format

compositor-drm: refactor to avoid unnecessary allocation of KMS FBs

Currently, the drm backend will create and destroy a KMS FB for each
frame. However, the bos for a gbm surface are reused (at least with
mesa) so we can store the fb_id on it and destroy it only on the bo's
destroy callback.

To use the same path for scanning out client buffers, some refactor
was needed. Previously, the bo for the client buffer was destroyed
early so that gbm_surface_release_buffer() would not be called with
it, since at the page flip handler output->scanout_buffer can be
NULL even if the current frame is a client buffer.

This was solved by adding a drm_fb structure that holds a gbm_bo,
an fb_id, and information about the fb coming from a client buffer
or not. A drm_fb is created in such a way that it is destroyed
whenever the bo it references is destroyed. The fields current_*
and next_* in drm_output are changed into only two pointers to
drm_fb's.

window: Set window title and class

shell: Handle set_title and set_class

Use the dummy pbuffer we create to make current

compositor-drm: call EGL cleanup functions on exit

shell: Always configure when surface type changes

We can drop the force_configure hack.

shell: Don't move or reconfigure surfaces until we get the new buffer

Going from fullscreen to toplevel will restore the surface position
immediately.  This will move the fullscreen surface to where the toplevel
surface was before, which will flicker for a frame of two before the
resized, non-fullscreen buffer is attached.

Instead, only change the surface geometry when we get the new buffer.

window: Only set toplevel window type if nothing else is set

shell: Fix not remove rotation for fullscreen surf

Remove rotation when entering fullscreen and restore rotation when exiting.

https://bugs.freedesktop.org/show_bug.cgi?id=48906

shell: allocate enough memory for shsurf->ping_timer

Merge remote-tracking branch 'pq/simple-fixes'

tests: Add event-test, which tests for a few incoming events

We check that we get surface.enter_output and move the pointer into
the window and make sure we get input_device.pointer_enter with
the right coordinates.

There's a lot of code for a very simple test here, so we need to
figure out how to reuse most of the event handling and such.  It's also
not clear that a custom, text based protocol is practical here, we might
just use a wayland extension after all.

tests: Generalize test client and add helpers for launching it

tests: Add test case for basic client lifecycle

tests: Add an initial weston integration test

The idea here is to make weston load test cases as a module and then
run test cases from within weston.

compositor: Initialize xserver variable

This variable is used unitialized if --xserver is not passed to weston.

Signed-off-by: Pekka Paalanen <ppaalanen@gmail.com>

fix build for --disable-clients --enable-simple-clients

Simple clients were relying on AM_CFLAGS and AM_CPPFLAGS set for
toytoolkit clients. With toytoolkit clients disabled, the build fails
with missing wayland-client.h.

Move AM_CFLAGS and AM_CPPFLAGS outside of conditional sections, since
they are meant to be global settings.

Let simple clients override AM_CPPFLAGS with their own
SIMPLE_CLIENT_CFLAGS, which the configure script already sets up for us,
but was unused until now.

Signed-off-by: Pekka Paalanen <ppaalanen@gmail.com>

simple-shm: no need for wayland-egl.h

Signed-off-by: Pekka Paalanen <ppaalanen@gmail.com>

simple-shm: handle shm buffer failure

Do not segfault, if creating the shm buffer fails.

Signed-off-by: Pekka Paalanen <ppaalanen@gmail.com>

compositor: report .so path when load_module() fails

I want to know which module is failing to load with unresolved symbols.

Signed-off-by: Pekka Paalanen <ppaalanen@gmail.com>

simple-shm: render meaningful diagnostics

Previously, simple-shm was rendering an image that looked like stride
gone wrong somewhere, and was quite confusing if you did not know it was
supposed to look like that.

Replace the drawing code. Two circles, inner and outer, now delimit
three co-centric areas. The outmost area from surface borders to outer
circle contains horizontal gradients that move (animate) to the left.
The area between outer and inner circles contains vertical gradients
that move upwards. The center disc has circular gradients moving towards
the center.

The circles are not ellipses.

Diagnostics:
The X-channel is manipulated so, that if a compositor takes the XRGB
image, and uses the X channel as alpha instead of ignoring it, the whole
image will be crossed out by two lines that either quickly saturate to
white or show through with additive blending. Does not work on black
background.

Signed-off-by: Pekka Paalanen <ppaalanen@gmail.com>

gears: Allow model rotation with pointer

shell-animation: add conf option in shell

Now we can choose zoom or fade in weston.ini
to fulfill and customize the user experence easier.

Signed-off-by: Juan Zhao <juan.j.zhao@intel.com>

animation: add fading-in animation

To ful-fill user experience, add the fading-in animation
when mapping a window.
v2: update that westom_surface_damage to repaint
    remove that transform part in fade struct

Signed-off-by: Juan Zhao <juan.j.zhao@intel.com>

Add COPYING

xserver: Move window property reading to set_window_id handler

This will work for override-redirect windows too.

xserver: Read window properties in map request handler

shell: Don't ping internal shell surfaces

This is a hack for now to get xserver surfaces working again.

compositor: introduce internal shell_interface

We're able now to create shell_surfaces inside Weston. This makes possible the
glue needed between shell and xserver-launcher.

On the desktop-shell, it was split the protocol part from shell_surface
specific functions to make this possible.

Signed-off-by: Tiago Vignatti <tiago.vignatti@intel.com>

compositor: Set pointer surface input region to empty on attach

We set up an emtpy region for the initial NULL-cursor, and then going
from no buffer to a 32x32 buffer didn't trigger the undeffing of the input
region.  So when something sets a cursor of a different size, the cursor
surface input region gets initialized and starts swallowing events.

shell: Move binding-modifier to shell section

shell: modifier bindings configurable

This patch makes the main modifier configurable.  We used to hardcode super
(windows key) for most bindings, but now that can be changed.

The change affects two key bindings: rotation moves to mod+right click
and backlight moves to from ctrl+f9/f10 to mod+f9/f10.

Signed-off-by: Tiago Vignatti <tiago.vignatti@intel.com>

screenshooter: Don't assume output offsets start at zero

screenshooter: Move buffer assembly to client side

screenshooter: Grab pixel data directly before buffer swap

Send surface enter/leave events

These new protocol events allow us to tell which outputs a surface is on, and
potentially update where we allocate our buffers from.

Signed-off-by: Casey Dahlin <cdahlin@redhat.com>

Add an id field to outputs

All outputs now have a unique integer ID, allocated from a bitfield pool in the
compositor.

Signed-off-by: Casey Dahlin <cdahlin@redhat.com>

Change find_resource_for_surface to find_resource_for_client

This function has generic applications and should be made generic.

Signed-off-by: Casey Dahlin <cdahlin@redhat.com>

shell: Cleanup ping_timer code.

 - Added ping_timer_destroy() to simplify cleanup.
 - Changed timeout and fade step to more realistic values.
 - Renamed ping_timeout_fade_frame() to unresponsive_fade_frame().