window.c: Add fullscreen handler to keep fullscreen state consistent

dnd: Clip flowers to window content area

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

Update src/weston-egl-ext.h

evdev: Release weston_seat with underlying evdev device

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

Fix memory leak on compositor exit

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

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>

wcap: Fix typo in usage output.

wcap: Clarify help message, don't dump yuv4mpeg2 to terminal

desktop-shell: Declare grab_cursor as enum cursor_type.

wcap: Add GCC_CFLAGS and fix more compiler warnings

wcap: Declare variable with same sign as convert_to_yv12() expects.

Handle new transform argument in wl_output.geometry event

xwm: Make override-redirect windows opaque

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>

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>

xwm: Initialize window decoration always as !override

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

weston.ini: Move binding-modifier to shell section.

The variable has no effect as part of the screensaver section.

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).

text-backend.c: Whitespace fixes

Use a temp variable to break a couple of long lines, break a few long
lines.

text-backend: minor fixes

* make sure deactivation is done consistently when a text_model
  is destroyed
* zero out allocated objects

editor: deactivate old text model before activating the new one

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.

configure.ac: Don't require vpx now that we don't build the webm encoder

shell: Remove unused pointer_end_grab

shell_grab_end already does that for us.

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

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>

shell: Center fullscreen surface whenever new buffer is attached

Flesh out README a bit, link to building instructions

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

compositor: Update to use new tokens for EGL_WL_bind_wayland_display

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.

shell: Set shell_interface.shell at initialization

compositor-drm: Remove cursor debug message

compositor-drm: Only update hw cursor contents when it changes

We avoid the memcpy, gbm_bo_write and the drmModeSetCursor ioctl per frame.

compositor-drm: Only call drmModeMoveCursor() if hw cursor position changed

Saves an ioctl per frame.

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.

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.

shell: Default to maximize/fullscreen on current output

compositor: Free the right EGLImage

We forgot to index the image array and freed image[0] multiple times

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.

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.

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.

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.

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.

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.

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>

xwm: set appropriate cursors for the frame window

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

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>

xwayland: Look up 32 depth rgba render format as well

xwayland: Use the right length for iterating through the formats array

Spotted by Tiago Vignatti, minimal fix to just use the right length.

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.

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.

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.

compositor: Don't send wl_surface.enter/leave if we didn't find a resource

shell: Scale fullscreen surface without cropping them, center on output

toytoolkit: Remove obsolete function window_set_custom().

wscreensaver: Fix --demo mode.

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>

Remove dead code

window->title is always NULL inside window_create_internal

image-loader: Use pixman_image_t, not struct pixman_image_t

image-loader: don't leak pixman_image bits data

Set the pixman_image destroy function to free bits data

clipboard: add seat destroy signal handler

compositor: handle YUV buffers.

Signed-off-by: Gwenole Beauchesne <gwenole.beauchesne@intel.com>

Update weston-egl-ext.h with EGL_WL_bind_wayland_display changes

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>

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>

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>

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>

window.c: Only set input region when we have a decoration

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

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

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

wcap-snapshot: add return value

Distro build with rpmlint aborts due to missing exit status.

toytoolkit: Remove redundant XKB free

We already unref these in fini_xkb().

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

Add cscope.out to .gitignore

clients: Free resources on exit in eventdemo

clients: Silence 2x gcc 4.6.3 "warn_unused_result" compiler warnings

window.c:1173:6: warning: ignoring return value of ‘read’, declared with attribute warn_unused_result [-Wunused-result]
desktop-shell.c:305:6: warning: ignoring return value of ‘read’, declared with attribute warn_unused_result [-Wunused-result]

wcap: Plug memory leak in wcap_decoder_create()

xwayland: Don't leak fd while reading lock file in src/xwayland/launcher.c

compositor-x11: Add error handling in x11_compositor_create()

compositor-wayland: Add error handling in wayland_compositor_create()

compositor-android: Add proper error handling in android_compositor_create()

shared: Don't leak file descriptor in shared/image-loader.c

clients: Don't leak "menu" on error path in clients/window.c

clients: Add clients/weston-screensaver to clients/.gitignore

compositor: Always empty the input region of the pointer surface

Otherwise a surface.attach request might cause the input region to be
reset to the default value (the entire surface) causing it to receive
focus.

Tiago ran into this problem with xwayland.

weston: Clip window title text to the frame

Fixes an issue with text overflowing the available frame space.

compositor-x11: Set make and model so we don't send NULL strings

shell: Use desired output when sending configure events when maximised

With shell_surface.set_maximised the caller can provide an output to maximise
to or the default output will be used. With the corresponding configure we
should use the dimensions from the chosen output not the output the surface
was currently on.

Similarly when calculating the position for the window in the map() function
we should use the desired output again.

There is no need to assign shsurf->output to es->output since that happens in
the map() function later.

compositor: Correctly calculate changed sub image rectangle

When calling glTexSubImage2D for sub image updates for SHM surfaces the
changed rectangle was being wrongly calculated. This resulted in interesting
redraw artefacts for clients using SHM.

terminal: Extend selection to end of line when selecting past last char

terminal: Insert newlines when copying multiple lines

clipboard: initialise clipboard memory after creation

In case we don't, weston segfault in clipboard_set_selection
on:
if (clipboard->source)
clipboard_source_unref(clipboard->source);

terminal: Handle selection for pointer positions outside widget correctly

That is, don't crash and select entire first/last line when the pointer
is above or below widget.

window.c: Don't allocate proxy manually

With the wayland change to automatically allocate the client side proxy
manually, we can now drop the code (and the FIXME) that did that and just
receive the proxy from the callback arguments.

terminal: Add a terminal enter handler for setting I-beam cursor

Now that the mod+click shortcuts change the cursor, this was suddenly obvious.

shell: Don't move fullscreen surfaces

shell: Show approprite cursors during move and resize grabs

This patch expands the busy cursor mechanism so that it is possible for
the desktop-shell client to show the appropriate cursor during grabs.

compositor: Always allow owner of pointer surface to change the hotspot

Make pointer_set_cursor() succeed if a client that doesn't have pointer
focus but is the owner of the current pointer surface calls it.

compositor: Move repaint debug binding to mod-shift-space

Alt-space or Super-space are too likely to conflict with application
bindings.

compositor: Update xkb state with key releases on focus out

This happens when vt-switching away from the compositor (drm) or
giving keyboard focus to a different X window.  Release the modifiers
so we don't get stuck modifiers.  We'll update with the new keys down
when we come back.

compositor-drm: Improve initial mode picking

We now pick the driver preferred mode for our initial mode.  If no preferred
mode is available we default to the current mode.  We also have a command
line option now to keep the current mode if it differs from the preferred.

This commit also drops the built-in mode and insteade uses the current mode
if the connector doesn't report any modes.