efl_access_component: resolve property name conflicts

this class is a giant FIXME for anyone looking to do refactoring work.
the only reason it seems to exist is to provide wrappers around
efl.gfx functions and provide screen position adjustments--something
which isn't even viable under wayland

note that a test was removed here due to the corresponding method being
removed

Change-Id: I6ee1c14c6a56945ff7f551af4e89705d216018ed

efl.access: remove parent property

most of this is just a reimplementation of efl_provider_find

efl.access: name -> i18n_name

efl: rename player.position property -> player.pos

position conflicts with gfx.position; this could maybe be named better?

eio: sentry.del -> sentry.remove

ecore_audio: remove "name" property

efl-net: rename conflicting "name" properties

eldbus: rename conflicting "name" properties for some classes

eldbus: rename custom constructor methods

"constructor" conflicts with the base efl object constructor

some of the params for these should maybe be refactored into properties
and use finalize?

evas_textgrid: size -> grid_size

this conflicted with efl_ui_base.size

evas_table: remove mirrored property

use efl_ui_base.mirrored instead

evas_grid: size -> grid_size

this conflicted with efl_ui_base.size

evas_grid: remove mirrored property

use efl_ui_base property

evas: move smart_objects_calculating to Efl_Canvas

evas: remove Evas_Canvas.smart_objects_calculate

also implement Efl_Canvas method

evas: move more canvas method implementations into evas_main.c

evas: remove Evas_Canvas.object_top_in_rectangle_get

also implement Efl_Canvas method

evas: remove Evas_Canvas.objects_in_rectangle_get

also implement Efl_Canvas method

evas: remove Evas_Canvas.object_top_at_xy_get

also implement Efl_Canvas method

evas: remove Evas_Canvas.objects_at_xy_get

also implement Efl_Canvas.objects_at_xy_get

note that any function which returns an iterator cannot be @const since
it's necessary to wref the object to ensure the iterator's lifetime

efl: merge list->iterator creation function into evas_main.c and reuse

this deduplicates a large amount of code

evas: move more canvas functions into evas_main.c

typechecking is an important part of compilation

evas: move evas_stack.c -> evas_stack.x

this file contains canvas methods which should be subject to compiler
type checking, requiring that they be included in the file which contains
the eolian generated code

evas: properly implement evas_image_max_size_get() as a wrapper

ref 457dc16cffdd5f9c6ce42e72beddad06fa36f0c0

evas: move canvas eolian functions from image.c -> main.c

this reveals existing breakage which went unnoticed due to the compiler
being unable to do type checks since the generated eolian code was not included

Elementary: remove needless structs in Efl_Ui_List

elementary: efl_ui_list change SegArray tree to Efl.Object

elementary: efl_ui_list model_set update values fixed

elementary: efl_ui_list_segarray fixed remove last block item

elementary: efl_ui_list focus manager fix
fixed focus changed with keyboard
fixed scroll in focus
fixed focus in example

ecore_wl2: Take a library reference on surface create

Since surface handling is now done via modules, we need to ensure
the library can't be shutdown while a surface exists.  Otherwise,
we get a segfault trying to call a function we've unmapped.

Fixes a bug on shutdown for some wayland clients using software
rendering.

efl_ui_focus_manager_sub: remove needless call

Summary: focus manager is not assigned in constructor

Reviewers: bu5hm4n

Reviewed By: bu5hm4n

Subscribers: cedric

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

elm_hover: remove focus_layer related code

Summary:
because elm_hover implements Efl.Ui.Focus.Layer, focus_layer related code
dosen't needed.

Test Plan: elementary_test - hover

Reviewers: bu5hm4n

Reviewed By: bu5hm4n

Subscribers: cedric, jpeg

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

efl - fix build again after removal of install of public header

put elm_code_widget_legacy.eo.h back in the includes to install as it
is required by other public headers...

test_ui_table: fix uninitialized variable warning

maybe right, maybe wrong, maybe compile warnings shouldn't be added
in reviewed patches

ref D5668

gl_drm: NULL check img

If we set native surface on a new object we can get here with NULL
engine_info from the caller.  We should still create a native surface.

elementary: fix legacy widget type name for backward compat

Summary:
For example, the widget type of elm_button was "Elm_Button".
But, the object which is created by elm_button_add() will
return its widget type "Efl.Ui.Button_Legacy".
It is not legacy name. It should be fixed to return "Elm_Button".

I don't know when but eolian start to make class name with ".".
So, it should be converted to "_" for all widgets.

@fix

Test Plan:
All test cases are included in this patch.
Run "make check"

Reviewers: raster, cedric, jpeg, taxi2se

Reviewed By: cedric

Subscribers: taxi2se, woohyun

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

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

eina: improve documentation for eina_bezier.

Summary:
Adds missing @param docs and fixes an incorrectly documented one.
Clarifies difference between 'length' and 'position', specifying the
latter is a number between 0.0 and 1.0.  Improves verbage here and there
for grammatical correctness and internal consistency.

Subscribers: cedric

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

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

eina: make doxygen @notes consistent in eina_hash

Summary:
These routines all have slight permutations of the same basic note, but
are each formatted a bit differently.  Fix that and a few punctuation
irregularities.

Similarly for a couple @warnings, and escalate one @note to a @warning
since what it describes might be a security issue so deserves
highlighting.

Subscribers: cedric

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

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

eina: fix minor spelling mistake in eina-private.h doxygen

Reviewers: cedric

Reviewed By: cedric

Subscribers: cedric

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

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

eina: use matrix compose in place of equivalent code in matrix multiply

Summary:
The eina_matrix3_compose and eina_matrix3_multiply API's are
mathematically identical (even though the implementations are
reversed... weird), except that the latter also includes a fastpath for
identity matrices.

Having two functionally equivalent APIs is redundant, so ideally one or
the other would be dropped.  But in order avoid API breakage, just have
one routine wrapper the other and eliminate the internal redundancy.

(Note that the parameter signatures of the two routines are different -
eina_matrix3_compose() takes the two input matrices first, and the
output matrix last, while eina_matrix3_multiply() takes the parameters
in the reverse order.  This inconsistency in the API style could result
in accidentally erroneous usage and would be an argument for deprecation
of one of the two APIs.)

Signed-off-by: Bryce Harrington <bryce@osg.samsung.com>
Reviewers: cedric

Reviewed By: cedric

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

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

ecore_file: modify check for wd in file monitor del

Summary:
Make checks consistent in ecore_file_monitor_backend_del.
inotify_add_watch can return 0 as well on success.
So, even if wd is 0, inotify_rm_watch need to be invoked.

Reviewers: cedric, jpeg

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

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

eina: cleanup doxygen for eina_rectangle.

Summary:
Some spelling/punctuation/grammar/formatting fixes to make the
rectangles doxygen more consistent.  Simplify wording in several places,
including removing redundant documentation of return values.  Reword the
rectangle cutting API's to be more clearly worded.  Make clearer mention
of parameters that get changed by the function call, and ones that
allocate memory.

Reviewers: cedric

Reviewed By: cedric

Subscribers: cedric

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

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

evas/vg: reset the path data in destructor.

Summary: As the gfx_path mixin class destructor is not called hence explicit call of reset.

Reviewers: jpeg, SanghyeonLee

Subscribers: cedric

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

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

efl: general distcheck fixes

Summary:
- Added missing C++ header
- Added missing elementary header
- Removed generated header from elementary_SOURCES
  (Was added by raster in 42dfee37)
  Not sure of what would be the best place for it, though.
- Removed previously removed files from elementary examples Makefile.

Test Plan: Run 'make distcheck'

Reviewers: stefan, felipealmeida

Subscribers: cedric

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

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

efl.ui.layout: change signal name for icon/text

Summary:
when set/unset icon and text,
signal "elm,state,[part],visible/hidden" is emitted.
This is wrong because visible/hidden should be handled by
Efl.Gfx.visible, not Efl.Text nor Efl.Content.
This should be changed into elm,state,[part],set/unset"
All relating edc should be fixed.

Test Plan: run elementary_test->button, Efl.Ui.Button

Reviewers: jpeg, cedric, woohyun, Jaehyun_Cho

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

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

efl_wl: Update native surface for dmabuf test too

Should've added this to the last commit - though this change is purely
cosmetic as the test can never trigger direct scanout.

efl_wl: Update native surface usage

We should be zeroing out the dmabuf scanout stuff.

efl-wl: check for existing keyboard enter before sending leave

ref 4a32e2456eaca80cee0b1ce70cea00cb41e11979

ecore_wl2_surface_dmabuf: fix memory leak on surface destroy

We need to free the surface private data on destroy.

ecore_wl2: Fix memory leak on shutdown

The output list wasn't properly cleaned up on shutdown/disconnect.

efl_selection_manager: shut up shut up shut up ecore-x when running in wayland

fix T6643

efl_gesture_manager: free hash in destructor

every canvas object has had this leak for some time:

==25217== 64 bytes in 1 blocks are definitely lost in loss record 1,669 of 3,515
==25217==    at 0x4C2FB6B: malloc (vg_replace_malloc.c:299)
==25217==    by 0x688D839: eina_hash_new (eina_hash.c:755)
==25217==    by 0x688D940: eina_hash_string_superfast_new (eina_hash.c:791)
==25217==    by 0x6265F8C: _efl_gesture_manager_efl_object_constructor (efl_gesture_manager.c:66)

eolian: properly initialize warned (to avoid random parse failures)

elua: add eina in environment variable library path

In-tree libeina.so library path is missing while running elua.

http://autobuild.buildroot.net/results/2bb/2bbe89694c206d764d51a5dd9c75bc4aa844e98d/build-end.log

@ref T6670

Cc: Daniel Kolesa <d.kolesa@samsung.com>
Signed-off-by: Daniel Kolesa <d.kolesa@osg.samsung.com>

evas drm engines: Fix variable type for output

We already include the Ecore_Drm2 header for these engines, so there
is no need for the 'output' field to be a void pointer here.

@fix

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

ecore-evas-drm: Use output viewport in ecore_evas_animator_tick calls

@fix

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

ecore-drm2: Add API functions to get/set output relative mode

These API functions will be used inside Enlightenment in order to
determine the proper extended screen size based on outputs relative
position.

@feature

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

ecore-evas-drm: Include Evas_Internal header

We need to include this header so that we can use the new
efl_canvas_output functions

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

efl_ui_nstate: make "changed" callback is called properly

- "changed" callback is not called when theme_apply is called
- elm_check_state_set does not call "changed" callback to keep
  backward compatibility

Revert "elm: do not call "changed" callback when setting style"

This reverts commit b1ee35e35ce3132566244abbbd3a804167ef9c00.

Revert "Efl.Ui.Nstate: do not call "changed" callback"

This reverts commit fd8e9f9c4e8dbda6c379dc464f562f5b2c681d31.

eolian: print all duplicate warnings before aborting build

Instead of aborting at the first warning, accumulate the warnings
and stop at the end of validation.

elm focus ... set focus in focus in if focus is not already on something

let's actually set focus in focus_in on window (ecore evas) IF focus
is not set. this seems to make terminology still work...

efl ui win - if job runs after destruction of evas so dont flush pending

elm - dont get top widget for non-ui objects

efl ui win - fix focus break for terminology (at least)

at least terminology was broken by this. perhaps other efl apps. if an
efl app has decided to manually play with focus itself, like set focus
to some obj then leav ie, this line breaks that and interferes with
the object focused whenever the window gets focus set. if there is a
focus manager at least this is a new thing and so let the focus
manager set focus, but dont set old evas focus.

this fixes the break added in 56beb861e83cbc830c82db7473e0a88461e2e213

evas_vg: implement interpolate from Efl.Gfx.Path

interfaces: merge all bounds_get methods into efl.gfx.path

ref 884fc35160152712decdf7fd43082ef3912c0856

evas3d: use a type setter function with finalize instead of a custom constructor

resolves a conflict with the 'constructor' method name

evas3d: add "unknown" node type for use with errors

interfaces: remove Efl.Text.Properties

this is provided in Efl.Text.Font

efl_text_font: copy docs from efl_text_properties

efl_vg: use efl.object's name property instead of implementing a new one

ector: rename ector_renderer "bounds_get" method to "boundary_get"

this conflicted with a method from Efl.Gfx.Path

ector: fix property/method name conflict for "fill"

Shape implements a property, sw/gl/cairo implements a method

rename method to op_fill

eolian: convert all of cedric's tabs to spaces in .eo files

eolian: stop build with EOLIAN_WARN_FUNC_DUPLICATES > 1

ecore_wl2: Send window hide in response to xdg popup done

Clients should be sent this event when the compositor sends popup done.

efl_ui_win: move inital focus set to focus_in handler

it seems that focus changes to FOCUS=FALSE are causing autodel windows
to kill themself, so we only set the focus on the window if the window
manager calls focus in AND we dont have anything to focus and nothing is
focused yet.

edje - entry - fix empty item handling

tjhis should fix empty items which might have a null item string...
deal with it

fixes T6668

@fix

eina - tiler -= return true if rect alreadt there as its not a failure

docs say return true on succesas, false on failure. adding a rect we
already added is not a failur. it's an optimization to a NOP. so fix.
this was brought up by and fixes T6669 ... but in the opposite way.

elm - fix new em_config func prototypes in header to have real param decl

efl ui format - fix format checking to be correct

the format string checking was just ... wrong. it didnt' handle %%. it
didn't handle constant strings with no format in them... now it does.

fixes T6697

@fix

elm progressbar - fix val set

set initial min and max to 0 and 1 respectively so val set works at
the start...

fixes T6697

@fix

elm - make progress test code a little more solid and nice to read

use sizeof the buffer rather than constant in snprint and 0.5 rather
than .50 which almost looks like a typo... :) also use correct escape
for % (%%%% so it becoems %% which is correct for a fmt string yto
produce %)

elm: Fix double shutdowns of ecore_x and ecore_wl2

This is now done elsewhere.  wayland clients were ERRing on shutdown.

ref 9cf61c2f6d215e3adfa1567154a229eb49116769

wayland_imf: Clear hide timer when deleting context

Otherwise the callback can fire after the module is unloaded leading to some
very entertaining to debug crashes at shutdown.

Elm Progressbar test: Add in examples of efl regressions to progressbar.

ecore_wl2: Destroy offscreen event handler in surface destroy

Or we can get the event after destroying the surface and crash.

efl_mono: Distcheck fixes

- Export required sources
- Avoid generated sources being passed as static ones

eina_mono: Avoid calling eina_init directly from the modules.

They must be initialized from the eina.Config.Init() method.

ci: make sure we fail the build and pass on the non zero exit code

So far we just kept going on when one of the make targets failed with
the next one.

efl_part: call Efl.Text of each class' part

Summary:
Macro ELM_PART_TEXT_DEFAULT_IMPLEMENT calls
super class' Efl.Text interface, rather than this class.

Reviewers: jpeg, woohyun, Jaehyun_Cho, herdsman

Subscribers: cedric, id213sin

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

efl_part: use efl_ui_widget's default_part_get

Summary:
efl_part macros are using each widget's internally defined
default_part_get() functions to get default part name.
This might potentially cause errors when future widgets
inherits the widget but not overriding Efl.Text.text and
Efl.Content.content.

Reviewers: jpeg, cedric, woohyun, Jaehyun_Cho

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

ector: Updated the ector_buffer_pixels_set() api with stride info

Reviewers: jypark, jpeg

Subscribers: cedric

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

elm: remove ecore-x init from quicklaunch post-fork

this is invalid for basically every possible case. allow components
to naturally call this function as needed

@fix

elm: call ecore_app_args_set() after quicklaunch fork

ensure that apps get expected behavior when calling args_get()

@fix

ecore-imf: do ecore-x init during ctx creation instead of module init

this avoids a number of issues with efl under wayland, msot notably the
automatic triggering of xwayland during every app startup

@fix

elm: move ecore-x and ecore-wl2 connection init to selection manager

this seems to be the only place where the related components are
explicitly used

neither of these components have fork-safe connections, so there is no
benefit to calling them during quicklaunch init