when maximizing a wayland client with csd, use window geometry

wayland clients which have csd must be resized according to window geometry,
not client (surface) geometry. this is somewhat tricky to handle because x11
clients which have csd work the exact opposite way and must continue to be
managed using client geometry

this is not my ideal solution for this issue, but I can't think of a
better one at this time which fully fixes wayland client maximization

reject frame theme changes for clients which have CSD

only apply frame geometry deltas for CSD if the CSD region exists

ref e931e86161d7f419e44652b533fea4dc9b16241b

set client input rects more accurately, accounting for SSD

enforce cw->obj pass_events state more accurately

changing the edje file of cw->shobj can change the pass_events state
of its child object, cw->obj

clip client input rect to client geometry, not window geometry

when using csd, the window geometry will be smaller than the surface,
resulting in an unusually small input region

defer all module error dialogs

due to the (current) synchronous creation of wayland surfaces in efl,
creating these dialogs during startup will result in a crash. the only
way to prevent this and still display the dialogs is to wait a bit and
show the dialogs after init has completed

completely remove access module

as promised almost a year ago, this module has been unmaintained to
the point where it cannot function in any way, and attempting to keep
it in the tree is a futile effort

this module can now be found at devs/discomfitor/e_module-access.git

ref 77717523f3e6c327ddd797ad064a97c467e6f90b

remove dead modules from whitelist

unconditionally show opengl option in compositor settings

enlightenment is not always running in x11, so making this option
dependent on xlib will break other environments

add hacks to work around pulseaudio+xwayland integration deadlocks

enlightenment is (I think) the first wayland compositor to run with
in-process pulseaudio integration for audio playback and not just mixer
support. hooray.

this results in a fun issue: if DISPLAY is set, as it must be for x11
clients to function, pulseaudio will unconditionally attempt to use a
blocking socket connection to create a connection to the running xserver.
the only exception here is if x11 support has been compiled out of pulseaudio,
but probably no distro will do that ever.

so, what happens when the compositor thread tries to create a socket connection
to the xserver that the compositor thread has not yet started? absolutely nothing.
forever.

the easiest solution which continues to provide the key press sounds that everyone
loves is to ensure that the pulseaudio connection is created before DISPLAY is ever
set, namely in the xwayland module init.

this will now occur automatically now in the case when the mixer module detects
pulseaudio support.

TL;DR: don't disable mixer module if you use xwayland

require efl >= 1.16 for wayland support

the fact is that nobody developing this has tested with 1.15.x recently,
and almost none of the changes have been backported to that branch. better
to require a newer version than to present a failing user experience

make comp config unresizable...again

ref 5178197f602da6cff94e03f20c522d5b1a631ae7

do not use ecore-x functions in scale config while running in wayland

append new keyboard resources for wayland focused-client to the focused list

fixes all cases where applications would fail to receive focus initially
and then be permanently broken

fix typo in wayland keyboard resource unbind list management

do not use x11 input windows for grab dialogs when running in wayland

Fix e_menu trying to swallow an object from a different canvas

_e_menu_realize was trying to swallow the menu->container_object into
the menu->bg_object, but the menu->bg_object was being created on the
compositor canvas, and the container object was being created on the
e_comp->elm_win.

With recent EFL changes, this causes an abort inside Evas.

To fix this, set the menu->evas to be the Evas from the e_comp->elm_win.

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

update csd using deltas of previous values

when applying new csd to a window which already has csd, the previous
csd must be removed in order to apply any new csd offsets in order to
avoid unwanted moving/resizing

e-comp-wl-input: don't leak the initial keymap.

Summary: remove xkb_map_ref() for keymap in _e_comp_wl_input_keymap_update().

Test Plan: N/A

Reviewers: devilhorns, zmike

Subscribers: ManMower, cedric

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

mouse_config: fix accel threshold and numerator limits.

man xset =>  If the `threshold' parameter is provided and 0, the
`acceleration' parameter will be used in the exponent of a more
natural and continuous formula, giving precise control for slow motion
but big reach for fast motion, and a progressive transition for motions
in between.  Recommended `acceleration' value in this case is 3/2 to 3,
but not limited to that range.

queue pending buffers for wayland shm clients

due to how deferred rendering works, it's possible for a client to
send a second buffer before enlightenment has rendered the first one.
in this situation, it seems that the best solution is to queue successive
buffers (frames) and pop the queue after each render

ref T2784

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

since ICCCM requires that clients be unmapped while iconified, it's necessary
for the compositor to perform one last render prior to the unmap in order to
ensure that mirror objects will still appear as expected. this render must use
the pixmap buffer data in order to avoid timing issues due to async/deferred
rendering, and it is only necessary for the case of clients rendering with a
native surface

fix T2788

clear x11 client pixmap when returning from iconic

ensure pixmap remains usable during the entire hide process

check systray dbus name existence before attempting to recover session

in the case that a system error occurs the dbus method for returning the
session name can fail, leading to a null return

CID 1327399

use dbus unique name for tracking systray sessions

the env var DBUS_SESSION_BUS_ADDRESS may not be set for all platforms
where enlightenment+dbus are usable, so it's necessary to get the dbus
id to ensure that this value exists and is properly tracked

reject deleted clients from x11 sync alarm event callback

E_Client->comp_data is null after a client has been deleted, so
attempting to handle events which require the dereferencing of this
pointer after a client has been deleted will result in a crash

these events should be rejected after delete regardless, since at this
time the compositor has stopped handling events for the client

ref f42c6aa1871cca6c6ffb39b65e2bfa3815bff35c

e - comps sync - ec client was null - i think, so protect against crash

i got a crash here and the bt was broken and i couldnt check if
_e_comp_x_client_data_get() returned null, but it's the only thing
that would make sense, so protect against this to avoid a crash. as
this was a one-off, i can't find out more,

@fix

systray: search for the best available resolution of a icon

For example in telegram the first icon which was transmitted was in a
pretty bad resolution, the new code searches the best res.

cache systray items for each dbus session

the current spec does not directly require any behavior from clients
when a systray host, leading to an issue where clients do not re-register
their items when a new host appears

when using an in-process systray watcher, as the current implementation does,
the best choice for maintaining consistency for systray items across restarts
is to cache them according to the current dbus session. the process of setting
up the item will validate it on subsequent restarts, and changes to the session
will clear the cache

fix T2786

 e_flowlayout: typecast return value to int

Summary:
return value is small enough, so there is no risk in casting from unsigned int to signed int.

CID: 1267211

Reviewers: devilhorns, raster, cedric, zmike

Subscribers: seoz, sachin.dev

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

trap shaped x11 clients and prevent compositor blocking with high rect count

in the case where a shaped window with many rects exists, there is a high
probability of the damage rect count being huge, leading to massive blocking for
each frame as the compositor attempts to fetch all of these rects from the xserver.

instead, the compositor can shortcut this by forcing a full-window damage any time
the rect count is sufficiently high, trading a blocking socket operation for some
amount of (potential) overdraw.

testing in affected scenarios has shown huge improvements: where previously the entire
compositor would lock up, things work as expected now

see https://bugzilla.mozilla.org/show_bug.cgi?id=1214746 for a sample case

do not return when x11 damage event returns no rects

ensure first_damage flag is set in all cases

pulseaudio: pass big data as pass by reference

CID: 1308394

enlightenment: Make sure we have a zone before calling
_bgpreview_viewport_update

@fix CID1324753

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

enlightenment: Use 'fabs' to compute absolute value of floating point
types

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

enlightenment: #if 0 out an unused variable

NB: The function which actually uses this variable is also #if 0'd out

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

enlightenment: Use 'fabs' to get absolute value of floating point types

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

enlightenment: Fix E_CONFIG_LIMIT to use the proper min & max values

e_config->backlight.idle_dim is actually an unsigned char in the
structure, so use a 0 & 1 for min & max

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

enlightenment: Add missing field initializers for Eina_Rectangle

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

enlightenment: Add missing field initializers for Evas_Point

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

enlightenment: Add missing EINA_UNUSED for unused function paramaters

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

enlightenment: Add missing EINA_UNUSED for unused function parameters

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

enlightenment: Add missing EINA_UNUSED for unused function paramaters

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

subtract x11 client damage region even when forcing full override damage

fix T2785

force full damage for first frame of x11 override clients

it seems that damages for popup windows in some applications,
namely blink-based browsers, are reported incorrectly, resulting
in sometimes having a blank window

Fix make distcheck

mixer: start app from bin directory

before the app was installed into the modules directory, now its
installed to the bin directory

mixer: install icon and .desktop file

mixer: install app to bin directory

use correct param ordering for setting wayland client frame geometry

function uses l,r,t,b and not x,y,xx,yy

fix T2624

remove duplicate surface size update for wayland clients during commit

this has already happened in e_comp_wl_surface_attach()

don't forget wl_desktop_shell's header in tarball

Summary: Merge branch 'master' of git://git.enlightenment.org/core/enlightenment

Reviewers: zmike

Subscribers: cedric

Maniphest Tasks: T2776

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

e mixer - be more robust if pulse exits/crashes and def sink is null

ensure there are protections everywhere for when default sink is null
- mixer did crash when pulse crashed and a popup was to be shown...

explicitly disable screensaver for wayland urgency wakeups

make wake-on-urgent apply for all cases of window urgency

in the case that an urgent window is set as urgent again, wakeup should
still occur

apply _GTK_FRAME_EXTENTS for xwayland clients

not entirely sure this is worthwhile but may as well

Intl: fix potential stack buffer overflow.

CID 1298051

@fix

Client: fix potential out of bounds read.

This is an unsigned int underflow that could lead to an out
of bounds read.

CID 1291837

@fix

enlightenment: Update wayland readme

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

don't crash desktop gadget systrays when adjusting image sizing

desktop gadgets (gadman) do not have a shelf, so derefing that pointer
without checking will guarantee a crash

e comp data - actually ... NO visibility effect by default is right

so a bit of a config wipe had my comp config reset (some dev
shenanigans) and i got to see what current e "out of box" config is -
and it was horrid for menus. visibilitiy effect was broken. vertial.
default was none - always. forever. like empty. so go back to that. in
fact why do this visibility effect? that was the point of having a
different shadow in the theme - each has its own look + effect. this
makes things far more complex...

when removing VERTICAL maximize state, also remove LEFT and RIGHT states

LEFT and RIGHT are supersets of VERTICAL which serve no purpose on their own,
so remove them when removing VERTICAL

fix T2770

e comp - default config was rather broken - fix

default config didnt select menu style shadow - was selecting
vertical/still which meant animation was rather horrible. fix.

only use fast-path pixmap size updating after a resize for x11 clients

updating these values after other operations will result in the pixmap size
being set inaccurately, leading to rendering inconsistencies

update wayland readme

repopulate app list dialogs when efreet deskop cache updates

should improve usage when installing/uninstalling apps

attempt to load default e_bindings if initial load fails

do not set frame geometry on frame-possessing wayland clients

throw an ACK when trying to set comp frame geometry if frame exists

clear menu cache when saving a menu file

force regeneration of menus after editing them

ref T2271

make icon selection dialog resizable in .desktop editor

ref T2271

move configure init after intl init has completed

gettext calls will not return the correct language string until after
setup has completed, leading to untranslatable strings

fix T2760

clear the tiler for input region after setting to comp object.

Summary:
there is no need to set repeatedly input region even if it's already applied.
and this patch remove the code to clear tiler from client's unmapped case.
this fixes that tiler for input region is removed before applying it to comp object in case client is unmmaped yet.

Reviewers: devilhorns, zmike

Subscribers: cedric

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

Add support for the ISO 8601 date format to the clock module

Reviewers: zmike

Subscribers: cedric

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

 e fm: Use correct string length when copying path.

 Sometimes, trying to go to the parent directory wouldn’t work and end up showing some garbage. Unfortunately, the code ended up not NULL-ending the path, which made searching for the path separator buggy.

Add "(drag only)" to title for drag only edge bindings

Reviewers: zmike

Subscribers: cedric

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

 e menus: Fix invalid comparisons.

 Fixes CID 1267215 and 1267216.

fix systray to work with spec-breaking apps, eg. steam

according to the StatusNotifierItem specification, applications
register "service org.freedesktop.StatusNotifierItem-PID-ID" on the
session bus, and then "must register the unique instance name
to the StatusNotifierWatcher".

some applications, such as steam, instead register the path that they
will run on (/org/ayatana/NotificationItem/steam) and then expect the
watcher to register the method call's send id bus: this is totally bogus.

to catch this, when registering the new item the enlightenment watcher must
first determine if the item is spec-conforming. if yes, proceed as normal.
if no, pretend the application knows what it's doing and try to make things
work as expected anyway

for more details, read the full spec here
http://www.freedesktop.org/wiki/Specifications/StatusNotifierItem

fix T2763

move client post-update adding into render function

in some cases, it's possible for a client which expects to render on
the next frame to actually render on the frame after. in these cases,
the compositor must not clear the pixmap image until after the render
has occurred in order to avoid inaccuracies. for this reason, the best
place to flag a client for post-render work is at the time of the client's
render

ref T2762
ref D3120

unify comp render returns for argb and non-argb clients

fix comp render return value for failed draws of non-argb clients

fix return value of comp render for wayland clients

add RENDER_DEBUG for forced renders when setting dirty flag

should remove a frame resource from pending.frames list also

Summary: Signed-off-by: Boram Park <boram1288.park@samsung.com>

Reviewers: stefan_schmidt, gwanglim, raster, zmike, devilhorns

Reviewed By: zmike, devilhorns

Subscribers: cedric

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

md5: fix wrong sizeof argument (SIZEOF_MISMATCH)

Summary: fixes CID: 222382

Reviewers: raster, zmike, cedric, devilhorns

Reviewed By: devilhorns

Subscribers: cedric, seoz, sachin.dev

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

unset release mode

20.0-alpha NEWS updates

20.0-alpha release

always apply all randr changes after a screen event

fixes case where plugging an external monitor would cause the screen
to expand onto that display without the display being effectively usable
until after a restart

wl_desktop_shell: return void

Summary:
_e_input_panel_cb_surface_get() returns void.

Signed-off-by: Boram Park <boram1288.park@samsung.com>
Reviewers: zmike, stefan_schmidt, gwanglim, raster, devilhorns

Subscribers: cedric

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

remove remote colorclass editor usage

match current elm function signatures

merge E19 branch NEWS items

update wayland readme

redo wayland client buffer referencing

the previous methodology was effectively:
attach -> ref(new buffer) x2 / unref(old buffer) x2
...
...
attach -> ref(new buffer) x2 / unref(old buffer) x2

this resulted in buffer management failures and crashing. now the
buffer gets 1x ref before render and 1x unref after render, ensuring
that the lifetime is accurate (assuming evas doesn't lie to us)

now we still have random crashing during resize, but not as much as
before

redo CSD for wayland clients to use comp object frame geometry

this fixes issues with client placement and is a more native solution
than the previous hacks

never mark wl client pixmaps as dirty or refresh them

this has no meaning for a wl pixmap in the current implementation

free x11 damage region rects

do not roundtrip for x11 client message DBG if it will not be visible

also free fetched atom name

unset E_Client->internal_elm_win before deleting the client in e_win

==24509== Invalid write of size 8
==24509==    at 0x502D00: _e_elm_win_trap_del (e_win.c:39)
==24509==    by 0x509BFC2: _elm_win_evas_object_smart_del (elm_win.c:1886)
==24509==    by 0x91F4643: evas_obj_smart_del (in /usr/lib/libevas.so.1.15.99)
==24509==    by 0x91F5B5C: evas_object_smart_del (evas_object_smart.c:1021)
==24509==    by 0x91E9107: _evas_object_eo_base_destructor (evas_object_main.c:739)
==24509==    by 0xE54A8A3: eo_destructor (in /usr/lib/libeo.so.1.15.99)
==24509==    by 0x5086715: _elm_widget_eo_base_destructor (elm_widget.c:5744)
==24509==    by 0xE54A8A3: eo_destructor (in /usr/lib/libeo.so.1.15.99)
==24509==    by 0xE5443EC: _eo_del_internal (eo_private.h:221)
==24509==    by 0xE5443EC: _eo_unref (eo_private.h:295)
==24509==    by 0xE5443EC: _eo_do_end (eo.c:546)
==24509==    by 0x4D5B1A: _e_obj_dialog_free (e_obj_dialog.c:125)
==24509==    by 0x4D61FB: e_object_free (e_object.c:152)
==24509==    by 0x4D61FB: e_object_unref (e_object.c:152)
==24509==    by 0x4EDC54: _e_sys_logout_after (e_sys.c:750)
==24509==    by 0x4ED7AC: _e_sys_action_do (e_sys.c:925)
==24509==    by 0x4EE348: e_sys_action_raw_do (e_sys.c:311)
==24509==    by 0x4EE43F: _e_sys_comp_done_cb (e_sys.c:66)
==24509==    by 0x6097348: _edje_emit_cb (edje_program.c:1476)
==24509==    by 0x6097348: _edje_emit_handle (edje_program.c:1405)
==24509==    by 0x60924EE: _edje_message_queue_process (edje_message_queue.c:787)
==24509==    by 0x60926A6: _edje_job (edje_message_queue.c:154)
==24509==    by 0xCC5087A: _ecore_job_event_handler (ecore_job.c:121)
==24509==    by 0xCC4B204: _ecore_call_handler_cb (ecore_private.h:390)
==24509==    by 0xCC4B204: _ecore_event_call (ecore_events.c:565)
==24509==    by 0xCC52AE7: _ecore_main_loop_iterate_internal (ecore_main.c:1927)
==24509==    by 0xCC52CD6: ecore_main_loop_begin (ecore_main.c:983)
==24509==    by 0x4383F4: main (e_main.c:1047)
==24509==  Address 0x14fb1a28 is 1,176 bytes inside a block of size 1,352 free'd
==24509==    at 0x4C2A65B: free (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==24509==    by 0x4D61FB: e_object_free (e_object.c:152)
==24509==    by 0x4D61FB: e_object_unref (e_object.c:152)
==24509==    by 0x502CED: _e_elm_win_trap_del (e_win.c:37)
==24509==    by 0x509BFC2: _elm_win_evas_object_smart_del (elm_win.c:1886)
==24509==    by 0x91F4643: evas_obj_smart_del (in /usr/lib/libevas.so.1.15.99)
==24509==    by 0x91F5B5C: evas_object_smart_del (evas_object_smart.c:1021)
==24509==    by 0x91E9107: _evas_object_eo_base_destructor (evas_object_main.c:739)
==24509==    by 0xE54A8A3: eo_destructor (in /usr/lib/libeo.so.1.15.99)
==24509==    by 0x5086715: _elm_widget_eo_base_destructor (elm_widget.c:5744)
==24509==    by 0xE54A8A3: eo_destructor (in /usr/lib/libeo.so.1.15.99)
==24509==    by 0xE5443EC: _eo_del_internal (eo_private.h:221)
==24509==    by 0xE5443EC: _eo_unref (eo_private.h:295)
==24509==    by 0xE5443EC: _eo_do_end (eo.c:546)
==24509==    by 0x4D5B1A: _e_obj_dialog_free (e_obj_dialog.c:125)
==24509==    by 0x4D61FB: e_object_free (e_object.c:152)
==24509==    by 0x4D61FB: e_object_unref (e_object.c:152)
==24509==    by 0x4EDC54: _e_sys_logout_after (e_sys.c:750)
==24509==    by 0x4ED7AC: _e_sys_action_do (e_sys.c:925)
==24509==    by 0x4EE348: e_sys_action_raw_do (e_sys.c:311)
==24509==    by 0x4EE43F: _e_sys_comp_done_cb (e_sys.c:66)
==24509==    by 0x6097348: _edje_emit_cb (edje_program.c:1476)
==24509==    by 0x6097348: _edje_emit_handle (edje_program.c:1405)
==24509==    by 0x60924EE: _edje_message_queue_process (edje_message_queue.c:787)
==24509==    by 0x60926A6: _edje_job (edje_message_queue.c:154)
==24509==    by 0xCC5087A: _ecore_job_event_handler (ecore_job.c:121)
==24509==    by 0xCC4B204: _ecore_call_handler_cb (ecore_private.h:390)
==24509==    by 0xCC4B204: _ecore_event_call (ecore_events.c:565)
==24509==    by 0xCC52AE7: _ecore_main_loop_iterate_internal (ecore_main.c:1927)
==24509==    by 0xCC52CD6: ecore_main_loop_begin (ecore_main.c:983)
==24509==    by 0x4383F4: main (e_main.c:1047)