efl_ui_popup: use "elm.swallow.content" name instead of "default" for scroller

Summary:
efl_part don't support legacy aliases.

ref 495edbd9958ff077e44a62bf83f1a9032875f423

Test Plan:
1. elementary_test -to 'efl.ui.popup.alert.text'
2. elementary_test -to 'efl.ui.popup.alert.scroll'

Reviewers: Jaehyun_Cho, herb

Reviewed By: Jaehyun_Cho

Subscribers: stefan_schmidt, cedric, #committers, zmike

Tags: #efl

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

efl ui win - fix legacy theme support correctly

this fixes T6918. during development legacy theme support was b roken.
473e657120ffd9626b6ee8832c75f2f66c9a5351 broke it.

eet - check for null edd's and don't decode junk data

in the process of fixing a bug in elm config i found the eet will
decode junk data with a null edd. check for these where it was missing
and return null so it's easier to see a problem. not really a fix as
it's invalid use of the api, but it's nicer to debug...

elm config - fix derived save crash

derived shutdown nuked the edd's and eet still decoded something...
that was junk. oddly. this fixes T6879.

Revert "ecore-evas-x: set draw_block until the window receives a configure event"

This reverts commit 7b80038fa7b54cff27b463382283211727aaf104.

JackDanielz asked nicely, but this hasn't been reverted. As this
totally breaks enlightenment (it's black) and this happens on
everythng I've tested (1 laptop, desktop and Xephyr) I'm calling this
patch a dud.

Now... what kind of review is going on here? This hasn't been tested.
What kind of review doesn't build + run things?

for the reasons of poor review and massive horribler fully complete
desktop like breakage ... this gets reverted as master should not be
broken like this.

ecore_con: efl_net_socket_ssl_adopted_get is EFL_NET_SOCKET_SSL_PROTECTED.

elm_spinner: Spinner value adjust when round enabled.

Summary:
The round value was not being applied when input the value as an internal entry.

@fix

Test Plan: elementary_test -> spinner sample.

Reviewers: cedric, woohyun, Jaehyun_Cho

Reviewed By: cedric

Subscribers: cedric

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

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

ecore-evas-x: set draw_block until the window receives a configure event

Summary:
drawing a non-override window before receiving a configure event results
in an unsized window, breaking spec. it also prevents ecore-evas resize
callbacks from triggering, yielding undefined returns from functions which
attempt to get the geometry of the ecore-evas

fix T6907

Reviewers: cedric, raster

Reviewed By: cedric

Subscribers: #committers, vtorri

Tags: #efl

Maniphest Tasks: T6907

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

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

eina: Improve eina_array function documentation

Summary:
Define return values when errors occur.  Eliminate use of the term
'vector' to avoid confusion with eina_vector.  Cleanup grammar
throughout.

Reviewers: cedric, Hermet

Reviewed By: Hermet

Subscribers: #committers, zmike

Tags: #efl

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

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

Revert "efl_ui_win: simplify checking theme version when adding frame"

Summary:
This reverts commit f37b96a1de80c6f65306c96c6ebcdc69bafdbfa1.

as pointed out in D5957, this would break use of theme_set during the
efl_add constructor once this is implemented in the future

Reviewers: raster, cedric

Reviewed By: cedric

Subscribers: #committers

Tags: #efl

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

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

elm_notify: remove allow_events_set call in destructor

Summary:
4976f1fc5276835940610b72ca4fa48776fc9410 erroneously flipped the
bool flag on this call, accidentally creating another object during
deletion which is illegal now as the notify object is now unparented
at this point which will result in a CRI error

this call was never required in the first place as the corresponding
layout object would have been deleted automatically due to widget parenting

@fix

Reviewers: cedric

Tags: #efl

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

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

edje: don't give a wrong Edje data when an Edje object has group parts

Summary:
_edje_part_fetch() function gets an Edje which has the requested Edje_Real_Part.
Basically, it gets main Edje of the given object.
But, if a requested part is in a GROUP part, it gets the Edje of GROUP part.
It shouldn't be passed to _edje_efl_text_text_get() function directly.
@fix

Test Plan: N/A

Reviewers: herdsman, raster, cedric, woohyun

Reviewed By: cedric

Subscribers: #committers, zmike

Tags: #efl

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

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

elementary widget: Do super's efl_gfx_color_set to apply color to widget itself

Summary:
The efl_gfx_color interface was not applied properly.
The implementation code of evas_object_smart_color_set was moved
to efl_gfx_color_set implementation code. But, these two functions are not same.
In efl_gfx_color_set impl, it has to call super's color set to apply
the given color values to widget object itself.

This bug caused color_set/get test failure and the following bug.
1. elm_image_add
2. evas_object_color_set
3. elm_image_file_set
4. show. See the given color is not applied. It was applied in the past.

Test Plan: color_set/get to elm_image object is included in test suite.

Reviewers: raster, cedric, herdsman, woohyun

Reviewed By: cedric

Subscribers: #committers, zmike

Tags: #efl

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

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

ecore_con : Fix class check to check inner_socket class

Summary:
cl->socket is a Efl.Net.Socket_Simple. it is not inherit the Efl.Loop.fd class.
and The target of the class check have to be the inner_socket.
But inner_socket is Efl.Net.Socket_Ssl. Efl.Net.Socket_Ssl class is not inherit FD class.
Efl.Net.Socket_Tcp is inherit Efl.Loop.fd class. So, Need to add Efl.Net.Socket_Tcp to inheritance.
(The server side is a similar hierarchy. (ssl -> tcp -> ip -> fd))

Test Plan: N/A

Reviewers: cedric, jypark, myoungwoon, zmike, barbieri

Reviewed By: cedric

Subscribers: #committers, bowonryu, zmike

Tags: #efl

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

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

elm_interface_scrollable: change return value for post events.

Summary:
1. Scroller in scroller case, callback process should continue.
     If child scroller does not scroll, parent scroller should be checked whether scroll or not.
     So returning EINA_TRUE is necessary.
2. down.dragged flag is always EINA_FALSE, so scroll is never unlocked.
@fix

Test Plan:
precondition
  elementary_config -> scrolling -> Set Scroll Smooth Amount to 0
elementary_test -> pagescroller

Reviewers: woohyun, zmike, singh.amitesh, cedric

Reviewed By: cedric

Subscribers: #committers, cedric, zmike

Tags: #efl

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

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

efl_mono: Fix efl_add_internal_end call.

Summary:
It was missing setting the 'is_ref' parameter to 1 to mirror the
internal_start call.

Test Plan: make check

Reviewers: cedric

Subscribers: #committers, zmike

Tags: #efl

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

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

eo_cxx: Fix do_eo_add after lifecycle branch.

Summary:
Also changed the previously failing tests to use ck_assert_int_eqinstead
of fail_if for better reporting.

Test Plan: make check

Reviewers: cedric, felipealmeida

Reviewed By: cedric

Subscribers: #committers, zmike

Tags: #efl

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

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

elementary textpath: improves text rendering quality of curved text

Summary:
There was wrong logic for calculating # of slices, dt, dist of each segment.
It caused bad rendering quality by putting too much slices on small text.

In addition, textpath didn't care about smoothness of curve's slope changing.
The patch fixes to check differences of previous points and next points for Evas Map.
So, textpath can show more smoothly curved text.

Also, it fixes "autofit" bug when text is much huge than given circle's size.

@tix

Test Plan:
- I'll attach screenshots for comparing rendering quality.
- To see "autofit" bug.
1. Run the following command.
   ELM_ACCEL=gl ELM_SCALE=2.0 elementary_test -to "efl.ui.textpath"

2. Toggle "autofit" check box.
3. See the bug case. Textpath can't show all text properly.

Reviewers: raster, cedric, thiepha

Subscribers: #committers, zmike

Tags: #efl

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

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

elementary: fetch the model explicitely.

T6939

efl_ui_win: fix wrong comparison in icon_object_set

@fix

elementary: prevent model from being invalidated when their parent model get destroyed.

elementary: ctxpopup is using legacy Elm.List which use legacy Elm.Object.Item, fix accordingly.

elementary: fix elm_list items lifecycle.

Elm.List.Item lifecycle where an exception in Efl. They were trying to prevent the death of
there parent, to avoid dealing with safely walking on items list. This has been on the todo
list for years and is now fixed by this patch.

evas: make smart object deletion more robust to bad child behavior.

elementary: refactor filesector target cleanup to not leave function handler behind.

elementary: make sure to use the model that generated the event to fetch information from.

elementary: make sure fileselector button model has a parent and self destruct properly.

eio: enforce eio model to have a parent that provide main loop access.

ecore: during shutdown their is no need to delay anything.

eo: an object is invalidated if it is invalidated or if any parent is invalidated.

ecore_con: disconnect from the server as soon as it request it.

Ecore_Event can still be in flight while the client get freed by the server.
We are protected agaist that and keep the client partially alive until the event
are processed, but we need to disconnect from the server as soon as possible
as we might outlive the server.

evas: disable del hook when pushed inside the hash.

This is necessary in case of cycling init/shutdown as the hash will
destroy the Event, but the hook will instantly reinsert it.

eo: match raise of error level in tests.

eo: we are actually testing internal function too here.

eo: add infrastructure to test critical Eina Error.

eo: improve tests suite debugging by always displaying the error message before abort.

eo: move wref set to NULL to append after last event.

Making wref disapear before the last event require additional work to use
them during the FREE event. As I think we should introduce a wref that
disapear after the invalidate state, it makes sense to move that to a later
stage. Especially because their is no specific event related to them
being set to NULL.

evas: improve test to actually test something meaningful.

elementary: fix configuration destruction by actually calling the proper way of destruction.

evas: actually we expect gradient to live as long as the part that is using them.

ecore_con: allow for NULL to do nothing.

elementary: the config subsystem expect a stringshare.

evas: multiple call to evas_object_del are allowed even after the first one succeeded.

eo: allow efl_part during object invalidate.

eo: remove parent_sink as this hack should not be necessary anymore.

eo: try to make efl_add and efl_add_ref refcounting sane in all case.

ecore_file: on shutdown properly set to NULL to avoid confusion when cycling init/shutdown.

ecore: allow for NULL file handler request.

elementary: destroy timer as they could tick during shutdown.

elementary: properly destroy block events object during destruction.

evas: do not crash if constructor fail and invalidate is run on a partial object.

eo: improve error reporting to display object pointer (without accessing it).

elementary: cleanup genlist destructor to order content destruction correctly.

evas: make evas_object_free work in case of lost reference to Eo object.

This is clearly a workaround to a bad situation, but there is no case
that I can find that solely lead to object being NULL nor is there any
code that should do that, but still it does in some case...

evas: detect incoherency in object content and report problem.

evas: handle Evas_Object constructor failure case better.

edje: simplify part handling as we are now guaranteed that a part as no parent when reaching the del intercept.

eo: make sure that reporting object refcount during efl_noref, doesn't lead to recursive loop.

eo: add warning in case of a terrible situation happen.

eo: efl_invalidate should also be triggered for object with no parent before the destructor.

ecore: properly cleanup variable on shutdown, so that cycling another init doesn't crash.

evas: improve evas child destruction order.

Technically I do not thing it is a correct behavior to force destroy
reference that evas do not hold, but evas_object are deaply tied to
the canvas they are build on and even after invalidate it is hard
to not have function call that would lead to crash. Making the pointer
incorrect thanks to eo indirection seems safer here.

evas: do not override lifecycle by overriding efl_parent_set.

evas: reindent code properly.

eo: actually enforcing the efl_parent_set code path is necessary to enforce the predictability of the code path.

eo: as efl_del is not an Eo API call anymore, manually protect call with refcount.

eo: update refcount before sending efl_noref event to avoid double update during events propagation.

ecore: because fd_handler do not have the main loop as parent, the main loop can die before them.

evas: move the del event to match invalidate and free event to the end of the destructor.

elementary: when there is no file listed yet, no need to request them.

eina: make it possible to return an EINA_VALUE_EMPTY by just requesting a new value of type NULL.

eina: check the type before looking at anything else to avoid triggering valgrind warning.

elementary: enforce walk to protect the object existence by using eo refcounting.

elementary: reorder operation to only call efl_parent_set when the object still has a chance to exist.

evas: destructor can not be called recursively, we should rely on Eo refcount.

eo: move destruction of the link with children to be always after invalidate.

eo: propagate invalidate before destroying the link to the parent.

evas: actually we do not even need to NULL it, if we wref it.

edje: do not reparent when the object is dead already.

eo: improve message for debugging life cycle issues.

edje: forgot to propagate invalidate upward.

cxx: Fix examples after part changes

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

examples: Elm: Add dependency on efl.la due to efl_part

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

efl_mono: Fixes after efl_part/del changes.

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

ecore_evas: we shouldn't do anything when we are deleted.

evas: preserve evas_object_del logic to not really destroy an object until it is fully unref first.

evas: enforce legacy parent to be the canvas.

elementary: refactor by using elm_legacy_add.

evas: add a way to figure out the evas class that should be the real parent for all legacy object.

elementary: make toolbar 'more_item' properly self destruct.

elementary: no need to recalculate the layout during invalidate.

elementary: the pan object of the map object was created with efl_add, adjust accordingly.

elementary: due to lifecycle difference between legacy object and unified object, we need to track their lifecycle more explicitely.

It is my understanding that some items view are created with efl_add directly
and manipulate VIEW directly with Eo new API. This clash with the inconsistent
behavior that evas_object_del expect. To work around this, we track object life
by explictely relying on efl_wref_add while holding the pointer to the object.

eo: efl_del should not be something anyone can override this can break so many things.

Allowing override of efl_del allow for messing with object life cycle which can create
problem for binding where an object was expected to be invalidated, but isn't really.

evas: try to protect old legacy lifecycle without overloading Efl.Del.

eio: properly handle Eio.Model lifecycle inside Elm.Fileselector.

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

elementary: dayselector use efl_add to create the object, calling efl_del during destructor make no sense.

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

elementary: no need to check for part during destruction.

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

elementary: no need for resize an object during destruction.

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

elementary: colorselector item are created with efl_add, so they are dead when they reach the destructor.

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