move opaque region (image border) apply to render fn

this seems to provide a more accurate region for rendering in my tests,
as various weston-apps no longer show black rects during resize

don't mangle wl client opaque region tiler during commit clip

improve wl compositor tiler usage

pixman <-> tiler api naming seems similar but functionality is totally
different

only perform keybind grabs on root window for x11 compositors

block compositor keybinds using more accurate x11 detection

blocking based on event window here is only valid for true x11 compositors
and not just compositors with root windows

fixes keybind activation when running xwayland clients

wayland: Also link with libwayland-client.so

Summary:
E is both a client and a server, so it should be linking
both libraries.

 #Kansas

Reviewers: zmike, devilhorns

Reviewed By: devilhorns

Subscribers: cedric

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

e - randr allow you to turn off screens and keep them off if config says so

e - unbreak client pixmaps when native surf is used

this unbreaks disappearing client content introduced by
5a001b1759445c1461341c7d1e069ac74e411f04.

do not set opaque regions on comp object mirrors

based on testing, this breaks all rendering of related objects. I
suspect that the image border needs to be manually scaled based on
image::mirror proportions in order for this to work as expected, but
adding the required code seems like too much complexity for nearly zero
gain

render surface opaque regions as opaque when image border is set

it seems that just setting the opaque region is not enough for evas;
we must also inform it that the opaque region is opaque and should not
be blended

zero out wl surface opaque region on commit in all cases

it's possible that the opaque ptr may exist but have no regions,
which will result in the previous frame's opaque region being used

add e_client_unignore() for triggering events when unignoring a client

mainly for use in wayland compositing since all surfaces are ignored
until they get a shell surface or a commit

fix wl_fb define name in comp init

enforce image border/size/pixels_dirty setting on comp mirrors during create

this should provide a more accurate first-render for mirrors as they are added

ensure wayland parts of shot module use the correct wayland display objects

the ecore-wl global objects used here will be wrong for the case of nested
compositors

do not update client list hint when creating a new wayland client

always set new wayland E_Clients to be ignored during client hook

add wl_wl wl output module

it is now possible to create a xephyr window in a drm-enlightenment session,
launch wl-x11 enlightenment in xephyr, and then launch wl-wl enlightenment
inside that enlightenment

the primary limitation on this output module is that all internal windows will
appear in the outer compositor due to the current restriction of ecore-wayland
with regard to only having a single global display server connection

 #Inception

remove a kind of typo in E_ZONE_GEOMETRY atom string.

Test Plan: N/A

Reviewers: devilhorns, zmike

Reviewed By: devilhorns, zmike

Subscribers: cedric

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

e-comp-wl: Fix region_add and region_subtract functions to make sure we have accurate opaque regions

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

...and also ungrab the compositor when internal drags end

only use input xwindows for internal drags in x11 compositor

this fixes input region detection for dnd operations

fix T1831

screenshooting: Avoid void pointer arithmetic

Summary:
If we use unsigned char pointers instead of void pointers we actually
conform to the C standard.

This patch removes a reliance on a gcc extension and, as an added bonus,
also quiets a warning in the default build.

Reviewers: zmike

Subscribers: cedric

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

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