e_module: Add event handler for ECORE_WL_EVENT_INTERFACES_BOUND

Summary:
- Add event handler to make sure e_module_all_load() done after wayland compositor interfaces are bound
- Delete the "return" code if still loading modules, since "wl_desktop_shell" module create wl-shell and xdg-shell interfaces needed to be bound.

Test Plan:
(1) Run enlightenment on wayland
(2) If e_module attemps to make the "Error loading Module" popup window, you will see deadlock waiting for the wayland sync "done" event to be received. This means the wayland global interfaces including wl.compositor is not ready.

below is the mainloop we are blocked
---------------------------------------------------------------
_ecore_wl_init_wait(void)
{
   while (!_ecore_wl_disp->init_done)
     wl_display_dispatch(_ecore_wl_disp->wl.display);
}

Change-Id: Id1207dfdf40b0fe078ed9700c63d1203fb26ec5e
Signed-off-by: Duna Oh <duna.oh@samsung.com>

wl_drm: add check bind wayland display api to prevent segfault when enlightenment is trying to run on sw mesa

Change-Id: Ief1172a4816a01ac91b52b0c3996eafe152647e5

Add NULL check for function pointer of GL Api

Change-Id: I986a59a047168c52b8f0c124d0a888102bffc90e
Signed-off-by: MinJeong Kim <minjjj.kim@samsung.com>

Force free e_client when its buffer is destroied

The destory request of any surface will trigger to hide its evas object
and to delete e_client object of the surface. Those hide process of evas
object generates hide animation corresponding to its shadow style. In the
case, long animation can cause SIGSEGV by trying to access shared memory
which is already munmapped. This revision prevents the situation by free of
evas object immediately when the buffer is destroied.

Change-Id: I2b404270cec581354c367390fb0040f868cbe556
Signed-off-by: MinJeong Kim <minjjj.kim@samsung.com>

Add support for wayland opengles compositing

Change-Id: Ibcfe29ba861863b7738e80401fca5825bb5b4d14

To show underlay plane, added render_copy option for evas object of 24bit window on x11

Change-Id: Ie3d2222d19e6c9541ba76c07106ba81e29fc3f38

e_comp_object: set client size to requested size for wl_pointer-cursor

Change-Id: Icfb2250f2ef45f415b08aa0e6654be9d18e20a7a
Signed-off-by: MinJeong Kim <minjjj.kim@samsung.com>

e_comp_object: resize wl-pointer-cursor object directly

Change-Id: Ia80fa8822adb714449e97f7f1bfc78c25339da37
Signed-off-by: MinJeong Kim <minjjj.kim@samsung.com>

e_comp_wl: fix not sending a mouse enter event.

Change-Id: I21cb031e293fe281e35ba89f3a96116a28a48856

decrement e_comp->new_clients counter when unsetting new_client flag on wl cursor clients

Change-Id: If35c45cea17b947bbc3d6b320ee08fee24cd91af

mobile policy desk set hook needs to ignore new_client clients

desk set happens during client init, so these will always be new clients

ref D2242

Change-Id: Icf83859410916d2a4f6a0d9ac9679661d6347669

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

Change-Id: I7fdfb1ad76152a65af9dbedf147c0ce8dae13c75

Merge "comp_wl: Don't sync client size with buffer if client-side resize was locked" into tizen

comp_wl: Don't sync client size with buffer if client-side resize was locked

Change-Id: Idd4cbe75cec8bf55b132ae38f5dc88aaf2660d68
Signed-off-by: MinJeong Kim <minjjj.kim@samsung.com>

configure: do not require ecore-x on wayland only env

Change-Id: I9b4fa78b4623096053846793f80e16ebe1638e42
Signed-off-by: MinJeong Kim <minjjj.kim@samsung.com>

packaging: version to 0.19.99

Change-Id: I22c9a257ea62b3c54875e78d3c3ffd98a69252f8

remove temporary code

Summary:
    To avoid build-break, we removed version check part from configure.ac
    Now wayland is 1.7.0, We don't need temporary code any more.

Change-Id: I6550b72fbc7606968db133b40f0a5e7d57407d4f

Fix wayland client cursor set and client cursor move.

Change-Id: I4ca08d0deaaa7cc09e8b608b1f63a953c8233d14

Fix build break

Change-Id: I95f1c3325c40181eb3905873b4d20761dc7da92f

e_comp: Added skip_first_damage

For supporting the case that X doesn't send DamageNotify event of
damage creation, added skip_first_damage value.
This skip_first_damage will be used for determining skip of first DamageNotify.

Change-Id: I5d4c616110b9a0ed53a0b1781c02c0300481fede
Signed-off-by: MinJeong Kim <minjjj.kim@samsung.com>

test_helper: Fixed appending NULL and Remove hide process of registrant

Change-Id: I80c8508f63b983f6d176fba7ffecaffdf067d986
Signed-off-by: MinJeong Kim <minjjj.kim@samsung.com>

fix tty problem when enlightenment(drm-backend) is killed by SIGSEGV

Summary:
    When enlightenment is working as wayland display server, enlightenment
    changes KDSETMODE to KD_GRAPHICS in _ecore_drm_tty_setup(). However,
    when enlightenment is killed by SIGSEGV, it doesn't changes KDSETMODE
    to KD_TEXT because englightenment process doesn't call ecore_drm_tty_close().

    When enlightenment is killed by SIGSEGV, it should get drm devices with
    ecore_drm_device_get_list(), and will call ecore_drm_launcher_disconnect(),
    and ecore_drm_launcher_disconnect will call ecore_drm_tty_close() internally.

Reviewers: zmike, cedric, raster, gwanglim, devilhorns

Reviewed By: devilhorns

Subscribers: cedric

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

Change-Id: Iffe4279a50bce0cf3baf419c959bd1fef8010feb

Fix null dereference. while e_comp_shutdown is going, e_client's object could be unfocused.

Change-Id: Ib8734941b48468ca60b93cb5eaca5115d79f227a

fix build-error for wayland-only env

Change-Id: I949cc283c2bbe2f85d8a600846acbbaad0ff2b88

Disable x11 dependent modules with options for wayland-only

Change-Id: I5c86f1332882864d157a1ca3fc6cdf8f7951915f
Signed-off-by: Duna Oh <duna.oh@samsung.com>

move enlightenment.service file to e-tizen-data

   Enlightenment can run as wayland display server without X.
   In this case, englightenment will be launched by display-manager
   -run.service. It looks better e-tizen-data offers systemd service
   files.

Change-Id: Id86e5d0b9dcdb800fe52c2026e5e97d64c442a79

Added extended focus policy :Top visible window takes focus.

Change-Id: Ic92f1bda9eecfe0104ceee9ee3d76d433959eda4

replace e_comp_list with e_comp.

Change-Id: I230a15be9e93900d4d45337b8014a83c27bcb2af

Fix indentation on e_comp_object.

Change-Id: I08b5f22fe0d9e9b81413a5f576489630c8afef24

Delay first showing of client object while damage count up to 2.

Change-Id: Idc0d3031673fc7171780fb8e556cb9af39b708f2

Avoid freeing pixmap from unsetting parent window of pixmap

When some E_Client is deleted by hide event of its window,
comp_x tries to unset parent window of client's E_Pixmap.
This unsetting is causing freeing pixmap even if the pixmap is still pending.
this revision is for avoiding those unwanted free.

Change-Id: Id7568b57b4d274534915920c115ac59ae94e746e
Signed-off-by: MinJeong Kim <minjjj.kim@samsung.com>

add client-notification-level to E_Layer

Change-Id: I5781943db2440c1e5dc268287420b3fd3122da09

Ignore configure requests for windows which do not have relevant E_Client yet

Change-Id: I2555246058460da538f61daa0496a18995ea6d43
Signed-off-by: MinJeong Kim <minjjj.kim@samsung.com>

Added check damage count and pixmap validate of e_client object showing.

Change-Id: I99e0956043978b09064b2e8d2dcb3cd2c36cc075

exec enlightenment instead of enlightenment_start

    Refer to below commit of mainstream. enlightenment_start can't
    be used to start enlightenment in systemd.

    -------------------------------------------------------
    commit 6f955263fa76649682066adc82f12d4249d9c3d0
    Author: Cedric BAIL <cedric.bail@samsung.com>
    Commit: Cedric BAIL <cedric.bail@samsung.com>

    enlightenment: do not propagate the systemd socket to E17.

    In a proper systemd user session setup, enlightenment_start will be
    gone. That is still not doable today and as I plan to roll more feature
    in EFL regarding systemd, we need to prevent enlightenment to believe
    it is the process started by systemd.

Change-Id: I8cfcef9abf9318da3babd96c7f9191796fe0fabb

Added initial test case framework

1. Visibility calculation with comp_object status.
2. Added e_test_helper for supporting external TC modules through dbus

Change-Id: Ib994d99e179579c97aa7a59ccfa63b9bb3ea7d4d
Signed-off-by: MinJeong Kim <minjjj.kim@samsung.com>

Disable display backlight control to avoid camera flash

Change-Id: I78c6287aaca9856c84d2a7da0ad4a8dd6b3b1c4d

Do NOT set layer again for a client that have 'transient for window'

Change-Id: I6d0d1d3f75480f7877280bc576bc486193617de3
Signed-off-by: MinJeong Kim <minjjj.kim@samsung.com>

Added client_type config and E_CLIENT_PROPERTY_CLIENT_TYPE.

Change-Id: I6aafc020437c7aaec500a8c1a4db378f9ed87dcf

Adjust version number to upstream tag version for fixing build break

Change-Id: I53a36f1a3d87b16914e6e2885658b0c2b2a7354b

Packaging: update version to 0.19.1

Change-Id: If110b40d0430693cd1bba7f7f18638df3229cf42

Added check for deleted status of e_object before refering it

Change-Id: I9b76d6b4bb1ccb3fb8db8d5c7a44312882862e7d
Signed-off-by: MinJeong Kim <minjjj.kim@samsung.com>

Added 'wm_win_rotation' config value for ZONE_WINDOW_ROTATION Feature

Change-Id: Ic223e2aae9cbd101604f037b422f2f3d3b85c7cd

Added e_comp_object_intercept_hook feature on e_comp_object

Change-Id: I9236a012e27af8ba7356863fb76cebba76497bdf

Added E_CLIENT_HOOK_NEW_CLIENT_POST to E_CLIENT_HOOK

Change-Id: Ibe464690522e362dd0d6e52c445e848b5648fd73

Added E_VIRTUAL_KEYBOARD_TYPE type and value to E_Client

Change-Id: I7a2b9378f528dab636acfb5b773e9a243ed11a4b

Add first step of Zone_Window_Roation Feature

Change-Id: Ife577b104b8bd665e50d066d2b450e26f6146ebd

fix crash when activating menu that is not in range

Change-Id: I5582a562e4607e5b7aee58ca50a451b0ff4c9c60
Signed-off-by: Duna Oh <duna.oh@samsung.com>

Added 'effect_style' config value to determine effect style

Change-Id: Ic50d54b2d1404909b601f3d42dd2c69b7f1d7606

Added a config value 'effect_file' on E_Comp_Config to load specific effect edc

Change-Id: I508a0c31c3a0e4d176fbf504bb0b14e8c4c82bdf
Signed-off-by: MinJeong Kim <minjjj.kim@samsung.com>

Add home directory to environment of systemd service file

Change-Id: Ia14988d50c64539aac7366297faad04d346a0180

add enlightenment.service temporarily to run enlightenment in system session

Change-Id: I3566c731e9a4f2e73738497ef8d6df1a00b58b4c

e_config: NOT install profile.cfg and Ignore line feed when it gets profile name

Change-Id: Ia0d64070647a1cf14547773b9c186d274b6fdfdc
Signed-off-by: MinJeong Kim <minjjj.kim@samsung.com>

fixed installation path of "enlightenment.service" to /usr/lib.

%{_libdir} is converted /usr/lib64, when build based on x86_64 arch.
since systemd is installed in /usr/lib, and enlightenment.service file is installed that path,
it occurs rpm packaging error.

Change-Id: I4a72a3cd6c208874ae768b127e011dd146043017

packaging: add a runtime requirement to "monotype-fonts"

Running Enlightenment explicitly requires the "Sans" font,
which is provided by the "monotype-fonts" package.

Change-Id: I918d8645705da05403a00d39a9c5dda00771a62e
Signed-off-by: Manuel Bachmann <manuel.bachmann@open.eurogiciel.org>

Update Version to 0.19.0, and spec file to packge the rpm properly.

Change-Id: I9483f97e7afcf567823d784fa6b54f1ebb253996

add packaging directory

Change-Id: I1751c9619d96b3c9846c5bb994e009f240e4cf21

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>