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.

terminal: Scroll selection with terminal contents

terminal: Make font size configurable

terminal: Implement word and line based selection

Double-click to select by word, triple click to select by line.

shell: Take a wl_surface as parent for transient and popup windows

shell: Make input_panel just a regular wl_surface

shell: Make screensaver just a wl_surface

shell: Make the lock surface just a plain wl_surface too

shell: Make panel just a wl_surface too

shell: Don't make desktop_shell background a wl_shell_surface

We don't gain anything from taking a wl_shell_surface in
desktop_surface.set_background, except making wl_shell_surface
gratuitously dependent on wl_shell.  In shell.c we can also handle
backgrounds in their own background_configure function which simplifies
the mapping and placement logic.

terminal: Tab should not output spaces, just move cursor

Emacs uses tab and backspace to move the cursor as well as the regular
cursor movement escape codes.  When it's less bytes than the escape code,
emacs will use a tab or tab + backspace to move the cursor forward.  The
effect is that as you're  moving around in the buffer, emacs will
(seemingly) randomly insert spaces and overwrite what's in the terminal.
Making tab just move the cursor as it should fixes this.

compositor-drm: Don't leak FBs when the buffer of a sprite is detroyed

As a side effect, the overlay is disabled when the buffer is destroyed.

compositor-drm: Remove destroy listeners when disabling planes

When an unused plane is disabled, the destroy listener for a previously
used buffer needs to be removed. This fixes a crash when an overlay
would be reenabled using the same buffer as before, causing the destroy
listener to be inserted twice.

compositor-drm: Wait for vblank events before starting next frame

Besides the fact of the frame not being done, assigning planes before
the vblank handler is called will make the state inconsistent, leading
to a crash.

compositor-drm: Check surface's outputs when assigning overlays

Check if the output passed to drm_output_prepare_overlay_surface() does
actually contain the surface and fail if it doesn't. Also fail if a
surface spans multiple outputs since clearing the damage will cause the
portion that is not in an overlay to not be updated at all.

simple-egl: Make window moveable

configure.ac: Bump version to 0.94.90

shell: Fix inverted test for locked in workspace bindings

Makefile.am: Don't list wcap sub directory twice in DIST_SUBDIRS

tests: Add weston-test to EXTRA_DIST

Move text-cursor-position.c into zoom.c

wcap: Add vpx_config.h to Makefile.am

xwm: Add xwayland.h to Makefile.am

Install screensaver by default

shell: Hide workspace layer when locking screen

compositor: Send ping to keyboard focus surface for key press

We were sending to the pointer focus surface.

terminal: Ignore more modifier keys

Don't want these to leak into the terminal.

terminal: Only hide cursor for keypresses that generate data for the process

compositor-drm: Add proper error handling to init

This way, if initialisation fails (say, udev or Mesa are broken, or we
couldn't find any devices), we'll at least take you back to where you
were, rather than leave you at a totally broken VT you can't escape
from.

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

Split weston_compositor_init into base and _gl

weston_compositor_init is always called late because most
implementations can't initialise GL until fairly late in the game.
Split it into a base version with the same name, followed by
weston_compositor_init_gl which can be called later on.

This simplifies compositor-wayland, which no longer needs a separate
global handler just for wl_seat.

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

compositor-wayland: Synchronise keyboard state with parent

Use the wl_keyboard::modifiers events our parent helpfully sends us to
make sure our views of the keyboard state are always identical, rather
than relying on key press events to do the right thing.

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

compositor-x11: Attempt to track state without XKB

This isn't very pleasant, but it's pretty much the best we can do in the
absence of either XCB XKB support, or XCB XI2 bindings (argh!).

We get a state mask with most significant X11 events (which inexplicably
includes EnterNotify but not FocusIn), but unfortunately it's only a
single flat set of effective modifiers rather than the more granular
sets we want, so we still update the state with every key, but then also
use the core X11 state as a mask to make sure we don't get any stuck
modifiers.

Ugh.

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

compositor-x11: Use XKB StateNotify to synchronise state

Make sure that we always have the exact same view of the keyboard state
as the host server by using XKB StateNotify events to update our state
exactly rather than relying on key events.  In particular, this fixes
key state during grabs, where we either miss modifiers completely or get
them stuck permanently, depending on the nature of the grab and the
implementation of the X window manager/compositor.

The downside, however, is that Weston wakes up on every modifier change,
regardless of whether or not it has focus.

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

Rename modifier_state_changed to notify_modifiers

notify_modifiers will now synchronise Weston's internal state with the
XKB state, and send a modifier event if necessary.  This eliminates the
need for update_modifier_state to have a return value at all.

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

notify_key: Add update_state argument

If update_state is true, then notify_key will continue to call
xkb_key_update_state to update the local state mask, as before this
commit.  Otherwise, it will rely on the compositor to manually update
the state itself, for nested compositors.

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

compositor-x11: Use XKB detectable autorepeat if possible

If we have XCB XKB support, use XKB's detectable autorepeat, which
generates repeat sequences as a series of
press-press-press-[...]-release events, rather than
press-release-press-release-[...].

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

compositor-x11: Add base XCB/XKB support

Try to find the XKB extension, to be used in later commits.

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

compositor-x11: Move input creation before output

So we can set up XKB masks and be sure to get the right events before
we set up our window.

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

weston_xkb_info: Cache all modifier indices for X11

X11 has a set of eight modifiers which we want to represent.  Cache
their indices when we create a weston_xkb_info, so we can use this from
compositor-x11 to keep the state synchronised exactly between the host X
server and a nested Weston instance.

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

Run key bindings on keyboard focus in

This is mainly for X11's benefit; the common case is using Logo+S to
take a screenshot, where GNOME Shell has grabbed Logo, and replays the
event down to the nested compositor after S is pressed.  This means we
get an enter event with both Logo and S down, and even if Shell delivers
the key press event for S (which isn't mandatory, and not all window
managers do), then we never run the binding since notify_key realises
that S is already down and exits early.

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

Ignore repeat keys in notify_key

Let compositors just blithely post through every event they get,
including repeating keys.

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

Let wayland-server send modifier events for focus

When the focus changes, wl_keyboard_set_focus (and, as an added bonus,
wl_pointer_set_focus) will now send wl_keyboard::modifier events for us
if we store the modifier state in the right place, so we don't have to
worry about that anymore.

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

Split notify_keyboard_focus into in/out variants

Since the two functions had nothing in common but the local variables.

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

.gitignore: Ignore ctags and cscope output

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

window.c: Always set cursor after pointer enter

If the cursor didn't change since last time we had pointer focus we just
wouldn't change it.  But whoever had pointer focus in the mean time could
have changed it, so make sure we always set the cursor after pointer enter.