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

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

efl_ui_focus_manager_sub: flush the border if we cannot find a candidate

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

ecore_wl2_window: Silence some spurious warnings

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

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

ee_wayland: Set window visible before trying to render it

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

eolian: turn some state APIs into inlines over unit APIs

pyolian: remove unneeded bindings in eolian_state

elua: inherit state from unit in eolian bindings

eolian: refactor doc ref resolution API

eo: fix invalidate to always happen.

genlist: fix "insane" order [BUG COMPATIBILITY]

This patch implements bug compatibility.

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

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

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

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

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

@fix

fix T5938

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

tests: add genlist test to verify group item api behaviour

ref T5938

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

efl_ui_clock: do not locally store clock module api

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

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

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

this is a no-op case which causes unnecessary errors

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

efl_promise: avoid invalid calls when unsetting parent

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

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

tests: remove invalid api usage from elm layout test

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

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

tests: disable elm glview tests when running in buffer engine

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

elm_map: remove unnecessary error message in destructor

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

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

elm: call ecore_event_shutdown during elm_shutdown

ref 4e14ba3f4874b03185d5216829ecc1876d11c48e

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

tests: check whether ecore is properly shut down after elm_shutdown

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

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

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

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

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

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

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

this should add some consistency to tests

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

tests: properly destroy elm_win timers to prevent crashes

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

efl_ui_win: track the finalize future for del to avoid crash

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

ecore_evas: implement the move method for basic buffer canvas

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

efl_threadio: always return a value to silence warning

efl_loop: fix exit code of the loop

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

eo: cleanup future on invalidate.

eo: add an helper to get an object from an Eina_Value.

eo: allow setting value eo Eina_Value from an Eo pointer directly.

eina: fix promise scheduler lookup to find it when given a resolved chain of future.

T6738

eina: avoid warning when accessing first item and just return an error.

eolian: update type/var name APIs

eolian: make more name APIs into inline helpers

eolian: update class name APIs

pyolian: update name APIs

elua: update eolian name APIs

eolian: expose short_name/namespaces via object

eolian: use short name for function pointer name

eolian: only merge units on success when parsing all files

doc: check - correct the ingroup name for legacy apis

Pyolian generator: adjust to follow eolian changes

eolian: simplify namespace handling

This is mostly a preparation for rework of name(space) APIs.

Pyolian: remove the useless Object.type binding

This is useless in py as Objects are never returned
directly, you always get the correct subtype (Class,
Variable, etc..). It was also clashing with Class.type

Pyolian: add tests for Eolian_Object

Two of the new tests are failing, the problem is that now
we have name clashes between Eolian_Object and Eolian_Class (at least)
For the moment I spotted:
 - Object.name clash with Class.name
 - Object.type clash with Class.type

Also fixed a typo in eolian_lib.py spotted by the new tests,
and removed the old tests for Declaration.

Pyolian: build the correct type from generic Object

This way all the generic Eolian_Object types are automatically
converted to the correct types (Class, Variable, etc...)

elm test - adapt to argc/v being regular argv 0 being binary name

ecore - efl thread - remove unused local vars

ecore - efl exe/thread - only allow run once ever

eolian: resolve to Eolian_Objects instead of voids

eolian: return Eolian_Object_Type from ref resolver

This needs only one enum and allows for better interop.

ecore: do not remove a timer that has already been removed.

csharp: update examples

Summary:
They were still using the old SetSize(w, h) API instead of receiving
eina.Size2D structs.

Test Plan: Build examples and run them.

Reviewers: felipealmeida, cedric

Reviewed By: cedric

Subscribers: cedric

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

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

csharp: generate helper constructors for structs.

Summary:
C# does not have a literal form for structs (like C++'s {} aggregate
initialization). Before this commit the user would need to explicitly
instantiate a struct and assign the required values to it, like:

   eina.Size2D size;
   size.W = width;
   size.H = height;
   widget.SetSize(size);

As a workaround, this commit generates helper constructor with
parameters corresponding to the struct fields in the order they are
declared. These parameters have default values if one does not want to
explicitly initialize all fields directly. With these constructs, the
above code could be translated to:

   widget.SetSize(new eina.Size2D(width, height));

It should be noted that the constructed struct will live on the managed
memory (GC) instead of the stack.

Test Plan: run "make check"

Reviewers: felipealmeida

Subscribers: cedric

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

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

eo: ERR some warning when illegally using invalidated object.

ecore: avoid calls to provider_find on invalidated objects in timer class

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

eolian cxx: fix typo

eo: when an object is reused, reset the invalidate flag.

eolian: remove old declaration APIs

eolian cxx: replace declaration api with obj api

eolian: tests for new object apis

pyolian: add support for object retrieval

pyolian: wipe out declaration api

elua: remove declaration api from eolian bindings

eolian: remove obsolete file_get APIs

eolian cxx: remove usages of obsolete file_get APIs

pyolian: add Eolian_Object APIs and remove obsolete file_get ones

elua: update eolian bindings for latest APIs

Added support for Eolian_Object, made other luaified types
"inherited" from Eolian_Object, removed type-specific file_get
as it was replaced by Object's file_get, declarations API still
in place

gl_drm: Use gl extension string helper instead of strstr

eolian gen: switch from decl api to object api

eolian: give objects names, and reduce duplication

eolian: add APIs to retrieve declarations as objects

This will replace the old declarations API.

gl_drm: Don't NULL init a variable we're about to set

minor clean up with no functional change.

gl_drm: Check EGL client extensions before using platform_base

The comment here was right, we shouldn't just trust these without testing
if we should trust these.

gl_drm: Rename eng_gl_symbols to eng_egl_symbols

It now only deals with egl.  gl extensions must be queried later, so it
was previously a bit weird/confusing.

gl_drm: Move some extension queries to after context creation

Anything non-EGL we might query would have to be queried here, so
I'm moving the call here to protect us in the event that we need GL
extensions in the future.

I'm still a bit confused as to what string I should be passing to
evas_gl_symbols, though.

eolian: add API to query information about Objects

As nearly every Eolian handle is backed by an Eolian_Object,
this information is now publicly exposed and has an API. This
opens up an array of new possibilities for tooling, as you
can now externally query file names, line numbers etc.,
as well as cast arbitrary handles to Eolian_Object pointers
and back.

This will be expanded later and it will replace the Declaration
system, as it's cleaner, better integrated and more versatile.

@feature

gl_drm: use EGL_IMG_context_priority if available

This is a hint that we want a high priority context.  Since gl_drm is
likely a compositor or a full screen app, it makes sense that it try to
use this (but other engines probably shouldn't)

Based loosely on Chris Wilson's weston patch to do the same thing.
(weston commit b678befb6ed055e6c66466505d9195a3cebf8073)

As this extension appears to have been around for years, I haven't
added fallback defines for:
EGL_CONTEXT_PRIORITY_LEVEL_IMG  0x3100
EGL_CONTEXT_PRIORITY_HIGH_IMG   0x3101

gl_drm: Move the gl symbol check to immediately after display init

We don't actually need a context first, just an initialized display.

eo: fix warning.

eo: adjust test suite to take invalidate state into account.

ecore: rely on Efl.Object.invalidate instead of a hack for Efl.Model.Container.Item.

eo: introduce invalidate and noref state.

This is just a first step. All user of destructor should be updated to
move the code that rely on their efl_parent and on efl_provider_find to
invalidate. Then we will be able to change the way efl_add and efl_del
work to properly refcount things. efl_noref won't be triggered at the
moment until both efl_parent_set(obj, NULL) and the last user ref are
set to NULL. This is not what we want, but due to how user refcount is
accounting parent at the moment, until all the code is move to rely
on invalidate we can not fix this.

ecore_wl2: cleanup input code to string logic to reduce compiler warning and useless memory access.

elementary: fix backward compatibility bug introduced by 9c8749b99a03d3601321da6d16071dd7b631d1ae.

ecore-wl2: Check for valid string before passing to strlen()

As strlen() cannot accept NULL (segfaults), we should check for valid
key, keyname, and compose strings here before passing to strlen().

@fix

Signed-off-by: Chris Michael <cp.michael@samsung.com>

ecore-wl2: Reduce calls to strlen

Small patch to reduce calls to strlen when sending key events. This
patch is loosely based on Phab D5567

@fix

Signed-off-by: Chris Michael <cp.michael@samsung.com>

eolian: get rid of old APIs for typedecl retrieval

eolian: remove old variable retrieval APIs

eolian: remove old class retrieval APIs

efl_ui_focus_manager_calc: performe refocus after the node is deleted

if node is the last element, the element will be focused again, and
later deleted, without cleaning up the history

examples: cxx: fix the compilation for slider example.

efl check - fix env var for CK_FORK=no

elementary: entry - recover cursor position when theme is changed

Summary:
When theme is applied text is reset and cursor position is set to 0.
This patch is recover the position to previous after text is reset.

Signed-off-by: Minkyu Kang <mk7.kang@samsung.com>
Reviewers: cedric, jpeg, woohyun, herdsman

Reviewed By: cedric

Subscribers: cedric, jpeg

Tags: #efl

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

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

elementary: bg - keep file path and key string for legacy bg widget

Summary:
If a file path and key string was passed to elm_bg, we could get
the same file path and key string. Even if it failed to load the image file.
And the file path also remained its original form.
ex) Setting file path "~/image.png" => Getting file path "~/image.png"
                                       (Not "/home/user_name/image.png")

@fix

Test Plan: Included in elementary test suite.

Reviewers: jpeg, cedric, raster

Reviewed By: cedric

Subscribers: woohyun

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

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

elementary: entry - fix to generate "activated" and "aborted" event in single line entry only

Summary:
"aborted": The escape key was pressed on a single line entry. (since 1.7)
"activated": The enter key was pressed on a single line entry.

Reviewers: id213sin, jpeg, cedric, herdsman

Subscribers: cedric, jpeg

Tags: #efl

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

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

elementary: add missed type for legacy.

Summary:
We've been missed type of class name.

Add legacy class name to maintain compatibility.

@fix

Reviewers: jpeg, woohyun, Jaehyun_Cho

Subscribers: cedric, jpeg

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

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

elemetary: spinner - free the longpress_timer when unfocused

Summary:
It needs to delete when widget unfocused.

@minkyu
Gets from tizen.
0bc1a19703dc204c8d347794bae0290006f4bde5

Reviewers: jpeg, Jaehyun_Cho, minkyu, cedric

Subscribers: minkyu, cedric, jpeg

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

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

elm: fix location of legacy include for elm_code.

Summary:
Instead of including the legacy header on the main header, just include
it where it is actually used (diff_widget.c), like in elm_code_widget.c.

Test Plan: Try to compile edi with the tarball from `make dist`. It should not fail with elm_code related errors.

Reviewers: raster, stefan_schmidt, felipealmeida

Subscribers: cedric

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

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