update according to cedrics review comments

focus: update object definition

Summary: This introduces a new focus system and the migration to this system.

Test Plan: run elementary_test

Reviewers: cedric

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

elm_widget: add unfocusable to debug informations

efl_ui_focus_manager: make setting a order easier

elm_box,grid,table: take the resize object as evas object

efl_ui_focus_manager: if there is no normal item then just do nothing

efl_ui_flip: migrate to new focus system

elm_grid: migrate to new focus system

efl_ui_focus_manager: there is the case that n can be NULL

efl_ui_focus_manager_sub: do not update if parent is NULL

is is actually a dangerous case, and should NEVER happen

efl_ui_frame: migrate to new focus system

when collapsed the subtree is unfocusable

elm_table: migrate to new focus system

elm_interface_scrollable: fix bringing in of items

elm_scroller: migrate to new focus system

the scrollable interface is doing most of the work, if we know that
something is already focused we just leave the action_move.

elm_widget: evalulate show flag and call focus on super

elm_menu: mirgrate to the new focus

this just means the menu is registering a new manager, it is not really
doing anything, its just there so the children which are searching for
one are finiding one. Elm menu has not handled anything with keys in the
past and will later learn to do so.

elm_hover: migrate to new focus system

the hover will set itself as redirect once it is visible. And unset
itself once it is hidden.

efl_ui_win: migrate to the new focus system

this means, focus requests are redirected to the new focus manager.

elm_interface_scrollable: use the submanager to register the children

this adds the registered elements at the broder of the scrollable to the
parents manager. scroller and gengrid works for registering

elm_box: set the correct order

efl_ui_focus_manager: allow settings of subsets

those subsets will just be prepended.

efl_ui_focus_manager: handle logical end correctly.

efl_ui_focus_manager: fix error message

efl_ui_focus_manager: return the new candidate

efl_ui_focus_manager: handle redirect objects in logical better

efl_ui_focus_manager: be more secure

efl_ui_focus_manager: fix sub manager logical handling

efl_ui_focus_manager: better error messages

efl_ui_focus_manager_sub: focus the item in sub and main manager

efl_ui_focus_manager: add log domain for focus movements

efl_ui_focus_manager: restructure node struct

a redirect can happen for logical OR normal

efl_ui_focus_manager: more debug information

efl_ui_focus_manager: do not depend on elm widget events

efl_ui_focus_manager: unset redirect when focusing new item

efl_ui_focus_manager: make next and prev searching deadlock prove

efl_ui_focus_manager: unset redirect when destroyed

efl_ui_focus_manager: allow flushing of one single node

this gives a massive performance boost

efl_ui_focus_manager_sub: register the forwared itself

efl_ui_focus_manager: handle redirect managers in a better way

elm_widget: update focus register work

now it trackes changes of the parent manager. And registers more
reliable.

efl_ui_focus_manager: fix up tracking and returning of data

a logical parent should not be in the border elements. The child should
also be not deleted itself. If we do so correct deletion could fail.

tests: update to recent changes

efl_ui_focus_manager_sub: update to recenent changes

efl_ui_focus_manager_sub: register children correctly

elm_widget: add logical parent to interface

and evaluate all values correctly and emit the correct events

elm_widget: make sure we dont have a destructed logical parent

efl_ui_focus_manager_sub: register the root as logical

efl_ui_focus_manager: a logical item should never get focus.

efl_ui_focus_manager: use data iterator not key iterator

this saves one call to node_get

elm_widget: enhance focus handling of the widget

widgets will now use there widget parent as logical parent.

elm_widget: add debugging informations for a manager in a widget

efl_ui_focus_manager: new api for adding a logical widget

a widget which is registered logically will not be used for right left
top or down relations.

efl_ui_win: make it a focus manager

Added a focusmanager with the window itself as the root.

elm_widget: make it a efl_ui_focus_object

it will register based on it can_focus flag

efl_ui_focus_manager: add new api to receive child configuration

focus: fix compilation

it seems like eolian changed, so i need to import the eina_types

managersub: remove the select_set call

this can be done directly be inheriting border_elements and cutting down
the set.

focus: update documentation on all eo files

we should now have a complete documentation of those eo files

focus: update object definition

the event is now a bit better, since you only need to subscribe to one
event instead of two, the new state is passed in the event.

efl_ui: introduce a focus manager and sub manager object

The Efl.Ui.Focus.Manager abstracts the creation of a localization graph
and a logical tree. The localization graph is used to find a object
right left up or down of a given object. The logical tree is used to
iterate throuw the containers which are used to build a ui.

Those managers can be used bound to some layer in the ui, so for
example the window is a layer, the content of a scroller is a layer.
With those layers, we can make sure that movements of a scroller for
example just means that this graph of objects in the scroller needs to
be recalculated, and not the complete ui.

The advantage of having this to layer bound datastructures is that you
can easily debug those graphs, since the complete layer of this
managerobject can be calculated completly.

edje_cc: Keep script block during group inheritance

Currently, script block is removed when an edje group inherits from other group
after defining its own script block.
  group { "somegroup";
    script {
       ...
    }
    parts {
       ...
    }
    inherit: "othergroup"; // <= previous script block is removed here.
  }

If parent group doesn't have script block, it doesn't need to overwrite previous
one. This will keep script block and print warning when script block is overwritten.

elm_image: Fix file_set when preload is disabled

If we disable preload, then the second file set on an elm_image
object would not trigger a deletion of the first image. As a
consequence, both images would be visible... really bad if there's
alpha or different dimensions!

Thanks Anand Kumar for the report!

@fix

examples: Fix most trivial warnings

This fixes all warnings for "make examples" for:
 -Wunused-parameter
 -Wshadow
 -Wformat-security
 -Wenum-conversion

Some remaining warnings include:
 -Wdeprecated-delcarations

examples/evas: Use printf instead of fprintf(stdout, ...)

Summary:
Applies the correction purely mechanically using the following shell
command-line:

    src/examples/evas$ grep -sr 'fprintf(stdout' . | cut -d: -f1 | uniq | \
        xargs sed -i "s/fprintf(stdout/printf(/"

This fixes a few warnings about lack of a format string:

    warning: format not a string literal and no format arguments [-Wformat-security]
        fprintf(stdout, commands);

Reviewers: cedric

Subscribers: cedric

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

ecore_wl2: destroy dmabuf global at shutdown

Fixes a small memory leak.

ecore_wl2: refactor global cleanup in shutdown paths

Both session recovery and regular disconnect do this, so make
it a function to ensure they stay in sync.

ecore-drm2: Iterate plane formats properly

This patch addresses an issue where plane formats were not being
properly copied into our Plane State structure and causing any usage
of our atomic code paths to crash and burn

@fix

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

elput: Remove specific case for pointer wheel tilt

Since efl-jenkins does not have libinput >= 1.7.0, this commit broke
the jenkins build due to missing
LIBINPUT_POINTER_AXIS_SOURCE_WHEEL_TILT value. We should be able to
get by with letting 'default' case handle it....

@fix

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

elm.hoversel: highlight selected item when auto update is enabled

Summary:
When auto update is enabled, the label of the hoversel will be that of selected
item. This feature is usually used when changing state of something.
Highlighting item previous selected will show what is current state more
explicitly especailly hoversel has many items.

Test Plan: elementary_test -to hoversel

Reviewers: cedric, jpeg

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

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

edje_pick: fix a couple spelling errors

Reviewers: cedric

Reviewed By: cedric

Subscribers: cedric, jpeg

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

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

ecore_evas: fix some incorrect words

Summary:
 + whichever -> whatever
 + hyphenate pre-* and post-*
 + flowed -> followed

Subscribers: cedric, jpeg

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

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

evas: fix several incorrect words

Reviewers: cedric

Reviewed By: cedric

Subscribers: cedric, jpeg

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

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

evas: fix spelling for "ptimitive"

Reviewers: cedric

Reviewed By: cedric

Subscribers: cedric, jpeg

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

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

elput: Add case for wheel tilt when getting pointer axis value

libinput >= 1.7.0 added support for
LIBINPUT_POINTER_AXIS_SOURCE_WHEEL_TILT, however we do not yet support
that in elput, thus gcc was spilling a warning about an unhandled
enumeration value here. We'll add a case for the above, plus a default
to cover any future additions.

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

elput: Remove unused variable

Gcc reports 'flags' as unused here, so remove it

@fix

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

ecore-con: Remove unused variable

Gcc complains that 'flags' here may be used uninitialized. In looking
at the code, 'flags' does not seem to be needed in the debug prints
here. If we keep and initialize the variable to 0 during declaration,
it would only ever print out 0 anyway as 'flags' is never changed in the
code.

@fix

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

eolian: fix buggy generation of auto/empty on properties

We only want to generate auto/empty function bodies on what is
specified inside the braces for properties, not for the whole
property.

Evas.Table: Apply mirrored correctly when align is set

Summary:
When table items are left aligned and mirrored is set, items should be placed
from the right side, but align is not changed. (still left-aligned)

@fix

Test Plan: make and run attached example

Reviewers: cedric, jpeg

Subscribers: thiepha, woohyun

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

elm_map: Fix elm_map_overlay_color_get

Summary:
Add all supports types for overlay color (such as in Doc.)
@fix

Reviewers: cedric, Hermet, raster, NikaWhite, jpeg

Reviewed By: jpeg

Subscribers: jpeg, artem.popov

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

eo: Micro-optimize function calls

This is related to the previous changes to efl_super.

eo: Remove super bit from eo id

It is not required anymore as the super class and super bit can
be stored inside the object data. See the previous patches.

eo: Add docs for efl_super and efl_cast (doxygen)

eo: Add new API efl_cast

This is similar to efl_super but the specified class is the one
we want to call the function on. This is similar to dynamic_cast<>
in C++.

Note: both efl_super() and efl_cast() need documentation!

This is an experimental feature.

Fixes T5311

@feature

Maniphest Tasks: T5311

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

eo: Remove lock from efl_super

I've always really disliked this lock. If someone calls a non-eo
function by accident with efl_super() then you'll most likely end
up in a deadlock.

This adds the cur_klass pointer to the object itself, exploiting
the fact that we have 8 bytes of padding (on 64 bits, at least).

Also, this opens the door to efl_cast() which would be similar to
efl_super() except that only a dynamic cast is done, not a call
to the parent function.

make benchmark shows a performance improvement, surprisingly.

This is a bit experimental. See also the following commit (efl_cast)

ecore_pipe: Fixed dead lock with epoll

See a76ebea2d840ecc575efb025574c15503225b83f and the following
commits on this file.

The following test scenario let to easily reproducible
application hangs:

  elementary_test -to "Icon Desktops"
  # then scroll vigorously with the mouse wheel up/down

This patch was applied as a new revision on the below diff:

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

ecore: use new API eina_file_close_on_exec in example too.

evas: use new API eina_file_close_on_exec.

elementary: use new API eina_file_close_on_exec.

elput: use new API eina_file_close_on_exec.

eio: use new API eina_file_close_on_exec.

ecore_file: use new API eina_file_close_on_exec.

ecore_con: use new API eina_file_close_on_exec.

ecore: use new API eina_file_close_on_exec.

evas: use new API eina_file_close_on_exec.

eet: use new API eina_file_close_on_exec.

eina: use new API eina_file_close_on_exec.

eina: add an API to correctly do close on exec.

ecore-drm2: Add support for putting dmabuf clients into hardware planes

Small patch set which adds necessary API functions that we can use in
order to allow placing dmabuf clients into a hardware plane based on
supported format, size, etc.

Merge branch 'devs/devilhorns/planes'

ecore-drm2: Add fields to plane structure for current, next, pending
FBOs

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

ecore-drm2: Free any previous atomic request

If we fail the atomic request merge, cleanup any previous request and
just use the new one.

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