elm_config: force icons in preview to keep aspect ratio

#SizeHintsMess... I really cannot understand why aspect_fixed is not respected :(

Configure: 2 small in-string quote changes

This should be harmless and just fix syntax hilighting is some editors (geany at least)

eina: Fix magic check for eina_promise_owner_buffer_get

eina: Fix EINA_MAGIC and eina_promise_then NULL check

ecore: add a test for detecting the double free problem.

ecore: properly handle double free case of timeout promise.

The bug came from the fact we need to handle the destruction of the
main loop which destroy the underlying timer. The event handler that
catch the destruction of the timer can not make the difference between
eo_del call from the timeout code and eo_del from the main loop
destruction. By removing the event handler, the double free is properly
avoided.

ecore: use EINA_ERROR_PROMISE_CANCEL instead of a custom Ecore one.

ecore: do not generate error while rescheduling the last timer.

eina: protect promise death during callbacks cancel.

ecore: Ecore_Cb is used by elm_box.eo.

ecore: Efl.Timer should be Efl.Loop.Timer for coherence.

As we add more object in the main loop, they can't live in the top
namespace as they make little sense there (Efl.Fd !). For coherence,
everyone should in the loop namespace, so move timer there.

ecore: remove Ecore_Animator Eo object.

This is only a legacy API.

eina: Remove Eina_Promise* parameter to callbacks to promises in examples

eina: Add NULL checks and EINA_MAGIC checks to promises

Add NULL checks and EINA_MAGIC checks that avoid crashing when it
shouldn't.

eina: Remove more Eina_Promise* parameter in promise callback

Remove not very useful parameter to eina_promise_then callbacks.

wayland_shm: Fix more dmabuf failure corner cases

With the exynos allocator a failure can happen during a resize, and the
resize path wasn't properly falling back to wl_shm buffering.

psd_loader: Fix bad assumption about signedness of chars

Whether "char" is signed or unsigned is architecture dependent, so if we
know we need a -1 in a char type we must declared it to be signed.

ecore_anim: Fix bad assumption about signedness of chars

Whether "char" is signed or unsigned is architecture dependent, so if we
know we need a -1 in a char type we must declared it to be signed.

eina: Remove Eina_Promise* parameter in promise callback

Remove not very useful parameter to eina_promise_then callbacks.

eolian-cxx: Fix C++ example compilation

eina: Fix examples using new Eina_Promise semantics

eina: Remove unnecessary indirection to promises

Now when dealing with pointer types, we will not get pointer to
pointer semantics in callbacks and eina_promise_owner_value_set
for Eina_Promise.

It will work as expected:

Eina_Promise_Owner* promise = eina_promise_add();

void* p = malloc(sizeof(T));
eina_promise_owner_value_set(promise, p, &free);

autotools: use DISTCLEAN rules to properly destroy files.

ector: variable in _SOURCES do not work.

This fix the left over etc .deps during make distcheck.

wayland_shm: Add exynos allocator for dmabuf

Allows clients on exynos hardware to allocate GEM buffer objects
to back DMAbuf buffers.

wayland_shm: Abstract out dmabuf management

Split out the intel specific stuff into an abstraction in preparation for
adding exynos support.

Eina: Fix eina_promise_all refcount ordering problem

The call to eina_promise_then steals the first ref'count, so it is
possible that the promise is freed after the eina_promise_then,
so we need to eina_promise_ref before eina_promise_then.

evas: Get pixels from rendered 3D scene

Summary:
Implemented interface Efl.Gfx.Buffer functions bufer_map/unmap for Efl.Canvas3D.Scene.
Added function e3d_drawable_texture_rendered_pixels_get to module evas_gl_3d
to getting pixels from FBO. Added wrappers for functions
e3d_drawable_texture_rendered_pixels_get and e3d_drawable_texture_id_get
to have possibility call it through engine functions.

Reviewers: cedric, Hermet, raster, jpeg

Reviewed By: jpeg

Subscribers: jpeg

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

evas: fix huge memory leak for non-async rendering

So... I had issues with evas-fb engine which was massively leaking,
one image per frame.
After investigating a bit with @cedric on IRC, the reference count
of the cache entries was always 2 before the engine dropped.
So, for each frame with an animation, we could never drop a cache
entry, leading to a trumendous amount of memory leaking.

Now for non-async rendering, we copy the behaviour of
evas_render_pipe_wakeup() which is called in async-mode,
and actually drops a reference in the cache entry.

Fixes T3763

edje_edit: add API for set/get part item

ecore timeouts - dont crash on cancel

this fixes a nasty double deletion on cancel of timeouts. at least for
now as long as we have promises.

fix warning in build with uninit variable in generic loaders for xcf

elm_image: fix break related with elm image error enum

Efl.Pack: Replace pack_direction with Efl.Orientation

Since I first made the pack_direction before orientation
existed, the names didn't match. This should improve API
consistency.

Fix legacy name of elm_win_efl_screen_dpi_get()

...spank again goes to Ji-Youn Park, that insist to not even try his work

elm_win: remove main_menu, window_id, fake_canvas property from eo

elm_test: Add test case for event refeed

Not sure if this test case is really good, but it was
necessary to prove that input event refeed can work from
application side. This is a feature that should work
but shouldn't be used :)

elm_win: Forward events between window and evas

This does:
1. Forward keyboard events from evas to win
2. Allow feeding external input events

Input events can be faked by apps by simply forging
eo objects of the proper type (key or pointer evt) and
calling eo_event_callback_call().

Such events will be forwarded to the internal Evas, and
some bool flags prevent infinite refeeding loops.

efl_event_dup() returns fake events for this to work.

@feature

elm: Partially revert "elementary: avoid random memory access."

This reverts commit ac5a0b7e79d6b32285bd7b794f050826f38cc7c2.

I added assert(pd == sd) in the modified code and couldn't find
anything suspicious. The proper class is used in eo_data_scope_get().
I also can't find a crash or a bug currently in hover. I know there
was a problem earlier, but that was fixed in 3092e08a83f754ba8d49c.

Now, if there is still a problem, we should check it carefully.
Pinging @cedric

Efl.Ui.Box: Fix direction set

VERTICAL means UP but UP was ignored...

edje_edit: add API for get part item attributes by item index

Cedric revert patch 6f7608befd006f8076d237bee5d17c9e9e1e09ac same days
ago.

<quote Cedric>
Your patch 6f7608befd006f8076d237bee5d17c9e9e1e09ac did break elemines
which does relly on the proper index to be given to it. I think
Edje_Edit should be fine and use the index if no name is provided (See
edje_load.c around L1180). In all case this was clearly an ABI break and
needed to be reverted.
</quote Cedric>

So now edje_cc not guarantee name for part items. This is bad, because
edje_edit use item name for access to item attributes. If item have not
name we cann't get it attributes, because edje_edit compare the given
name with part items name from edj-file, strcmp dislake NULL args.
For now I see one way mark all edje_edit_part_item* APIs as deprecated
and add new APIs for manage part items by index.

For clearly and history:
The eason for revert - https://phab.enlightenment.org/D3820
The proposed solution - https://phab.enlightenment.org/D3907

examples: elementary: fix the wrong spell in comment.

elementary: avoid random memory access.

It seems that this content_get function are not getting the data pointer
they expect at all.

elementary: make eo_provider_find lookup in both eo and elementary parent hierarchy.

We can not assume that the elementary parent will be also somewhere in the eo parent
hierarchy. We need to explore both when doing an eo_provider_find in elementary for it
to be really useful.

evas: attempt to fix massive memory leak in the fb engine.

T3763

emotion: properly fix build of vlc generic player.

evas: correctly handle dependencies of evas generic loaders.

autotools: we can't destroy this files as it break a make clean all cycle.

Revert "Revert "elput: add missing dependency on libudev""

Apparently, libudev is not thread-safe which means we cannot use Eeze
for getting the udev but rather need to actually use libudev, so those
revert needs to be reverted.

This reverts commit 354bc66d0d58a74b0464275e865a9cc6c920a538.

build: use the normal efl tree macros for dependency handling

This should hopefully be the last fix for the merged emotion generic player.

Revert "elput: Use Eeze function to get the udev to create context with"

This reverts commit 7b3d20371e8095207c4bafb3684b8a30d4ca6895.

using a separate context was intentional here since libudev is not threadsafe
and this will crash

Revert "elput: add missing dependency on libudev"

A better fix to not use libudev functions has been pushed previously,
so there is no need to add an extra dependency on libudev as we use
Eeze already.

This reverts commit 9d429a7c1ff059e2e4da961af73a6aecec3b5b98.

elput: Use Eeze function to get the udev to create context with

As we already link to Eeze and use it throughout this code, there is
no point in adding an extra dependency on libudev itself when we can
just use Eeze functions to get what we need. This is a better fix for
T3712

@fix

Signed-off-by: Chris Michael <cpmichael@osg.samsung.com>

build: use internal dependency for eina and ecore for generic players

On a system without efl installed these two packages would not be found as.
Need to be internal now that we merged them inside efl.

elementary: add missing dependency on libwayland-client in elm_cnp

lib/elementary/.libs/lib_elementary_libelementary_la-elm_cnp.o: In function `wl_data_offer_accept':
/usr/include/wayland-client-protocol.h:893: undefined reference to `wl_proxy_marshal'
/usr/include/wayland-client-protocol.h:893: undefined reference to `wl_proxy_marshal'
collect2: error: ld returned 1 exit status

Based on a patch by aerodynamik. Thanks!

Fixes T3398

elput: add missing dependency on libudev

Elput is using udev directly so we need to amke sure we link against it.

lib/elput/.libs/lib_elput_libelput_la-elput_input.o: In function `_elput_input_init_thread':
/<<PKGBUILDDIR>>/src/lib/elput/elput_input.c:317: undefined reference to `udev_new'
/<<PKGBUILDDIR>>/src/lib/elput/elput_input.c:326: undefined reference to `udev_unref'

Patch by aerodynamik. Thanks!

Fixes T3712

evas generic svg loader - dotn continue if rsvg handle is null

should avoid complaints of null rsvg handles

efl - build again after adding generic players

eolian: utilize the new void_ptr builtin across eo files

This lets me narrow down the remaining cases of pointers across the EFL.
The void pointers will later need to be reevaluated on per-case basis and
replaced appropriately where possible/feasible.

edje_cc: improve code readability when parsing external params

Reviewers: Jaehyun_Cho

Reviewed By: Jaehyun_Cho

Subscribers: cedric, jpeg

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

Merge evas generic loaders and emotion generic players

We decided to merge these into the efl. They are now part of the repo
and build as port of the normal makefiles. History is fully retained.

emotion: remove legacy files from emotion_generic_players.

evas: remove legacy file from evas_generic_loaders.

evas: integrate evas generic loaders into our single tree build system.

emotion: integrate generic legacy vlc support in the build system.

Merge in emotion generic players.

This commit only merges the repository in (including history) but
doesn't actually build anything. This is the first steps towards the
merge.

Merge in evas generic loaders.

This commit only merges the repository in (including history) but
doesn't actually build anything. This is the first steps towards the
merge.

Evas: Rename Evas.Rectangle as Efl.Canvas.Rectangle

elm fileselector - make ok/cancel in selector configurable per os

so the theme build can order ok/cancel based on preference for an os,
so make configure have an option for this and build the theme
specifically based on that option. enable the option if you want mac
style cancel, ok or default ok, cancel as is common elsewhere.

  --enable-cancel-ok

is the option

@feature

elm_win: remove elm_win_layer_set and inlined image get

elm_win_layer_set is not supported int wayland.
inlined image get api should alternative method to help app save, get pixel for TODO

elm_win: remove illume command send from eo

1. remain elm_win_type_set
2. remain elm_win_alpha_set
3. remain elm_win_available_profile_set/get
app usually set availble profile into window, and app fw or other lib can read it.
so I remain get/set api both.
4. remove elm_win_profile_set/get from eo
5. remove illume feture in eo and move it to legacy

Efl.Gfx.Size.Hint: add Efl.Gfx.Size.Hint skeletton

create Efl.Gfx.Size.Hint skeletton and change elm_win eo.
remove size_step and size_base API from elm_win
and inherit from Efl.Gfx.Size.hint

ecore_evas: Fix mouse in/out events

Apparently I broke some inputs in E (efm) like mouse wheel.
Somehow the list of objects where the pointer is in was NULL.
This was because the mouse_in/out events were not matched to
the proper window ID.

Fixes T3760

ecore_evas: Fix mouse inputs

Yup, I broke everything again. This time, mouse move inputs
would not move the cursor, since I was bypassing the regular
_ecore_evas_mouse_xxx callbacks.

Fixes T3766

Evas: Simplify eo event mechanism

So, I was stupid. I was relying on legacy callbacks to
trigger eo events, which means that only when a legacy
callback was registered would my new eo events be triggered.

Instead, I can pass the eo event desc & info whenever
calling evas_object_event_callback_call().

Evas: Send Efl_Event_Key data to evas objects

Evas: Send key events through eo callbacks

Evas: Add storage class for Key events

This is for key up/down events' info.

eina: fix compilation on OpenBSD/Windows of eina_btlog

We only care whether the process terminated successfully or not.
Non-zero values will be anything but a success, so we don't even care
about the exit code of the program.

This should fix compiling on OpenBSD (where WEXITSTATUS needed
to be defined by including a system header) as well on Windows.

Fixes T3733

elementary: fix runtime warning about missing fixed part

ecore_cocoa: fix bad copy/paste in documentation

Thanks to Stefan Schmidt.

Switch to eo_del() where it should be a del and not an unref

This code was written when eo_del() was removed and eo_unref() was the
recommended practice. Since we added eo_del() back we now need to adjust
this new code accordingly.

edje_cc: fix SIGNAL_EMIT action inheritance error

Summary:
When edje group inherits edje program whose action type is SIGNAL_EMIT
with target, unnecessary parse error occurs.

Reviewers: raster, cedric

Reviewed By: cedric

Subscribers: jpeg

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

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

elm_win: force wl cursor set on every mouse-in

handle the case where the mouse-in is not over a resize edge more effectively

@fix

build: fix distcheck by removing files manually

These files are not getting cleaned up as they are not integrated into the rest
of our build system. These are the extra makefiles to build build only eina, eo,
etc.

The trick with -C clean does not work here for me and this is the only way I
found to get a distcheck going through. Which is kinda important given we want to
do the first alpha tarballs for 1.18 next Monday.

build: keep our CLEANFILES tidy to avoid argument list too long during clean

This is a new incarnation of 0a03e633500466b86cca3b7a5f088735afd7cb56. Our list
has grown to big again as people insist of adding the generated eolian files to
DISTCLEAN while BUILT_SOURCES will get removed durign the clean anyway.

Adding this file list twice will just make the argument list for rm to long to
work.

ecore loop args - go back to eo_del not eo_unref.

Revert "Eo: Remove eo_del() and make eo_unref() the replacement."

This reverts commit 546ff7bbba788ec834c5608361c0834853f2d5d7.

It seems that eo_del() is useful and removing it was creating bugs.
The issue is that the way we defined parents in eo, both the parent and
the programmer share a reference to the object. When we eo_unref() that
reference as the programmer, eo has no way to know it's this specific
reference we are freeing, and not a general one, so in some
circumstances, for example:
eo_ref(child);
eo_unref(child); // trying to delete here
eo_unref(container); // container is deleted here
eo_unref(child); // child already has 0 refs before this point.

We would have an issue with references and objects being freed too soon
and in general, issue with the references.

Having eo_del() solves that, because this one explicitly unparents if
there is a parent, meaning the reference ownership is explicitly taken
by the programmer.

eo_del() is essentially a convenience function around "check if has
parent, and if so unparent, otherwise, unref". Which should be used when
you want to delete an object although it has a parent, and is equivalent
to eo_unref() when it doesn't have one.

elm_win: move shaped & title property from eo to legacy

1. remain borderless property.
2. shaped property was removed because wayland not support this.
3. remain noblank property.
4. remain role, name properties.
5. remove title property because it can be supported using name property

elm_win: add enum for elm_win_modal set and elm_win_urgent set

1. remain iconified, maximized, fullscreen property
2. remain sticky property
3. change return value type of modal from bool to enum for the future.
4. merge urgent_set and demand_attention
5. remain activate method

elm_win: remove elm_win_icon_name from eo.

1. remain focus_highlight_style,focus_highlight_enabled,
focus_highlight_animated,prop_focus_skip,focus. because each window need to set the focus
feature.
2. autohide, autodel is remained for convenient.
3. icon_name will be removed. elm_win_icon_object_get can used instaed of icon_name.

evas : evas_object_key_grab exclusive option logic is changed.

Summary:
 - added is_active property in Evas_Key_Grab.
 - Evas key down and key up events transferred to active grabs only.
 - If evas grabs has a exclusive grab, other grabs will be deactivated.
 - If a exclusive grab is ungrabbed, remained grabs will be activated.

Reviewers: jypark

Reviewed By: jypark

Subscribers: cedric, jpeg

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

efl_screen: create efl_screen interface

elm_win have three feature releated with screen.
1. screen_rotation_get
2. scrren_size_get
3. screen_dpi_get
so create efl_screen interface, and elm_win implement that interface

elm_win: move elm_win_screen_size/position get api to legacy

these kind of api are not supported in wayland.

evas: updated evas_vg_sample with the appen_arc() api as well as _shape_dup() api test.

Reviewers: Hermet, cedric

Reviewed By: cedric

Subscribers: cedric, jpeg

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

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

evas: fix recalculation coordinates for pick object from scene in Evas.Canvas3d

Summary:
We did the same for evas_canvas3d_scene_pick(see c850cc0d80b754be851576083eba27a72b58b9f2),
but forget for evas_canvas3d_scene_exist.

Reviewers: Hermet, raster, cedric

Reviewed By: cedric

Subscribers: jpeg

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

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

evas: avoid useless manipulation during pick object from scene in Evas.Canvas3d

Summary:
It is logically break cycle in case object is found. In case need pick all objects
that lay in screen ray, we should use evas_canvas3d_scene_pick_member_list_get
function.

Reviewers: Hermet, raster, cedric

Reviewed By: cedric

Subscribers: jpeg

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

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

elementary: initialize single_direction as none for elm_map.

Summary:
 - elm_map has a scrollbale interface and it is set
   as ELM_SCROLLER_SINGLE_DIRECTION_SOFT by default.
 - elm_map can be rotated by gesture or by an API
   elm_map_rotate_set, so this single direction
   makes scroll unnatural.

Reviewers: Hermet, cedric

Subscribers: conr2d, jpeg

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

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

emile: fix build warning of Emile_Image.

Summary: initialized structure 'opts_region' for uninitialized build warning

Test Plan: N/A

Reviewers: raster, woohyun, cedric

Reviewed By: cedric

Subscribers: cedric, jpeg

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

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

ecore: if you don't use the value, then don't set it.