efl: use efl_add_ref to create objects which have no parent

Signed-off-by: Mike Blumenkrantz <zmike@osg.samsung.com>

eo: start reporting abuse of efl_del

csharp: Add support for efl parts as Properties

Instead of
   var bg = efl.ui.Background.static_cast(myobj.Part("background"));

Now do
   var bg = myobj.Background;

Also a couple helper functions were added.

eolian_cxx: Fill documentation of part_def.

eolian-cxx: Add helper method to get all parts

Line get_all_methods, gets everything from the inherits too.

csharp: Use the library mapping on linux too.

Now the generated DllImport call will have the actual library name
instead of placeholders on Linux. Before we relied on dlsym being able
to search for the symbols on all loaded so's while Windows needed you to
specify the correct library to call GetProcAddress.

This fixes the case when someone call an function from an efl interface
defined in efl.so (e.g. efl_part) and the library is not loaded yet.

ecore_wl2: Send a mouse-up event on reentry after client initiated resize

Apparently when we initiate a client side move in ecore_wl2 we flag that
and send a mouse-up immediately on the next pointer enter.

Do the same for resize.

At some point this might need to be revisited, we should probably be
sending a "cancel" at the start of client initiated move/resize instead
of an up at the end?

Fix T6422

eolian: make use of panics for certain alloc errors

eolian: no need for printf-style for internal log funcs

eolian: use new error logging for all recoverable errors

efl.task - move to returning future insead of bool + exit event

title says it all...

eo: slight improvement of invalidate code as parent_set(NULL) will be valid on an invalidated object.

eo: automatically cancel a future that is attached to an invalidated object.

elementary: make sure windows are not born orfan and properly have at least a main loop as parent.

elementary: make global object lifecycle depends on the main loop.

evas: use efl_add_ref for the container, might consider using a parent later on.

evas: technically the evas canvas is using the main loop, so let's make canvas depend on it for their lifecycle.

edje: global class should actually be bound to the main loop.

ecore: the main loop has no parent and only one ref, let's properly create it.

eo: update tests to also use efl_add_ref correctly.

eo: check return in efl_add before performing more operations

this fixes a crash on objects which fail to be created using efl_add_ref

Signed-off-by: Mike Blumenkrantz <zmike@osg.samsung.com>

eo: start printing warning when instanciating object with efl_add and no parent.

eldbus: ensure the lifetime of the eldbus object path is monitored for proxies

the object is instantly destroyed when connection is lost

Signed-off-by: Mike Blumenkrantz <zmike@osg.samsung.com>

eldbus: use efl_del when clearing the model connection child list

it is an error to delete an object with an existing parent

Signed-off-by: Mike Blumenkrantz <zmike@osg.samsung.com>

tests: do not call unref on eldbus message objects after sending them

Signed-off-by: Mike Blumenkrantz <zmike@osg.samsung.com>

Pyolian: fix __repr__ for Documentation

Pyolian doc generator: update wrt name API changes

Argh, this has been difficult... hopefully I have done
this right

Pyolian: let ctypes automatically convert types

Not a big improvement and not so much automatically...
but for sure a bit less to type :)

Pyolian: use tuple instead of list in declarations

This is a bit more correct and (maybe) a bit more faster

Pyolian: new API eolian_unit_state_get

Pyolian: new API eolian_object_unit_get

Also implemented __repr__ for Unit and State

Pyolian: fix tests wrt name/short_name/full_name API changes

Pyolian: add support EOLIAN_SO_DIR env var

Thanks goes to @lauromoura for proposing this in D5840

ecore_wl2: Silence a commit ERR on window hide

Cosmetic fix, should be functionally identical.  Hide can be called
asynchronously, so it's ok, and the ERR can be silenced.

eolian: add APIs to get unit from object and state from unit

eolian: intial API for a new error mechanism

Unlike panic, this will be used to handle regular errors
such as parse errors. There will be no jumps and you
will be able to pass in a pointer to get the error
data into some local memory. That way you will be
able to override printing error messages.

eolian: initial API for a new panic mechanism

This will be used to handle unrecoverable errors.
For robustness, you will be able to set a custom
panic callback, jump and try to recover manually.

csharp: Fix support for ptr(structs)

In general, ptr(struct) parameters behavior depends whether the
parameter has the @owned modifier or not.

If there is no @owned parameter (meaning no transfer of ownership
happens) and it is a "complex" struct, with reference type fields
(like strings), the struct is converted to the respective
<Struct>Internal struct and passed with "ref" to the DllImport'd
function. For @in parameters, after the function it returns, this
intermediate struct is converted to the public struct type and
assigned to the original parameter, updating it to the external
world.

When we have ownership transfers, the structure is copied to unmanaged
memory and given to the callee. We can't send managed memory directly as
the callee may try to free it. On the managed side, the original struct
is left to be garbage collected normally.

csharp: Change policy on ptr(struct) owned calls

When transferring the ownership of a ptr(struct) from Unamanaged to
managed, we should marshal the reference-typed fields or they can point
to Managed memory that would make no sense to access from C (for
example, strings would return garbage).

The downside is that it can cause potential leaks if the receiver of the
memory do not free it. In the current state of the EFL api this can
happen in two places:

- Efl.Ui.List.SegArray::remove
return -> ptr(Efl.Ui.List.LayoutItem) @owned
- Efl.Ui.Focus.Manager::fetch
return -> ptr(Efl.Ui.Focus.Relations) @owned

The resources copied by both structs may leak when those functions are
overriden in C# and the values returned to the C code.

Also hide some internal stuff instead of exporting it and generate
implicit conversion operators for struct external/internal.

eolian_cxx: Add 'unknown' to parameter direction

It'll be used in eolian_mono as the 'direction' of return types in some
helper functions.

efl-cxx: Fix installation of efl_part_impl header

It was being installed directly into efl-cxx instead of efl-cxx/cxx.

Change-Id: I769c1b526ae2da465a1f4108117b2c18e30db01b

eolian: simplify lexer init

eolian: event/function_get_by_name -> by_name_get

eo: Orphan an object that is being invalidated.

Summary:
This will avoid infinite loops and errors when the parent tries
to orphan an invalidated child.

Fixes T6780

Test Plan: Run `make check`

Reviewers: cedric

Maniphest Tasks: T6780

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

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

eolian: more stringent and fine-grained validation

eolian: fix unit child management

Also pass correct unit to validation for performance.

eolian: merge objects in units

eolian: reduce the number of database traversals during validation

This significantly improves performance by not iterating certain
paths multiple times.

ee_drm: prevent extra flips

Similarly to commit 8b962d24acf192bd6c22de779c57eac73dd2f4db prevent
extra flips in the drm engine.

These are potentially more harmful to DRM as they can cause frame
drops and missed render opportunities.

ee_wayland: Prevent extra commits

It's possible that we unregister then reregister an animator so quickly
that the ecore_evas idle enter/exiters haven't had a chance to run yet.
In this case a render will come shortly anyway and we shouldn't use a
'false' commit to kick off a frame callback.

Silences an ERR and unmeasurably improves protocol utilization.

eolian: fix minor memory leaks

eolian: make sure dtors are always run correctly

eolian: remove the temps system

eolian: manage exprs through node system

eolian: manage typedecls through node system

eolian: manage class/vars through new node system

eolian: store types in node hash

eolian: generic dtor management for lexer

eolian: unified node allocation api

eolian: in-lexer-state current class storage

efl_ui_focus_manager_root_focus: switch to focus adapter

efl_ui_focus_manager_calc: set redirect before setting focus

settings the redirect might result in unfocus calls, that might unset
evas_object_focus on a object that might be registered in two managers
due to border linking. This fixes mysterical disapearing focus.

efl_ui_focus_manager_calc: do not set focus on a deletion manager

otherwise objects might already be on the way out, without a elementary
parent.

elm_interface_scrollable: make sure to have the content always focusable

elm_widget_item_static_focus: move canvas_object_set to later

We dont delete the adapeter when we are still focused, to set anyway the
correct view to it, we need to set the view to the adapeter as often as
possible

elm_genlist: support immidiate focus once the item is realized

this supports the case of item.focus = true before the item is realized.

elm_genlist: move focus recovering to first_touch instead of
focus_update

this means that the function is called more reliable, and not only based
on focus state, but also based on the redirect situation.

Change-Id: Ic23534aa8d6022f2c2b7341f61c98b28a1b7de0c

efl_ui_focus_manager_calc: add onedirection to dirty

elm_genlist: performe elm_item_focus_set

this means all neccessary fields with last focus and current focused
item are updated correctly

efl_ui_focus_composition: redirect canvas move events to the adapter

otherwise a moving canvas object will not be recalculated, as the
manager does not know about the new focus geometry

elm_widget_item_static_focus: fix fetching of the item

elm_widget_item_static_focus: delete adapter when deleted or unrealized

somehow genlist leaks the view sometimes, thus the adapter is not
deleted when the item is deleted. This resulted in strange ghost focus
objects in the window.

elm_genlist: set focus to the item when a user calls the function

elm_genlist: set focus to the item when a user calls the function

efl_ui_focus_manager_sub: flush the border if we cannot find a candidate

after that someone probebly wants to check where the last focus was on,
if the set of boder elements has changed since the last border flush, then the
element might not be registered in the parent manager

ecore_wl2_window: Silence some spurious warnings

ecore_wl2_window_commit() must be called during window size negotiation,
but this currently trips a warning when no frame callback has been
received for the first commit.  We can't even have frame callbacks at
that point because no buffer is attached.

Don't set up the commit_pending logic until after we have a buffer.

ee_wayland: Set window visible before trying to render it

if we try to render before setting ee->visible the results aren't
likely to be... visible.

eolian: turn some state APIs into inlines over unit APIs

pyolian: remove unneeded bindings in eolian_state

elua: inherit state from unit in eolian bindings

eolian: refactor doc ref resolution API

eo: fix invalidate to always happen.

genlist: fix "insane" order [BUG COMPATIBILITY]

This patch implements bug compatibility.

genlist internally uses both a tree structure with Eina_List and a flat
Eina_Inlist to track its items. ALL of the items are in the inlist while
subitems appear in their parent's list. As a consequence both lists must
be kept in sync pretty tightly. Obviously this is not done at all and
has led to countless bugs, as soon as tree or groups are used:
 - Invalid order of items (visually)
 - Invalid order of items with sorted_insert
 - Glitches in the matrix
 - Crashes with sorted_insert
 - Odd/even styles not properly set
 - Promote/demote functions broken by design
 - Developers send to psychiatric hospitals
 - Etc...

Legacy genlist (1.19 and before) used an inlist order that basically
didn't make sense, as it didn't follow the logical order of elements (as
they appear visually). Unfortunately this has "worked" (really, that's a
huge stretch to use this word here) for a long time this way. As a
consequence, some applications (*cough* empc *cough*) have relied on
this order to implement "next album" or "previous album" where the
album title is a group node.

By changing the order of items in the inlist, this has broken the
assumptions made above, and ends up in cases that return NULL, leading
to SEGV. Sure, the app should have checked NULL, but that's not really
the point here. The behavior has been changed.

This patch implements "fixes" for the following functions:
 - elm_genlist_first_item_get(): Don't return a parent
 - elm_genlist_last_item_get(): Return a parent
 - elm_genlist_item_next_get(): return a parent upon reaching the last child
 - elm_genlist_item_prev_get(): return a child when a parent is passed

Important notes:
 - This does not cover 100% behavior compatibility here. The only way to
   have it would be to simply revert the entire genlist code to its
   original version and never touch it again, ever.
 - An explicit API is required for an application to specify which API
   level it targets, so that we can cherry-pick which bug compatibility
   features we want to enable. We are already doing this for EDC,
   unfortunately.

@fix

fix T5938

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

tests: add genlist test to verify group item api behaviour

ref T5938

Signed-off-by: Mike Blumenkrantz <zmike@osg.samsung.com>

efl_ui_clock: do not locally store clock module api

there is no corresponding code to unset this global pointer when the module
is unloaded, so this will crash if elm_shutdown is ever called

Change-Id: Id296a265f899394d74812f8d79c14778127a5638
Signed-off-by: Mike Blumenkrantz <zmike@osg.samsung.com>

elm_gengrid: do not zero item cache if object is not finalized

this is a no-op case which causes unnecessary errors

Signed-off-by: Mike Blumenkrantz <zmike@osg.samsung.com>

efl_promise: avoid invalid calls when unsetting parent

if promise has no parent then the promise's loop is null

Signed-off-by: Mike Blumenkrantz <zmike@osg.samsung.com>

tests: remove invalid api usage from elm layout test

the comment here indicates that the author knew it was an invalid api use
so it should not have been here in the first place

Signed-off-by: Mike Blumenkrantz <zmike@osg.samsung.com>

tests: disable elm glview tests when running in buffer engine

Signed-off-by: Mike Blumenkrantz <zmike@osg.samsung.com>

elm_map: remove unnecessary error message in destructor

there is no check for the existence of this file so this will error most times
from it not existing

Signed-off-by: Mike Blumenkrantz <zmike@osg.samsung.com>

elm: call ecore_event_shutdown during elm_shutdown

ref 4e14ba3f4874b03185d5216829ecc1876d11c48e

Signed-off-by: Mike Blumenkrantz <zmike@osg.samsung.com>

tests: check whether ecore is properly shut down after elm_shutdown

Signed-off-by: Mike Blumenkrantz <zmike@osg.samsung.com>

efl_ui_win: do not trigger elm quit policy twice for same win

only check each policy for the case which should trigger it to avoid borking
main loop

Signed-off-by: Mike Blumenkrantz <zmike@osg.samsung.com>

tests: avoid test failures for elm_win when not running in X

Signed-off-by: Mike Blumenkrantz <zmike@osg.samsung.com>

tests: use buffer engine for elm_suite if engine is not specified

this should add some consistency to tests

Signed-off-by: Mike Blumenkrantz <zmike@osg.samsung.com>

tests: properly destroy elm_win timers to prevent crashes

Signed-off-by: Mike Blumenkrantz <zmike@osg.samsung.com>

efl_ui_win: track the finalize future for del to avoid crash

Signed-off-by: Mike Blumenkrantz <zmike@osg.samsung.com>

ecore_evas: implement the move method for basic buffer canvas

Signed-off-by: Mike Blumenkrantz <zmike@osg.samsung.com>

efl_threadio: always return a value to silence warning

efl_loop: fix exit code of the loop

For numeric types, eina_value_set() accepts values instead of references
on the value to be set. Hence, we were affecting as the exit code of the
loop a garbage value, yielding to invalid results.