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>

fix some formatting ugliness

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

fix compiler warning about pointer targets differ in signedness

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

update wayland README file to remove build confusion

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

Provide wl_output events on hotplug

Summary:
Clients that have bound wl_output now receive wl_output events on hotplug

NOTE: We don't handle removal yet

Reviewers: devilhorns, zmike

Reviewed By: devilhorns, zmike

Subscribers: cedric

Maniphest Tasks: T2131

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

Provide wl_output interface to clients

Reviewers: devilhorns, zmike

Reviewed By: devilhorns, zmike

Subscribers: cedric

Maniphest Tasks: T2131

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

rename extremely frustratingly-named E_Screen_Limits enum and members

this came in a patch that I take full responsibility for not adequately reviewing at the time.
the names are confusing and counterintuitive, and it does not properly include the client namespace.

clamp client size to screen when client limit policy does not allow offscreen

fix T1779

fix crashing when changing desklock settings and using pin/personal auth

escape notification popup text...always

fix T2053

don't send unfocused signal to override clients

fix T2099

use direct path for setting icons in util theme set function when icon is a path

fix T1888

Allow building without ptrace

Summary: Building without ptrace (on OSes which do not support it, like OpenBSD) did not work, because the fallback code had small typos.

Reviewers: devilhorns

Projects: #enlightenment-git

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

cpufreq: freebsd fixes

This fixes a sysctlbyname lookup and adds a stub for the "pstate"
command. Based on a patch contributed by Dmitry Luhtionov.

@fix

block linear desk flips that would wrap if desk flip wrapping is disabled

fix T1948

don't revert X focus during nocomp

use elm scroller for xkb options

fix T2007

revert some comp object debugging bits that snuck into another commit

Ibar: Emit the given origin to the Ibar Icon.

Summary:
This fixes T1828.

The theme also would have to support this!

Reviewers: zmike

Subscribers: myoungwoon, cedric

Maniphest Tasks: T1828

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

enlightenment: Fix building for wayland-only

We don't have e_randr2 calls for wayland yet, so ifdef those out if we
are building for wayland only

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

add MIN define for non-linux systems

actions - add smart suspend + hibernate actions for laptop usage

this adds 2 actions that only suspend a laptop (or hibernate) if there
are no external screens enabled. if there is no lid screen eg like a
desktop, then it will still go on as usual.

Revert "fix dbus unit installation with DESTDIR"

This reverts commit 3afaafe506d0f8c2ee95934cff6224cbc00ee35d.

broken test machine = broken commits :(