evdev: reuse code for device removal

Just like device_added, now the routines to close the compositor and vt switch
leave are using the same code to remove a device.

This patch also closes properly a mtdev device, bug spotted by Christopher
Michael.

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

compositor: move force_configure field to shell_surface

compositor: make surface_configure() a vfunc

This allows us to move the logic that calls shell->map() or
shell->configure() into shell while allowing it to be overriden for
surfaces that should not be handle by the shell, such as drag icons.

This patch adds a pointer function called configure to weston_surface,
moves the currsent surface_configure() code into shell and implements
a separate configure() for drag surfaces.

compositor: refactor surface_attach()

surface_attach() does two things: sets up a new buffer as the contents
of the given surface and then calls into shell so it can setup the
position of the surface and map it if necessary. However we do not want
the shell to meddle with some internal surfaces such as drag surfaces.

The intention of this refactoring is to make room for making the part
that calls into shell a virtual function that the compositor can
override for these internal surfaces.

This changes weston_buffer_attach() so it handle all the logic of tying
a buffer to a surface, including unmapping it when the buffer is NULL.
The shell map() vs. configure() logic is then split into a another
function: surface_configure(). In a later commit, this function will be
turned into a function pointer in struct weston_surface.

compositor: use new drag icon listener for setting up drag surfaces

That way we can get rid of the call to
weston_compositor_update_drag_surfaces() on shell->map().

compositor: make es->pitch consistent between shm and drm surfaces

For shm buffers, es->pitch is set using the stride of the buffer. If
the shell happened to set the surface width to something different than
the buffer width, the contents of the surface would be cropped on the
width during redraw. However, for non-shm surfaces, es->pitch was set to
the surface width. That caused the contents of the buffer to be scaled
on the width when the buffer was wider than the surface.

This makes the behavior on both cases the same: crop on the width and
scale on the height. (which is weird but consistent)

compositor: add weston_surface_to_global_float helper

compositor: add a weston_surface_is_mapped() helper

compositor: Use GL_BGRA_EXT for glReadPixels in screenshooter

This needs GL_EXT_read_format_bgra.

Simplify shm buffer handling

There was a lot of code here to do a lot of work we didn't need to do.
If we damage a surface with a shm buffer attached, all we need to do
is to re-upload the damaged region to the texture.  As for drm buffers,
we don't assume anything changes on attach and only update the
regions the client tells us to update in the damage request.

window.c: Do resize work from the repaint idle callback

This way we always make sure we handle any resizing before we start drawing.

compositor-drm: Check es->image, not es->buffer when looking for overlays

es->buffer can be NULL if the client destroyed the buffer, but es->image
will be non-NULL if and only if we have a buffer we might use with KMS.

config-parser: consolidate shell files into weston.ini only

The shell choice happens in the configuration file now.

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

window.c; Only redraw once per frame

Based on a patch from Martin Minarik <minarik11@student.fiit.stuba.sk> who
tracked down the excessive redraw problem.

Secure screenshooter protocol.

shell: fix matrix invertible bug in rotation handler

While activating and deactivating rotation mechanism without moving
the pointer, rotation matrix from rotate_grab object is not being
initialised and damage shell surface rotation matrix in
rotate_grab_button handler, making it invertible.

This patch initialise rotate matrix in rotate_binding
and moves surface position check to rotate_grab_motion handler.

Add surface opacity controls.

Switch zoom binding to <Super>+Scroll.

Modify bindings for axis event detection.

Hook up axis events.

drm: plane src coords are 16.16 fixed point

shell: correct position of a surface before rotating it.

How to reproduce: rotate the surface to something like 45 degrees,
resize it drastically, continue to rotate. The surface will jump
some space and the rotation point will not be in the center
of the surface.

Fix is to shift the surface position to match the rotation point

window.c: Fix the stretch scaling code

window.c: Increase drop shadow fluffiness

window.c: Tweak window decoration appearance

The old orange decorations were starting to look like clown shoes.

Revert "automake: Don't fail the build if we can't setuid."

This reverts commit e7ad5cdcd2eb8a307ad9cf2efdfde76f2e83aeb7.

If you ask for setuid install and that fails you didn't get what you
asked for and we shouldn't just silently carry on.  If installing weston
somewhere in your home directory and don't want the setuid bit set,
disable that at configure time.

evdev: fix input lag when processing input from output repaint

When the compositor is in a repaint cycle, input is processed only once
per frame. However, a call to evdev_input_device_data() would handle at
most 8 events at time. When there was more than 8 events pending for a
given frame, input lag would occur. This was most visible with multi
touch input.

This patch changes the evdev_input_device_data() so that it will handle
all the events available in the fd. In order to do that, the fd is put
in non-blocking mode, so that it is possible to loop on read and stop
on EAGAIN instead of blocking.

compositor: Fix pointer type mismatch warnings

weston_surface: initialize pitch

Initializing pitch to 1 to avoid xxx/pitch errors
This won't influence the valid texture coordinate calculation, because
 in that case buffer_attach will provide the correct value.

weston: don't hang in wait()

Pausing weston (with ^Z in the shell) and putting it in the background
made it hang.

weston: Don't ask for EGL depth buffer if not needed

This makes the compositor and demo clients work on the current nouveau
nvfx driver. Obviously does not fix any clients that actually want a
depth buffer, but this does allow more people to at least try wayland.

evdev: use mtdev for multitouch devices

mtdev library translates all multitouch based devices to the slotted evdev
protocol. It provides an uniform interface for Weston, which eases mt
implementation when dealing with a big variety of devices.

Weston on drm now directly depends on such library.

Signed-off-by: Tiago Vignatti <tiago.vignatti@intel.com>
Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>

compositor: properly restore keyboard_focus in notify_keyboard_focus()

Commit f992b2fc removed the saved keyboard focus logic to fix a crash
when the saved surface is destroyed. However, setting keyboard focus to
the first surface on the list ends up trying to set the focus to the
cursor surface most of the time. The end result is a NULL keyboard
focus.

This patch restores the saved keyboard focus logic and fixes the crash
mentioned above using a destroy listener.

compositor: allow multiple input_device_attach()'s with a nil buffer

Without this change, weston would crash whenever a nil buffer was
passed to input_device_attach() if the cursor sprite was not mapped.
While at it, change the unmapping code to use weston_surface_unmap().

compositor-wayland: use image loading code from shared/

Function weston_load_image() was deleted in f02a649a but the wayland
backend was not adapted to the new interface. This probably went
unoticed because the prototype for the missing function was not deleted
from compositor.h so the backend would compile without warnings.

cairo-util: load_cairo_surface returns NULL if loading the file fails.

Otherwise a non-existent file in the configuration crashes the desktop shell.

Signed-off-by: Ustun Ergenoglu <ego@ustun.fi>

compositor: use smoother range for backlight control

now it goes from 0 to 255.

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

tests: add backlight test

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

shell: Fix 2 switcher crashes.

In case of no regular window, switcher will crash in switch_next()
due to "next == NULL" and in switcher_destroy() due to
"switcher->current == NULL".

automake: Don't fail the build if we can't setuid.

The configure default is to setuid root the weston compositor.
However, if installing as non-root (say, to your prefix in homedir),
the install fails anyway, even if you didn't need setuid to run weston
in your configuration.

configure: Fix build on debian by avoiding bashisms in the test command.

Consolidate image loading code and move to shared/

Add an option parser

On one hand, getopt (in particular the -o suboption syntax) sucks on the
server side, and on the client side we would like to avoid the glib
dependency.  We can roll out own option parser and solve both problems
and save a few lines of code total.

x11: Move the xcb fd to the input loop

This way, we handle input just before repaint, similar to the drm backend.

Fix picking crashes

The surface list is only valid after repaint, so we need to do picking and
input delivery there.

drm: fix issue copying supported plane formats

Originally on the first 1/4th of the plane formats were copied, leading
to problems if AR24/XR24 appears later in the list.

clients/window: fix build issues w/ gles

compositor-drm: use udev_device_get_property_value() helper

Udev provides a convenient helper. Use it instead of working with the
property-list directly.

Signed-off-by: David Herrmann <dh.herrmann@googlemail.com>

compositor-drm: ignore hotplug-events from other devices

If we have multiple video devices on the system (card0, card1), we should
ignore hotplug events for cards that we do not use. This avoids calling
update_outputs() if the event was not generated by our device so we avoid
refreshing the DRM information if it didn't change.

Signed-off-by: David Herrmann <dh.herrmann@googlemail.com>

drm: Fix drmModeRes leak on error paths

When creating outputs in the drm compositor, if allocating crtcs
fails, then free the drm resources. Also, if the base output list is
empty, free drm resources

compositor: Fix compilation without xserver launcher

shell: don't assign output for surface of type none

If map is called with a surface of type none it will call
weston_surface_assign_output, even though the surface will
not be mapped.

This change was lost with the changes for using wl_layers.

shell: Fix segment fault when clicking to activate a NULL pointer_focus

This can happen for instance running a toytoolkit application, launching
the popup menu and then clicking on a surface that does not belong to
the client that create the popup surface.

shell: Fix not destroy black surface when fullscreen surface exit abnomally.

When a client with fullscreen surface displayed was aborted by Ctrl-C, the
black surface still be there. Destroy the black surface in
destroy_shell_surface().

shell: Put popups and transients on top of their parent

In other words, don't map the panel menu under the panel.

window: Compute frame boundary for all windows except fullscreen

We were only computing it for toplevel type windows, which broke
the unlock dialog size allocation and repainting.

desktop-shell: Simplify unlock dialog redraw function

drm: Disable sprites if they're broken

Current sprite implementation has a color format bug.  Disable sprites
if they don't work and avoid spamming stderr and syslog.

compositor: Only process input once per frame

When we're repainting, there's no point in polling for input events.
We just read input events once before each repaint and send out events
as needed.  The input events come with an accurate timestamp, so this
doesn't affect the timing information and client should always look at
the event timestamps if they're trying to determine pointer motion
speed or double click speed.  If we go idle (stop repainting) we add the
input devices back into the primary main loop and wait for the next event.

This avoids waking up the compositor separately (one or more times per
frame) to handle input events.  We also avoid updating cursor position
and other compositor state after the client has rendered its new
frame, reducing lag between what the client renders and the pointer
position.

compositor: Move input devices to their own event loop

First step towards only handling input at repaint time.

compositor: Swallow binding key for bindings that don't set up a grab

This prevents passing the key press and release to the keyboard focus client.

window: Only set opaque and input regions if they change

Setting these regions damages the entire window or causes a repick, which
we don't want to trigger if we don't need to.

Use wl_array_for_each() where possible

compositor-drm: fix libbacklight build

Bad inclusion of DRM headers, missing some system headers.

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

Use typesafe server-side stubs

compositor: Add a repaint debugging mechanism

Toggle on/off with Super+SPACE.

Merge branch 'layers'

I was integrating patches on the 'layers' branch and had a fixed version of
the fullscreen patch there.  My master branch had the fullscreen patch with
the resize bug, and when I pushed I meant to push the layers branch to master.

Conflicts:
src/shell.c

shell: Don't launch a new screensaver if the old one is still running

compositor: Damage sprite surface on attach

Normally the client is responsible for damaging the surface after attach,
but the sprite surface is an internal surface.

compositor: Post keyboard leave before destroying surface

We may want to move this into wayland-server.

compositor: split drag surface update into smaller functions

compositor: fix crash when a drag surface is destroyed during the drag

This can happen for instance if the client that started the drag
crashes. Weston would crash because of the invalid surface pointed by
device->drag_surface.

Fix this by reseting the drag surface to nil on a destroy listener.

compositor: Organize surface stack using new struct weston_layer

The surface data structure is now a list of list of surfaces.  The core
compositor defines the fade and cursor layer, and it's up to the shell to
provide more layers for the various surface types it implements.

compositor: Just damage compositor->damage in weston_output_damage()

compositor: Implement damage_below by using the new compositor->damage

We can now clip the surface bounding box against the previous frame
opaque clip, and then just union the result (visible damage) into
compositor->damage immediately.

compositor: Store opaque clip for previous frame in weston_surface

shell: Add implementation of fullscreen.

Undo fullscreen in shell_unset_fullscreen(), do all the stacking order
in shell_stack_fullscreen(), and configure black surface, method in
shell_configure_fullscreen().

Signed-off-by: Alex Wu <zhiwen.wu@linux.intel.com>
Signed-off-by: Juan Zhao <juan.j.zhao@linux.intel.com>

shell: Add implementation of fullscreen.

Undo fullscreen in shell_unset_fullscreen(), do all the stacking order
in shell_stack_fullscreen(), and configure black surface, method in
shell_configure_fullscreen().

Signed-off-by: Alex Wu <zhiwen.wu@linux.intel.com>
Signed-off-by: Juan Zhao <juan.j.zhao@linux.intel.com>

window.c: Drop support for rgb contexts

wscreensaver: Use the argb config, rgb is not compatible with window.c

Fixes the config/surface mismatch, spotted by Scott Moreau.

compositor-drm: Just pull in libbacklight source for now

libbacklight is 300 lines of code in one .c file, and we're relying on
udev changes that aren't yet upstream.  For now, let's just keep a
copy in weston and if the Xorg DDX drivers start using libbacklight and
it becomes more widely available, we'll make it an external dependency.

compositor: add dpms and backlight support

DPMS kicks in only when wscreensaver is launched, in the moment that shell
call lock() for the second time. Backlight control internals are managed by
libbacklight:

http://cgit.freedesktop.org/~vignatti/libbacklight/

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

zoom: Don't damage_all here

Setting geometry.dirty = 1 is enough.

compositor: Insert dnd icon surface below device sprite surface

compositor: Set an empty input region for the fade surface

Otherwise we don't set the cursor until the fade is done.

window: Set opaque region after child widget adjusts size

weston: Change path for default terminal launcher to ${prefix}/bin

If you don't have anything at ~/.config/weston-desktop-shell.ini and
have weston installed somewhere other than /usr, then this patch will
help.

Cheers,

Signed-off-by: Rodney Lorrimar <rodney@rodney.id.au>

clients: Use wl_display_disconnect()

compositor: make the input region of drag surfaces empty

With the input region changes, drag surfaces were being picked and
receiving events which led to problems on the client side.

shell: Mark geometry dirty when we change surface alpha

We need to recompute the opaque region when the alpha changes.

compositor: Dont set the opaque region if we have an overall surface alpha

terminal: Don't queue a resize after setting fullscreen

We end up doing an attach at the non-fullscreen size before resizing to
fullscreen, causing the terminal to jump to the center for a frame before we
render the fullscreen image.

compositor: Export functions and add weston_suface::force_configure for fullscreen

This will facilitate the implementation of fullscreen.

Signed-off-by: Alex Wu <zhiwen.wu@linux.intel.com>

Restructure output zoom key handling.

This effectively eats the keybinding events, as we don't want them sent
to clients.

drm: Remove redundant cursor region math

We already check this in weston_output_set_cursor().

drm: Remove redundant weston_surface_update_transform() call

compositor: Remove surface overlapped attribute

We need to compute this as we assign surfaces to overlays, so that
if we assign a surface to an overlay, it isn't considered overlapping.

Move weston_output_set_cursor into compositor-drm.c

First step towards moving hw cursor handling into compositor-drm.c a its
general overlay handling.

compositor-drm: Destroy sprites on shutdown, turn off on vt switch

compositor: Remove unused compositor vfunc create_cursor_image