e-comp-wl: Fix formatting

Signed-off-by: Chris Michael <cp.michael@samsung.com>

e passive window grabs - fix side-effect leave/enter events on clients

when you have click to focus we have a passive grab set up. somewhere
that window changed to the parent window instead of the client. this
leads to a side effect of a leave and enter event on clients for every
click. generally clients are ok with this, but some seem to have buggy
event handling. these enter/leave events are a side effect of the
passive grab even though we allow/replay the event.

this fixes that by placing passive grabs on the client window itself
instead of the parent.

@fix

don't select mouse wheel events on x11 reparenting window

this fixes mouse eventing/focus on windows, though I'm not entirely sure why...
probably something with binding grabs

mixer: Fix resource leaks

This fixes Coverity CID1308395: Resource leak. Basically, don't bother
allocating 'source' if we are just going to end up returning due to
'eol' variable tests

Signed-off-by: Chris Michael <cp.michael@samsung.com>

set shm engine for wl internal windows

this seems to be necessary for now to prevent crashing

add wrappers for window/pixmap/comp_data fetching in x11 compositor

these helper functions automatically account for "swapped" xwayland
clients and return the expected value from the wl client comp_data.
in this way, all of the current x11 compositor code can be reused with
minimal changes

reject non-x11 clients from _e_comp_x_client_zone_geometry_set()

add hooking for WL_SURFACE_ID atom on XWayland windows and composite them

in order to maximize the amount of reused code the following details the current
process for xwayland compositing:

* get map request from window
* force reparenting
* show window
* await WL_SURFACE_ID x11 message
* move x11 client data + pixmap onto corresponding wayland client
* business as usual with wayland compositing

this is pretty similar to the method of the reference code in weston,
except that there's no x11 compositor in weston

fix grabinput to work better with multiple compositor backends

check root window to determine existence of X11 compositor,
use both wayland and non-wayland define codepaths where available

reject non-X clients from _e_comp_x_object_add()

block xkb activation for non-X11 compositors

correct e_comp->new_client count for overrides in x compositor

move X idle flusher to comp_x

don't call wl comp delete function twice on shutdown

comment out grabinput focus calls in wayland compositor

these did nothing, remove for now

set client visibility for shell-less wayland clients appropriately during commit

reject ignored clients from being processed in client idler

reject stacking clients from frame updates in client idler

use ctrl+alt+backspace hotkey in wayland compositor for all non-release builds

unused variable--

set wl client surface pointer in surface create method

remnant from when clients were created by the shell

remove unnecessary header includes from e_comp_x.h

don't undef NEED_X in wl compositor headers

no idea why this was here

remove E_Comp_Data typedef, use backend-specific typedefs unconditionally

this turned into a bit of a charlie-foxtrot due to constant pointer aliasing
in the wl compositor codebase

no functional changes

redo wayland-only configure option and define

HAVE_WAYLAND_ONLY is now only set when there is only wayland. this means there
is no x11 support. zero. no x compositor. no xwayland. no x11 output module.

simplify e build and add X headers for xwayland module

add setup for WL_SURFACE_ID X atom

use c array size for e atom init

remove wayland-clients configure option

this has been dead for a long time and is no longer necessary since
xwayland support exists

bump wayland version requirement to 1.8

HAVE_WAYLAND_CLIENTS || HAVE_WAYLAND_ONLY -> HAVE_WAYLAND

these are semantically identical

finish xwayland module implementation

note that this basically does nothing other than start an xserver and connection

bisect harder if looking for bugs

fix xwayland fd dup conditionals

fix xwayland configure check

load wl_desktop_shell and xwayland after wl comp load

update xwayland for api changes

fix xwayland linkage + makefile formatting

remove xwayland .desktop and .edj files

xwayland: Add checks for xcb libraries needed for xwayland support

Signed-off-by: Chris Michael <cp.michael@samsung.com>

xwayland: Add code to cleanup xwayland on shutdown

Signed-off-by: Chris Michael <cp.michael@samsung.com>

xwayland: Check for sigusr1 in event handler

XWayland servers sends us SIGUSR1 when it has finished initializing,
so we should be checking the signal number when we get the event.

Signed-off-by: Chris Michael <cp.michael@samsung.com>

xwayland: Fix build breakage with recent e_comp_wl changes

Signed-off-by: Chris Michael <cp.michael@samsung.com>

xwayland: Add some debugging code while trying to work out failures
with this

Signed-off-by: Chris Michael <cp.michael@samsung.com>

xwayland: Fix launching of XWayland server and pass proper display to
wl_client_create

Signed-off-by: Chris Michael <cp.michael@samsung.com>

Whitelist the new xwayland module

Signed-off-by: Chris Michael <cp.michael@samsung.com>
Conflicts:
src/bin/e_module.c

xwayland: Create the wl_client which represents the window manager
process.

Signed-off-by: Chris Michael <cp.michael@samsung.com>

xwayland: Create and bind to unix & abstract sockets

NB: XWayland server needs the sockets setup prior to launching it so
we add some code to create & bind the needed sockets before starting
the XWayland binary

Signed-off-by: Chris Michael <cp.michael@samsung.com>

xwayland: Fix makefile to use xwayland binary path

Signed-off-by: Chris Michael <cp.michael@samsung.com>

xwayland: Fix configure.ac to substitute the xwayland binary path

Signed-off-by: Chris Michael <cp.michael@samsung.com>

Add start of xwayand module code

Signed-off-by: Chris Michael <cp.michael@samsung.com>

add xwayland module edj file for icon

Signed-off-by: Chris Michael <cp.michael@samsung.com>

add xwayland module desktop file

Signed-off-by: Chris Michael <cp.michael@samsung.com>

Add xwayland Makefile

Signed-off-by: Chris Michael <cp.michael@samsung.com>

Add xwayland module to modules Makefile

Signed-off-by: Chris Michael <cp.michael@samsung.com>

add xwayland to configure.ac

Signed-off-by: Chris Michael <cp.michael@samsung.com>
Conflicts:
configure.ac

wl-desktop-shell: Don't crash when trying to place popup menus in QT apps

This is a partial fix for T2517. It fixes the crash, but does not
address popup menu placement.

Signed-off-by: Chris Michael <cp.michael@samsung.com>

wl-desktop-shell: Remove useless return at end of function

Signed-off-by: Chris Michael <cp.michael@samsung.com>

wl-desktop-shell: Fix formatting

Signed-off-by: Chris Michael <cp.michael@samsung.com>

Logging: Fix broken logging macros.

This is macro 101, you don't EVER put multiple statements in a macro
like that.
See Chris's commits, these broken macros already introduced (subtle)
bugs. Always surround macros in "do {} while()" unless you absolutely
can't (like when you declare a new variable to be used in the scope).

Why is it even there? I think we can safely assume eina log is available
for usage in E...

@fix

Fix Coverity CID1298053

Signed-off-by: Chris Michael <cp.michael@samsung.com>

Fix Coverity CID1298052

Signed-off-by: Chris Michael <cp.michael@samsung.com>

handle compositor animation race condition when changing effects

this is more of an academic case than any existing scenario, but
it's possible that an effect may be stopped by something attempting
to trigger another effect during the animation

fix animation refcounting when toggling object visibility very quickly

previously the animating flag would receive an additional increment for
every effect, even if it was currently animating a prior effect, leading
to objects which were never deleted

add comp object bool flag to indicate whether a compositor effect is currently running

make e_comp_object_effect_stop() return bool for object validity

sanitize bool return of _e_comp_object_animating_end()

only apply deskmirror client visibility logic for non-deleted clients

return an accurate value for override clients in e_comp_object_util_zone_get()

use current zone for visibility effects on override windows

e-comp-wl: #if 0 out a currently unused function for pre_frame assignment.

Signed-off-by: Chris Michael <cp.michael@samsung.com>

remove a stupid elm win hack flagged for deletion

may trigger crashes on non-updated elementary

Fix misspellings in E's files.

Summary: Fix misspellings and spaces in E's files.

Reviewers: zmike

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2676

allow layer-blocked clients in deskmirror to restack

added quick tile to corner feature

Summary: added quick tile to corner feature, Fixes T2408

Reviewers: zmike

Subscribers: abyomi0, cedric

Maniphest Tasks: T2408, T2419

Differential Revision: https://phab.enlightenment.org/D2638

Center a dialog window when it is displaying off-screen

Summary: Center a newly created dialog window when it is displaying off-screen.  Fixes T2419

Reviewers: zmike

Subscribers: cedric

Maniphest Tasks: T2419

Differential Revision: https://phab.enlightenment.org/D2646

only play visibility effects for allowed clients

add utility function for determining if a comp object should animate

use new visibility effects by default for menu + combo windows

tooltip effects by default...probably too powerful for average users?

send client/screen/pointer geometry to visibility effect edje

unify comp object visibility effect animation code

add flag for detecting whether an effect is currently set on a comp object

also improve refcounting for effect apis and add return values for use in
determining whether an effect was successfully set/started

add flag to E_Comp_Object for detecting show animation state

this allows improvements to the code which provides hide animations,
allowing clients to begin hiding during their show animations instead
of rendering a black rectangle

use a safe iterator when deleting clients during comp shutdown

it's conceivable that, were there a bug in client refcounting,
this could become an infinite loop and prevent shutdown/restart.
since, at this point, we don't really care about deleting anything,
ensure that the loop will end

e - e client, comp win etc. reffing fix ... part 2

this follows 56cabf59c672af27af87eda82079de4e2ee2d38e then
4e5521b4d82063a2fbf6632a117fda201cc1687d where i have been trying to
fix a crash with e client and comp win references etc. i have gone
over all referencing with a fine tooth comb and found all the nigglies
i can., no leaks now, no crashes, no valgrind complaints etc. so i
call this fixed now. as best i know this is new in e20, so not a
backport fix

e - bring back fullscreen crash bug while fixing refcount correctness

refs were inconsistent - thus this fixed the fullscreen quit bug by
never freeing a client. this brings the bug back by fixing this client
leak. i'll look again at this later.

e - fix crash in desktop lookup+fixup where exe_inst is null

e comp - fix refcounting for e_comp and comp clients

the refcoutning for e_comp and e comp clients seemed to be a bit off -
i read over every ref and unref carefully and fix it. this leads to
the com-_data being null (properly now), so now check for that too.

e - mixer - allow to 150% volume with some "stickynedd" from 100 to 120

fixed window focus and keyboard input issues

Summary: fixed window focus and keyboard input issues if the to-be-focused window is in an iconized state

Reviewers: zmike

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2644

Changed search pattern to focus window by name to case-insensitive.

Summary: Sometimes a window has a name that's not like the application name, for example.  chromium-browser application will have a window name: Chromium-browser.  Most users will try to match the window name with "chromium-browser", but it wont work b/c the e's window name match is case sensitive.  Most users, would not know that the window name of chrome is "Chromium-browser", so it is pretty much impossible for them to setup a focus-to-window-name binding properly.

Reviewers: zmike

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2645

rename visibility effect struct member effect -> visibility_effect

feeble attempt at making future added effects more intelligible

add compositor visibility effects and matching configuration

these are specific types of animation for use when toggling window visibility.
they combine with existing compositor window animations to provide nicer integration
for very specific types of windows

see https://www.youtube.com/watch?v=hIVdd0Z2K00 for a demo

allow comp matching for UNKNOWN type windows

sometimes this is a valid type and we want to match it

Updating italian translation

e menu - an extra object ref causes menus to never be freed

this has been in e for ages - someone not noticed, but this fixes
visual artifacts of left over menus on the top-left. this extra ref
really makes no sense. it's not like this ref is then accomoanied by a
matching unref somewhere else (after much debugging).

@fix

wl-desktop-shell: Fix popup menu placement for xdg surfaces

Signed-off-by: Chris Michael <cp.michael@samsung.com>

e - mixer - clean up popup a bit with mute next to slider etc.

e mixer - fix segv on desklock if default sink not set yet

mixer - dont make list scrollable horizontally for nicer look

Revert "Revert "fix mixer gcc class name""

This reverts commit 6bacf2070532d0bdf22afa4d303ac41a0ce52047.

oops - i was wrong. totally misread that patch i thinkg it changed
mixer -> emix