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>

correctly fix check for x11 compositor in e_xkb

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

correctly fix check for compositor in e_grabinput

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

e_grabinput: Don't call ecore_x functions if we are not an X compositor

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

Don't call ecore_x_xkb_select_group if the compositor is not X11

This fixes T2533 where the startup wizard would crash when run under
DRM due to the change in build options (xwayland support).

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

use client window for x11 button ungrabbing

ref afb62b498997ca038687c0cd0ca97aa361aaa1b7

copy more attributes from x11 -> wl on xwayland client swap

xwayland sets a wrong size on some (eg. menus) clients and wayland
cannot provide geometry or stacking information, so ensure that all
of this is copied over

also remove overrides from focus stack

clean up xwl data on client deletion

remove the extra comp_data, the pending client list entry, and the pixmap ref

don't create first draw timers for input-only x11 clients

don't attempt to change layer of override clients with parent set in x11

overrides will stack how they want, leave them alone

add debug info when x11 client messages are missed

temporarily toggle ignored flag during wl surface commit

ensures canvas operations succeed during commit in cases where
subsequent commits do not occur (eg. menus, popups, ...)

fix typo in iterating pending xwl clients on wl surface creation

free all ec->comp_data ptrs in _e_client_del

need this to live through multiple del hooks, so simplify all code by
freeing in one place

only reject ec->ignored clients from wl mouse events

fixes input on overrides

add special case for xwayland override clients in show event handler

use separate client hashes for x11/wl clients

e-comp-wl: Release keys on key_up regardless if client has focus

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

fix compile from previous commit

I am not a smart man.

disable uuid store

currently seems to fill logs with infinite error messages

ref T2532

unset ec->ignored during xwayland client data swap

attempt to create xwayland x11 socket path before creating socket

e-comp-wl: Don't crash from a missing keymap

This fixes T2531 where E would crash if the keymap could not be
fetched from xkb. Now if no keymap rules, model, or layout are passed
in we will default to a US keymap.

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

wl_x11: Mark data in callback as unused

add focusgrab test

this test demonstrates one of the more annoying aspects of x11 focus

enforce pstate's extremely-confusing no_turbo option in cpufreq

fix T2411

trigger client mouse-in on x11 mouse movement for non-action clients

this should fix the case of mouse-based focus policies trying to reapply
focus after another client has stolen it away without the pointer leaving
the window

trigger fake mouse-out on clients when unsetting focus

this will allow a more reasonable determination of pointer location
when attempting to apply mouse-based focus policies

unset changes.visible when forcing visibility during no-effect desk flip

this is set for new clients and creates visibility issues on startup

fix focus FIXME regarding focus-setting on clients from other desks

fix T2259

check for possible parent window on x11 mouse button events

in the case of different window <-> event_window, window is a child window
of event_window, and thus checking event_window here is valid (and necessary)

stop rejecting possibly-valid x11 focus events

fixes case where window assumes focus without wm interaction

set pager_plain to fill area

fix T2526

fix ecore-x detection with wayland support, also fix wl-only detection

fix T2527

disable wl-fb output module

this has never worked and is not being actively developed

remove E_Comp_Data reference from wl-x11 module

fix T2528

fix wl-x11 configure checks

rename --enable-wayland-only to --enable-wayland

wayland-only is only enabled if xwayland and x11 output are disabled

session_recovery: Add wayland extension protocol prototype for session recovery

Like with all other wayland protocols I add the files generated with wayland
scanner here. Also the xml so we have the source around for updating and
modifications. We might want to think about wayland-scanner support in our build
system but this works for now.

The protocol prototype is hold simple and does only have a uuid signal and provide
call to handle the uuid assignment from compositor to app and app providing its
uuid if present already.

e_pixmap: Create uuid when being used as a wl surface

e_main: Initialize UUID store during E startup

I have been running with this enabled for a while and it should not make
trouble but if it does simply reverting this one if totally fine while I'm
away.

e_remember: Update UUID store properties from e_remember

The e_remember infrastructure already hooks into all needed places to keep
a record of the given properties for an e_client. We use this to update the
UUID store.

Signed-off-by: Stefan Schmidt <s.schmidt@samsung.com>

e_client: Remove deleted client from UUID store

Signed-off-by: Stefan Schmidt <s.schmidt@samsung.com>

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.