ecore-drm: Handle various touch events

Summary: This adds code to handle various touch events (up, down,
motion) so that touchscreen devices can work with ecore_drm.

@feature

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

ecore-drm: Remove libinput log print callback

Summary: This was causing crashes in libinput when trying to print out
via eina_log. Disable/Remove it. Let libinput print it's own logs.

@fix

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

elua: always install cmake stuff for elua, even when not being built

ecore_drm: Setup correct linker flags for usage with systemd-login

After the introduction of the non-systemd launcher the linking with systemd
enabled failed for me.

lib/ecore_drm/.libs/libecore_drm.so: undefined reference to `sd_pid_get_session'

Better we make sure that we actually setup the linker flags for the systemd use
case as well.

modules/tgv: Switch from obsolete LZ4_uncompress function to LZ4_decompress_fast

The function is declared as obsolete and in some newer headers it is no
longer defined. The new function is the same besides the name change.

configure: Fix typo in systemd-login check

eet: Switch from obsolete LZ4_uncompress function to LZ4_decompress_fast

The function is declared as obsolete and in some newer headers it is no
longer defined. The new function is the same besides the name change.

ecore-drm: fix passing wrong argument to open().

Summary: remove unintended argument.

@fix

Reviewers: devilhorns

Reviewed By: devilhorns

Subscribers: cedric

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

ecore-drm: restore VT to text mode when close tty.

Summary: if ecore-drm failed to be run after changing tty mode, need to restore VT to text mode.

Test Plan:
- run enlightenment(drm backend) without systemd.
after changing tty graphics mode, if it's failed to be run by any reason,
you'll see black screen and coulnd't control session.

@fix

Reviewers: devilhorns

Reviewed By: devilhorns

Subscribers: cedric

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

ecore-drm: cleanup data when failed to setup tty.

Summary: added code to free string of tty's name when failed to setup tty.

@fix

Reviewers: devilhorns

Reviewed By: devilhorns

Subscribers: cedric

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

ecore_drm: Removed unused variables in ecore_drm_evdev

Summary: Signed-off-by: kabeer khan <kabeer.khan@samsung.com>

@fix

Reviewers: devilhorns

Reviewed By: devilhorns

Subscribers: cedric

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

ecore-drm: Added cleanup data code as a error handling of connect logind.

Summary: need to clean also dbus, when failed open VT as a connection progress of logind.

@fix

Reviewers: devilhorns

Reviewed By: devilhorns

Subscribers: cedric

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

ecore-drm: Fix formatting.

Summary: No functional changes, just formatting fixes

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

edge: fix memory leak and removed redundant return statement.

Summary:
The name was freed after exit, so freeing it before and also freeing in a error case. There was a return statement after exit, thought it was redundant and removed.
@fix

Signed-off-by: Srivardhan Hebbar <sri.hebbar@samsung.com>
Reviewers: devilhorns, cedric

Reviewed By: cedric

Subscribers: cedric

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

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

ecore_audio: fix memory leak in ecore_audio_pulse.

Summary: Signed-off-by: kabeer khan <kabeer.khan@samsung.com>

@fix

Reviewers: devilhorns, cedric

Reviewed By: cedric

Subscribers: cedric

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

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

edje: Edje_Edit - generate missed properties of 'text' block

Summary:
missed souce, text-source, repch, size_range, style are added.
TODO is added for filter property.

Reviewers: Hermet, raster, cedric

Subscribers: cedric, reutskiy.v.v

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

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

Evas GL: Fix direct rendering with client-side rotation

There was a problem when checking whether the current surface
is compatible with direct rendering. In case of client-side
rotation (it's a flag set on the surface by the app), a surface
can be directly rendered even if the rotation is not 0.

But, before this patch, it was assumed that the surface was
current. Which doesn't make sense because make_current is
called by the pixel callback, from the application, and this
happens *after* we check for direct rendering.

As a consequence, it was not possible to mix directly rendered
surfaces with FBO-based ones, and use client-side rotation.

This patch should solve that issue.

evil: add getpwnam() function

@feature

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

ecoreodrm - fix build if systemd is not enabled

ecore-drm: Merge port to libinput

Merge branch 'devs/devilhorns/libinput'

Summary: These commits port the current ecore_drm input code to use
libinput for handling of devices, input events, etc

@feature

ecore-drm: Port ecore_drm to use libinput

Summary: This ports the input code of ecore_drm to make use of
libinput for handling of devices, events, etc

@feature

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

ecore-drm: Fix ecore_drm_launcher_device_close to not cause segfaults

Summary: We can only call _ecore_drm_logind_device_close if we have an
actual device string. It is possible that this Could get called with
NULL, and thus cause a crash.

@fix

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

ecore-drm: Cleanup structures in private header

Summary: With the port to libinput, some fields in the input & evdev
structures are not needed anymore, so this commit cleans those up.

@fix

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

eeze: Add API function for eeze_udev_find_by_subsystem_sysname

Summary: This adds an API function to find a device by
subsystem_sysname. This is needed for the port of ecore_drm to use
libinput.

@feature

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

eeze: Add API function declaration for find_by_subsystem_sysname

Summary: This adds an API function declaration for
eeze_udev_find_by_subsystem_sysname function which is needed for the
porting of ecore_drm to use libinput

@feature

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

ecore-drm: Include libinput header in drm private header

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

ecore-drm: Add libinput dependency check

Summary: ecore-drm will now require libinput for handling input
devices, so this commit adds a configure check for libinput

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

ecore-drm: Fix formatting

Summary: No functional changes, just fixing formatting

@fix

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

ecore-drm: Fix error message typo

Summary: This just fixes a typo in the error message string

@fix

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

ecore-drm: added drm launcher that is allow to determine whether to launch with logind or root privilege.

Summary:
- allow to launch drm backend without systemd-logind with root privilege.
- allow to open drm device node via logind, not directly open it, in case exist systemd-logind.
- fixes issue that couldn't switch session, because ecore-drm couldn't drop master to drm device with no permission. (allow to switch session appropriate.)

Reviewers: gwanglim, devilhorns

Subscribers: torori, cedric

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

ethumb: Fixing eina_init to eina_shutdown.

Summary:
In the end we should call eina_shutdown, but by mistake eina_init is being called. So have corrected that.

@fix

Signed-off-by: Srivardhan Hebbar <sri.hebbar@samsung.com>
Reviewers: devilhorns

Subscribers: cedric

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

evas_xcb_outbuf: fix memory leak

Summary: This fixes a leak in the xcb engine where the Outbuf_Region
was not being freed.

@fix

Signed-off-by: kabeer khan <kabeer.khan@samsung.com>
Reviewers: devilhorns

Subscribers: cedric

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

eina: removed multiple eina_init call.

Summary:
I observed that eina_init() was called twice in this function and eina_shutdown() once. So thought this might be a mistake, and so have cleaned it.

Signed-off-by: Srivardhan Hebbar <sri.hebbar@samsung.com>
Reviewers: devilhorns, cedric

Reviewed By: cedric

Subscribers: cedric

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

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

Evas GL: Fix scissors with direct rendering

If an app calls glDisable(SCISSORS) and uses direct rendering,
then the DR scissors were dropped and so glClear would erase
the contents of the entire canvas, instead of being restricted
to the image object.

Evas GL: Fix current FBO after make_current(0, 0)

Example scenario:
- Create a direct rendered Evas GL 'sfc' 'ctx'
- Create a PBuffer dummy surface, make it current
- Do some stuff
- Make current (NULL, NULL) to go back to no target
- Make current (sfc, ctx)
--> glClear() will not render anything on screen

Reason:
The current FBO binding is still set to the implicit FBO
bound to the PBuffer surface (it could be any surface, really).

edje/edje_cc: don't limit the bezier cubic minium value to 0.

No need to limit the minimum value to 0.
let user have a choice.

ecore/animator: improve the doc description.

ecore/animator: code refactoring.

renamed to more meaningful name.
don't stupid computaion twice.

ecore/animator: fix the wrong computation of bezier cubic.

Previous beizer cubic finds t value approximately.
In this sequence, there were 2 problems.

1. Previous guess_t value should be passed to differential equation to get the more accurate t value.
2. Guessing time count is not enough. I found 6 is enough time to get the t value experimentally. Previously it just tried 4 times on the other hand.

@fix

Revert "ecore/animator: fix the bezier cubic compuation."

This reverts commit 2e1032e28f0cd704ffaf28cea0b5ad533ae2b6c1.

This patch is wrong. On reviewing the origin implementation.

Revert "ecore/animator: simpler code"

This reverts commit 44f0f3e1707f0ade71eb7ccab5ecc70494c00ad8.

edje: Edje_Edit - generate 'perspective' block

Summary: Adds focal and zplane perspective to the part when generating an EDC.

Reviewers: Hermet, raster, cedric

Reviewed By: cedric

Subscribers: cedric, reutskiy.v.v

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

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

edje: Edje_Edit - add generation of missed properties of 'image' block.

Summary: add generation of missed parameters: scale_hint, border_scale and border_scale_by.

Reviewers: Hermet, cedric, raster

Subscribers: cedric, reutskiy.v.v

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

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

Evas font: Improve RLE rounding alpha8 to alpha44

This should ensure that the difference between the original
pixel value and the rle4 encoded one is <= 8.

The previous fix was a bit stupid as it was not taking into
account the conversion a4 to a8 (which is a8 = (a4 << 4) | a4).

elocation test binary variables should not be defined if tests don't exist

src/Makefile_Elocation.am:32: warning: variable 'tests_elocation_elocation_suite_SOURCES' is defined but no program or
src/Makefile_Elocation.am:32: library has 'tests_elocation_elocation_suite' as canonical name (possible typo)
src/Makefile.am:82:   'src/Makefile_Elocation.am' included from here
src/Makefile_Elocation.am:41: warning: variable 'tests_elocation_elocation_suite_LDADD' is defined but no program or
src/Makefile_Elocation.am:41: library has 'tests_elocation_elocation_suite' as canonical name (possible typo)
src/Makefile.am:82:   'src/Makefile_Elocation.am' included from here
src/Makefile_Elocation.am:42: warning: variable 'tests_elocation_elocation_suite_DEPENDENCIES' is defined but no program or
src/Makefile_Elocation.am:42: library has 'tests_elocation_elocation_suite' as canonical name (possible typo)
src/Makefile.am:82:   'src/Makefile_Elocation.am' included from here

build fix for elua binary when using -jX

libtool: link: cannot find the library `lib/elua/libelua.la' or unhandled argument `lib/elua/libelua.la'
make[2]: *** [bin/elua/elua] Error 1

upower module now errors and kills itself off when unsupported new version detected

0.99.0 removed the OnLowBattery property and added the per-device WarningLevel property. this requires what will effectively be a full rewrite of the module to track all the power levels of all the attached batteries and set the ecore power level somehow based on a combination of their levels

since I have no desire to spend any more hours working on and debugging this module which is based on a known-unstable api, I'm making it disable itself if it detects a version >= 0.99.0. hopefully someone will decide to maintain both this and eldbus in the future so that we can more accurately track upstream when they make changes to these things

ref T1908

ref T1909

eldbus hackaround for proxy deletion during pending callback

this is merely a way to avoid crashing immediately when the referenced bug is triggered. it does not fix the problem.

ref T1908

elua: add pc file into gitignore

elua: add elua.pc to configure.ac and add cmake definitions for elua

elua: add pkg-config for the elua library

eina: Refactored code and removed unwanted assignment.

Summary:
Removed unused assignment for length.

Signed-off-by: Srivardhan Hebbar <sri.hebbar@samsung.com>
Reviewers: devilhorns

Subscribers: cedric

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

elua: initial skeleton for the Elua library

Because of Bob and other things, it is required to separate the functionality currently
provided by the elua binary into a library. This library will then be used by the elua
binary as well as any other project.

evas/gl_drm: Fix warning

Using int instead of uint32_t.

Evas font: Compress font algo should round instead of floor

This is an attempt at having higher quality font rendering
while still using RLE-based font compression.

eina_thread: fix memory leak

Summary: Signed-off-by: kabeer khan <kabeer.khan@samsung.com>

Reviewers: devilhorns

Subscribers: cedric

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

eina: corrected documentation of ecore_hash_destroy function in benchmarj.

Summary:
ecore_hash_destroy function return nothing, still in the documentation of the function, it was mentioned that it would return true or false. So corrected the documentation.

Signed-off-by: Srivardhan Hebbar <sri.hebbar@samsung.com>
Reviewers: devilhorns, cedric

Reviewed By: cedric

Subscribers: cedric

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

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

ecore-wayland: make possible to disable xdg-shell at runtime

Summary:
Since xdg-shell is not yet supported officialy by wayland
It makes sense to have it disabled by default
and enable it on purpose.

Using env variable : EFL_WAYLAND_DONT_USE_XDG_SHELL
(name is inspired from Qt implementation of xdg-shell
but in the opposite way)

There is no EFL_WAYLAND_USE_XDG_SHELL in efl,
please use EFL_WAYLAND_DONT_USE_XDG_SHELL instead.

By default xdg-shell is enabled since it's supported by efl

Since there is only one shell supported at runtime
it's easier to switch to wl-shell (wayland's fallback shell)
by change this env variable.

Note, this patch can be reverted once xdg-shell replaces wl-shell
in wayland which is not the case in weston-1.6
but could integrated into upcoming wayland-1.7
(to be confirmed on release)

Change-Id: Id3732492397df9abe4a7c9e6e92a8f2c993c8395
Bug: https://phab.enlightenment.org/T1901
Bug-Tizen: TC-1353/part
Forwarded: https://phab.enlightenment.org/T1901
Signed-off-by: Philippe Coval <philippe.coval@open.eurogiciel.org>
Tizen Test Plan: echo 'export EFL_WAYLAND_DONT_USE_XDG_SHELL=defined' > /etc/profile.d/ecore.sh

Reviewers: seoz, devilhorns

Reviewed By: devilhorns

Subscribers: cedric

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

eeze_scanner now has separate return codes for various startup-related failures

ref T1612

eeze_scanner needs to hash on the address of the pointer and not use direct_add()

this is an @fix for the constant crashing

ref T1612

ecore-con now prints error messages when local socket creation fails

ecore-evas-wayland: Fix issue with ELM_PROFILE=mobile and windows not
showing up correctly.

Summary: If elm config profile is set to Mobile, then the
auto_norender_withdrawn is set to 1, which was causing Elm windows to
not draw properly due to the ee->withdrawn property not being updated.
It was not getting updated due to these checks for override.

NB: Tested this with all elm profiles, tested in weston, and tested in
Enlightenment (Wayland).

@fix

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

eldbus: Use the timeout parameter on eldbus_proxy_send_and_block()

Sorry by bad I had forgot to use the new parameter.

eldbus: Add timeout parameter to eldbus_proxy_send_and_block()

As all other send functions have, we need to have here.

@fix

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

eldbus: Fix _eldbus_connection_send_and_block()

Some problems with the actual implementation:
- the reply should not be writable, as it can only be read.
- if an error happen dbus_connection_send_with_reply_and_block()
will return NULL so we need check before use it
- all other send calls remove one reference of the message

Now also it is creating a error message, so the caller can know why it fail.

@fix

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

ecore/animator: simpler code

ecore/animator: fix the bezier cubic compuation.

The previous compuation is totally wrong.
Even it doesn't work correctly.
(I have no idea what the orignal author was thinking?)

Here we just need a simple and clear fomular to get the current progress frame.

If i'm wrong, please ping me.

@fix

Eo: fix error handling when too many deletions invocations occur.

Before this fix, when a deletion was invoked twice on an object, a
wrong message ("...You wrongly call eo_unref() within a destructor...")
was printed. This was caused by the del_triggered flag that was not
resetted when the destruction finished.

This patch fixes this behavior by printing the right message on a double
deletion.

eldbus: Add actual API function code for eldbus_proxy_send_and_block

Summary: This adds the actual code to send a dbus message and block
while waiting for a reply.

@feature

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

eldbus: Add code for _eldbus_connection_send_and_block function

Summary: This function will send a message to dbus and block while
waiting for a reply

NB: This is needed for our 'port to libinput', and for our 'opening up the
drm card without systemd' efforts

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

eldbus: Add API function declaration for eldbus_proxy_send_and_block

Summary: This adds a public facing API function to make dbus calls
which will block and wait for a reply. This is needed for a couple of
use cases in our Wayland efforts (libinput, etc).

@feature

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

eldbus: Add function prototype for send_and_block

Summary: This just adds the function prototype into the eldbus private
header. It will be used in the new proxy function
"eldbus_proxy_send_and_block"

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

ecore: Fix @since version to 1.13 for new API

1.3 is acient so we really will release this with 1.13.
Removed some trailing whitespaces devilhorns added. :)

ecore: Add code for new API function 'ecore_main_loop_nested_get'

Summary: This commit adds the actual code to the function, which
returns the 'in_main_loop' variable so that we can detect if the
ecore_main_loop is actually running.

NB: Will be needed for new eldbus API function (yet to add).

@feature

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

ecore: Add a new API function ecore_main_loop_nested_get

Summary: This commit adds a new function 'ecore_main_loop_nested_get'
so that we can detect if the ecore_main_loop is running.

NB: This is going to be needed for a new eldbus function that we have
to add in order to handle a use-case on the Wayland side. Spoke with
cedric for a while wrt to all this, and he gave it his 'ok' ;)

@feature

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

configure: Add note about automake 1.14 problem

Maybe we can remove this once it is fixed and the fixed version it widely used.

build: support automake-1.14

Summary:
This was needed to install example on purposes

The problem is that the install_sh* variables
in generated makefiles are expanding to ~/install-sh
which is wrong and should be the one in root source tree

Change-Id: I5311298864a5f042de90071cb6dfaefbefdf001a
Bug: https://phab.enlightenment.org/T1889
Bug-Tizen: TC-2177
Signed-off-by: Philippe Coval <philippe.coval@open.eurogiciel.org>
Reviewers: seoz

Subscribers: stefan_schmidt, cedric

Maniphest Tasks: T1889

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

evas/examples: fix warnings.

removed unused.

evas/examples: fix build break

add dependency libefl to call efl_file_set().

evas/gl_drm: add native surface for wayland_egl_client.

Reviewers: zmike, devilhorns, cedric, gwanglim

Reviewed By: cedric, gwanglim

Subscribers: jpeg, torori, cedric

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

evas - gif loader - fix technically "wrong" sizeof (doesnt cause a bug)

go from sizeof(char **) to sizeof(char *) ... effectively. so no real bug.

ecore-drm: Remove erroneous debug messages

Summary: No real function changes, just some cleanup of old debug
messages

@fix

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

elua: support is_c_only in lualian

eolian: eolian_function_is_c_only tests

eolian: add eolian_function_is_c_only

This function allows us to mark functions that are not bindable.
Also remove some obsolete code.

ecore con: Fix GnuTLS build error.

By making the respective changes on the GnuTLS side for upstream commits
d9b5f192d4883193f79cd3e43ed1da52521825dc and
d72f809fb874c8f14b1461949802905d41d648a1
which only take care of OpenSSL.

@fix

edje/edje_cc_handler - updated internal doc.

Evas: Evas_3D - Add check of visibilty node.

If the node is not visible, it is not rendered, which improves performance.
@feature.

Reviewers: raster, Hermet, cedric

Reviewed By: cedric

Subscribers: cedric

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

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

autotools: fix compiling/linking gl engines on OSX.

Summary:
Until now, it was necessary to set global LDFLAGS and CFLAGS to allow
compiling (and linking) engines using OpenGL.
gl_generic used to complained about missing headers or unkown libraries.
A problem on OSX is that there is CGL (Apple's Core OpenGL) on which the whole system
relies on and GLX, when X11 is installed; and they cohabit together.
When gl_cocoa is enabled, gl_generic is now linked against CGL.
When it is not, gl_generic is compiled with and linked against GLX as a fallback.

@fix

Test Plan:
With --enable-cocoa: software_x11, opengl_x11 and opengl_cocoa are working as expected.
With --disable-cocoa: software_x11 and opengl_x11 are also working as expected.
No compiling nor linking problems have been issued.

Reviewers: cedric, raster, raoulh

Subscribers: cedric

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

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

evas: Evas_3D - fix typo.

Summary:
A little fix of copy-paste, there were problems while changing texture coordinates of indices.

@fix

Reviewers: raster, Hermet, cedric

Reviewed By: cedric

Subscribers: cedric

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

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

Evas GL: Clarify doc about Evas_GL_API struct version

Evas GL: Add preventive padding after Evas_GL_API

Since this struct is likely to grow in size over time, client apps
built against future versions of EFL might start indexing fields
that are not present in the current form.

Also, don't reset the struct memory as this would break
multithreaded GL applications.

While this is not exactly a fix, I'll backport this.

@fix

evas-gl-drm: remove initalizing gbm on engine setup

Summary: Removed evas_drm_gbm_init() on the initial setup of evas gl_drm engine.

Reviewers: gwanglim

Reviewed By: gwanglim

Subscribers: cedric

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

Evas GL: Update evas gl api version number to 3

evas gl api supports wayland extensions as being part of a struct
of version 3.

Evas GL: Add support for bind_wayland_display extension

This provides EGL_WL_bind_wayland_display EGL extension to bind
and unbind the wl_display of a wayland server to an EGLDisplay.

@feature

ecore-wayland: Check for xdg_surface before sending ack_configure

Summary: Calling wayland library functions with NULL typically leads
to an abort, so add a safety check for valid xdg_surface before
calling function

@fix

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

ecore-wayland: Fix formatting

Summary: No functional changes, just formatting

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

ecore-wayland: Add safety check for xdg_popup creation

Summary: xdg_shell_get_xdg_popup could return NULL, in which case any
calls below would cause an abort in the wayland libraries, so this
adds a trap for that.

@fix

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

ecore-wayland: Add safety check for window surface creation

Summary: We cannot call wl_proxy_get_id if the surface creation failed
else that would lead to an abort in the wayland libraries. This commit
adds a safety check

@fix

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

ecore-wayland: No need for setting the input here as it's done again
directly below.

Summary: Not much of a functional change, just removing an extra
unnecessary line. Input is being set directly below this with an if
(!( line, so no point in this one.

@fix

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