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 :(

add ctrl+alt+backspace to kill wl-only e

 #barrel

only reject client mouse out for X clients

not sure this is still applicable since systray xembeds are gone

Update the error message shown if a theme file is invalid.

Summary:
Update the error message shown if a theme file is invalid. This isn't E17 any more. rEec1cd25cfe666665b9e5641c91d3ad1fd2f9f11c

{F13161 size="full"}

Reviewers: zmike, raster

Subscribers: cedric

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

remove wl override clients from focus stack

no more client cursor clients in winlist hoorayyyyy

add first YOLO: e_client_focus_stack_set()

to quote ecore docs:

DO NOT use this function unless you are the person God comes to ask for
advice when He has trouble managing the Universe.

add YOLO define to throw a compile warning for Extremely Dangerous Functions

a deprecated warning isn't ideal, but at least maybe someone will look at why there's a warning

allow focus on normal wl clients again

remove unneeded client attr setting in wl shell

these are always set by the wl new client hook

set wl popups to appropriate layer in shell module

remove some futile attempts at variable setting in wl client new hook

none of these attrs are available at this point so the client attrs can't be set

prevent client cursor clients from receiving focus

remove unused var

Conditionally ignore wayland cursor set events

Wayland cursor set events should be ignored if the client sending them
doesn't own the surface the pointer is currently in.

Signed-off-by: Derek Foreman <derekf@osg.samsung.com>

fix typo in wayland readme file

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

e - randr - handle missing relative display and use priority for zone

this makes highest priority screen the lowest (0) zone. this also
handles missing screesn that you are relative "of". missing clones are
not working atm. also zone reconfigure moves windows now too

add note about logind to wl readme

update wl readme

fix dbus unit installation with DESTDIR

manage client pointer client hidden states when changing pointer cursor objects

fixes sticky cursors

don't clear wl surface input tiler during commit

this broke input regions for successive frames

add E_Client->mouse.in for determining mouse-in status

remove unused E_Client->input_object

this is a relic from E18

unset wl client placed value on first commit to force placement

many clients still have not had a first commit prior to running new_client placement,
so this will force a second run of that algo to prevent placing as though a client's
size = 1x1

don't add wl evas callbacks multiple times

 #sigh

don't use garbage values for new_client wl surface configures

don't set geometry during wl client surface mapping

this happens automatically in other places, calling it here breaks client placement

 #barrel

reapply accurate ec->placed value for wl clients during surface commit configure

fixes client placement when client has not been placed prior to first commit

use pixmap size for wl subsurface commit configure

move map before configure for wl clients during commit

this ensures visibility is set correctly before attempting to set geometry

reenable new_client for wl clients, but force moveresize actions to succeed