fix e_screensaver_update compiler warnings when building for
wayland-only

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

fix unused variable if building for wayland-only

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

remove call to show pointer during canvas_new function

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

destroy pointer when destroying window since we create new pointers
for every window again.

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

add fixme note

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

Tiling: Respect min size hints (set to floating if doesn't fit).

This fix floats all the windows that don't fit because of their min size
restrictions.
It can be made better, at the moment it is very simple. An example
broken case would be the following:
Assume we have 3 windows, A, B and C.
B and C both hand a min width of half the screen.

First we create A, then B. Everything works as they share the screen,
but when we add C, both B's and C's restrictions "fail" so they are both
set to float although in reality, floating C is enough. This can be
fixed by doing a live scan of the tree every time a window is floated,
though it's not essential at the moment.

Fixes T952.

fix pointer for stupid X11...but this breaks pointers in wayland (for
now) :(

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

create window pointer based on compositor type

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

fix X compositor to use e_pointer_window_new

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

add back 'pixels' field structure for pointer

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

Check whether attached buffer exists, before using it.

Summary:
Because attached buffer can be NULL for removing surface content,
check of pending.buffer should be done before getting resource value of the buffer.

Reviewers: devilhorns

Subscribers: cedric

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

add missing 'pragma GCC diagnostic push'

use a new e_pointer for the window canvas

This fixes an issue where internal dialogs were not using the e
cursors but would default back to an x cursor

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

be sure to send the wl_data_device_send_enter when changing surface
focus and add some more debugging for dnd

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

trap return value of shm_buffer_get

This function Can fail to return an shm_buffer if the buffer_resource
is an EGL buffer.

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

Add checks for client being a wl pixmap in some functions

If we don't do this, then when we run in X11 with wl client support,
we crash in various places (iconify, etc) when trying to work with
normal X clients.

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

fix dereference of incomplete pointer type

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

Fix formatting

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

wl-x11: Declare keyboard strings as NULL (initially) to avoid crash
when updating keymap

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

move wl client_resize protocol calls to idler

automatically enable pointer and keyboard for wayland-in-X

reuse existing render post callback for wayland clients

Fix elm apps crashing from invalid e_remember

Don't set changes.icon during surface creation. There are functions
that get called when icccm.class changes and That is the place to set
changes.icon.

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

Fix inproper setting of window type during transient_set.

This fixes the issue of some wayland applications always showing up
with the e_border visible.

If we get a transient_set from wl_shell or xdg_shell, don't reset the
client's netwm.type. Some wayland applications are sending us
transient_set with parent to NULL....so don't explicitly set the
netwm.type to dialog.

@fix

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

Reduce usage of EC_CHANGED.
Lock subsurface border type to borderless.
Remove commented out code.
Don't mark border.changed during pre_frame callback.

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

reorder client idler and stacking functions so we can call them after
post_move.

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

Reduce potential of unnecessary function calls:
 - When we are doing client eval, don't check for Just changes in
property OR fetching. Rather we will check using an AND.

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

Simplify both transient_set functions (wl_shell and xdg_shell).
Remove debugging noise.

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

Remove NULL checks for valid client->comp_data ... let the crashing begin !!

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

Add ERR messages for EINTERN & EAPI functions.
Remove NULL checks ... let the crashing begin !!

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

Add error messages in EINTERN & EAPI functions.
Remove NULL checks ... let the crashing begin !!

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

Remove unwanted NULL check in internal function

Summary: Remove unwanted NULL check in internal function.

Reviewers: gwanglim, devilhorns, zmike

Subscribers: cedric

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

clear parent's list of transients, when set parent to NULL.

Summary: we also need to clear parent's resource.

Test Plan: N/A

Reviewers: devilhorns

Subscribers: cedric

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

wl-desktop-shell: Avoid runtime ERR noise for something that is not
really an error.

When we call shell_surface_parent_set with a NULL parent resource,
then there is No reason to try and find the pixmap window of a NULL
parent. Avoid that function call and just set the appropriate
properties and get out.

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

Fix building Enlightenment for X11-only, Wayland-Only, and X11 with Wayland clients.

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

handle setting any pending opaque regions during surface commit and
cleanup the input region code also

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

fix return variable type from e_pixmap_window_get function

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

change client transient_for id to uint64_t

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

wl_drm: Add short block to mark setup for drm egl rendering

When the new drm hw accel engine is ready, we can replace this little
bit of code and have hardware-accelerated drm.

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

Add simple clipboard based on wayland protocol

Summary:
Whenever any client requests new selection_set,
the data is saved to server-side clipboard directly on this revision.
So data device is able to keep data which is copied from client app have closed already
and is able to provide those data to new focused surface before coming new selection_set request.

Reviewers: devilhorns

Subscribers: cedric, gwanglim

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

Implement wayland subsurface support based on E_Client.

Summary: Support subsurface interface of wayland.

Test Plan:
1. Build enlightenment on devs/devilhorns/e_comp_wl branch.
2. Run enlightenment.
3. Run weston-subsurfaces
   - should be built after removing code of gl surface (not yet implemented)
   - just comment out the line egl_state_create in function of demoapp_create()

Reviewers: gwanglim, devilhorns

Subscribers: cedric

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

Add missing wl_resource_destroy

Summary:
There are no resource free codes in xdg shell destroy handler
and xdg popup destroy handler. e should delete wayland resource as
client requested.

Test Plan:
1. run e as wayland server
2. run weston-stacking in e
3. click with right mouse button to create xdg popup surface
4. click main surface area to delete xdg popup surface

Reviewers: devilhorns, zmike, raster

Subscribers: cedric

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

Don't run xrdb or xmodmap when we are wayland only

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

move the part of code for fetch transient_for in eval time to _e_shell_surface_parent_set().

Summary:
we need to set ec->parent before eval time.
that's because client can request to commit surface calling shell.configure() before eval time.
in this case, ec->parent isn't set yet, configure(move) will be failed.

in order to fix it, we have to set ec->parent in _e_shell_surface_parent_set() before eval time,
but we should check if ec already have parent, and if so, remove its parent's transients before change the value of ec->parent.

Test Plan: N/A

Reviewers: devilhorns

Subscribers: cedric

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

remove EINA_UNUSED in move and resize callbacks

Summary: remove EINA_UNUSED from used function parameters

Test Plan: N/A

Reviewers: devilhorns

Subscribers: cedric

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

Fixed trying to create damage area with 0x0 size

Summary:
This revision fixed failure to create damage area using Eina_Tiler on surface damage callback.
It was caused by trying to create Eina_Tiler with 0x0 size which is from size of E_Client.

Test Plan:
1. Run elementary_test.
2. Select one of the test apps.(no more magic fail message)

Reviewers: zmike, devilhorns

Reviewed By: devilhorns

Subscribers: cedric, gwanglim

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

remove setting ec->parent before evaluation.

Summary:
the value of "ec->parent" should be set in evaluation time,
therefore ec->parent->transients will be reset automatically.

Reviewers: devilhorns

CC: cedric
Differential Revision: https://phab.enlightenment.org/D1155

Add focus surface setting of keyboard for copy and paste between clients

Summary:
To enable copy and paste between clients,
wayland data device requires to know what surface is focused.
So this revision have make keyboard get focused surface when client is focused or unfocused,
and create data_offer for new focused surface in order to let new focused surface be able to get data from current selection data_source.

Test Plan:
1. Run wayland server
2. Run elementary_test -to entry5 on wayland server.
3. Run elementary_test -to entry5 on wayland server again. (preparing two clients)
4. Copy text on one of clients, and paste to the other.

Reviewers: devilhorns

CC: gwanglim, cedric
Differential Revision: https://phab.enlightenment.org/D1157

fix crash when client in wayland request to set parent to null.

Summary: Clear the E_Client's parent if client request to set parent to null.

Test Plan: run weston-stacking.

Reviewers: devilhorns

CC: cedric
Differential Revision: https://phab.enlightenment.org/D1154

Added conversion for image format to ARGB from XRGB for wayland. evas_object_image_data_set will be used instead of evas_object_image_data_copy_set.

Summary: this patch will provide to launch the application based on XRGB8888 like weston-simple-shm.

Test Plan: launch weston-simple-shm

Reviewers: devilhorns, zmike, raster

CC: cedric
Differential Revision: https://phab.enlightenment.org/D1137

Change time unit to milli-second from second when send event of frame done for wayland client.

Summary: that's because weston sample client supposed that the time passed by frame callback is milli-second.

Reviewers: devilhorns, zmike, raster

CC: cedric
Differential Revision: https://phab.enlightenment.org/D1133

map/unmap surface in committing surface should be valid, only when buffer is newly attached. check if buffer is newly attached, when do map/unmap surface.

Summary:
There is a case that client do commit twice after one wl_surface_attach.
In this case, even if no buffer attached, unmapping the surface.
Because server already clear the buffer in first commit.

This patch fixes it.

Reviewers: devilhorns, zmike, raster

CC: cedric
Differential Revision: https://phab.enlightenment.org/D1115

Add missing function

Summary:
Added missing function _e_comp_wl_data_find_for_client.
@fix

Reviewers: devilhorns

CC: cedric
Differential Revision: https://phab.enlightenment.org/D1102

Fix formatting.
Rename data_resource_list to data_resources.

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

Implement selection copy & paste using data interfaces of wayland

Summary: It supports selection copy & pasete with wl_data_device_manager, wl_data_device, wl_data_offer and wl_data_source.

Test Plan:
1. Build enlightenment on devs/devilhorns/e_comp_wl branch.
2. Run enlightenment.
3. Run elementary_test -to entry5 (or any entry samples).
4. Select any block of text.
5. Do copy the block and Copy anywhere you want.

Reviewers: gwanglim, devilhorns

CC: cedric
Differential Revision: https://phab.enlightenment.org/D1094

pass the compositor ecore_evas to e_pointer_canvas_new

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

pass the ecore_evas to e_pointer_canvas_new and use Eina_Bool for
true/false values

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

use Eina_Bool for true/false values

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

use the pointer from the compositor for e_win and don't free the pointer when the window closes

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

create the compositor e_pointer on the ecore_evas

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

add new e_pointer files

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

fix crash and memory leak when press and release any key repeatedly.

Summary:
since the size of wl_array is determined based on one byte,
so in order to compare with uint32_t, the size of wl_array should be divided by uint32_t's size.
and when calculate the size of wl_array by difference between two address,
address should type cast char* as one byte.

Test Plan:
(1) run terminology
(2) input any key several time.

Reviewers: devilhorns, zmike, raster

CC: cedric
Differential Revision: https://phab.enlightenment.org/D1089

remove old e_pointer code

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

flush clients Before we dispatch main loop events as clients may queue
events onto the loop.

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

add EGL headers for wayland compositor

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

fix define check for shutting down wayland compositor.

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

enable checking of windows going outside of e_config->screen limits

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

re-enable checking of current client geometry when get a configure
message.

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

add support for mouse wheel events and pass them to wayland

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

check if client is getting destroyed before we try to unmap it.

NB: This is done for wayland clients where e_client delete function
has already freed the client wl_comp_data Before we get a shell
surface destroy message from wayland. (occurs due to async nature).

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

check for valid comp_data before trying to use it.
Don't set pointer to 'k' in keydown event (we don't need it).

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

unify surface parent setting code for use with wl_shell and xdg_shell.
When setting transient, tell the compositor that we need to reset the
window type for this surface.

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

update some fixme/notes

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

fix building theme module with wayland-only

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

Fix state of popup windows to skip taskbar, pager, etc, etc.

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

wl-desktop-shell: Unify popup_parent code and fix placement of popup
windows.

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

fix compiler warning about unused variables when building for X11 or Wayland.

NB: Each different build (x11/wl-only) had various unused variables.
This is a squash of the commits to fix all that.

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

fix formatting

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

remove debug/warning message

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

e-comp-wl: Handle setting window parent for internal dialogs.
Move parent->modal setting from the shell module to the compositor.
Support setting of various window types (dialogs, menus, etc).
Handle setting of internal properties (min size, max size, etc).

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

wl-desktop-shell: Simplify transient_set functions and move the
client->parent->modal code to the compositor side.
Also, after setting netwm.type, change the set_win_type in the
compositor data field so that the compositor will handle parent->modal
on next property fetch.

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

remove dead comments

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

Implement focus_down_set function to grab mouse & wheel bindings.

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

Implement focus_check function and call on delete_request.
Don't try to ping clients that we are forceably killing.
Implement e_grabinput calls during focus_set.

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

Fix defines for building with wayland-only or wayland-client support.

NB: This fixes internal dialogs not having frames

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

Release array of keyboard keys when we shutdown input

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

Implement client stacking.
Ungrab mouse & wheel bindings when we delete the client.

Signed-off-by: Chris Michael <cp.michael@samsung.com>
remove client stacking code (apparently handled in e_client already)

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

fix define check for wayland client uuid store

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

Implement setting of client priority during focus events.

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

fix formatting

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

Resize input/damage/opaque pending tilers when client is resized

Summary: When the client resized, it need to resize tilers for surface region also.

Reviewers: devilhorns, zmike, raster, gwanglim

CC: cedric
Differential Revision: https://phab.enlightenment.org/D1020

remove debug noise

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

add support for xdg_shell surface transient_for

Summary: add support for xdg_shell surface transient_for

Test Plan:
1. run e wayland server
2. run weston-stacking and set focus it
3. press 't' for creating transient_for surface

Reviewers: devilhorns, zmike, raster

CC: cedric
Differential Revision: https://phab.enlightenment.org/D1016

@feature

add client property event into wayland eval fetch handler

Summary: add client property event for wayland client

Reviewers: devilhorns, zmike, raster

CC: cedric
Differential Revision: https://phab.enlightenment.org/D1015

use calloc to allocate memory of E_Comp_Wl_Buffer which is set to zero, instead of malloc.

Summary: E_Comp_Wl_Buffer's busy value should be updated after initialization as zero.

Reviewers: devilhorns, zmike, raster

Reviewed By: devilhorns

CC: cedric
Differential Revision: https://phab.enlightenment.org/D1011

use eina_hash_int64_new to create hash table "clients_win_hash" in case wayland, instead of eina_hash_int32_new.

Summary:
since the data as a key of hash is 64bit integers, clients_win_hash should be created to int64.
if not, loss of data can occur when add or delete data from hash table.

Test Plan:
(1) run any application. (added ec to hash table)
(2) close the application. (try to delete ec from hash table, but couldn't)
(3) after that, if _e_comp_wl_cb_render_post is called, since refer to ec which was already freed, enlightenment had been crashed.

Reviewers: devilhorns, zmike, raster

CC: cedric
Differential Revision: https://phab.enlightenment.org/D1006

fix geometry restore for maximized and fullscreen windows

Summary:
there are no geometry restore codes in handler of
wl_shell_surface toplevel_set.

Test Plan:
1. run e wayland only server with devilhons's branch source (devs/devilhorns/e_comp_wl)
2. run terminology
3. make terminlogy surface to be maximized
4. make terminlogy surface to be un-maximized
5. move terminlogy surface

Reviewers: devilhorns, zmike, raster, stefan_schmidt

CC: cedric
Differential Revision: https://phab.enlightenment.org/D965

cleanup unused code

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

Switch Wayland Compositor to use Eina_Tiler for damages, opaque, input, etc.

Add code to hold references to 'buffers'.
 - Release 'buffers' After they have been rendered by the compositor.

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

Add structures for a buffer and reference to a buffer.
Switch client_data structure to use eina_tiler for input, opaque,
damage regions.

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