evas_drm: Make the fbs an array of pointers

Summary:
Use pointers instead of an array of structures, since we're going to
replace the array with a list shortly.

ref T7099

Reviewers: devilhorns

Reviewed By: devilhorns

Subscribers: cedric, #committers, zmike

Tags: #efl

Maniphest Tasks: T7099

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

tests/ecore: remove ecore-drm test

Summary:
this caused DSO linker issues when enabled and was only testing
init+shutdown for a deprecated component which has not been actively developed
in some time

Reviewers: devilhorns, ManMower, bu5hm4n

Reviewed By: devilhorns, ManMower

Subscribers: bu5hm4n, ManMower, cedric, #committers

Tags: #efl

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

ecore_wl2&elput: correctly fill the key structure

Summary: the key structure has two fields key and keyname, those should be identical to the keystructures from x. the method xkb_keysym_to_utf8 however returns different values for keys like "minus" thus only relying on xkb_keysym_name files this issue.

Reviewers: eagleeye, devilhorns

Reviewed By: eagleeye

Subscribers: cedric, #committers, zmike

Tags: #efl

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

efreet: include buildsystem.h in efreet_cache.c

Summary: lib/efreet/efreet_cache.c:121:6: warning: implicit declaration of function 'bs_binary_get' is invalid in C99 [-Wimplicit-function-declaration]

Reviewers: devilhorns

Reviewed By: devilhorns

Subscribers: cedric, #committers

Tags: #efl

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

ecore_wl2: Include window_v6.x in the dist even if not configured for wayland

Summary:
Release tarballs need this file regardless of whether --enable-wayland is
specified or not.

fix T7120

Reviewers: devilhorns

Reviewed By: devilhorns

Subscribers: cedric, #committers, zmike

Tags: #efl

Maniphest Tasks: T7120

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

eio_model: Fix deleting files that don't have assigned types yet

Summary:
Trying to delete a file from a creation notification callback can
fail.  Sometimes the eio model test would sit forever in select()
waiting for events that will never occur because of this.

This happens since d84a268a71b09d585f1672b44ade698d7babe28d broke
deleting of files that haven't yet been assigned a type.  Before
this commit a delete_me flag would be set before attempting to
build a stat buf asynchronously, and then on completion the file
would be deleted.

I think this was changed because that could potentially race with
other async calls and delete the file sooner than expected.  So
instead of reverting I've made a special delete path that shouldn't
race with non-delete paths.

Reviewers: devilhorns, zmike

Reviewed By: zmike

Subscribers: cedric, #committers, zmike

Tags: #efl

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

Revert "edje: clean up internal objects during invalidate"

This reverts commit 78cd61034185f07779d70163a38f5ffdffe5507b.

this patch needed a bit more review+testing; it fixes some issues but
creates others

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

evas gl-x11 engine - nvidia driver - fix performance drop

this has been going on for a while. on nvidia drivers in gles mode on
x11 there is a massive perf drop to like a few fps with enough windows
if we build for egl/gles instead of opengl. it was the re-creating of
eglimages every frame. put a vendor specific workaround for this and
avoid it. it's not needed there anyway. framerate back to 60fps
smoothness afterwards.

@fix

ui scroll_manager: update doc.

Ui.Scroll_Manager: restrict to Efl.Ui.Pan type for 'pan' property

Summary:
The implementation calls efl_ui_pan_xxx on what was assumed to be any
Efl.Canvas.Object. This changes the type to be Efl.Ui.Pan.

Also, reordered headers due to the introduced dependency.

Reviewers: devilhorns, Hermet

Reviewed By: Hermet

Subscribers: cedric, #committers, zmike

Tags: #efl

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

edje: clean up internal objects during invalidate

Summary:
these objects are destroyed when invalidate is called as a result of the
evas smart object class destroying the smart object at this time

ref D6222

Reviewers: bu5hm4n, devilhorns

Subscribers: cedric, #committers

Tags: #efl

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

tests/ecore-con: add a global timeout timer for all tests

Summary:
it's possible for many tests in this suite to hang indefinitely, so ensure
that they terminate eventually instead of holding up the build

ref T6838

Reviewers: stefan_schmidt, bu5hm4n, devilhorns, Hermet

Reviewed By: Hermet

Subscribers: cedric, #committers

Tags: #efl

Maniphest Tasks: T6838

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

updated AUTHORS

eina: Improve documentation for binbuf/strbuf/ustrbuf

Summary:
These three classes have essentially the same API, just different
underlying data types.  This patch seeks to improve the docs for all
three while retaining or improving the consistency between them.

Several functions had completely incorrect documentation (looks like
cut-and-paste mistakes), others had missing or incorrectly named
parameter documentation, typographical errors, or other similar issues.
This also cleans up a lot of spelling and grammar errors, defines return
values as part of @return, and reformats/revises doxygen code for
consistency.

There are no changes to code, except some whitespace cleanup.

Reviewers: devilhorns

Subscribers: cedric, #committers, zmike

Tags: #efl

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

elm/ctxpopup: check for content object before removing callbacks during del

Summary: ref T7030

Reviewers: devilhorns

Reviewed By: devilhorns

Subscribers: cedric, #committers

Tags: #efl

Maniphest Tasks: T7030

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

tests: Add a password callback for eet_test_identiy

Summary:
For whatever reason the test is stalling for me waiting for me to
entire a password at make time.  Provide one internally.

Reviewers: devilhorns, zmike

Reviewed By: zmike

Subscribers: cedric, #committers, zmike

Tags: #efl

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

elm_focus_legacy: remove wrong EINA_UNUSED

Summary: Depends on D6532

Reviewers: devilhorns, segfaultxavi

Reviewed By: segfaultxavi

Subscribers: cedric, #committers, zmike

Tags: #efl

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

elm_focus_legacy: check if top is really a window

Summary:
if not, things are going to fall apart, as manager_top then can be NULL
or invalid. Top has to be a window element, if this is not the case,
then the widget tree of the given widget is dangling somewhere in the
void. Calculating the next object in there or even the active manager
will result in errors, since the active manager is not really the active
manager, but rather just a manager object somewhere in a danging widget
subtree.

Moving the focus into such a dangling widgettree might result in a stuck
focus rect on this object, since the DFS of the focus manager
implementation cannot backtrack anymore into the widgets that are still
part of the widget graph.

Depends on D6531

Reviewers: devilhorns, segfaultxavi, zmike

Reviewed By: segfaultxavi

Subscribers: cedric, #committers, zmike

Tags: #efl

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

elm_focus_legacy: fix focus moving for legacy

Summary:
elm_object_focus_next was not working correctly for objects where obj is
not the focused object.

fix T5940

Reviewers: devilhorns, segfaultxavi, zmike, stefan_schmidt

Reviewed By: segfaultxavi

Subscribers: cedric, #committers, zmike

Tags: #efl

Maniphest Tasks: T5940

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

elementary: Avoid passing NULL in some win focus methods

Summary:
This one is rather harmless, as the NULL returned by this fuction is handled
correctly in all places, but this commit removes the runtime warning.

ref T7030

Test Plan: After this commit the warning disappears. Got the warning with win_example.

Reviewers: zmike, bu5hm4n, devilhorns

Reviewed By: devilhorns

Subscribers: netstar, cedric, #committers

Tags: #efl

Maniphest Tasks: T7030

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

tests/eina: make inclusion for define explicit and more compatible

Summary:
signal.h is not included with all build options somehow, making this
a bit unreliable. also check for define existence for random platforms
which don't define this symbol

Reviewers: devilhorns, ManMower

Reviewed By: ManMower

Subscribers: ManMower, cedric, #committers

Tags: #efl

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

tests/ecore: check uid on ecore-file tests

Summary:
these tests will fail if run with root permission, so avoid checking them
when run as root

ref T7094

Reviewers: devilhorns

Reviewed By: devilhorns

Subscribers: cedric, #committers

Tags: #efl

Maniphest Tasks: T7094

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

eina: remove no longer used function _eina_thread_queue_msg_block_real_free

Summary:
It seems this function is no longer used since some rework. Clean it up.
lib/eina/eina_thread_queue.c:127:1: warning: ‘_eina_thread_queue_msg_block_real_free’ defined but not used [-Wunused-function]

Reviewers: devilhorns

Reviewed By: devilhorns

Subscribers: cedric, #committers, zmike

Tags: #efl

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

C# bindings: Make efl_loop_timer.eo available to bindings

Summary:
This file contains both legacy and new API code, this makes it a little
difficult to deploy.
In Makefile_Ecore.am files can be marked either as legacy or public.
If it is not in legacy, make distcheck fails because Ecore_Legacy.h includes
efl_loop_timer.eo.h.
If it is not in public, bindings are not generated for it.
It cannot be in both places, or make install fails, refusing to install the
same file twice.

Fortunately, there is an ugly place to put files like this one, and it's
already full of other outcasts, so...

Fixes T7114

Test Plan: make distcheck still works, and C# bindings are generated for efl_loop_timer.eo

Reviewers: zmike, bu5hm4n, devilhorns

Reviewed By: zmike

Subscribers: vitor.sousa, cedric, #committers

Tags: #efl

Maniphest Tasks: T7114

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

popup: fix popup sizing when scroll enabled.

Summary:
Force immediate calculate on main_layout after sizing hints set.

@fix T6886

Test Plan: Elementary_test: popup -> select scrollable -> use popup examples.

Reviewers: #committers, zmike, stephenmhouston, bu5hm4n, devilhorns

Reviewed By: #committers, zmike, stephenmhouston

Subscribers: cedric

Tags: #efl

Maniphest Tasks: T6886

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

Revert "edje_cc: temporarily disable aborting compile when namespace validation fails"

Summary:
This reverts commit 0dc492087ed048f153bc45f488f1504034fdcd30.

there are no longer any errors here so this should fail the build any time errors
are introduced

Reviewers: devilhorns

Reviewed By: devilhorns

Subscribers: cedric, #committers

Tags: #efl

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

selection_manager: Silence silly ERR messages

Summary: These should be DBG, they're not errors.

Reviewers: devilhorns

Reviewed By: devilhorns

Subscribers: cedric, #committers, zmike

Tags: #efl

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

selection_manager: Remove unused variable

Summary:
gcc somehow didn't notice this until an unrelated use of the variable
later was removed.  Now it's a warning.
Depends on D6523

Reviewers: devilhorns

Reviewed By: devilhorns

Subscribers: cedric, #committers, zmike

Tags: #efl

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

ee_wayland: Remove extraneous unregister

Summary:
ecore_event_window_unregister(ee->prop.window) is actually *exactly*
the same as ecore_evas_input_event_unregister(ee)

So this sequence just uselessly tries to remove something from an empty
hash table.

Reviewers: devilhorns

Reviewed By: devilhorns

Subscribers: cedric, #committers, zmike

Tags: #efl

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

build: always check for fork() and clearenv()

these are used in places other than quicklaunch and must be correctly
detected in order for functionality to be enabled

@fix

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

efreet: simplify and fix efreetd launch

this previously used an entire eina prefix to determine where to find
efreetd, when a simpler approach would have been to just pass the directory
where it's being installed

this also inhibited running the correct efreetd during in-tree builds and tests,
as it was using the install prefix instead of the in-tree wrapper script

@fix
fix T6713

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

ecore-con/proxy_helper: fix in-tree run path

this is located in the bin/ecore_con directory, not bin/ecore_con/utils,
so ensure that we pass the correct path in order to avoid not finding the
file

@fix
ref T6713

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

edje_cc: disable efreetd connection

edje_cc does not use any of this functionality, and it's problematic
at build time since efreetd is not run in-tree and does not use in-tree
libraries

ref T6713

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

release: Update NEWS and bump version for 1.21.0-alpha1 release

Revert "elua: disable this for now"

This reverts commit 7b4ff708dc4063731829cc919e056fdc431d6de1.

This had a side effect of breaking the distcheck build (which I need for
the upcoming alpha today). I talked to Marcel and we agreed on reverting
this for now and come back with a fully working patch afterwards.

elementary textpath: code refactoring.

Don't twist code unnecessarily.
logically same, round() is enough.
Differential Revision: https://phab.enlightenment.org/D6501

theme: skip namespace validation for "e/modules/music-control/popup" group

as this group is used by an external application, it can never be modified,
and thus namespacing issues cannot ever be resolved. for this reason there is
no other option except to skip validation of this group

fix T7072
Depends on D6388

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

edje_cc: add 'skip_namespace_validation' keyword

this pairs with the newly-added -N option to denote groups in edc files
which should not be checked for correct namespacing

the option is useful for cases such as music_control.edc,
where a group has been shipping for years with API signals like
"btn,clicked" which cannot be changed since they are used in an external
application and compatibility must be maintained

the documentation for this option explicitly states that it must only be used
inside an #ifdef SKIP_NAMESPACE_VALIDATION block, allowing this keyword to
be easily removed at a later point

ref T7072

@feature

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

ecore_evas: skip rendering evas and making animator tick

Summary:
Some ecore_evas such as ecore_evas_extn_plug doesn't have evas.

ecore_evas_extn_plug seems to be Ecore_Evas, but actually it is Evas_Object_Image.
ecore_evas_extn_plug makes new ecore evas, but it only exists to communicate with ecore_evas_extn_socket.
newly ecore evas  only open and close file(ecore_evas_extn_socket). so it doesn't have evas.

```
EAPI Evas_Object *
ecore_evas_extn_plug_new_internal(Ecore_Evas *ee_target)
{
...
   ee = calloc(1, sizeof(Ecore_Evas));
...
   o = evas_object_image_filled_add(ee_target->evas);
...
  return o;
}
```

Reviewers: zmike, Hermet, woohyun, raster, devilhorns

Reviewed By: zmike

Subscribers: cedric, #committers

Tags: #efl

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

evas/main: call correct destroy function in evas_free

Summary:
an evas may or may not have a parent; this is legacy api and it's all
confusing

Reviewers: bu5hm4n, devilhorns

Reviewed By: bu5hm4n

Subscribers: cedric, #committers

Tags: #efl

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

tests/eo: add explicit test case for setting an object as its own parent

Summary: Depends on D6489

Reviewers: bu5hm4n, devilhorns

Reviewed By: bu5hm4n

Subscribers: cedric, #committers

Tags: #efl

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

eo/base_class: explicitly refuse to set an object as its own parent

somehow this has been possible until now. no.

@fix

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

elm_popup: ensure focus is emitted on this object

Summary:
This mirrors the focus property from the notify to the one from popup.

This is needed to establish legacy behaviour, as earlier popup was able to get focus.

ref T6707
Depends on D6510

Reviewers: zmike, devilhorns

Reviewed By: zmike

Subscribers: cedric, #committers

Tags: #efl

Maniphest Tasks: T6707

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

efl_ui_focus_manager_root_focus: mirror the focus properties correctly

Summary:
this object takes a internal dummy object, and thus the properties
should be mirrored to the real root.

The focus property on the root of the manager is required that the outer world really sees that the focus is on the root or not.

Depends on D6506

Reviewers: zmike, devilhorns

Reviewed By: zmike

Subscribers: cedric, #committers

Tags: #efl

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

elua: disable this for now

Summary:
the files that are generated are calling files that dont exist anymore.
This cannot work and is only eating time when building, disable this for
now. It can be reenabled late.
The api missing was removed in fc88037977dcc39dfd6d817c522cce01f5bfa024.

Reviewers: devilhorns, ManMower

Reviewed By: ManMower

Subscribers: ManMower, cedric, #committers, zmike

Tags: #efl

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

elementary: Add missing EINA_UNUSED for unused function parameter

ecore: fix public/private class separation

Summary:
A few classes were added in the wrong place in the makefile, rendering them
neither public nor legacy, and unaccessible to bindings.

Test Plan:
The Efl.Model_Item class is now accessible to C#, and previously it wasn't.
This class is used in the EO tutorials, which would have needed a rewrite.
make check, examples & distcheck still work.

Reviewers: bu5hm4n, zmike, devilhorns

Reviewed By: zmike, devilhorns

Subscribers: cedric, felipealmeida, lauromoura, #committers

Tags: #efl

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

Migration: Fix Build Break

[Title]    Fix to the bug which is the shared evas_gl_context with multi window

[Problem]  When multi windows are shown, the latest window does not shown.
[Cause]    The latest window can not call glViewport with the updated coordinate
[Solution] Force call glViewport when window resizing occurs
[Team]     Graphics

Change-Id: I6df3f021385fe24190854abe84cb180d236a9dda

emile image: close file properly.

@fix

Change-Id: Ied6436f4bc4cbb048a5c136ee000909a8bbd3aeb

ecore_evas_tbm : add tbm_surface_map and unmap at tbm_pixels_acquire()
and release()

Change-Id: I6522c3d7730591cce7975b9d15d6ebc066e4174d

evas : modified to share resource by each gl backend

Change-Id: I5dfaa8ea94a840f3e1512707699bda8b8e2abf19

evas_tbm : Add more thread safety

Change-Id: I093469f2f081205f9fa633487313c5b9bded4521

evas : Change tbm_bufmgr to tbm_dummy_display.

Change-Id: I7d9d88f01a6ff9960b6e0149ee0a55c2e80b41e3

Revert "Revert "evas_gl: fix a TC fail of EGL_SYNC""

This reverts commit e491817c99e5a5d909af8b11453523dcede495ee.

Change-Id: I55bd46ebbfbff5390a88fa7c09de042fff2ac436

Revert "Revert "evas : remove return null""

This reverts commit 0c6502556008708c8441784525e2015c521c91d8.

Change-Id: I85b4f7e9cf9b3401336e634f7d52e572d8ea80fb

Revert "evas : remove return null"

This reverts commit 87eba6aa9eb2d31f496f845df4c3a79d73405ec1.

temporarily revert for SR.

Change-Id: I70deb73fba407212d585b01dc2cff728a328f0c1

Revert "evas_gl: fix a TC fail of EGL_SYNC"

This reverts commit a757cd4e390f3f01261f1d3179632be0588354bc.

temporarily revert for SR.

Change-Id: I26d6feb0cae96f38e266df48e66c88f17776c576

evas_gl: fix a TC fail of EGL_SYNC

Change-Id: I55e5972f82b33fc5af155cb85d6e7550f47f08ca

Fix crash in tct tests

Change-Id: I5148a0791288b195ee67f6227ffb0118fd615799

evas : remove return null

Change-Id: I539c597e91da10f9e2d5efcf76b61ac68fccba41

Fix for invalid navigation on list widget

Change-Id: I0bc8b2855c44fb8075651e993d0f3c5e4869ec3c

evas_gl: fix EVAS_GL_EGL_SYNC_ON

Change-Id: I6ae26b9fff0db48a4ce001fd006d613b83b11c6a

evas textblock: adds missing legacy types

*upstream patch(need a review)
 https://phab.enlightenment.org/D6529

@tizen_fix

Change-Id: I5360694dcad68003c0444c1643d81e92e1f40e67

elm_toolbar: fix flickering issue when item resized.

resize the box first to avoid flickering of
saved state of the toolbar when ratation changed but not in case of scroll.

Change-Id: I1ba904a8c9a346f707b869f03ebd15e07284566c
Signed-off-by: Bowon Ryu <bowon.ryu@samsung.com>

gengrid: fix component_highlight_grab

Change-Id: I759fcadd0f0403ae5898c77840857c41780a1aae

Fix navigation at CtxPopup

Change-Id: I20604a51f69fb10e064435fd52bbb1a7e36ee34c

Fix of invalid scrolling from first to last

fix scroll from first to last at apps-screen

Change-Id: I3926b8302ed28924d232811213ff27ac3a4dcbfd

evas : fixed downscale sampling issue

    Fixed an issue where image wave due to differences in sampling
    coordinates when overlapping images

Change-Id: Ice000379c0397b52fd3bd44c84e8a7bb517df6bc

multibuttonentry: Add Tizen Only features to maintain compatibility.

Change-Id: I64684105f79a6d06a473ff4d889d1f706311f054

Revert "Fix structure of AT-SPI genlist subtree"

This reverts commit e17303010bfe5fe53127c708fc7fa56fc346630b.

Revert "[evasgl] change swap mode when rect size not match tile size"

This reverts commit 9a7a006c5e38e5991ed7707e7fb35e8ceb1a3839.

This stopped partial rendering that breaks tv performance.
We need to make sure the precise scenario that requires this patch.
There may be a better solution to clip the drawing region
 with exact multiple of tile size. see...

Change-Id: If9a792d9d3ed93c5fe92e9862f29f4345f3571bd

Revert "eo: Turn thread-validation off for unstable Tizen 5.0 apps which are violating thread-safety."

This reverts commit 017b38c5e98bfa2906782ed172d5d9c997d39414.

No more requirement by VD, so forget this.

Change-Id: Icda4cc71147f0a3cae0c85666fc83243b7186469

Fix structure of AT-SPI genlist subtree

Change-Id: I3eefa7a6ebcd2f69004032c1d2b706f8545365cc

update res folder that has been changed in upstream.

Change-Id: I12efe0ae0fcad4fdf65fbb96cf49cdb843012493

eina: Improve eina_binbuf function documentation

Summary:
Define return values as part of @return.
Cleanup grammar.

Reviewers: devilhorns

Subscribers: cedric, #committers, zmike

Tags: #efl

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

evas: add more error messages when passing invalid objects to evas callbacks

Summary:
this should only happen if the user has made a mistake regarding the
existence or type of an object, so ensure that an error message occurs to
help debug any failures which result

fix T6326

Reviewers: bu5hm4n, Hermet, woohyun, devilhorns

Reviewed By: Hermet

Subscribers: cedric, #committers

Tags: #efl

Maniphest Tasks: T6326

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

eina: Improve eina_benchmark function documentation

Summary:
Define return values as part of @return.
Clarify that the returned list of files are gnuplot filenames specifically.
Cleanup grammar throughout.

Reviewers: zmike, devilhorns

Reviewed By: devilhorns

Subscribers: cedric, #committers

Tags: #efl

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

efreet: use correct path when generating error message about stale efreetd socket

Summary:
using runtime directory in all cases for this is wrong, as ecore-con has a number
of fallback codepaths for the case where runtime directory is not set or not valid.

by using the same ecore-con function which ecore-ipc uses to generate the socket
string, the error message path should always be the same as the path which is
used by efreetd

extra linkage was required by efreet in order to use ecore-con functions, so
the internal lib variable in the build system was modified to provide this

@fix

fix T7045

Reviewers: devilhorns

Reviewed By: devilhorns

Subscribers: cedric, #committers

Tags: #efl

Maniphest Tasks: T7045

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

tests/eina: disable eina debug signal handling test in non-fork mode

Summary: signal tests can only be used in fork mode or else they just fail

Reviewers: devilhorns

Subscribers: cedric, #committers

Tags: #efl

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

tests/ecore-con: fix dns test to use correct ordering

Summary:
it's unclear to me why this test was implemented with the expected
event ordering of SERVER_DEL -> SERVER_ERROR; going as far back
as efl 1.7, the behavior has always been ERROR -> DEL

this fixes the test to verify the expected event ordering on a dns
resolve failure
Depends on D6481

Reviewers: devilhorns

Reviewed By: devilhorns

Subscribers: cedric, #committers

Tags: #efl

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

ecore-con/legacy: kill server with DEL event if a dialer error occurs

Summary:
in the course of the efl-net rewrite, the previous relied-upon behavior
sequence of error -> disconnect -> del was broken, and error events no
longer triggered disconnects or server deletion.

the failure was caused by a check for the 'connecting' flag, preventing
DEL events from being triggered

further, the ordering of the events was flipped from ERROR -> DEL to
DEL -> ERROR; this is also fixed now

this caused any failed connection to leak the entire server connection
since existing handlers for the del event were never triggered and the
server was never destroyed

@fix
fix T6330

Reviewers: devilhorns

Reviewed By: devilhorns

Subscribers: cedric, #committers

Tags: #efl

Maniphest Tasks: T6330

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

elm_test: fix invalid read in dnd image drop

Summary:
this selection is data with a specified length, not a terminated string

@fix
Depends on D6484

Reviewers: devilhorns

Subscribers: netstar, cedric, #committers

Tags: #efl

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

efl/selection_manager: fix some invalid memory reads

Summary:
these selections are data with specified lengths, not strings
Depends on D6483

Reviewers: devilhorns, bu5hm4n

Subscribers: bu5hm4n, netstar, cedric, #committers

Tags: #efl

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

efl/selection_manager: make selection manager a child of the app

Summary:
a selection manager is application-wide, not per-window. creating separate
managers for each window duplicates all callbacks for the window's display
server, guaranteeing broken behavior at any time when more than one window
exists

fix T6937

Reviewers: bu5hm4n, devilhorns

Subscribers: cedric, #committers

Tags: #efl

Maniphest Tasks: T6937

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

eina/debug: remove global lock for session list access

this list is now only accessed from the main thread so there's no reason
to have a lock for it

eina/debug: rework session thread destruction to avoid deadlock on shutdown

when debugging was active, this would frequently result in the monitor thread
blocking in a read() call for a fd which had already been closed by the main
thread. the main thread would be waiting for the monitor thread to fail its read()
and destroy itself, but the read() would never end without an interrupt

this reworks the monitor thread destruction to stop relying on the thread to
remove its session from the session list and instead only access the list from
the main thread, allowing sessions to always be immediately destroyed

fix T7103

efl_ui_focus_layer: do not call on NULL objects

This fixes the following backtrace

#5  0x00007ffff4ee87d0 in efl_ui_focus_manager_root_get (obj=0x0) at lib/elementary/efl_ui_focus_manager.eo.c:17
#6  0x00007ffff4ef50cb in _efl_ui_focus_layer_enable_set (obj=0x40000004adfe, pd=0x12fde40, v=0 '\000') at lib/elementary/efl_ui_focus_layer.c:127
#7  0x00007ffff4ef52dd in efl_ui_focus_layer_enable_set (obj=0x40000004adfe, v=1 '\001') at lib/elementary/efl_ui_focus_layer.eo.c:4
#8  0x00007ffff4ef4df0 in _efl_ui_focus_layer_efl_gfx_entity_visible_set (obj=0x40000004adfe, pd=0x12fde40, v=1 '\001') at lib/elementary/efl_ui_focus_layer.c:35
#9  0x00007ffff0f366a3 in efl_gfx_entity_visible_set (obj=0x40000004adfe, v=1 '\001') at ../src/lib/efl/interfaces/efl_gfx_entity.eo.c:15
#10 0x00007ffff5d832bb in evas_object_show (eo_obj=0x40000004adfe) at lib/evas/canvas/evas_object_main.c:1917
#11 0x00007fffcc365c74 in _bar_icon_preview_show (data=0xfaa7e0) at ../src/modules/luncher/bar.c:906
#12 0x00007ffff6d4305a in _ecore_call_task_cb (func=0x7fffcc36549c <_bar_icon_preview_show>, data=0xfaa7e0) at lib/ecore/ecore_private.h:442
#13 0x00007ffff6d434d2 in _ecore_timer_legacy_tick (data=0x12ca950, event=0x7ffffffc57c0) at lib/ecore/ecore_timer.c:160
#14 0x00007ffff0d06e90 in _event_callback_call (obj_id=0x4000001f25b2, pd=0x923820, desc=0x7ffff6f673e0 <_EFL_LOOP_TIMER_EVENT_TICK>, event_info=0x0, legacy_compare=0 '\000') at lib/eo/eo_base_class.c:1671
#15 0x00007ffff0d07162 in _efl_object_event_callback_call (obj_id=0x4000001f25b2, pd=0x923820, desc=0x7ffff6f673e0 <_EFL_LOOP_TIMER_EVENT_TICK>, event_info=0x0) at lib/eo/eo_base_class.c:1755
#16 0x00007ffff0d07208 in efl_event_callback_call (obj=0x4000001f25b2, desc=0x7ffff6f673e0 <_EFL_LOOP_TIMER_EVENT_TICK>, event_info=0x0) at lib/eo/eo_base_class.c:1758
#17 0x00007ffff6d4478e in _efl_loop_timer_expired_call (obj=0x4000000003dc, pd=0x890bf0, when=333436.894054887) at lib/ecore/ecore_timer.c:634
#18 0x00007ffff6d445bd in _efl_loop_timer_expired_timers_call (obj=0x4000000003dc, pd=0x890bf0, when=333436.894054887) at lib/ecore/ecore_timer.c:587
#19 0x00007ffff6d0b423 in _ecore_main_loop_iterate_internal (obj=0x4000000003dc, pd=0x890bf0, once_only=0) at lib/ecore/ecore_main.c:2317
#20 0x00007ffff6d08e66 in _ecore_main_loop_begin (obj=0x4000000003dc, pd=0x890bf0) at lib/ecore/ecore_main.c:1175
#21 0x00007ffff6d11757 in _efl_loop_begin (obj=0x4000000003dc, pd=0x890bf0) at lib/ecore/efl_loop.c:83
#22 0x00007ffff6d13e6e in efl_loop_begin (obj=0x4000000003dc) at lib/ecore/efl_loop.eo.c:28
#23 0x00007ffff6d08fe0 in ecore_main_loop_begin () at lib/ecore/ecore_main.c:1248
#24 0x000000000054817c in main (argc=2, argv=0x7fffffffdc78) at ../src/bin/e_main.c:1090

fix T7034.

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

elm_toolbar: handle the cleanup in the invalidate call

there is actaully no need to add the event, we can cleanup the pointer
in the invalidate call. Further more, the event was executed before the
method, thus checks against more_item failed because that was already
NULL. The result have been error messages, this fixes the error.

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

elm_toolbar: fix _item_visiblity

'more' should indicate when a more item is needed, this is needed if one
item is set to visibility false, thus checking this here is required.

 #FocusBug

fix T6806

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

efl_ui_bg: set scale manually to hand by hand

according to 09d3d5b85ad4a0b7441b5b1e508db0e595bef395 elm scale should
be ignored. However, setting no_scale breaks efl_ui_bg as no_scale
disables scale types, which are required to work.

fix T7080

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

efl_ui_image: apply the whole sizing eval

orientation no_scale etc. can have impact on the min max calculation.
Thus that should be called on every property set.

ref T7080

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

efl_ui_focus_graph: improve results based on lvls

This improves the results from the graph calculations.
Everything is documented in the comments.

The genlist of focus 6 is placed in a way that the button
at the top is not in reach, before we only had lvl1 and lvl2
(as described in the code comment), thus the button was only
reachable via right not via top. lvl3 makes it accessable
via top AND right.

fix T7098

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

Evas Textblock: Don't convert <br/> to <ps/> after changing text

Summary:
It was only happened when legacy newline is enabled. By default,
legacy newline is enabled. As I know, legacy newline option has
to change textblock's internal behavior. But, it shouldn't change
the given original text. It fixes T3399.

Test Plan: A Test case is included in Evas test suite.

Reviewers: Jaehyun_Cho, z-wony, tasn, woohyun, herdsman, Blackmole, devilhorns

Subscribers: #committers, zmike, raster, cedric, jpeg

Tags: #efl

Maniphest Tasks: T3399

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

elementary: fix buffer overflow in efl/calendar widget

I missed this one in the previous part name refactor, where the efl
namespace was added.
Differential Revision: https://phab.enlightenment.org/D6498

data: install icon according to FDO specification.

install to icons/hicolor/128x128/apps instead of icons/
Differential Revision: https://phab.enlightenment.org/D6497

efl_ui_focus_parent_provider_gen: provide results for adapters

Summary:
a adapter is used whenno real widget is created during realization, when
a adapter is used, the parent cannot be fetched via
elm_widget_parent_get, there we need efl_parent_get

Reviewers: ManMower, devilhorns, stephenmhouston, zmike, Hermet

Reviewed By: Hermet

Subscribers: Hermet, cedric, #committers, zmike

Tags: #efl

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

Evas textblock: clear ellipsis item at start of layout

The ellipsis item was tracked so it can be inserted at the correct
position. It was not cleared properly, leading to a randomly failing
test suite.

Actually, it seems that there is no need to even store the ellipsis item
as the object data, so it was also moved to be in the layout context,
instead.

A special shoutout to ManMower for lending me his machine, as the test
suite didn't fail on mine.

Fixes T6986

@fix

Evas font: simplify fallback parsing step

@fix

Evas font: add comparison for fallbacks in evas_font_desc_cmp()

Summary:
fallbacks string also has to be compared to load proper fdesc.
If a font which does not have fallback fonts is loaded,
fallback fonts can't be appended to the same font.
@fix

Reviewers: tasn, woohyun, herdsman, zmike, devilhorns

Reviewed By: herdsman

Subscribers: #committers, zmike, raster, Blackmole, z-wony, cedric, jpeg

Tags: #efl

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