remove e_comp_wl struct keyboard repeat members

duplicating these from e_config serves no purpose

ref D3364

(1) e_config.c/e_comp_wl.c: add code for making repeat values configurable
(2) e.src(s): add keyboard.repeat_delay, keyboard.repeat_rate into e.src files

Summary:
As of now, the default values of repeat delay/rate are being set in e_comp_wl.c.
Those values need to be configurable and will be used in e_comp_wl_init().
The limit of each of the values is defined from -1 to 1000. (maximum 1s).
If one of the two is negative, it means default repeat delay/rate are going to be used.
(e.g. delay:400, rate:25)

Test Plan:
N/A

Signed-off-by: Sung-Jin Park <input.hacker@gmail.com>
Reviewers: raster, stefan_schmidt, gwanglim, devilhorns, zmike

Subscribers: Jeon, ohduna, cedric

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

Stop using MIN macros when choosing versions for wayland resources

Summary:
libwayland-server.so will post an error if the requested version
is higher than the supported one anyway, so there's no point in
doing this.

Using MIN() to pick versions is a client side idiom.

 #kansas

Reviewers: zmike, devilhorns

Subscribers: cedric

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

e_menu: use Eina_Bool for one-bit bitfield

Make sure we are using a unsigned var here. Best go with a Eina_Bool directly.

e_gadcon: use Eina_Bool for one-bit bitfield

Make sure we are using a unsigned var here. Best go with a Eina_Bool directly.

pulse backend: correct declaration of  function with no parameters

We have to use void in a function declaration if we want no function
parameters. Using just empty parenthesis means the function takes an
unspecified number of parameters.

We had it correct for most declarations and this series fixes it for
the rest.

Thanks for the sparse semantic parser for pointing this out.

e_fm:correct declaration of  function with no parameters

We have to use void in a function declaration if we want no function
parameters. Using just empty parenthesis means the function takes an
unspecified number of parameters.

We had it correct for most declarations and this series fixes it for
the rest.

Thanks for the sparse semantic parser for pointing this out.

re-swallow bg_object after wallpaper transition has completed

fixes eventing/stacking on bg object

allow more client hooks to bypass delete check

failing to propagate these signals can result in a broken compositor
when a client is deleted before/during an operation

do not apply max client geometry restriction if max geometry is not set

default values for these is set to a very high value, but it's possible
(and guaranteed) for some clients to set them to a value <= 0, which results
in a broken window

use CSD geometry during move/resize display

this should provide more user-relevant information

adjust for SSD geometry during wayland client resize

client moveinfo includes the frame geometry, so this must be taken
into account when resizing a client which has a frame

fix T2863

avoid setting incorrect geometry during new_client frame calc

in the case where a client has no geometry set, attempting to update
the client's geometry during frame recalc will guarantee that wrong
geometry is set, resulting in a bad first frame

declare xwayland functions prior to using them

 #CompileFail

use separate handler for x11 fatal errors when using xwayland

a fatal error with xwayland is not a fatal error for the compositor,
so this should not result in a dead session

add mechanism for preventing unload of important modules during runtime

if a wayland output module is unloaded while the compositor is still
active, bad things will happen

fix all shadow warnings in msgbus module

ref cdb436ee8602728e18a63244988da22b25477819

reenable restart/shutdown dbus methods under x11

e - efm - rate limit dbus requests to open dirs or files to avoid dos

in a wayland wold we dont want to trust clients on the other side of a
dbus connection - so rate limit what you can do with efm dbus requests

@fix

e msgbus - move almost all of core dbus methods to msgbus module

for security reaons, all the dbus methods that allow you to mess with
e are now in the msgbus module - load at your own risk. this is
irrelevant in x11, but in wayland this matters as wayland is actually
secure.

this also disables restart and shutdown dbus methods still in core.
they are there but non-functional due to possibly being able to be
abused in a wayland universe to "dos attack" the wm.

@fix

only perform drm shutdown during signal handler if drm engine is in use

only create wayland screensaver timer in wayland mode

probably fix T2867

Fix invalid compositor type check in e_dpms_force_update

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

Fix formatting and remove extra blank line

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

e - fix compositor fade out ghosting

due to enabling manual rendering (and als animator frametiem to 10
secons) in e_comp_canvas.c when screensaver is active (blanking is
finished totallly - eg the fade to black) evas weill nto render the
last frame of the animation - skipping it and not rendering another
update until screensaver is disabled. this leaves a subtle ghost of
pixel data which is 1 step before black on the screen (until dpms
turns the monitor off).

this fixes that. this delays enabling manual render for 1 more second
after we have been told the screensaver is active. this is plenty of
time to update all the way to black.

@fix

ref clients while a wayland shell surface exists

in the case where a client is deleted, it's possible that the shell
surface may persist longer than the duration of the normal client delete
cycle, so it's necessary to ensure that the client will continue to exist
until the shell surface has been destroyed

remove fwin popup size hint change callback on win deletion

this callback also crashes if called during delete

ensure fwin efm delete callback does not trigger crash

if this is called during fwin delete, dereferencing the passed data
will result in a crash

e/core: Fix minor mem leak when reading module paths

Summary:
Found this with a quick valgrind session.  For paths that are not directories, if we fail the `ecore_file_is_dir` condition memory will not be freed.

The change is to use the default e_path freeing function.

Test Plan:
Run enlightenment in valgrind.  It should no longer show a mem leak like below.

```
==6912== 8 bytes in 1 blocks are definitely lost in loss record 186 of 5,940
==6912==    at 0x4C28C50: malloc (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so)
==6912==    by 0x4D8574: e_path_dir_list_get (e_path.c:326)
==6912==    by 0x4D1BA4: e_module_init.part.0 (e_module.c:183)
==6912==    by 0x4D1DA8: e_module_init (e_module.c:153)
==6912==    by 0x4371ED: main (e_main.c:868)
```

Reviewers: zmike

Subscribers: cedric, seoz

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

always set wayland E_Client->redirected during client creation

fix T2846

ensure that a deskmirror object does not orphan child objects

ref T2846, T2830

Revert "e deskmirror - fix loss of mirror object from deskmirror"

This reverts commit bd0c590489a7ae1c3185578389f19bde4bacbcff.

the m->mirror object can never be deleted except in the case of a client
being removed from the owner desk. removing it at any point will break any
users of the deskmirror api and create other issues

ref T2846

block task gadget refills while a drag is active

it's impossible to use a pager effectively with a task gadget due to
the constant resizing of the tasks gadget during any kind of window move

always create compositor mirror objects, ignore pixmap image state

checking pixmap image state in this case resulted in mirrors failing to
be created in some cases when there was no reason to fail

ref T2846

e/imc: fix issue with input method import segv and small window size

Summary:
The imc import dialog at "Settings > Language > Input Method Settings >
Import .." had issue where the size was almost zero.  When trying to
close the window it would SEGV.

The SEGV seems to be do the with dialog->cdata not being set.

The resize issue I am not so sure, but I upgraded to ELM and if fixed
the issue.

Test Plan:
Open "Settings > Language > Input Method Settings > Import ..."
 - The size should not be really small
 - When closing it should not cause E to SEGV
After applying the patch the above issues are fixed.

Reviewers: zmike

Subscribers: cedric, seoz

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

add safety checks to e_client_focus/raise_latest_set() fns

passing a null client here breaks the universe, so ensure that it's
easily catchable

further improve csd calculations for x11 clients

there was previously a frame in which maximized x11 csd clients would
reposition themselves offscreen before rendering

this is now fixed by adjusting the client geometries either upon
unsetting the csd or re-setting it, allowing the client to retain its
expected positioning without jitters

e mixer fix - fix default mixer to pulse if there

this makes e use the correct mixer backend if pulse is available as
opposed to using alsa going behind pulses back.

@fix

L10N: Update Finnish translations

Includes translations from Launchpad.

update E_Client->desk member upon focusing a sticky client

a sticky window previously would always have the desk set for where
it was set as sticky, meaning that anything which tries to access it
will be reading wrong data here.
more useful information to provide is the last desk which the sticky
client was focused on, so update that upon focusing it

do not refill all tasks bars when a sticky window changes desks

do not add new deskmirror clients on desk_set event if client already exists

improve x11 client focusing with mouse-based focus policies on mouse move

in the case that a mouse move event occurs, the compositor should validate
the event to ensure that the mouse cursor is actually over the window that
the event claims to be from

fix T2594

improve readability of E_FIRST_FRAME environment variable detection

functionality remains the same, but this is more clear for readers

install emixer icon to $(datadir)/pixmaps instead of /icons

fix T2834

unset release mode

20.0-rc NEWS updates

20.0-rc release

null efm icon eio handle in error callback

ref T2829

ensure that all mixer volumes resolve to the appropriate gadget icon

reported by Jef91 in #e

e screens/zones/randr - store randr2 id in xinerama and zones for lookup

so e has a bit of a problem. we mostly deal with zones, BUt these
zones come from our old xinerama code (this likely should just die
some time) and THIS code gets fed info from e's randr code. we
re-fill/modify as randr finds new screens or things get reconfigured.
thus zones adapt. the problem is now all our zone code really has a
hard time reverse mapping the zone back to where it came from -eg the
randr screen data. you literally can't do a whole bunch of things like
know if that zone was an internal laptop lid or an external screen, or
if it was rotated or even what the dpi is... as you ave no deasy way
to map it back other than by guessing geometry matches.

this fixes that by storing the randr screen id (which should be
unique) fromt he original src randr screen in the xinerama screen and
then in the zone. with this you can do a quick lookup in the e randr
data should you ever need to find the info. this should pave the way
for some other fixes/improvements, but without this they cannot be done.

@fix

use unsigned int for client focus tracking freeze state

there's no reason why this should ever be negative

Revert "clamp client to ABOVE layer when ending nocomp"

This reverts commit 8c6bbe0d44e7355a43e40062baed73ac013c03a6.

This Breaks Everything™.

clamp client to ABOVE layer when ending nocomp

should fix case where an "Always on Top" window toggles fullscreen

Fix null dereference issue

Summary:
@Fix
possible null dereference issue while referencing cs2->id. So added a condition
to check it

Signed-off-by: Vivek Ellur <vivek.ellur@samsung.com>
Reviewers: zmike

Subscribers: cedric

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

Fix memory leak issue

Summary:
@Fix

Signed-off-by: Vivek Ellur <vivek.ellur@samsung.com>
Reviewers: zmike

Subscribers: cedric

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

e: Fix realloc issue in e font module

Summary:
@Fix

Signed-off-by: Vivek Ellur <vivek.ellur@samsung.com>
Reviewers: zmike

Subscribers: cedric

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

e deskmirror - fix loss of mirror object from deskmirror

this fixes T2830

@fix

ibar - fix shutdown if ibar client menu is up

valgrind pointed out a referencing issue with the ibar client menu on
hover and if e is shut down while this is up. it ends up referencing
an already deleted ibar icon, so delete the menu objects that
reference the icon if icon is deleted.

@fix

check client comp data existence in x11 mirror visibility callbacks

enforce fullscreen client stacking during nocomp end IFF client is fullscreen

previous behavior would result in the nocomp window becoming stuck at a fullscreen
layer when ending nocomp, even if the client was no longer fullscreen

fix T2827

disable client menu stacking functions for unstackable windows

any client stacked higher than E_LAYER_CLIENT_ABOVE should never be
user-stacked

e/desk_wallpaper: Fix setting wallpaper from within Virtual Desk dialog

Summary:
It seems this was missed when doing the manager/container id removal
back in march.  The param validation was failing and causing the "Set"
button to not work in "Screen > Virtual Desktops > Click to Change
wallpaper".

Test Plan:
1. Open "Screen > Virtual Desktops > Click to Change wallpaper"
2. Click "Set"
  - Expect that a wallpaper dialog be shown
  - Before this patch nothing happens

Reviewers: zmike

Subscribers: cedric, seoz

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

Revert "force a software render in all cases when finalizing x11 client iconify"

This reverts commit 1a87463083fa9e343aa38b4c8824db0c92645b8e.

no longer necessary

toggle x11 client iconic/mapped state based on mirror visibility

in order to continue rendering an iconic client without breaking icccm,
it's necessary to map the client's window again and unset iconic state
whenever rendering is needed, then re-set states when rendering
stops

ref T2788

add e_comp_object_mirror_visibility_check()

can be used to check whether visible mirrors exist for a comp object

add counter and smart callbacks for comp object visible mirrors

ref T2788

emit ibox icon signals to the internal edje object of the e_icon

move client iconic hint setting into x11 hide function

e_hints should probably be merged into the x11 compositor entirely
at some point

break out x11 client visibility setting into util functions

set e_comp->gl for wayland compositors based on gl display bind success

e/ilist: Fix issue with multiselect returning wrong index

Summary:
I noticed this because the description dialog in "Apps > Startup Application"
was never getting properly populated.

Test Plan:
When selecting items in the "Apps > Startup Applications" config the
description should update if a "Comment" field is available in the
.desktop file.

Reviewers: zmike

Subscribers: cedric, seoz

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

e/conf_apps: Fix resizing of "Startup Application" dialog

Summary:
Currently the config dialog in Apps > Startup Applications does not properly
resized as one of the objects is not set to horizontal fill.

Test Plan: 1. Resize "Apps > Startup Applications", it should resize the list with the window

Reviewers: zmike

Subscribers: cedric, seoz

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

adjust efm multi-icon drag positioning by the current pan coords

fixes positioning when dragging multiple icons after scrolling

account for window position in efm window drag initial positioning

fix T2819

e/win_config: allow content of some dialogs to resize with the window

Summary:
Currently the dialogs in Winows > Window List Menu and Window Display are
resizable the their content does not resize.  This patch fixes that.

Test Plan: Open windows in window config dialog and ensure they can resize propertly

Reviewers: zmike

Subscribers: raster, cedric, seoz

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

remove note about window close crashing

fixed courtesy of the majestic compiler #warning for shm buffers

check for wayland egl extensions before starting a gl compositor

when running in software mode, evasgl performs no checking to determine
whether gl is available, leading to a crash due to missing extensions

Revert "Adds eglfs to Enlightenment"

This reverts commit ff1e7b50834e183f336fd427298981c6a9ebf7e8.

Revert "configure.ac: fixes wayland-only build"

This reverts commit 829a12ef86c6ab32b3f42a6440a5ed584a4fe10b.

Revert "e_signals.c: Inclues Ecore_DRM only if WL_DRM is built and not only if WAYLAND is built"

This reverts commit 96e304bc904c49df406b34108d7442835025349d.

Revert "wl_eglfs: enable gl acceleration, mouse, touchscreen and keyboard"

This reverts commit c99d46ae666a753c36313826747ef1f72dab07ca.

Revert "[HACK??] Fixes wayland-only build"

This reverts commit bc3464e98c94638cb4cbe0065fa512413ac51c91.

Revert "Fixes build complaining about missing ABOUT-NLS"

This reverts commit b39ec87150c9ccb06b3ab934a52ffa8b08cb29bf.

Revert "Don't load xwayland if it isn't compiled"

This reverts commit 496ae3746f94e2fab2c40d6bdf16e7f953ddba1a.

Don't load xwayland if it isn't compiled

Fixes build complaining about missing ABOUT-NLS

[HACK??] Fixes wayland-only build

wl_eglfs: enable gl acceleration, mouse, touchscreen and keyboard

e_signals.c: Inclues Ecore_DRM only if WL_DRM is built and not only if WAYLAND is built

configure.ac: fixes wayland-only build

Adds eglfs to Enlightenment

add wayland egl client support

definitely not a feature, I promise™

use e_comp->gl flag to set native surface state

e_comp_gl_get() is for determining if gl exists, not if it's being used

fix comp object native surface config checking for non-x11 compositors

add native surface (gl) clients to post render list

forgot to add this when I was redoing it for sw

ref 4d30674ab87001d6a1515908341d756d75a22a89

Revert "queue pending buffers for wayland shm clients"

This reverts commit 1ee825d29c2ca6d35da5f0860838d57019bca3f2.

this was fixing the wrong problem and was not functioning accurately
in any case

e exec tracker - fix tracking to update desktop files on efreet change

efreet will send change events meaning out desktop file icons are
invalid now. we have refs but content is junk. update these to new
desktop ptr handles by using the exec key to look up new ones on the
change event. this fixes ibar gainign a bunch of blank unknown icons
when any desktop files are touched on the system or user dirs

@fix

simplify client SMART/EXPAND maximize

calling e_client_resize_limit() here was misleading since the values
were never used

adjust non-x11 client's saved frame geometry when changing csd

this seems to be a more comprehensive solution for retaining previous
window sizes after toggling various csd-affecting window states in wayland