_GTK_FRAME_EXTENTS changes using a property, not a client message

reject identical calls to e_comp_object_frame_geometry_set()

e - efm fix. fix eio error handler that is called after eio cancel

so you have to have an error cb - so make it empty as even after a
canel, it is called, and thus accesses an invalid icon we've freed.

efm - handle cancel of async eio if icon is freed and eio still active

@fix

force pixmap clear when hiding an x11 window

failure to refetch this upon remapping the window will result in
a broken pixmap which cannot be rendered

don't call e_client_unignore() when un-withdrawing an x11 window

these clients were not ignored on creation, so they've already triggered
all the relevant events

bump required efl version

map/unmap x11 client windows when toggling iconic state

ICCCM 4.1.4

initialize pixmap size vars in comp resize interceptor

these are never uninitialized when they are used, but this
is not obvious to the reader

CID 1324956, 1324956

set x11 supported atom array size correctly

CID 1324958

fix handling of x11 ICCCM WithdrawnState

according to ICCCM 4.1.4:
Only the client can effect a transition into or out of the Withdrawn state

withdrawn windows cannot be shown under any circumstances. the best that can
be done is to try mapping the window and hope it decides to appear.

to prevent any inadvertent showing of the window before it leaves the
withdrawn state, we play games with the E_Client->ignored flag in order
to skip client evals until we get notified that maybe we want to stop
skipping those evals

ref T2745

check warp_client existence during pointer warp before dereferencing it

ref T2679

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

This reverts commit 2373b69c61d7d6ab74675a4facff54601831ce9a.

stop crashing on restart

add csd case for compositor frame adjust messages

not 100% perfect, but close enough

move compositor frame adjust messages to pixels_get callback

native surfaces don't call e_comp_object_render()

set comp object client_inset.calc based on frame_object existence

block border-affecting client functions for csd clients

block border-affecting entries from showing up in client menu with csd

fix e_shelf_position_calc() to not set struct members before applying them

fix T2743

add util function for determining if a client is allowed to have a frame

returns false if mwm borderless is set or if csd exists

fully support _GTK_FRAME_EXTENTS

gtk apps set an atom which provides information about the area
where non-window content (eg. shadows) may be drawn; this area
must not be used in placement calculations.

the easiest method for implementing this functionality was to add
a case to the compositor geometry interceptors which effectively
flip the client struct geometry values such that the E_Client->client
is outside of the more commonly used E_Client->x/y/w/h

fix T2744

broadcast support for _GTK_FRAME_EXTENTS

this is not actually supported yet, so behavior of windows using this
feature will be more wayland-like, eg. geometry determined by area
of window+shadow

fix T2744

Creating wl_shm global object before creating other global objects

Summary:
  It's more useful for client to bind wl_shm before receiving other global
  object's events. Then, App can quickly prepare some buffers. i.e. cursor,
  etc.

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

Reviewed By: devilhorns

Subscribers: cedric

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

add note for starting wl-x11

fix systray implementation of notifier watcher (StatusNotifierItem)

previously, this would throw dbus errors (or not) and then do nothing in
many cases. now it manages bus/path names more effectively and falls back
to binary image data when an icon path is not available

fix T2626 and probably some others

improve client exe_inst creation

ref T2679

add weekeyboard edj files

attempt to maintain currently-selected kbd layout when modifying kbd list

the previous behavior would just set up the new layouts, resulting in
the first layout in the list being applied. now it should be the case
that if the current layout has not been deleted, it will continue to
remain in effect; alternatively if the current layout has been modified,
it's now more likely to be picked up and used

only init E_EVENT_XKB_CHANGED during xkb init if xkb init hasn't already occurred

allows xkb init to be called repeatedly

fix shot module compile without x11 support

update xkb settings and send xkb update event when calling e_xkb_layout_set()

resolves issue where setting a specific kbd would fail to make settings permanent
as well as not propagating the kbd change to the rest of enlightenment

fix T1810

calculate comp object visibility regardless of pending damages

in the case of clients on non-visible vdesks, this improves the reliability
of mirrored rendering

do not emit client iconify signal when reapplying compositor theme

this has no effect other than breaking the animation counter for the
client and preventing it from being deleted

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

This reverts commit 6bef668a8fb37104f1e9e4bb1bb0dd379cb68ef5.

redo client maximization when a non-overlap shelf changes geometry

this fixes the annoying case of needing to remaximize all windows when
a shelf resizes in order to account for new geometry

clamp client geometry to zone during geometry calc

somehow it was possible for client sizes to overflow the zone geometry here
which would end up breaking maximization limits and result in clients
not respecting various geometry boundaries

only unpopulate a shelf when applying new settings if the shelf won't be recreated

block gadcon thaw on unpopulate when gadcon is deleted

saves a huge amount of unnecessary calc time

unset E_Client->want/take_focus flags during client eval

ref 4a33cd7ba0c51b911b1d7845411d2b70e01a3d55

e config - cache sizes in config don't actually work, so remove

remove several config vars in advanced performance that frankly don't
work (unless you change them int he dialog). they are not set up on
startup or not even used at all. remove things that don't work anymore!

@fix

e - fm - flush all evas caches to get rid of open file handles b4 umount

this should fix open file handles on unmount by flushing caches first.
not great, but works. long-term have evas not keep file handles open
for 0 refcount cached items.

apply vertical maximize algorithm for LEFT/RIGHT maximized windows

move focus setting on restart into desk restore function

the focused client on restart needs to be set after desk states
have been restored in order to avoid focusing the wrong client

only set focus on clients which are visible during client eval

focusing a client will automatically uniconify and desk flip, so
setting focus on a hidden client should be avoided during eval since
these focus-sets are not "user triggered"

this fixes issues where clients could randomly grab focus from other
desks and also restores expected behavior when restarting e on an
empty vdesk

do not return non-visible clients for e_client_under_pointer_get()

a hidden client should not be considered "under the mouse" since its
geometry is undefined until it becomes visible again

fix T1069

remove backlight update calls from backlight module

these are no longer necessary since the backlight value should be
kept up-to-date by the core backlight infra

add eeze watch for backlight events

when using "system" backlight mode, ensure that events are processed
as they occur so that the available backlight level is in sync with
the actual backlight level

fix T2255

only lower x11 layer windows for layers which have windows during init

decode .desktop link and directory uris in fileman menus

fix T1922

always set alpha for internal wins

due to the presence of color classes on internal windows, e must
ensure that users who set translucent colors for window base objects
get transparency as expected on those parts of the window. it's impossible
to determine in advance whether alpha is needed, so it's best to just
set alpha in all cases

fix T2050

make bgpreview widget work with panoramic wallpapers

send FLOAT_SET edje messages during desk flip

panoramic wallpapers are created to expect this type of message

ref T2219

distribute weekeyboard edj files

creating these during dist is problematic due to filename length limits
so they will need to be precompiled

Add initial module of weekeyboard as a wayland virtual keyboard.

Summary:
Depends on D2275

this module is a virtual keyboard used in wayland.

Test Plan:
<prerequisite>
- Configure with --enable-wl-weekeyboard.

1. run enlightenment as a wayland display server.
2. run weston-editor which has text entry using wayland protocol.

Reviewers: devilhorns, raster, ManMower, jonc, zmike

Subscribers: bryceharrington, jonc, jihoon, cedric, ManMower

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

add function to blank out a client's rendered image

when working with Extremely Serious effects, it may be the case that
a user is rendering at such an advanced level that any attempt by
enlightenment to perform rendering will be like a child trying to
reproduce a masterpiece of art while using fingerpaints

https://www.youtube.com/watch?v=tY6qag5KFx0&hd=1

always send CLIENT_SHOW event when clients are shown

I think it was the case for <E19 that uniconifying a client would
not send a show event, but this is not ideal and so the event should
still be sent

remove unused variables in ilist widget

set comp object mirror alpha flag more accurately on creation

add smart callback for comp object damage calls

related to the previous commit, this will allow api users to determine
when a client needs to re-render with the least amount of overhead

add function for overriding the native surface of a client's comp object

it's a pretty trivial thing to hand-composite a client, so this will
allow someone to do something like render out a gaussian blur to an fbo
using a client's texture and then render the fbo onto the compositor
canvas with minimal overhead

add smart callback for comp object hiding animation begin

it's impossible to determine this at the time of calling without adding
some sort of callback here; edje signals are deferred, meaning that
an interested user will not be able to check the state of a client
when it begins to hide

create util function for enforcing comp object image alpha setting

this can be set in a couple places, so move all the logic into a single
caller to ensure correct setting

alias x11 pixmaps to parent windows during reparent

failure to allow pixmaps/clients to be retrived by parent window will
result in api users being greatly inconvenienced after a reparenting has
occurred

add function for getting x11 pixmap from E_Pixmap

add pre-render callback list for compositor canvas

someone doing fancy effects might want to add a callback to do various
tasks only when a render is about to occur

enable depth+stencil bits for x11 gl compositor canvas creation

rename E_Comp_Grab_Cb to E_Comp_Cb

this is a super generic global callback type which will never be used
elsewhere, so rename it to make it potentially useful in other places

enforce gathering client moveinfo before beginning move/resize action

add ilist method for setting disabled state

ilist widget adds items in a deferred queue, resulting in attempts to
immediately toggle the disabled state having no effect. now there is
a flag so that items added after the disabled flag have been set will
have the correct state

fix T2730

unset client hidden flag on config dialogs when activating from another vdesk

fix T2731

delete notification mirror objects upon deleting original popup

fix T2722

e winlist focus alt-tabbing - fix focus list changes while tabbing

this should fix T2700

@fix

ensure that non-fullscreen nocomp clients have their layers re-set

in the case where a window is fullscreen without having the 'fullscreen'
flag set, the previously-used layer must be reapplied upon nocomp end
in order to avoid breaking the compositor

set 'starting' flag for desk show in e_zone_desk_count_set()

as a result of earlier changes which prevented recursive desk flips,
e_desk_show() now rejects some desk show calls which are invalid such as
a show where the "current" desk does not have the visible flag set. this
behavior is overridden in the case of startup, which is functionally the
same effect as changing the desk count

fix T2717

e comp - set rects to null to silence compiler warning

even though n is 0 and we don't use rects, compiler doesn't know, so
to avoid noise - set to null so we can focus on warnings of real errors.

e mixer - shut up coverity CID 1323098

add extreme hacks to enforce nocomp stacking policies

if windows set to "Always on Top" exist while the option to allow
windows over fullscreen windows is enabled, enabling nocomp will
result in the above windows being stuck over the nocomp window

instead, force the nocomp window to be the top-most window in all cases,
and then put it back if another object appears on the screen over it

fix T2703

ensure that normal clients are stacked below overrides when raising

when a client is set to "Always on Top", it will be on the same layer
as override clients. this can cause strange stacking and mouse eventing
in cases where these windows occupy the same space and the normal client
is stacked over the override

unshade clients when activating

fixes issue where an activated client on another vdesk which was shaded
would appear to be unshaded but was, in fact, shaded

force-disable refocus during desk flip on client activation

this prevents an infinite focus loop where focus will be constantly
reapplied between multiple windows if the activated window is not the
refocus window

block forced mirror render for clients with native surfaces

wl_text_input: initializes input_method's context when destroy context.

Summary: input_method's context set to NULL when context is freed.

Test Plan: N/A

Reviewers: zmike, devilhorns

Reviewed By: devilhorns

Subscribers: cedric

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

wl_text_input: remove duplicated code.

Summary: remove redundant code.

Test Plan: N/A

Reviewers: zmike, devilhorns

Reviewed By: devilhorns

Subscribers: cedric

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

wl_text_input: initializes global handle after destroying.

Summary: variable for handle should be initialized.

Test Plan: N/A

Reviewers: zmike, devilhorns

Reviewed By: devilhorns

Subscribers: cedric

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

Binding input method will be allowed to wl_client that is created by enlightenment.

Summary:
Added code to check whether wl_client's pid is equal to enlightenment's.
this means, only enlightenment modules can bind wl_text_input interface.

Reviewers: zmike, devilhorns, ManMower

Reviewed By: devilhorns, ManMower

Subscribers: cedric

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

fix x11 compositor to use damage events

it seems that since the first version of the enlightenment compositor
in e17, damage events in x11 have never been used correctly. using
the event struct members will only give the bounding box/area instead
of the damaged regions; the real regions must be explicitly fetched
from the server

this removes the need for a lot of hacks which were added over the years
to make override windows render correctly, and also probably reduces
rendering overhead slightly

deskmirror: check if ec is not null before accessing it

Summary: ec can be NULL

Reviewers: devilhorns, zmike

Projects: #enlightenment-git

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

e mixer - save state when we get change events too

this plugs a gap where wrong state is saved.

update mailmap

enlightenment: Fix formatting of Wayland compositor code so it is easily readable in 80 columns

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

e mixer - fix up saving to save everything

as pointed out by jackdanielsz and bu5hm4n - this doesnt save
everything. like all the outputs and ports and... so now it does.
everything is saved and restored is "remember" is enabled. now
everything should be fine.

e mixer - discovered bug as a lack of feature - does not remember volume

i have never seen this before until last night. on some systems audio
starts up volume 0 and muted (either or) and thus on login the volume
is not where you left it and you have to manually fix it every time.
this fixes this by having mixer remember the last volume and mute
state you set (option to enable/disable too) and handles "upgrading"
to remember by default if you have old config

@feature / @fix

e - mixer - fix leak when seting volume

fixes a lieak i spotted in mixer after moving to epulse core. fix for
e20

e pagers - fix full desktop ddnd (eg middle mouse) to finish the drag

drag state was not properly shut down on mouse up

@fix

e_comp_wl: create global and bind  session_recovery interface

Before we can use the session_recovery protocol extension we need to register
its global and bind the interface to our implementation.

The callback for the provide_uuid call from the client is just a stub for now.

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