elm: use Dbus securely

The Efl.Access.Attribute is using key and value.
The value could be NULL. If the value is NULL, then following error occurs.

*error:
arguments to dbus_message_iter_append_basic() were incorrect,
assertion "_dbus_check_is_valid_utf8 (*string_p)" failed
in file ../../dbus/dbus-message.c line 2712.
This is normally a bug in some application using the D-Bus library.
Array or variant type requires that type string be written, but end_dict_entry
was written.
The overall signature expected here was 'a{ss}' and we are on byte 3 of that
signature.

elm: fix for invalid attribute matching

Summary:
Fixes invalid attribute matching in at-spi's collection interface's GetMatches***
functions.

Reviewers: stanluk

Reviewed By: stanluk

Subscribers: lukasz.stanislawski, cedric

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

evil: final step : remove completely the binary, useless now that the unit test is theree

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

efl ui did - dont segv if createicon func is not provided

Use ERR instead of CRI if *DATA_GET* returns NULL

This patch set is for remains.

elm: Use ERR instead of CRI if *DATA_GET* returns NULL

eina tests: add missing semicolumn

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

evil: remove useless binary tests

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

eolian: no need to store 'parsed' table anymore either

eolian: simplify parsed checking

eolian: no need to track currently parsing file anymore

eolian: all dependency parsing is now deferred

efl_ui_focus_parent_provider_gen: code refactoring

Summary: remove needless function call and else statement

Reviewers: bu5hm4n

Subscribers: cedric, jpeg

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

efl_ui_focus_manager_sub: avoid a call to NULL

Summary:
we need to consider the manager change to NULL

@ref T6616

Reviewers: bu5hm4n

Subscribers: cedric

Maniphest Tasks: T6616

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

elm_menu: avoid a call to NULL

Summary: the provider_find can be called before parent_set in constructor.

Test Plan:
1. EINA_LOG_LEVELS=eo:2 elementary_test -to menu
2. check that there is no warning message about efl_provider_find

Reviewers: jpeg, bu5hm4n

Reviewed By: bu5hm4n

Subscribers: cedric

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

efl_ui_focus_composition: remove elements_flush

this can also be done when calling prepare_logical

Efl.ui.progressbar: remove duplicate call

eina_value_set() was already getting called

Efl.Ui.Progressbar: implement range min max

wl2_surface_dmabuf: Trim the buffer queue after a while

If we have more buffers than we need for 100 frames then drop the oldest.

This can happen if we're on a hardware plane and then removed from it, or
really whenever the compositor feels like holding onto a few frames.

Trimming the queue too soon could result in having to do a costly full
frame redraw, so we wait a while to make sure we don't need one again.

Having more frames than we need costs us a little every draw since we
always use the oldest available.  It also wastes memory.

ecore_wl2_surface: Flush surfaces for offscreen windows

When a surface leaves all outputs we can discard its buffers to save
memory.

Currently most compositors don't send leave events for iconify, so this
pretty much just saves us a cursor buffer under weston for now, but in
the future it could be used for freeing resources of offscreen (fully
occluded or iconified) windows.

wl2_surface_dmabuf: improve surface flush

We used to abandon all buffers even if they were locked. This can't
actually free them until they're all released by the compositor.

Instead just free any buffer the compositor doesn't have locked, so we can
still re-use them when they're released without needing a full redraw.

There's probably room for additional cleverness here.  If we get a new
frame event before the buffer release we may want to keep it, and if we
get the release first we may want to abandon it.

ecore_wl2: Add new event for windows not displayed on any output

When a window leaves all outputs we can free its render buffers to save
memory.  This new event is generated when a window leaves all outputs.

ecore_wl2: Track outputs a surface is present on

Keep a list of Ecore_Wl2_Output * a surface is present on.

ecore_wl2: Add internal _ecore_wl2_output_find

This finds the Ecore_Wl2_Output * for a given struct wl_output *

ecore: add function for prepending an event handler

@feature

eolian: deferred parsing of inherits + better static checks

This change finally introduces deferred parsing of inherits to
Eolian, after a long time and many iterations. That means instead
of parsing inherits directly as part of the main file's parse pass,
they're pushed into a queue and parsed later. The validation engine
was modified to properly fill in the remaining info afterwards.

This may introduce breakages but I haven't noticed any. It also
properly unlocks cyclic dependency support in Eolian.

Additionally, this also introduces checks for duplicates in class
inherits (class Foo(Bar, Bar) is no longer possible) and it adds
stricter name checks, so you can no longer have a class Foo.Bar
and refer to it as Foo_Bar in implements. This was actually never
meant to be supported, but the check was previously broken.

@feature

eolian: move impl/ctor fill to validation stage

Eina test: delete tempory file on Windows in eina_test_file_mktemp

Evil: remove "symlink" code (it was for .lnk files anyway...)

Eio: disable the symlink code on Windows

Efreet_Mime: update comment about symlinks on Windows

Eina: ignore EINA_FILE_LNK on Windows

Ecore_File: remove symlinks support on Windows (library and test)

Windows: remove symlink() usage in edje_decc and evil binaries

eina: Cleanup grammar in eina_hash doxygen

Summary:
Corrects some grammatical errors, and rephrases wording of some passages
for better clarity.  Also fix a few doxygen formatting inconsistencies.

Subscribers: cedric, jpeg

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

edje_object: fix typo in EAPI edje_object_color_class_description_get

Reviewers: woohyun, jpeg, cedric, conr2d

Reviewed By: jpeg

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

elm_test: add elementary_test for Efl.Ui.Button

Test Plan: Run elementary_test->Efl.Ui.Button

Reviewers: jpeg, cedric, Jaehyun_Cho

Reviewed By: jpeg

Subscribers: jsuya

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

Efl.Ui.Image: remove previous image when NULL is set

Summary:
when Efl.Ui.Image has an image,
         efl_file_set(efl_added, NULL, NULL) is not working.
         I think this should remove prevous image and go back to empty image.
         @fix

Reviewers: jpeg, cedric, eunue, woohyun, Jaehyun_Cho

Subscribers: Blackmole

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

elm_index: fix documentation of wrong param name

Reviewers: jpeg, woohyun, eunue, conr2d, Jaehyun_Cho

Reviewed By: Jaehyun_Cho

Subscribers: cedric

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

efl_ui_bg_widget_legacy: ignore ELM_SCALE

Summary:
For legacy efl_ui_bg_widget or elm_bg, ELM_SCALE is ignored
when calculating internal image size.

Test Plan:
compare elementary_test->bg option with
ELM_SCALE=2.0 elementary_test->bg option

Reviewers: woohyun, jpeg, Jaehyun_Cho

Reviewed By: Jaehyun_Cho

Subscribers: cedric, id213sin

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

elm: make elm_layout_text_set use efl_text_markup_set

The elm_layout_text_get is using efl_text_markup_get by following commit.

    commit c07a40c745c5df1b1f6f0bbf666b233d8d072ca7
    elm: make elm_object_text_get return markup info as well.

    This commit solves following issue

    https://phab.enlightenment.org/T6642

    If I set object text as below
    elm_object_text_set(btn, "Some<br>text");
    then elm_object_text_get(btn) returns "Some text" not "Some<br>text".

So using efl_text_markup_set makes sense.

edje_cc: check HAVE_SYS_RESOURCE_H presence for rlimit usage

fix T6645

efl_ui_focus_manager_calc: keep the entry for the redirect seperated

logicals on the focus stack can cause there a lot of confusion.

This fixes a missed focus out event from the entry in elementary_test

ecore_wl2: Make surface managers modular

This allows loading modules to handle wayland surfaces, and makes the
existing dmabuf manager a module.

eina_module: Drop ridiculous ERR messages from module_list_load/unload

The documentation for these functions claims that passing a NULL array
results in doing nothing - that should also include logging nothing.

EINA_SAFETY_ON_NULL_RETURN() logs an ERR message and should be reserved
for usage when NULL is not actually a valid state.

Additionally, it's entirely possible to turn off EINA_SAFETY_CHECKS, at
which point these functions would stop behaving as the documentation
says they do.  Not great.

ecore_wl2: Use wl_buffer_get in dmabuf_surface_post

This is the final place where this code was reliant on internal structures,
it now solely uses (beta) public api.

ecore_wl2: fix ecore_wl2_buffer_wl_buffer_get

It was currently only used internally and had the side effect of
creating a new buffer instead of just returning the existing one.

Now it's useful to external callers, as it only returns the existing
wl_buffer and has no freaky side effects.

ecore_wl2: Add surface_window_get api

One of the last remaining blockers for compiling dmabuf support as a
plug-in.

ecore_wl2: Add API to remove surface managers

Someone might want to do this I guess.

ecore_wl2: Add a way to register new surface managers

And use it for the existing dmabuf surface manager.

ecore_wl2: rename _surface_flush for consistency

It was originally thought that this could be common code for multiple
back-ends, but that doesn't really make sense now, so rename it to match
the other dmabuf functions.

ecore_wl2: Move private data allocation into back-end code

The backend should allocate its own private data and return it instead
of a bool.

This assumes all back-ends will need some manner of private data, which
is certanly true for the one back-end we provide.

ecore_wl2: Move backend specific members into a private data struct

buffers and current buffers are really specific to the dmabuf backend,
another plug-in might not need them.

ecore_wl2: Move surface size update into generic code

This didn't have to be in backend code in the first place.

ecore_wl2: Add API surface_buffer_create

The specific surface code only needs these generic surface bits to pass
to buffer_create, so make a helper function for that instead of queries
for w, h, and alpha.

ecore_wl2: Remove force from surface reconfigure

It no longer has any users or need to exist.

wayland_shm: remove dirty bit entirely

Now that we're not using it for session recovery we can strip it out
entirely.

wayland_shm: Use surface flush instead of setting dirty

The dirty bit was a dirty hack to let session recovery force reconfigures
on startup.

Now that we have a surface flush we can achieve the same thing by just
discarding all buffers immediately.

ecore_wl2: Move smarts from surface_map to buffer_map

This simplifies the surface code so that it no longer needs access to
buffer structure members.

ecore_wl2: Add buffer_fit API and use it from surface code

This tries to resize the buffer's useable area to fit the specified size -
this is possible if the stride of the buffer is larger than the current
width.

ecore_wl2: Add buffer age inc and use it in surface code

Technically not necessary, since we could do this with get and set, but
it's a little more convenient to the user this way.

ecore_wl2: Add buffer_age_set and use it in surface code

ecore_wl2: Add buffer_age_get API and use it in surface code

ecore_wl2: Add API for setting buffer busy status and use it

Since surfaces will be managed by plug-ins, we need to make sure they
don't rely on internal structures.

ecore_wl2: Add API for querying buffer lock status and use it

Since surfaces will be managed by plug-ins, we need to make sure they
don't rely on internal structures.

ecore_wl2: Make the surface interface a pointer

ecore_wl2: Move surface manager interface into public API

So people can write plug-ins for this stuff.

ecore_wl2: Call check function through a pointer

We'll need this in the function array when we make surface managers
modular.

ecore_wl2: Move dmabuf surface specific tests into their own function

Refactoring to allow non-dmabuf surface managers.

ecore_wl2: Make session recovery more robust

Clear out the window callback when doing session recovery, and
make sure we have a valid on if we get a double commit.

This should stop a session recovery crash, and fix a small leak per
recovery.

edje_cc: simplify thread tracking for exiting on signal

edje_cc: make -thread option

this option was broken due to immediately triggering EMFILE errors
during thread initialization and then attempting to exit while threads
were still ongoing

some work is still needed to make this an improvement over the current
default of not using threads

@fix

quicklaunch: return child pid from elm_quicklaunch_fork() on success

efl_ui_focus_manager_calc: prepare everything before border elem eval

otherwise we might miss the elements that are registered on prepare.

efl_ui_focus_manager_calc: check earlier for LOGICAL

this safes us performance

elm_gen****: do not call realize on items

better rely on the adapter and wait for realization so the adapter is
wether created or not even created but used with content. This fixes
item content focus, crashes at startup and a freeze in genlist test!

pyolian: improve a bit the 2 example templates

ecore: prune existing thread_join callbacks during ecore_fork_reset

permitting these to persist and trigger at a later point will cause
the process to deadlock while attempting to join non-existent threads

@fix

efl_mono: fix DLL instalation path on efl-mono.pc.in

Destination of libefl_mono.dll is OS dependent, this was not translated
to efl-mono.pc.in. This commit fix this issue.

eolian_cxx: Fix "unused parameter" warnings

The Klass(Klass &&other) one was causing "synthesized method ... first
required here' warnings on Windows gcc-4.8.

examples: Remove duplicated build code for mono examples.

tests: Check return value in tests.

GCC was complaining about ignored return values from realpath, chdir,
etc.

eldbus: Uncomment tests

These were commented out during the promises changes but seems to be
working now.

efl_mono: Remove unneeded code from tests

eolian_mono: Add underlying pointer to ToString

It'll make easier to track where an object is coming from.

eolian_mono: Fix event argument for strings

eolian_mono: Fix visibility of generated events

Protected members do not make sense in sealed classes like the Concrete
ones.

Also document the placeholder field in extern structures.

mono: Fix AM_TESTS_ENVIRONMENT

The export commands were messing up with the elm_suite, causing the
layout do fail to be loaded.

ecore_con: missed compiler warning.

ecore_con: restore ability for negative port on local connection.

Thanks make check.

ecore-con pass port into err print

ecore_con - give error output if port num is invalid

elm: make elm_object_text_get return markup info as well.

This commit solves following issue

https://phab.enlightenment.org/T6642

If I set object text as below
elm_object_text_set(btn, "Some<br>text");
then elm_object_text_get(btn) returns "Some text" not "Some<br>text".

win: Fix legacy win type name

Window constructor is hijacked by the finalize method, which messes up
the normal order of operations. As a result, the legacy type name for a
window object was "elm_widget" rather than "elm_win".

This fixes a crash in E (as it checks the legacy type name to verify an
object's type).

See D5748

eo: only call costly function _eo_op_desc_name_get when in debug mode.

efl-wl: don't leak xkb mmaps when running under x11

ref a32735e9a7bc537430d155a4af853d498dda2009

pyolian: improve the doc generation

efl-wl: create and maintain xserver-based keymap and keyboard states under x11

when run in a non-wayland environment, it's necessary to do some extra work
in order to guarantee that the keymap remains the same in the compositor as
in the xserver and to also guarantee that modifier states are accurately
applied even when the compositor is not actively focused

fix T6631

efl-wl: use image border to optimize surface rendering with 1 opaque rect

efl-wl: unset kbd mods changed flag after sending modifiers

@fix