wl clients should not use edges during keyboard resize

ref T2452

add E_Client->keyboard_resizing flag

there is no other indication of whether a client is resizing using the
keyboard, so this should serve as an adequate hint

never use new clients for stacking part 2: the secret of the stacking

a client with this flag set here is unreliable to use as a stacking
reference since it has yet to be stacked and can be located anywhere
in the window stack.

fixes internal window stacking on startup

ensure that focus is set and focus stack is managed on winlist hide

if pointer warping is disabled, attempting to pointer warp with mouse-based
focus policies will fail here, preventing focus from being applied as expected

ref T2566

only trigger client mouse-in from x11 mouse move event if client+desk are visible

ref T2566

simulate netwm sync for xwl clients

blocks execution of resizes until the surface commit arrives. reduces
the race condition between resize and render and eliminates frame drops
during slow resizes

move xwayland socket connection to its own thread

this operation performs a synchronous socket connection inside xlib which can
block for an infinite amount of time. in order to avoid having a potentially
unlimited amount of time where the ui is frozen and polling on the socket connection,
move it to a thread where it can hang for as long as it wants and then use the
resulting display object for the ecore-x connection

e-comp-wl: Remove unused variable

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

e-comp-wl: Fix T2570: stuck key when closing efm windows

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

e-comp-wl: Fix getting end key during key down/up events

NB: This is actually an Oopsie from a previous commit that should not
have gotten in.

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

e-comp-wl: don't send keyboard modifiers twice.

NB: e_comp_wl_input_keyboard_modifiers_update already sends keyboard
modifiers to all keyboard resources, so no need to call this twice.

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

e-comp-wl: Update keyboard and modifier state AFTER we send them to clients

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

e-comp-wl: Add support for wl_keyboard_send_repeat_info to allow for configurable keyboard repeat rates

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

e-comp-wl: Update keyboard key state regardless of client focus or not

Key state updates need to be consistent. As we update the key state on
key up regardless of client focus, then we should be doing the same in
the key_down event handler so that key state remains consistent.

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

e-comp-wl: Don't call keyboard_modifiers_update twice

e_comp_wl_input_keyboard_state_update already makes a call to
e_comp_wl_input_keyboard_modifeirs_update so there is no need to call
it twice on key up/down events.

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

Revert "delete render updates for xwayland clients upon triggering a client resize"

This reverts commit f3f29a0cf83412ae4f09fb1fbed065daca7c4047.

upon further review, this was not at all helpful and breaks resizing eventually

enlightenment_remote: restore compatibility with older versions with bg-add / bg-del

Summary:
older versions of enlightenment_remote had a additional first param that did nothing
this change detects if that paramater is still passed in and ignores it. There are
a couple of applications including variety that rely on this behavior as well as some
user scripts

Reviewers: zmike, raster

Subscribers: cedric

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

remove unnecessary call to e_bg_del

I noticed e_bg_add calls e_bg_del so the additional call is not
required, it should also be noted the msgbus module doesn't call
this and works fine.

Reviewers: raster, zmike

Subscribers: cedric

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

e_bg_del should only delete the bg on the zone specified

Summary:
previously e_bg_del didn't take the zone paramater into
account

fixes T2572

Reviewers: raster, zmike

Subscribers: cedric

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

e randr2 - auto clone and use common res when doing so - make it work

tested this and made it work. should fix T2100

disable size debug on drags

delete render updates for xwayland clients upon triggering a client resize

fixes frame dropping during resize

only register dnd xwindow with compositor canvas in x11 compositor mode

registering it under wayland creates spurious mouse events

fix xdnd in (x)wayland

this is a real thing, so don't abort immediately if it happens

adjust layering of pointer and init splash

prevents cursor from getting stuck above screensaver fade

add wayland client post-render references in the render loop

add function to fill in x11 cursor image on creation

xwayland compositing requires that we set up a root window cursor image
immediately since we'll be getting that cursor surface to display as soon
as the pointer goes out of an x11 client's window

break up some pointer setting code into helper functions

always set x11 override client geometry on startup

this geometry isn't something we can change later, so apply it now.
also fixes an ACK when trying to show input_only clients early

disable netwm sync on x11 clients in wayland compositors

this has no effect

when e crashes in wayland, just crash instead of possibly freezing

unify crash handler functions

redo e_pointer to support xwayland cursors

this requires that both canvas cursors and window cursors be present for the same
E_Pointer object, even though only the canvas cursor is actually visible

 #kansas

shot: Enable feature of taking just window shots in wayland

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

return to using standard client util fns for getting client wins in x11

remove unnecessary E_COMP_WL #defines

this is only required for aliasing E_Client->comp_data as wayland compositor data.
if comp_data is never dereferenced, it is not necessary to declare the compositor
type

make wayland-specific shot protocol files conditional on wayland build

remove NEED_WL define from some makefiles

this isn't a thing and I don't know why it's here

shot: Disable window shots in wayland for now

NB: Feature still being developed

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

shot: Update with proper compositor checking tests

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

remove old wl_screenshot module

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

remove wl_screenshot from configure.ac

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

shot: Update shot module to now be able to take Wayland screenshots

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

shot: Update shot makefile to include client screenshot protocol files

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

wl-drm: Add function to readback pixels for compositor screenshots

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

e-comp-wl: Create compositor screenshooter global and add function to actually handle taking shots

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

e-comp-wl: Add screenshooter structure to comp_wl_data and add additional fields we need in E_Comp_Wl_Output

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

e-comp-wl: Add screenshooter files to build order

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

shot: Add screenshooter client protocol files

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

e-comp-wl: Add screenshooter protocol files

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

fix wayland-only build

remove NEED_X define from build

this used to be a marker for places where x11 functionality was needed,
but this has been simplified with the removal of wayland-only in configure
and so it is no longer needed

make e_client_util_.?win_get() fns return xwayland windows when available

fixes setting x11 hints in xwayland clients

set xwayland clients in x11 compositor client hash using parent window on flip

properly calculate client+frame sizes during wl surface commit

account for race condition when creating initial comp object updates tiler

under wayland, some surfaces (eg. cursors) would attempt to show prior to
having acquired their actual size. these show attempts should be rejected
until the size has been set to ensure that rendering can proceed as expected

fix T2557

feed mouse-in event to drm canvas on init

this is normally triggered by the engine/display server, but the drm
output is too powerful to be bothered by such trivial matters as
mouse events on startup

block client signal binding activation when mouse action is active

this prevents oddities such as activating [window shade] and [resize] actions
simultaneously from breaking the compositor

shot: Fix formatting for 80 columns

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

set CRITICAL urgency for notification internal notifications

ensure that presentation/offline mode toggles will display a notify

move grabinput focus fix timer to x11 compositor and fix it to Work Better

see inline comments about x11 focus eventing for details

fix T2547

use ref macros to silence more printfs in e_remember

use evas object callbacks for fwin mouse in/out, don't use ecore-x handlers in wayland

improve compatibility with wayland compositors

silence printf noise in screensaver and pkgkit module

make pager popups only trigger on urgency hint if client is not currently visible

the annoyance of having this popup constantly triggered while a window is visible
cannot be understated.

make e_client_util_desk_visible() work for overrides without desks

e desktop file properties - fix icon sel tab to expand icon and icon name

this fixes this tab after e20 dev changes

e -> menu -> fix games menu to be flat like others and include steam

this improves the menu layout to be simpler and nicer by far. steam
games go from other to games menu.

wizard: Prevent crash

Summary: If we get here when curpage is NULL, we'll crash later, so we should test for it.

Reviewers: zmike

Subscribers: cedric

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

wizard: Prevent crash

Summary: If we get here when curpage is NULL, we'll crash later, so we should test for it.

Reviewers: zmike

Subscribers: cedric

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

Init some vars to NULL in compositor create

Summary: In some cases these end up uninitialized and we crash.

Reviewers: zmike

Subscribers: cedric

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

set title/name/class on wl-x11 output canvas window

only apply xwl client deletion to xwl clients in x11 compositor

bump required efl version

reorder xwayland client swap attribute setting

unset x11 focus grabs on clients only if there is an xwindow present

fix T2539

return NULL when a client's xwayland pixmap is unreachable

block wl key events from reaching clients during client actions or key grabs

pop an error dialog when trying to load xwayland module under x11

this is impossible and will never be possible; ecore-x can only manage
a single x11 connection at any time, and so it will never be possible to both
manage the x11 compositor canvas on one xserver and manage xwayland clients
on a separate server

invalidates T2537

reject xwayland module from saved module loading

set initial surface attrs for internal wl clients

clean up xwayland lock/socket init code

fix typo in conditional for setting x11 hints

do not apply x11 focus grabs to internal elm clients

due to recent changes in ecore-input-evas, mouse events are propagated
differently; specifically, there are now "more" events than there previously were.
as a result, grabs on internal wins are no longer necessary, though they probably
never were necessary after the elm conversion

see 5cb6cdbc5e1a13ea0262e155983b494e6519abde in efl

reject x11 replay clicks when event window != client window

only ungrab x11 windows when appropriate focus options are set

allow hint setting on x11 windows in xwayland

replace comp_x usage of internal xwindow detection function with new util fn

add client util function for determining if a backing xwindow exists

replace some x11 compositor checks with calls to util function

fix e_scale wayland conditional defines

add util function for determining presence of x11 compositor

print xwl loading debug after checking compositor type

ensure x11 focus grabs are applied on client init when needed

straggler ungrab of x11 parent window from previous commit

redo all x11 client mouse grabbing for focus

I did an audit of this and it seemed that it no longer served the purpose
for which it was originally intended. specifically, this is for enforcing
click: raise/focus options, and so grabs must be in play on client windows
only when they are not focused to ensure that we get mouse events and can
then focus them. the grabs must then be removed once the window has focus
to avoid spurious mouse eventing

e-comp-wl: Update modifiers on key down/key up events

NB: Fixes Ctrl+D problem here.

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

e-comp-wl-input: destroy xkb keyboard state Before destroying the keymap

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

e-comp-wl-input: Format for 80 columns

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

xwayland: Use proper E_API for exporting module symbols

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