eldbus: manage death of children during invalidate.

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

eldbus: object created with efl_add should be destroyed with efl_del.

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

ecore: first invalidate the loop and its children, then destroy all the object capability.

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

eo: optimize efl object reuse by limiting the amount of call to pointer translation.

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

edje: properly handle lifecycle of edje global object.x

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

eo: prevent parenting to an already invalidated parent.

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

eo: only call efl_parent_set(NULL) at the end of invalidate if the object is not invalidating.

Add also a comment to justify why we do force parent to NULL during invalidate.

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

ecore_con: correct lifecycle of object used by Efl.Net.Dialer.Ssl.

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

ecore_con: trigger teardown during invalidate.

Delaying teardown might result in code expecting parent to still be valid,
especially call to efl_provider_find won't work anymore.

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

ecore_con: it is best to call the super destructor and not touch the object anymore after.

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

ecore_con: correct a typo wrongly calling super destructor instead of super invalidate.

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

ecore: correct lifecycle of object used by Ecore_Con legacy.

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

eo: improve error message by pointing to the class that is creating trouble.

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

eo: preventing infinite loop by safely iterating over children during destructor.

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

ecore: properly handle shutdown of Efl.Loop children by using invalidate.

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

ecore: disable signal during shutdown.

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

eo: invalidate at the end of efl_invalidate only and prevent infinite recursion with an invalidating phase.

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

ecore: force close during invalidate of Efl.Io.Buffered_Stream.

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

elementary: use invalidate instead of overriding del for widget items.

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

elementary: rely on inheritance instead of faking efl_del behavior.

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

elementary: make del_pre not return a bool and force rely on Eo ref counting.

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

elementary: prevent death of list item using Eo ref counting.

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

elementary: remove now useless walking in item.

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

elementary: use Eo ref counting to prevent death of genlist items.

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

elementary: use Eo ref counting to prevent death of gengrid items.

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

elm_list: prevent list item operations during item deletion

ref T6880

Reviewed-by: Cedric BAIL <cedric@osg.samsung.com>
Differential Revision: https://phab.enlightenment.org/D6066

elm_list: fix crash in item deletion

this is still broken

ref T6881

Reviewed-by: Cedric BAIL <cedric@osg.samsung.com>
Differential Revision: https://phab.enlightenment.org/D6065

Revert "efl_add_ref - fis to use efl_add properly with a parent."

This reverts commit 2fb5cc3ad09f6aaf82b5d1131ac5ed22ed848bd4.

Most of this change where wrong as they didn't affect the destruction
of the object. efl_add_ref allow for manual handling of the lifecycle
of the object and make sure it is still alive during destructor. efl_add
will not allow you to access an object after invalidate also efl.parent.get
will always return NULL once the object is invalidated.

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

eo: to avoid mistake in timing, efl_invalidate should always be triggered before any destructor code.

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

elm_ctxpopup: prevent a giant load of errors

Summary:
i)  There have been 2 sorts of errors, first the items have been deleted
    after the list was deleted, thus the list items have already been
    destructed. which lead to the error as the popup items destroyed the list
    items on destruction as well.

ii) sd->box was attached to the ctxpopup when getting into group_del the
    box has already been destroyed by the cleanup logic.

Subscribers: cedric, zmike

Tags: #efl

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

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

elm_main: call _elm_config_shutdown() after loop_iterate()

In _elm_config_shutdown(), _elm_config is freed.
So, if some ecore events call efl_ui_xxx API which refers _elm_config,
there can be crashes.

@fix

eina: path get actually allocate memory and can't be pure.

efl_ui_focus_composition: track the deletion state of child objects

Reviewers: cedric, zmike

Reviewed By: cedric

Subscribers: #committers, cedric, zmike

Tags: #efl

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

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

efl_mono: Generate support for accessors in .eos

Summary: Depends on D6190

Reviewers: felipealmeida, vitor.sousa

Reviewed By: vitor.sousa

Subscribers: cedric, #committers, zmike

Tags: #efl

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

efl_mono: Initial support for Accessors.

Summary:
Plain conversion to IEnumerable, which is the base of LINQ
Depends on D6189

Reviewers: felipealmeida, vitor.sousa

Reviewed By: vitor.sousa

Subscribers: cedric, #committers, zmike

Tags: #efl

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

efl_mono: Reject the promise when it's disposed.

Summary:
As futures are tied to a given promises and are usually handled in
a 'attach and forget' scheme, we can't cancel them as it would cancel
the whole chain.

Reviewers: felipealmeida, vitor.sousa

Reviewed By: vitor.sousa

Subscribers: cedric, #committers, zmike

Tags: #efl

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

ecore-evas-buffer: defer resize callback until pre-render

Summary:
this mimics other engines, all of which have the same mechanics due
to display server interactions. it also avoids unnecessary recalcs before
pre-render if the canvas size was changing repeatedly

fix T6924
ref D6019

Reviewers: cedric, JackDanielZ

Reviewed By: JackDanielZ

Subscribers: #committers, JackDanielZ

Tags: #efl

Maniphest Tasks: T6924

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

efl_mono: Fix events after Event_Description rename.

efl: rename Efl.Ui.Zoom.Mode -> Efl.Ui.Zoom_Mode

efl: rename Efl.Ui.Widget.Orientation_Mode -> Efl.Ui.Widget_Orientation_Mode

efl: rename Efl.Ui.Text.Change_Info -> Efl.Ui.Text_Change_Info

efl: rename Efl.Ui.Tab_Page.* -> Efl.Ui.Tab_Page_*

efl: rename Efl.Ui.Stack.Event_* -> Efl.Ui.Stack_Event_*

efl: rename Efl.Ui.Slider.Indicator_Visible_Mode -> Efl.Ui.Slider_Indicator_Visible_Mode

efl: rename Efl.Ui.Pager.Loop -> Efl.Ui.Pager_Loop

efl: rename Efl.Ui.Focus.Composition.Adapter -> Efl.Ui.Focus.Composition_Adapter

efl: rename Efl.Ui.Drag.Dir -> Efl.Ui.Drag_Dir

efl: rename Efl.Ui.*_Legacy.Part -> Efl.Ui.*_Legacy_Part

efl: rename Efl.Text.Bidirectional_Type -> Efl.Text_Bidirectional_Type

efl: rename Efl.Task.* -> Efl.Task_*

efl: rename Efl.Selection.* -> Efl.Selection_*

efl: rename Efl.Observable.Tuple -> Efl.Observable_Tuple

efl: rename  Efl.Net.*.* ->  Efl.Net.*_*

efl: rename Efl.Input.Device.Type -> Efl.Input.Device_Type

efl: rename Efl.Gfx.*.* -> Efl.Gfx.*_*

efl: rename Efl.Gfx.Color_Class.Layer -> Efl.Gfx.Color_Class_Layer

efl: rename Efl.Exe.* -> Efl.Exe_*

efl: rename Efl.Event.Description -> Efl.Event_Description

efl: rename Efl.Event.Animator_Tick -> Efl.Event_Animator_Tick

efl: rename Efl.Canvas.Gesture.* -> Efl.Canvas.Gesture_*

efl: rename Efl.Canvas.Object.Animation.* -> Efl.Canvas.Object_Animation_*

efl: rename Efl.Canvas.Animation.* -> Efl.Canvas.Animation_*

evas: fix a map rendering problem.

When map of a smart object is turned off, redraw-updation area could be incorrect.
This results in a mirror image problem.

for fixing this, let smart object request update-area in render_pre() properly only in case of map changed.

Thanks to akanad for reviewing together.

@fix

elm: fix backward compatibility for "default" name part

Summary:
If you use legacy content/text set/get/unset with "default" part name,
_elm_layout_part_aliasing_eval change the part name to "elm.swallow.content".
Because internal _content_set/get/unset checked only "default" string,
legacy content/test set/get/unset with "default" part name was not working.
This patch fixes that bug.

Test Plan: elm_object_part_content/text_set(obj, "default", content)

Reviewers: Jaehyun_Cho, woohyun

Reviewed By: Jaehyun_Cho

Subscribers: cedric, zmike

Tags: #efl

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

efl_ui_win: always force resize job on first pre-render

Summary:
ensure that a resize eval occurs after the frame edje has been thawed
so that sizing will be correct in engines which either still have
broken size handling (D6019) or have sub-optimal size handling (D6145)

Reviewers: bu5hm4n, cedric

Reviewed By: bu5hm4n

Tags: #efl

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

elm_entry: add elm_entry_prediction_hint_hash_set/del API

elm_entry_prediction_hint_hash_set API sets the prediction hint data at the specified key, and
elm_entry_prediction_hint_hash_del API is for deleting the prediction hint data identified by a key.

@feature

Signed-off-by: Jihoon Kim <jihoon48.kim@samsung.com>

ecore_wl2_window: Silence early commit ERR for mouse cursors

Summary:
Our wayland mouse cursor code can trigger commits with commit pending
when mousing into a window across CSD, which results in quickly setting
the default cursors then an animated resize cursor before the first commit
has finished.

Fixing this is non trivial, and the bug is just a harmless inefficiency
of little impact, so just disable the ERR for that specific case instead.

Reviewers: zmike

Reviewed By: zmike

Subscribers: cedric, zmike

Tags: #efl

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

eolian: push missing event type changes in tests

efl_ui_tags: Fixed bug when setting format string.

Summary:
efl_ui_format_string_set was not working well.

Changed default format text. ("++++ %d" text for test only.)

@fix

Reviewers: Jaehyun_Cho, cedric, woohyun

Reviewed By: Jaehyun_Cho

Subscribers: zmike, cedric

Tags: #efl

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

efl_ui_spin: Added special value feature.

Summary:
User may want to set as meanignful words for the value instead of numberic value.

@feature

Test Plan: elementary_test->test_ui_spin_button.

Reviewers: cedric, woohyun, Jaehyun_Cho, herb

Reviewed By: Jaehyun_Cho

Subscribers: zmike, felipealmeida, cedric

Tags: #efl

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

eolian: integrate event type requirement into eo file grammar

eolian: force-enable event type checks

Now that event types have all been fixed, force-enable the
validation check.

efl: Add missing event types

Summary:
All events must have a type now, otherwise bindings don't know how to handle
the event_info field.
Most of the missing event types were actually "void" (no event_info present).
Some struct definitions had to be moved to eo instead of h files, so they
are available to bindings. Some have not, and are marked with FIXME.
Some namespaces have been fixed (like Efl_Event_Cb -> Efl.Event_Cb).

In general, there are hundreds of changed files, but mostly to add a type which
was not present before, so there's no harm done.
Also, A lot of FIXMEs have been added which should be, like, fixed.
For example, some events can send different types of event_info, which is
very inconvenient (and error prone).

Test Plan: make with c# bindings works, make check and make examples work too.

Reviewers: cedric, q66, lauromoura

Subscribers: zmike

Tags: #efl

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

ecore_con: add check for invalid space("") name

Summary:
space("") name check had existed, but it was removed on f4306d654dc55e8a0449698da18f371644d656df
(in the _efl_network_connector_efl_object_finalize)
This patch restore the safety checks

Reviewers: barbieri, Hermet, Jaehyun_Cho, myoungwoon, zmike

Reviewed By: zmike

Subscribers: cedric, zmike

Tags: #efl

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

ee_wayland: Fix device leak on disconnect

Summary:
A reference was taken on these when they were added, so by passing NULL
here we're not allowing that reference to be dropped.  This results in
not only leaking the device forever, but also potentially leaving it as
a default device after it's removed so that no new attach will replace it.

Under weston all devices are removed on a VT switch, and when you switch
back the default device is wrong, which leads to problems with events
like "mouse in" which, for legacy reasons, don't take a seat, but
instead look up the default seat.

This allows the delete callback to be fired for the first time ever,
and while I've tried to fix some bugs this has revealed, I'm not
actually sure I've caught them all.
Depends on D6182

Reviewers: zmike

Reviewed By: zmike

Subscribers: cedric, zmike

Tags: #efl

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

evas_main: Fix deleting pointer data

Summary:
This appears to be called from a delete callback that takes place well
after the eo parent relationship is deleted, however
efl_input_device_get_seat() finds the seat by finding the parent.  That
will always be NULL during this callback, so we'll leak the data.

Instead, search all seats for the pointer.
Depends on D6181

Reviewers: zmike

Reviewed By: zmike

Subscribers: cedric, zmike

Tags: #efl

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

evas_device: Only remove pointer data on deletion of pointer devices

Summary:
This delete callback is shared for mouse, pointer, touch, and seat.  We
should only delete pointer data when we're called on a pointer.

Note that (at least under wayland) this particular bug is irreproducible
right now because, due to a ref leak, this deletion callback can never
fire.
Depends on D6180

Reviewers: zmike

Reviewed By: zmike

Subscribers: cedric, zmike

Tags: #efl

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

efl_input_device: Fix seat_get

Summary: This was testing the same thing twice.

Reviewers: zmike

Reviewed By: zmike

Subscribers: cedric, zmike

Tags: #efl

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

cursor: fix ABI break in widget item cursor APIs

Summary:
fix ABI break in widget item cursor APIs
by efl_ui_cursor interface implementing.

item's view were not elementary widget mostly,
reimplementing efl_ui_cursor on efl_ui_widget
cannot resolve item's cursor API call.

Test Plan:
check,
elm_object_item_cursor_XXX
API is working properly on elm_object_item.

Reviewers: cedric, jpeg, raster

Reviewed By: raster

Subscribers: stefan_schmidt, zmike

Tags: #efl

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

eo: do not expose callback,add and del events in bindings

These events have been removed from efl_object.eo. Bindings can
still use the EFL_EVENT_CALLBACK_ADD and EFL_EVENT_CALLBACK_DEL
events from C, but they won't be available to the bound language.
The rationale is that bound languages probably will have their
own way to handle callbacks and C function pointers will mean
very little to them. For example, the binding code could route
the native callback through a dispatcher so the received
function ptr would always be that of the dispatcher method.

Closes D6179.

efl_mono: Remove callback,add from event tests.

Test Plan: make check

Reviewers: segfaultxavi

Reviewed By: segfaultxavi

Subscribers: cedric, zmike

Tags: #efl

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

efl_mono: Start generating eina future in eolian_mono.

Summary:
Besides the normal methods returning Futures, we now generate
a wrapper with the "Async" suffix. This wrapper returns a
Systems.Threading.Tasks.Task which can be awaited on and reflect the
status of the Future.

When an eina.Future fails with ECANCELED, TaskCanceledException is
raised in the Task. Otherwise, an efl.FutureException(eina.Error) is
raised.
Depends on D6174

Reviewers: felipealmeida

Reviewed By: felipealmeida

Subscribers: cedric, zmike

Tags: #efl

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

efl_mono: Initial support for Futures/Promises

Summary:
Promise/Future cleanup:

In the promises, we use a wrapper Eina_Promise_Cancel_Cb to
invalidate the wrapper if it ever gets cancelled from outside. When
invalidating from C#, we can do it directly.

For the futures, likewise, in order to be able to invalidate the
wrapper when the chain it belongs to gets resolved we then() an
internal future with a callback to invalidate the wrapper we return to
C#. The return of this intermediate then() is the future we actually
return to the user.

Also added ECANCELED to the list of default eina.Errors
Depends on D6173

Reviewers: felipealmeida

Reviewed By: felipealmeida

Subscribers: cedric, zmike

Tags: #efl

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

efl_mono: Fix eldbus warnings and typo.

Summary: Depends on D6172

Reviewers: felipealmeida

Reviewed By: felipealmeida

Subscribers: cedric, zmike

Tags: #efl

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

efl_mono: Add support for Eina.Error/Empty in eina.Value

Summary:
eina.Value.Empty now means that we have an zeroed (empty) eina value.
For optional values that are empty use eina.Value.OptionalEmpty.

This was required to support the empty values passed with
EINA_VALUE_EMPTY in some Ecore futures.

Also, returning an eina_value by value is not supported in eolian
for safety reasons, so we removed some tests that tried to use this
behavior.
Depends on D6171

Reviewers: felipealmeida

Reviewed By: felipealmeida

Subscribers: cedric, zmike

Tags: #efl

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

efl_mono: Add documentation tag in part decl

Summary: Depends on D6170

Reviewers: felipealmeida

Reviewed By: felipealmeida

Subscribers: cedric, zmike

Tags: #efl

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

efl_mono: Fix generation of event arguments

Summary:
Event argument was getting its type name from another path, different
from the type_generator.

In later commits the other functions from name_helpers should use the
type_generator too.

Reviewers: felipealmeida

Reviewed By: felipealmeida

Subscribers: cedric, zmike

Tags: #efl

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

eolian: add code to run static checks but don't enable yet

This adds code to run the Eolian static checker as part of tests,
but considering how many failures there are at this point, does
not enable it. Once they're all fixed, it can be enabled by
removing the #if 0.

ci: travis: tweak the IRC notification message template to our needs

I wanted to have the commit subject in it. This could be further changed
if needed.

https://docs.travis-ci.com/user/notifications/#Configuring-IRC-notifications

ci: travis: enable IRC notifications to #edevelop

efl_ui_scrollable: apply scroll_hold_push/pop and scroll_freeze_push/pop

Test Plan: elementary_test -> Efl.Ui.Scroller -> move slider

Reviewers: jpeg, woohyun, akanad, cedric, Jaehyun_Cho

Reviewed By: Jaehyun_Cho

Subscribers: zmike, Jaehyun_Cho, eagle001, cedric

Tags: #efl

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

evas textblock: manage default style properly for new interfaces

Summary:
Calling efl_canvas_text_style_set() with empty key means
setting a default style to object. But, it counldn't store style
as default properly. It caused a crash issue from elementary_test.
@fix

Test Plan:
New test case is included. Run test suite. Or,
1. Run elementary_test
2. Find and launch "Image Zoomable animation" test.
3. Close the image test window.
4. See the crash issue.

Reviewers: raster, herdsman, jpeg, cedric, zmike

Subscribers: zmike

Tags: #efl

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

Committer's note: rebased and removed unrelated test.

evas textblock: update format nodes when a Evas Textblock Style is updated

Summary:
A style tag among a text has to be replcaed by its matched tag when
a format node is created. If the matched tag is changed, format nodes
should be updated.
But, if a style text for Evas Textblock Style is changed,
related format nodes are not updated without setting new text.
The patch changes to update format nodes when new style text is set.
@fix

Test Plan: Included in Evas Test Suite.

Reviewers: raster, tasn, herdsman, subodh6129, zmike

Subscribers: zmike, cedric, z-wony, Blackmole

Tags: #efl

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

Committer's note: rebased.

efl_ui_win: hand-ref-count the provider object

Summary: Depends on D6108

Reviewers: cedric

Reviewed By: cedric

Subscribers: zmike

Tags: #efl

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

Introduce efl_ui_widget_focus_manager

Summary:
The mixin encapsulates the correct
- creation
- composition attaching
- Lifecycle handling

of the focus managers that are assosiated with the object.

This fixes error messages on shutdown, and additionally lifetime issues
where the composite_attached object was deleted before the object was
deleted.

Reviewers: cedric, zmike

Reviewed By: zmike

Subscribers: segfaultxavi, zmike

Tags: #efl

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

build: ensure we delete all new check-results*xml files

When we split up the tests to run them in parallel we also created a lot
more xml report files. Make sure we delete them when cleaning up.
Nicely spotted by distcheck by complaining about non removed files.

ci: test the installed parts of efl

Summary:
ci for now was not testing if building against the installation of efl
was working, this builds the lifegame example which ensures that elm is
build correctly.

Reviewers: stefan_schmidt

Subscribers: cedric, zmike

Tags: #efl

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

ci: travis: move macos build out of allowed failures

It is working fine again and we want to know if something breaks it.

ci: travis: switch Ubuntu and Fedora builds to latest release

edje: Fix to support translation for textblock part

Summary:
Text translation in edc for textblock is missing.
Added to support it.

@fix

Reviewers: cedric, zmike

Reviewed By: cedric, zmike

Subscribers: shilpasingh, zmike

Tags: #efl

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