conditionally compile elm win trap wayland functionality

Revert "e - fix build when wayland not enabled"

This reverts commit 413ca8046e44786d086816a94d60492b92f6fac6.

broke wayland runtime: wrong define names, wrong usage

wl drm output module must set comp gl state more pedantically

disable wl client hide animations even with gl

broken until we start making textures for rendering clients

don't identify all wl shell clients as internal...you crackhead

fix ibar drop calc to not unexpectedly trigger epileptic seizures in users

when attempting to drop on an icon, the drop area geometry was never included
in the calc, resulting in the drop changing between positions on every frame

slight ibar dnd optimization when checking taskbar apps for drop site

Updating italian translation

e clients - make sure we handle client with a NULL zone

don't modify pixmap hash when changing pixmap parent window

moderately certain I originally wrote this to work in the other direction
and then failed to remove it when I switched to setting parents instead of
children. regardless, pixmap hash should not be changed here

CRI on attempting to change a pixmap's client when pixmap already has client

this should never occur, so do something if it does since it means there's a big problem

always create new client for wl shell surfaces

this should be a more correct way of handling (shell) surfaces since
there's never a time when we won't want to create a new client

don't reuse wl surface pixmaps without adding a ref

don't attempt to find wl shell E_Client when one cannot exist

Wayland: more checks to avoid consuming serials needlessly

Reviewers: devilhorns, zmike

Subscribers: cedric

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

fix invalid eina list usage when merging wl surface frame list

Wayland: Don't consume serials when not sending events

Reviewers: zmike, devilhorns

Subscribers: cedric

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

wayland: Check for potential OOM from wl_array_add()

Summary:
wl_array_add() returns NULL if it can't malloc/realloc.  This
is probably unlikely but check for it and issue a meaningful error if it
occurs.

Reviewers: cedric, devilhorns, zmike, raster

Reviewed By: raster

Subscribers: raster, cedric

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

Send modifiers to wayland clients on focus in

Summary:
If we don't update the modifiers on focus in we can end up with stuck
modifiers if a modifier is held when starting a client.

Reviewers: zmike, devilhorns

Reviewed By: devilhorns

Subscribers: cedric

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

Break e_comp_wl_input_keyboard_modifiers_update into two functions

Summary:
Separate the state serialization from the send so we can use the
serialization function elsewhere.

Reviewers: zmike, devilhorns

Subscribers: cedric

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

Don't update xkbstate when no app has focus

Summary:
We still update the keyboard array.  focus_in adds everything in the
array to xkbstate anyway, so adding it when there's no focus results
in a double update of state and potentially stuck modifiers.

Reviewers: zmike, devilhorns

Subscribers: cedric

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

redo ibar icon label position signalling to emit left/center/right

also calc based on icon position, not icon orientation

fix crash when using e_comp_object_util_center_pos_get() with non comp_objects

fix horizontal/vertical smart maximize

fix T2170

use current zone for shelf cfd setup

fix T2168

Revert "disable fileman popups in wl-only mode"

This reverts commit e6c47e8d90e75182cffd1d8cbdbb9e3821a4dbf1.

these no longer crash so reenable them

remove unnecessary wl client pixmap checks

these are direct object callbacks so they will only be triggered on wl clients

use stderr messages when failing to init X compositor

can't show error dialogs when we're about to exit, may as well print something useful

fix T2182

exit immediately if any unrecognized arg is passed on startup

ref T2182

disable wl clients in X

this is untested and undeveloped, no point in leaving it around to crash randomly

Fix stuck modifiers on client exit

Summary:
We still need to remove the active keys when focus_out happens, even
if the pixmap is already gone.  If we don't and a modifier was held
down during exit then the xkb state will have that modifier stuck
forever.

Reviewers: zmike, devilhorns

Subscribers: cedric

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

don't require client mirror object existence to hide dead comp object mirrors

only hide comp object images for dead clients on smart hide

flag non-internal wl shm clients as dead and delete mirrors on hide

no more crashing?

trivial move comp object function

add E_Client->dead to mark clients that should immediately be removed from the compositor

theme_shadow was unused

force non-internal wl clients to hide when gl is not available

hide comp object image on smart hide

ensure that this is not being rendered anywhere at the time of hide

Revert "when we get a request to close client app, defer resource deletion"

This reverts commit 1cf7c1f1511c446413fdc6fae35d17870bdd1c4e.

didn't fix the issue and obfuscated the code unnecessarily

re-setting same cursor object updates hotspot as expected

never resize client clip object

set ec->fullscreen before applying fullscreen geometry

reapply wl surface focus during commit when necessary

grab wl shell surface geom and use for adjusting resize operations

I think this still may not be 100% perfect,
but it's more like 95% perfect now instead of 50%

enlightenment: Implement support for wl_data_device_interface "release" event

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

enlightenment: Bump required Wayland version to 1.7.0

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

don't install SIGBUS handler if we are running wayland-only

Wayland shm sets up a sigbus handler for catching invalid shm region
access. If we setup our sigbus handler here, then the wl-shm sigbus
handler will not function properly

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

when we get a request to close client app, defer resource deletion
until After close animation has been run

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

remove ec->awaiting_hide_event

no longer used

improve wl grab resize calcs

update+clamp wl surface client size to buffer size during commit

this is the same as the X render loop behavior, just in a different place

send different values to wl clients for configure when performing a resize action

the calculated values here seem to not be sufficiently accurate to manage a calculated resize.
instead, calc relative pointer coords and send them

 #barrel

set output->scale to e_scale, and when we wl_output_send_scale
actually send output->scale

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

wayland: Make sure we're initializing output scale sensibly

Summary:
By default the E_NEW() will create our output objects with a 0.0 scale,
which doesn't make any sense and if anything will lead to crashes.
Instead use a scale factor of 1.0.

When updating output details, if the scale setting is invalid then set
it to 1.0 as a sensible value.

Note this doesn't actually enable scaling, just helps make sure we're
not injecting invalid scale parameters from the start.

Reviewers: zmike, devilhorns, cedric

Subscribers: cedric

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

wayland: Refactor output initialization code

Summary:
The code to initialize outputs has duplicate stanzas for updating the
data members of the output structure.  This set of refactoring steps
shuffles things around to eliminate the redundant code.  I think this
also makes the code easier to follow too.

This also breaks out a new internal routine for looking up outputs by
id.  I suspect this will have general usefulness later.

Reviewers: devilhorns, cedric, zmike

Subscribers: cedric

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

remove wl client resize idler, send ec->w/h instead of ec->client.w/h on resize

this idler makes sense in X where there is a third party display server but not in wl

noop when setting pointer object if re-setting same object

disable fileman popups in wl-only mode

these crash for now and I'm annoyed

remove missed use of E_COMP_WL_PIXMAP_CHECK macro

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

wayland:  Document the public API

Summary: Add doxygen comments for public API members

Reviewers: zmike, cedric, devilhorns

Reviewed By: devilhorns

Subscribers: cedric

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

wayland: Eliminate extraneous E_COMP_WL_PIXMAP_CHECK macro

Summary:
Many of the functions in the compositor do some preliminary checks, to
ensure objects are valid and the right type.  Most of the checks follow
a consistent pattern, for example:

    if (!(ec = data)) return;
    if (e_object_is_del(E_OBJECT(ec))) return;
    if (ec->iconic) return;

This style makes it pretty straightforward to review the pre-conditions
of the function.

The one exception is the check to see if an object is a wayland pixmap;
this is implemented as a C macro.

     E_COMP_WL_PIXMAP_CHECK;

The name of the macro is descriptive enough, but it conceals what is
actually being checked.  I think that it is clearer and fits better with
the style of the rest of the pre-condition checks for this check to just
be shown as regular C code:

     if (e_pixmap_type_get(ec->pixmap) != E_PIXMAP_TYPE_WL) return;

Reviewers: zmike, cedric, devilhorns

Reviewed By: devilhorns

Subscribers: cedric

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

wayland: Fix missing check on E_NEW return

Summary:
All other E_NEW calls in this file check the return value, except this
one place.  In this function, other calls are being checked for NULL and
handled so this one should as well.  The other handlers in this function
follow the style of issuing an error message, freeing objects, and
returning false; we don't need to free anything so just do the error
message and return.

Reviewers: zmike, cedric, devilhorns

Subscribers: cedric

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

update wl xdg states on maximize/fullscreen events

don't send wl mouse button events when client's menu is active

unify wl comp mouse button callback checks

wayland: Whitespace cleanup

Summary: Cosmetic

Reviewers: zmike

Subscribers: cedric

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

reject ignored clients from client mouse event functions

fix compile issue due to output change and fixup subsurface creation to match new "state" paradigm

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

Implement buffer_reference to fix client resizing issue

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

adjust wayland compositor scale when we adjust e_scale

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

add a surface_state structure to E_Comp_Wl_Client_Data

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

clamp to parent height, not parent y co-ordinate.

Summary:
Clamping to Y makes pop ups get shifted up if a window is
near the top of the screen.

Reviewers: devilhorns, zmike

Subscribers: cedric

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

Don't crash when an error pop-up occurs before the compositor is set up.

Reviewers: devilhorns, zmike

Subscribers: cedric

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

wayland:  Refactor out common mouse handling code

Summary:
These two functions are nearly identical so move much of the common code
into a helper function.

Reviewers: cedric, devilhorns, zmike

Subscribers: cedric

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

pre-escape all notification body texts before display or merge

this fixes notification merging again

ref T2053

build: allow building in pure Wayland mode (no xcb, no ecore-x)

Summary:
We should be building Enlightenement with "--enable-wayland-only"
even if xcb, ecore-x... are not present. The Evas wayland-shm
engine will then be a prerequisite, though.

E_Alert being too much X11-oriented, let us disable it completely
before it is ported.

Also, do not require the X11 rendering engines at runtime.

Reviewers: stefan, devilhorns

Reviewed By: devilhorns

Subscribers: stefan_schmidt, stefan

Projects: #enlightenment-git

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

e - menus - fix submenu popup direction when zones are not at 0 y

e - menus - fix autoscroll on zones not at 0 y...

cosmetic:  Whitespace cleanup of wayland client files

Summary:
cosmetic: Cleanup whitespace for wayland compositor

There is excess trailing whitespace in several of the files for the
wayland implementation.  These patches attempt to apply the spacing
rules for Enlightenment as documented at:
https://phab.enlightenment.org/w/coding_convention/

The whitespace rules are applied using the following emacs tabbing
settings:

 (defun start-efl-c-mode ()
  "C mode with adjusted defaults for use with Enlightenment
  development."
  (interactive)
  (c-mode)
  (c-set-style "gnu")
  (setq tab-width 8)
  (setq indent-tabs-mode nil)
  (setq c-basic-offset 3)
  (setq defun-block-intro 3)
  (setq statement-block-intro 3)
  (setq case-label 1)
  (setq statement-case-intro 3)
 )

This patchset also eliminates all tabs in favor of whitespace, as per
the coding style rules.

Reviewers: zmike, cedric, devilhorns

Subscribers: cedric

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

remove unused variable

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

append E_Comp_Wl_Output to compositor's output list during output_init
call

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

don't remove E_Comp_Wl_Output on unbind as this Could lead to nasty
crashes if a second client tries to bind wl_output

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

Fix wl_output tracking in wayland compositor

Summary:
We should keep a list of resources per output to propagate wl_output
changes to each client that bound an output.

The output list now contains all outputs, not just bound ones.

Reviewers: devilhorns, zmike

Subscribers: cedric

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

Code cleanup and check for OOM condition with wl_array_add

Summary:
wayland: Check wl_array_add return for OOM condition
cosmetic: Make error return check syntax consistent with rest of code
wayland: Reuse use_adj to track child-forced adjust
wayland: Refactor to collapse if chain
wayland: Refactor _e_comp_wl_client_priority_adjust to collapse if chain
 + This uses early returns to unwrap a couple if clauses to make the code a bit less nesty.
wayland: Refactor if statements for consistency with rest of file

Reviewers: cedric, zmike, devilhorns

Subscribers: devilhorns, cedric

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

create wl_outputs when in wayland-only mode

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

Fix T2131 (crash when creating wl_outputs)

This fixes the crash and creates outputs based on e_randr
configuration (when in wayland-client mode).

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

add E_Comp_Wl_Output structure and function prototype to create outputs

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

efm - make dir listing far faster - no file magic content and dont sync

this speed sup dir listing in efm drastically. first the e fm back end
uses an io thread that just spools through everything fast and sends
it over the mainloop to then send by ipc to e.

and on the e side we no longer use the heavy file internal magic using
api calls that wander all over a file for magic numbers - this is
insanely slow and brings listing to a crawl.

fix border frame color classes to be configured as solids, not text

fix T1637

don't clamp bgpreview vsize if bgpreview is a fullscreen overlay

fix appindicator icon loading when using IconThemePath key

fix T1888

set default text for edgebindings caption when no bindings exist

fix dialog sizing when icon setting fails

remove unnecessary icon setting and fix sizing in edgebindings

ref T2144

Fixed tense in presentation mode notifications.

Summary: Fixed tense in presentation mode notifications.

Reviewers: zmike

Subscribers: cedric

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

 Bugfix: e backlight: use saner default values.

 The brightness scale goes from 0 (well, 1) to 100: 0.1 thus seems too low a step, 10 being better and is actually already used.

e - actions - smart suspend/hibernate - also dont do when on mains power

randr2 - handle missing clone src and walk back config only structs

do not reapply focus on canvas object ungrab/close for pointer-focus enthusiasts

change output_bind function name

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