evas: Fix issue of wayland clients not drawing border icons

Due to commit 7ce79be1a1748c32c4d0e58358ce2242902cf89b, EFL Wayland
Client applications stopped rendering their window icons. This was due
to the code which tried to detect if an object is in framespace.
Previous version would just check the obj->is_frame flag ... which is
insufficient to determine if an object is in Framespace. This commit
fixes that issue by checking an objects geometry also.

@fix

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

docs: atspi: document text editable atspi interface

docs: atspi: fix some typos in text docs

docs: atspi: add missing docs for enums and structs in atspi text interface

docs: atspi: add missing docs to atspi value interface

docs: atspi: add missing docs for atspi selection interface

docs: atspi: add missing docs for atspi image interface

docs: atspi: add missing method docs for atspi component interface

docs: atspi: add missing method docs for atspi accessible interface

docs: atspi: document structs for atspi accessible interface

docs: atspi: add missing docs for atspi action interface

efl_net_server_udp: initial UDP server.

This is the initial UDP server that works similarly to the TCP one,
however under the hood it's widely different since the socket is
reused for all "clients", thus needs a new Efl.Net.Server.Udp.Client
(Efl.Net.Socket) as Efl.Net.Socket.Udp exposes the fd and options such
as 'cork', which would interfere in other clients.

The main socket will read the packets and find an existing client to
feed it. If no client exists, then it will create one if not overr
limit. Since there is no kernel-queuing as done by listen()/accept(),
the 'no reject' case will just accept the client anyway.

Next commits will improve UDP server handling with some advanced
features:

 - join multicast groups
 - bind to a specific interface (SO_BINDTODEVICE)
 - block packets going out of local network (SO_DONTROUTE)
 - specify priorities (SO_PRIORITY)

efl_net_dialer_udp_example: be more 'correct' using events to read/write.

instead of blindly reading and writing, which can lead to hangs due no
server, let's use can_read and can_write to do the operations, this
won't let us believe we can read when we can't.

it's also the recommended approach, so let's show that in the examples
so users don't copy&paste incorrect stuff :-)

efl_net_server_fd: allows custom socket 'read' event handling.

This allows UDP to reimplement it using recvfrom() instead of
accept().

Eo: Add test for refcount in efl_add_ref

This commits adds a regression test to the previous commit.

Eo: Fix references of objects

Eo test: Fix cleanup of objects.

docs: support for documenting variables

Elm_theme: parse theme name "<disk>:/file.ext" only on Windows.

Commit 86928a430c broke ability to use more than one theme file for
posix compatible systems. For case
ELM_THEME=/home/user/a.edj:/home/user/b.edj string was parsed
incorrectly. The result was a single string
"/home/user/a.edj:/home/user/b.edj", but expected two strings
"/home/user/a.edj" and "/home/user/b.edj"

This commit add additional check of the file paths like
<disk>:/filename.ext or <disk>:\filename.ext in Windows only.
This avoid wrong parse behaviour on Linux and macOS.

elm test: correct elm_entry_autocapital_type_set() argument

Summary: EINA_TRUE translates into ELM_AUTOCAPITAL_TYPE_WORD, so use the latter to be consistent with the API.

Reviewers: jpeg, minkyu, devilhorns

Subscribers: stefan_schmidt, devilhorns, cedric

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

New flip mode called ELM_FLIP_CROSS_FADE

Summary: Added new flip mode called ELM_FLIP_CROSS_FADE, which causes one content to fade in while the other fades out. Updated test_flip.c program to include a button for testing the new flip mode.

Test Plan: Run the elementary_test program, and open the Flip test. There is a new button labelled "9" that will activate the new cross-fade flip mode.

Reviewers: Hermet, bu5hm4n, raster

Reviewed By: bu5hm4n

Subscribers: bu5hm4n, cedric, jpeg

Tags: #efl

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

This series fixes two problems in the VNC implementation.
One deadlock and one segfault.

Patch 1:

Software X11 Evas Engine: Fix deadlock

Summary:
The patch bc6e8d269208 introduced a callback responsible to notify the pixels
that were sent to the X server. Since all EFL rendering is done by another
thread, the callback should be called from the main thread context.
To achieve this behaviour evas_software_x11_region_push_hook_call()
was using ecore_thread_main_loop_begin(), which may cause deadlocks, since
evas mainloop waits for the render thread and the render thread waits
the mainloop.

In order to fix this problem, the function
ecore_main_loop_thread_safe_call_async() will be used to
schedule the callback to run in the main loop context.

Since a callback is schedule to run in async manner, the pixels that
were sent to the X server must not be deleted until the user is informed.
In order to avoid more mallocs(), this patch adds the support for refcounts to the
X_Output_Buffer and Xcb_Output_Buffer.

Patch 2:

Ecore_Evas VNC: Use the image size to create the buffer.

In same cases they may differ and may lead to a segfault, since
memcpy() causes a buffer overrun.

Reviewers: bdilly, raster

Reviewed By: raster

Subscribers: cedric, jpeg

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

efl_net_server_tcp: use async getaddrinfo() to resolve server name.

this allows nicer usage such as 'localhost:http' as the address, which
will resolve to [::1]:80 (if IPv6 is enabled) or 127.0.0.1:80 if only
IPv4 exists.

efl_net_server_example: improve error messages (show text).

ipv6_only: use system default if unset.

only apply the value if IPv6 && set. If unset (now 0xff), then call
the getter so the property is in sync with the kernel default.

ecore anim - actually fix by checking fd not thread handle

this should remove a 0.5 sec or so pause at shutdown of efl apps.

@fix

eocre animator - the anim thread is polled on shutdown use volatile

we sit in a loop wating for animator thread to die on shutdown
(sleeping for 1000us), so make it a volatile so every check actually
checks and doesnt cache.

@fix

efl_io_copier: always cancel job on destructor.

It's already deleted on close(), but that's now optional
(close_on_destructor).

efl_net_server_tcp: allow IPv4 over IPv6 sockets.

Sometimes we want to handle both IPv4 and IPv6 in the same socket,
instead of spawning 2 servers, one for each protocol. That is achieved
by means of disabling IPV6_V6ONLY socket option, present in most
recent platforms.

gl_x11: Do KHR Partial Damage properly

KHR partial damage must be set once at the beginning of rendering with
buffer damage for the entire scene to be rendered.  Doing otherwise can
result in graphical anomalies on some GL implementations.

docs: eina_types: add descriptions for some structs for these eina beta api's

docs: edje: add some missing descriptions in edje enums

docs: eldbus: document all eldbus structs and its members

docs: ecore_con: add descriptions for the HTTP error codes

With this commit the documented globals reach 100%. Please keep it that way.

docs: ecore_con: add docs for HTTP authentication method enum

docs: ecore_con: be a bitmore verbose for the HTTP version description

docs: ecore_con: add description for efl net dialer errors

Elm theme: correct parse a theme filename on Windows OS.

Windows os uses ':'  as delimiter between disk name and file path.
<disk>:/<dir>/file.ext. In this commit added parse case for <disk_name>
symbol meets before ":/".

eolian gen: use EWAPI for events instead of EOAPI

Apparently only methods/properties can be EOAPI.

eolian gen: use EWAPI for all generated variables

ecore con: generate vars instead of writing them manually

eolian gen: automatically uppercase names for vars/constants

eolian gen: update tests to cover var generation

eolian gen: generate vars in source files for globals with value

eolian gen: initial generation of variables in headers

eo-cxx: Fix compilation and warnings in clang

elm: Remove elm app client/server implementation & APIs

This removes EO APIs related to an unmaintained client/server
model for applications. The reasons for this removal are the
following:
- unused
- no support in E
- relies on dbus as the sole transport layer
- unmaintained since the original patches
- only EO API (iow: beta, never released API)

I've also never seen the test cases (in elementary_test) actually
work.

According to Gustavo (k-s), the original author of this feature
is not involved in EFL at the moment, and unlikely to be in the
near future.

Note that terminology has in the past used those APIs when it
was still using some beta EO APIs. This code is now long gone,
removed in terminology commit 3ffcbadd6f9881472db6 (2014/12/13,
for version 0.8.0)

If someone wants to step in and maintain the implementation,
protocol and (EO) API, then feel free to revert this patch
and revive the feature. But it will need to be more solid than
this implementation.

eolian: fix build warning

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

wayland_egl: Remove weirdo buffer age kludge cargo culted from gl_x11

There's no reason why we'd need a full redraw if the buffer age for
this buffer isn't the same as the buffer age for the last buffer, so
let's cut out this thing.

wayland_egl: Use surface damage instead of buffer damage

We should always pass surface damage to eglSwapBuffersWithDamage.

gl_drm: Use surface damage instead of buffer damage

We should always pass surface damage to eglSwapBuffersWithDamage.

evas_engines: Send both surface and buffer damage to outbuf_flush callback

Some engines should using sending surface damage, until now we'd only ever
provided them with buffer damage.

The difference is that surface damage is the damage to the surface the
compositor is displaying, and the buffer damage is the damage to the
buffer the client has rendered.  These are different when the client
is using multiple buffers of different ages to render into.

Anything that calls eglSwapBuffersWithDamage, wl_surface_damage() or
wl_surface_damage_buffer() should be using surface damage, and not
buffer damage.

This patch is intended to make no functional change - any flush cb that
used buffer damage before still should.  Actual fixes to follow.

Apologies if I broke any engines - it's a bit of a copy and wasteland
out here.

Revert "wayland_shm: Add swap mode fallback when buffer age changes"

This reverts commit 62ca4486ea30fd63a5589ea7af373a5cb9982031.

Honestly, I think the gl_x11 code doesn't make sense either. :)

There's no logical reason that we have to do that, though it may be
hiding some other bug?  If that's the case, the bug should be fixed
properly.

ecore: test recursive case on efl_future.

ecore: allow efl_future_cancel on a promise currently being resolved.

This is necessary for allowing proper handling of recursive cancel.

Eo: Add a regression test for ref leak in function resolve

This adds a regression test for the last commit.

Eo: Fix reference leak when failing to resolve function.

When resolving a function for an object the object would get reference,
and then in some failure cases won't be freed.

I suspect this is a regression following the reshuffling that was done
in that function recently.

Thanks to zmike for investigating and reporting this.

Fixes T4740

@fix

Eo: Remove no longer relevant tests.

eolian: fix the nonsensical semantics for variables

Both variables and constants can have extern on them (it means the same thing
as in types, they won't get generated in C). This is different from the previous
semantics which only allowed extern on globals, which makes no sense.

Both globals and constants are allowed to have a value (previously only
constants were); constants are required to have a value. Globals having
a value is just a convenience for initialization during source generation.
This is unlike the previous behavior, where the value was optional for both
globals and constants and only allowed when not marked extern, which makes
no sense either.

Obivously, even when globals have a value, they're not allowed to be used
in expressions, as they cannot be evaluated at compile time (they're mutable).

gengrid: support for non homogenous items

Summary:
Dimensions of gengrid items can be altered for
non homogenity.

Care must be taken however to set the homogenous
size for items using elm_gengrid_item_size_set().

In horizontal mode only the heights will change
and in vertical mode only the widths. Fixed dimension
will be as set with elm_gengrid_item_size_set().

This is forked by https://phab.enlightenment.org/D2422

Test Plan:
elementary_test -to "Gengrid Resized Items"
Current test program provides focus autoscroll
and item looping options.

Reviewers: raster

Subscribers: cedric, jpeg

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

evas: Last fix... now it builds

evas: More compilation fixup with EGLImage

This is what happens when you can't test a patch yourself ^^

evas: Fix compilation with older EGL (maybe <1.5)

EGLImage was not introduced until very recently. EGLImageKHR was
the proper type. Thanks Roy for the report.

ecore: handle recursive trigger of promise by the cancel of a future.

This avoid double free/double callback call.

efl_net_dialer_udp: enable SO_BROADCAST before sending to 255.255.255.255

Like other toolkits, let's enable this automatically for users before
connecting to 255.255.255.255 IPv4 (IPADDR_BROADCAST), otherwise most
systems will just fail to connect and send packets.

efl_net_dialer_udp: "connect" to an UDP server to send and receive data.

Like existing ecore_con code, this does not use SOCKSv5 UDP
proxy. It's kinda cumbersome to add since requires a keep alive TCP
connection to the server, a second UDP channel and framing around the
original UDP frame.

Added UDP_CORK (if present) to match TCP_UDP present in TCP sockets,
this allows one to execute multiple write() calls that will result in
a single datagram, generated when CORK becomes FALSE again.

The efl_io_copier_example.c now accepts this as output. There is no
input UDP as there is no way to notify the server of a connection
(since such thing doesn't exit), usually servers react after a
datagram is received, replying to the source.

efl_net_socket_tcp: only emit error message if cork is being enabled.

by default we'll start with cork=0 and on adoption of a FD we'll apply
cached values, thus we'd try to apply cork=0 (default) and it would
error, which is annoying on platforms without such feature.

since users interested in TCP_CORK will enable it first, they will get
the error at that point.

ecore: replace arbitrary time with 32-bit safe arbitrary time

The end of time is much closer than you think.  it_value.tv_sec is
a signed 32-bit number on 32-bit machines. Using a negative tv_sec
causes timerfd_settime() to fail.

wayland_shm: Add swap mode fallback when buffer age changes

This copies the behaviour in opengl_x11 engine, where the buffer
age needs to be continuously the same to be taken into account.
If the age varies, then we fallback to a full redraw.

Apparently this fixes issues on actual devices. I tested this
patch in weston (I didn't have issues before and buffer age is 1).

edje_edit: avoid generate '(null)' value for image.normal

edje_edit: add correct tweens generation for inherit state

edje_cc: set correct value for attr set for MESH_NODE

On parce the image.normal token set attribute 'set' only for part type
EDJE_PART_TYPE_MESH_NODE.

@fix

elm_atspi_bridge: visible data changed signal not sent issue fix.

Summary:
_visible_data_changed_signal_send function checks for wrong object,
hence always returns with out actually sending signal.

The state is set to pd->object_broadcast_mask and not
pd->object_children_broadcast_mask in _set_broadcast_flag function.

Test Plan:
Send the visible data changed signal from any widget, but signal
is not sent.

Reviewers: cedric, lukasz.stanislawski, kimcinoo

Reviewed By: kimcinoo

Subscribers: rajeshps, govi, jpeg

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

edje_edit: refactor eet file usage

Summary:
Move opening for read/write/read-write and error message to internal method
Reuse eet file from Edje instead of opening it again in read-only mode
Add wrapper for eet_close to skip closing if internal file was used

Reviewers: NikaWhite

Subscribers: cedric, jpeg, #eflete

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

elementary: Fix call to offer_supports_mime function

As this function was misnamed in Ecore_Wl2, update the code here to
use the proper API function.

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

ecore-wl2: Fix typo in API function name

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

map: Fix elm_map usage (typo)

Emile: use stronger ssl cipher

Follows on from 356a1aa87a04a8d1c43e01fa861270d0947069c0 emile
didn't exist when this work was done originally

ecore_ssl: Use stricter cipher suites

Thanks to Victor Pereira from the SUSE Security team for auditing
this and recommending better options.
This has been discussed several times but knowone ever got to
commiting it.

evas/map: do nothing when same map is set again

Reviewers: cedric, Hermet, jpeg

Reviewed By: jpeg

Subscribers: cedric, jpeg

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

ELM: sync "clicked" usage with other places in the lib

Summary: In all other places "clicked" is defined as a local variable.

Reviewers: tasn, jpeg

Reviewed By: jpeg

Subscribers: cedric, minkyu

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

Emotion: prefer ratio to calculate sizes

Instead of calculate the ratio from the sizes, calc the width from the ratio,
as ratio seems always accurate.
Fallback to the old code if ratio not available (tbh I never see this case)

@Fix EMOTION_ASPECT_KEEP_BOTH with some rare strams, mostly DVD and online stuff

fix edje_cc segv when compiling bling bling theme

accessed parts out of range for the part table. check part count first
before checking. fixes segv. @fix

blinblng: http://enform.haxlab.org/files/blingbling_e21.tgz

evas gl x11 engine - Wrap eglCreateImage

eglCreateImage and eglCreateImageKHR have different parameters - the
attribute list is EGLint for one and EGLAttrib for the other.  Since
EGLAttrib is long, on a 64-bit system it's a different size than EGLint.

This was causing a crash in Enlightenment's wl_x11 on 64-bit machines when
trying to use a gl client.

Add a thin wrapper to use whichever is available and pass the appropriate
sized data.

ecore_evas_wayland: Don't use frame callbacks on windows with no shell surface

If we set a frame callback on a window with no shell surface (ie: and unmapped
window), the frame callback will never fire.  This was preventing some
applications like rage from ever posting a frame.

ecore_wl2: add API for testing if a window has a shell surface

eolian gen: better checks for whether to add space after type

examples: eolian_cxx: adapt code generation to changed eolian_gen commandline

The commandline options have changed for eolian_gen to generate the code. Adapt
the makefile helpers to fix the build break in examples

fix possible eina file shutdown issue

i am not sure as i cannto reproduce this, but i hope this fixes T4677
by ensuring if eina_file_shutdown is called it cannto double-free a
hash.

evas/module: add a new module in vg_saver for eet

Reviewers: cedric, jpeg

Subscribers: cedric, jpeg

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

evas: Fix evas_object_smart_clipped_clipper_get

This was broken for smart objects that are not "clipped smart
objects". This fixes the example evas_smart_object.

NOTE: This EAPI was removed in efl-1.18!
/!\ This was an uncaught API break between 1.17 and 1.18 /!\

@fix

evas: Improve render debug logs

Compiled out by default

ecore_evas - fix setting urgent immediately after show

fixes T4726

ecore-wl2: Fix input and opaque region setting

Bad y co-ordinate resulted in broken shadows after session recovery.

@fix

elementary: Fix issue of using uninitialized values

Coverity reports that Evas_Coord_Rectangles were being used here without being
initialized so declare those variables with some initial values

Fixes CID1362984

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

evas: Fix directly dereferencing pointer which may be NULL

Coverity reports this as a dereference before null check which implies
that 'cur' May be null here, so let's not use it before we check it.

Fixes CID1363765

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

evas: Fix directly dereferencing pointer which may be NULL

Coverity reports this as a dereference before null check which implies
that 'pd' May be null here, so let's not use it before we check it.

Fixes CID1364114

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

evas: Fix missing varargs cleanup

This patch fixes Coverity CID1364123 which reports that (in some
cases) va_end was not being called for var args. As they were
previously initialized with va_start, then va_end should be getting
called.

@fix

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

gengrid: call this on the pam class not on the normal class

this fixes gengrid.

eolian gen: use the new data type API to simplify the code