Revert "Revert "deskmirror - fix dangling reference to mirror by refcounting it""

This reverts commit 466dd8d57db6f281e7faebc4d532d5bc77e02dd7.

sorry - this bug is still there and this tracks references - the
current code DOES NOT DO THIS RIGHT.

No symbol table info available.
No symbol table info available.
out>) at src/bin/e_deskmirror.c:336
        mb = 0x1ed7220
        #3  0x00007f0e3c864a8d in evas_object_smart_del
(eo_obj=0x8000039ae0101eb8) at lib/evas/canvas/evas_object_smart.c:1016
        obj = <optimized out>
        sobj = <optimized out>
        s = 0x220d540
        i = <optimized out>
        __FUNCTION__ = "evas_object_smart_del"
(eo_obj=0x8000039ae0101eb8, obj=0x21432e0) at
lib/evas/canvas/evas_object_main.c:739
        proxy = <optimized out>
        l = <optimized out>
        l2 = <optimized out>
        texture = <optimized out>
        __FUNCTION__ = "_evas_object_eo_base_destructor"
...

and EXACTLY as before p *(mb->m) shows that m is full of garbage.
mb->ec is a garbage pointer, w amd h arte garbage coordinates etc. m
is not tracked. pretty simple. open and close a few windows - mayeb
switch some desktops, THEN restart and see the crashes. its easy
enough to reproduce

remove dirty callback for deskmirror clients upon creating visual

if another callback triggered the creation of a deskmirror visual while
the dirty callback was in place, a second mirror object would be created
leading to an orphaned mirror object which retained references to the dm
client and eventually resulting in a crash

simplify e_desk_show visibility unsetting, reject recursive desk flips

the currently visible desk for a zone is stored on the zone struct, so
iterating here is unnecessary. furthermore, at the time when a desk is hidden,
a client may begin receiving mouse events which could trigger a focus-set and
lead to another desk flip. at this time and only this time, the "current" desk
will be marked as not visible, and so this sort of desk show must be rejected

fix T2676

track focus time for x11 canvas focus, unset client focus on x11 canvas focus-in

in the case that the canvas window has just had focus set on it, apply this focus
and ensure that no client retains focus

this resolves a race condition where focusing the compositor canvas <-> client
extremely quickly would result in a client trying to steal focus when it was
not actually focused

a notable (but trivial) side effect is that now when flipping desks at high speed while using
mouse-based focus policies, the user is almost guaranteed to end on a desk which
has open windows on it

reject focus-set attempt if desk flip to client's desk was rejected

in the case of recursive desk flips, toggling a desk's visibility may
erroneously send queued evas events to the client's frame object, leading
to a focus-set (mouse-based focus models) which triggers a desk flip
inside the original desk flip. this "inner" desk flip is spurious and
should be ignored

add full damage for x11 override clients on resize configure event

it seems that the reported damage events upon resizing an override window
are not accurate, and so we must force a full damage here while avoiding a
render queue in order to ensure that the full contents of the override will
be rendered in the next frame

fix T2045

don't watch dirty smart cb for desmirror clients with unusable pixmaps

this seems to fix an extremely rare issue related to both deskmirror artifacts
and crashes in deskmirror during restart; I was only able to reproduce the crash
twice in the span of over an hour of testing and it seemed to disappear after
this change

Revert "deskmirror - fix dangling reference to mirror by refcounting it"

This reverts commit 5404adc54fb7f49b435e736e375bf7368321a1b0.

I'm strongly against adding refcounts/nullchecks to hide bugs which
are caused by failure to properly track object existence. let's try to avoid
this sort of thing in the future

menu: match file looked by code.

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

e - systray - add more sizes to icon hunt to fix missing icons

some systray indicator icons are not found because the sizes are not
in the list. fix this. this SHOULD actually use our existing efreet
icon theme finding to auto-switch file based on size changes.

deskmirror - fix dangling reference to mirror by refcounting it

add deskmirror refs to avoid segv on shutdown/restart due to mb->m
becoming a pointer to freed (now garbage) memory, so trace every
ref/unref and count them to get it right. crash gone!

e shelf - small missing set of signals - tell theme when inset scrolls

the shelf theme actually can't know if there is scrollable content in
that direction when a shelf item has autoscroll on and has content in
that direction. this is kind of a missing theme capability that is
incredibly useful if you want to do certain kinds of themes. this is
fairly minor and i'd consider ok for e20 release as we aren't even
doing alphas yet...

only check dirty smart callbacks on non-new_client deskmirror clients

remove dirty smart callback from deskmirror clients during client delete

remove pager mirror delete callback during pager win delete

add basic fullscreen window test

watch dirty smart cb for currently unredirected clients in deskmirror

these clients will be unable to create a comp mirror until this callback
is triggered

fix T2585

return false when failing to create a mirror during deskmirror obj check

add some smart callbacks for comp objects on redirect/dirty state change

force render queue and check for redirection on clients with 0x0 pixmaps

fixes an issue where resizing the current nocomp client could result in a
broken compositor until a render queue was triggered

add compile warning and block load of wl_text_input module

needs vkbd module merge

Add module "wl_text_input" for supporting *wl_text_input* and *wl_input_method* protocol in wayland.

Summary:
this patch allow to use virtual keyboard such as weston-keyboard.
it was tested in wayland verion 1.6.

Test Plan:
<prerequisite>
- Configure with --enable-wl-text-input
- edit configuration file, e.cfg to enable module wl_text_input.

1. run enlightenment as a wayland display server.
2. run weston-keyboard.
3. run weston-editor.

Reviewers: raster, Sergeant_Whitespace, devilhorns, zmike

Reviewed By: zmike

Subscribers: ManMower, Sergeant_Whitespace, cedric, jihoon

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

remove bz2 from dist

apparently people don't use this anymore or something

check compositor screen iface existence before deref in dpms

fixes crash in compositors lacking screen ifaces

disable xkbswitch module loading under wayland

NYI

feed mouse-up event during mouse down on menus

this fixes the case where the mouse is moved while the mouse is pressed
so that the selected menu item will continue to follow the mouse

fix T2127

redo fileman menus to only scan the first 100 items found

this was meant to be a quick way to navigate directories, not a fully
operational filemanager in a menu

fix T2462

do not relaunch relaunchable quickaccess windows during logout

fix T2492

set 'stopping' during logout action

this flag is used in some cases to determine whether to behave as
though enlightenment is stopping. in the case of logout, we are stopping.

end wl xdnd drags on the compositor canvas using the full dnd end codepath

fix T2652

don't set E_Client->changes.visible on successful show/hide

iirc this was a fix for something else that was going on early in
e19 development, now it creates issues when attempting to hide clients

update wayland readme with current information

enlightenment: Remove commented out reference to wl_screenshot Makefile

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

enlightenment: Remove wl_screenshot module from whitelist

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

only use deferred focus-setting on desk flip focus if client is not visible

unset E_Client->want/take_focus flags in hide callback

deferred focus should no longer be valid if a client has been hidden
before the focus-set could be triggered

fixes super fun infinite loop with desk flips

do not modify E_Client->visible value in show/hide callbacks

reject x11 NotifyVirtual and NotifyInferior mouse in events more accurately

these events are valid when moving from parent<->child window

fix T2674

temperature: let's finally allow the temp process to rest also when enlightenment die.

battery: let's finally allow the battery process to rest also when enlightenment die.

add event rect for deskmirror in pager/taskbar modes

ensures that events are properly triggered and consumed when this
widget is clicked on

fix T2645

simplify ibar focus function

explicitly grab compositor on ibar focus

make pager kbd/mouse grabs more effective in wayland compositors

use the same codepaths as x11 compositors, but explicitly grab the
compositor canvas

handle winlist kbd events more accurately in wayland compositors

key handlers here will pick up both wayland and drm engine type events,
so ensure that we only handle events matching the compositor canvas
window to prevent unexpected behavior

fix T2637

force xwl client to retain original x11 client's placed flag during swap

fixes race condition where xwl clients would sometimes erroneously place
at 0,0

enable shape cutting for xwl clients during flip

fixes xwl<->xwl dnd

remove some local aliasing of e_comp_wl global

replace existing usage of x/wl_comp_data with corresponding globals

add e_comp_x and e_comp_wl globals

constantly typing e_comp->x/wl_comp_data-> is tiring

delete data device resources from hash using client

fixes crash when currently-entered wl surface is destroyed

enlightenment: Replace old comp_type conditionals

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

enlightenment: Replace old comp_type conditionals

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

redo wayland shot init to account for async+slow global loading

previously it was possible for the shot module to init in a wayland
compositor and fail to grab the necessary globals, resulting in black
rects for all shots

add local event for wl_wl output to use when a global has been received

emit randr change event on randr init

wayland compositors have been failing to create any outputs for some
time as a result of not getting any randr events on init

enforce compositor canvas resizing/updating when emitting randr events

all compositors will want this to occur, so make it automatic

move x11 ee resize callback to comp_canvas, simplify x11 randr event handler

this should ensure that all compositors accurately perform resize-related
canvas operations

init null wl mime types array when new type is offered

reject xwl selection notify events which do not match our xwl property atom

don't toggle screensaver twice on x11 xdnd selection owner change

only delete current drag on x11 xdnd selection loss if x11 owned selection

convert E_Comp_Wl_Data_Source->mime_types to Eina_Array

this data struct fits the usage of the data more effectively

track button mask for all drags, only end drag when mask has depopulated

previously, beginning a drag with the left button, then pressing and
releasing another button would result in the drag terminating without
the original button being released

force sending of mouse button release on xwl drag end

seems to be necessary for the client to end its drag

make wl client mouse button handler fn external

break out wl client motion sending, force motion events during x11 drags

weird, but fixes xwl drag positioning

fix xwl pixmap unaliasing during client deletion

no more crashing, hooray!

enlightenment: Remove legacy unused e_pixmap_image_draw_done function

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

don't iterate damage rects during render in wayland compositor mode

this is a no-op that spins for no reason

force copy rendering for argb wayland clients

wayland protocol for shm handling is broken for now and I can't fix
real issues because of the constant crashing when buffers disappear
randomly

use unmodified client geometry for new xwl clients during first surface commit

fixes a race condition where a configure request occurring before the xwl
client swap could fail to take effect due to xwayland bug where buffer size
is reported incorrectly

do not add render update on resize fail due to unusable pixmap in wl

this will not have any effect

enlightenment: Update README.wayland to reflect things that do not work yet

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

modules/battery: Remove assign without effect

CID: 1316024

e_widget_filepreview: Remove assign without effect

CID: 1316023

finish xwl dnd operation immediately if drop occurred somehow

don't create pipe for x11->wl clipboard selections

this is never used

only create fd handler for wl clipboard data source if fd is passed

don't remove wl data source listener when xwl owns the selection

crash-- since xwl has no listener

add separate log domain for xwayland

xwayland: Add missing EINA_UNUSED for unused function parameters

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

Fix copy-paste error in comment, This page is tasks not connman

Summary: No functional change

Reviewers: zmike, raster

Subscribers: cedric

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

finish off x11<->wl clipboard support

a lot of this functionality is reused from dnd. basic selection owner
management in x11 and slapping bytes down a pipe to the wl client;
a bit laggy when pasting to wl clients sometimes, need to spend more time
debugging that...

 #Kansas

close xwl dnd pipe on operation end

make e_comp_wl_data_device_keyboard_focus_set() external

make wl clipboard source create/unref external

don't crash in wl compositor when data resource is destroyed in multiple listeners

only update wl dnd surface client if a wl drag is active

track xwl clipboard owner

use current ecore-x time when setting selection owner for xwl dnd operations

according to icccm spec, it is not recommented to use currenttime here

add comp util function for determining whether xwayland is active

account for state in xwl property change handler

we only care about events where the property was deleted here

Merge branch 'devs/bu5hm4n/mixer'

This merge fixes problems which where discovered by the mail "[e-users] alsa NOT mixer?"

mixer: Fix segfault when backend does not have any sink

mixer: only list playbackdevices if they are _real_ playbackdevices

mixer: update internal volume and call the changed event

mixer: get volume after setting channels

mixer: this if does not make much sense

README.Wayland: Update to latest configure options and mention xwayland

Many of them have been out of date. Remove what is no longer needed and
mention XWayland.