elm_part: Mark as auto-unref before finalize

This simplifies the IMPLEMENT macro, which can now be a proper function.
Also, do not set a parent for the part object, as it's not useful and
could mess up the refcount.

eo: Make sure auto_unref only applies after finalize

It would be pretty stupid to create an object that dies before it even
gets created. Auto-unref is for parts and should probably not be used
anywhere else :)

part bg: Fix up some unnecessary complexity

See previous commit :)

Note: right now the background part has a small 3d indent which comes
from the legacy theme being used. This will be fixed soon.

efl_ui_layout: create efl_ui_bg when efl_part for background is called.

Reviewers: woohyun, jpeg, cedric

Subscribers: felipealmeida, cedric, jpeg

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

Efl.Ui.Format: add string format checking for user input

widget: Add EINA_UNUSED

efl_ui_widget: secure elm_legacy_add flag

Summary:
_elm_legacy_add goes back to EINA_FALSE after setting sd->legacy.
if constructor get called again after going back to EINA_FALSE,
sd->legacy should remain EINA_TRUE.
also, elm_legacy_add() should not be called non-elm_widget.

Test Plan:
Run elementary test->Efl.Ui.Text.Label.
Check legacy flag in _elm_theme_object_set() for efl_ui_win.
Check legacy flag for efl_ui_text after scrollable text is added.

Reviewers: jpeg, woohyun

Reviewed By: jpeg

Subscribers: cedric

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

evas: remove warning from image_loaders/png/evas_image_load_png

Reviewers: stanluk, lukasz.stanislawski, cedric

Reviewed By: cedric

Subscribers: cedric, jpeg

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

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

efl_ui_bg: fix unclipped image on EFL_UI_IMAGE_SCALE_TYPE_NONE

@fix

Summary:
efl_ui_bg has internal image and image may be larger than
         resize_obj when EFL.UI.IMAGE.SCALE_TYPE is none.
         Now resize_obj clips the internal image.

Test Plan:
1.Run elementary_test, part background(will be added soon)
2.Create a bg in a box with other widgets.
  Set bg as EFL_UI_IMAGE_SCALYE_TYPE_NONE.

Reviewers: jpeg, cedric

Reviewed By: cedric

Subscribers: woohyun

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

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

elementary: fix typos in elm_win_legacy

Reviewers: stanluk, lukasz.stanislawski, cedric

Reviewed By: cedric

Subscribers: cedric, jpeg

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

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

elementary: fix typos in elm_gesture_layer

Reviewers: stanluk, lukasz.stanislawski

Subscribers: cedric, jpeg

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

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

elementary: fix typos in elm_atspi_bridge

Reviewers: stanluk, lukasz.stanislawski

Reviewed By: lukasz.stanislawski

Subscribers: cedric, jpeg

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

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

ecore_con: quiet 2 "clobbered" warnings in ecore_con

Reviewers: stanluk, lukasz.stanislawski

Subscribers: cedric, jpeg

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

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

evas: fix typos for canvas

Reviewers: stanluk, lukasz.stanislawski

Subscribers: cedric, jpeg

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

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

elementary: fix compilation of cxx examples

Summary: Missing ecore_con library

Reviewers: felipealmeida, jpeg

Subscribers: cedric

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

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

elementary: Windows requires EAPI for the legacy .eo.h

Summary:
After the reorganization of elm eos, some sources include the .eo
headers directly instead of through Elementary.h. This causes problems
on windows as the declarations won't be decorated with the dllexport
attributes.

Reviewers: cedric, felipealmeida, jpeg, vtorri

Subscribers: jenkins

Tags: #windows, #efl

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

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

ecore_evas: use access() instead of stat()

Summary: stat() wasn't working on Windows due to some msys2/mingw quirk.

Reviewers: felipealmeida, cedric, vtorri

Subscribers: jenkins, jpeg

Tags: #windows

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

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

elementary: fix compilation of elm_code test on windows

Summary:
Added some guards to avoid redefinition of functions.

Partially fixes T5866, as there is still the question whether we should
test internal functions or not, as stated by jpeg in the comments.

Reviewers: vtorri, felipealmeida, jpeg, cedric

Reviewed By: cedric

Subscribers: jenkins, cedric

Maniphest Tasks: T5866

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

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

ecore: during fd cleanup we didn't remove freed fd handler from the ready fd list.

elm: Add hack to make test case work

Because of the way elm_code test case is written, by directly including
the C file, we end up with two symbols for the internal _elm_legacy_add
flag. This makes some test case fail (after applying a pending patch).

Solution found by Sungtaek Hong.

elm_code test case needs to be fixed. Don't include the C files
directly, testing static inlines should be done through a common .x or
something, but not by including the C file itself. This has led and will
lead to many issues.

eolian: Add API's for part enumeration

@feature

bg: Remove trailing \n in ERR message

efl_access: Add reading_info_type set/get APIs

Summary:
Information obtained by atspi client is name, role, description, state of an object.
reading_info_type_set/get APIs give control to application to decide on the information
that can be exposed.

Test Plan:
The reading info is added as an attribute of an accessible object, on query of
attribute, reading_info_type and corresponding character buffer should be given to ATSPI clients.

Reviewers: kimcinoo

Subscribers: cedric, govi, rajeshps, jpeg

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

elm_code: don't crash on NULL scope

This could be triggerd with the cursor was not in the file

efl_ui_focus_manager: remove unnecessary debugging printf.

efl_ui_focus_manager: fixup previous commit

i forgot to git commit --amend ... i am sorry :[

efl_ui_focus_manager: try to error out instead of a infinit recusrion

elm_gengrid: jump back to the last selected item on gengrid focus jump

this moves the item into selection and focus that was last focused or
selected.

efl_ui_win: use new api

we can just call the manager to prepare itself in the direction we have,
this means we dont need this loop here.

efl_ui_focus_manager: move preparing of the manager to its own api

with this some imlpementations can decide if they want to fallback to
some older focused item once they need to prepare themself for this
call.

elm_gengrid: respect item config

elm_gengrid: remove on_focus_update

this hook is not executed anymore, code for remembering the last focused
item will be added in the following commits.

docgen: Correct title plugin usage

Efl.Ui.Calendar: fix wrong define - again!

efl_ui_calendar: fix wrong define

elm_genlist: Add signals to communicate drag start and stop to atspi client.

Summary:
Send signal EFL_ACCESS_STATE_ANIMATED when dragging starts and stops to atspi clients and also set EFL_ACCESS_STATE_ANIMATED
when reorder mode is enabled.

Test Plan: When reorder happens atspi client should receive object:state-changed:animated signal.

Reviewers: kimcinoo, shilpasingh

Reviewed By: shilpasingh

Subscribers: cedric, govi, rajeshps, jpeg

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

elm: efl access: fix the warnings.

There were many warnings while building elm.

--->
./src/lib/elementary/efl_access.eo.h:433:55: warning: duplicate ‘const’ declaration specifier
 -Wduplicate-decl-specifier]
 EOAPI void efl_access_attribute_append(Eo *obj, const const char *key, const const char *value);
<---

efl_access: Add attribute_append, attributes_clear APIs

Summary:
Add attribute append and attributes clear API, attributes of widget/widget_item helps in adding additional information
about the widget/widget item in the form of key-value pair.

Test Plan:
Query the attributes using atspi_accessible_get_attributes in atspi_client and an hash table consisting
of updates attributes should be returned.

Signed-Off By: Shilpa Singh <shilpa.singh@samsung.com>
Signed-Off By: Lukasz Wlazly <l.wlazly@partner.samsung.com>

Reviewers: kimcinoo, lukasz.stanislawski

Subscribers: cedric, govi, rajeshps, jpeg

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

wayland_shm: Fix partial redraws

I accidentally the age tracking.

ecore_wl2: Add dmabuf allocations for vc4

This should theoretically be better for software rendering on rpi3 since
it allows the compositor to use our sw rendered buffers as a texture.

elm_widget: remove focused unfocused events

the ones form efl.ui.focus.object are now taken.

prefs elm_entry: change to the new event

efl_ui_text: remove the event emits here

with the previous commit we emit the legacy events for each interface
event, and thus this event is not needed.

However, in elm_entry this means that changing editable will _not_
reemit the focus event (where i am not that sure if that is correct or
not).

elm_fileselector: forward focus changed from entry to fileselector

the old event is then emitted from elm_widget.c

elm_widget: move event emission to a callback

the callback gets triggered if the new event got triggered.

efl_ui_multibuttonentry: migrate to efl_ui_focus_object events

this merges the two two events callbacks into one

elm_layout: call the super for calling the events and methods

this simplifies the code

elm_multibuttonentry: on_focus_update is not needed anymore

the widget itself never gets focus, the elements of the button are
getting them. MBE is basically just a box. The text input for the entry
is setted by elm_entry itself when it gets focus.

The legacy "focused" event is emitted somewhere seperatly.

elm_spinner: lets use the new event

elm_popup: lets use the new event

elm_hoversel: lets use the new event

elm_fileselector: lets use the new event

gl_drm: Use the same gbm_device for multiple instances

If we want to share a gl context (we do) between multiple instances of
gl_drm, we need to make sure they all use the same gbm_device.

This resolves a blocker for multi-output on the gl_drm backend.

ecore: reformat ecore_exe_posix.c

ecore_wl2: Fix buffer manager refcounting

commit 0cf806005e32827f174f42cba27192c47ed9c4a7 correctly fixed a
leaked buffer.  However, other code was already accounting for the
leaked reference to the buffer manager, so an extra deref happened
and broke the universe - but only on hardware that no developer
has access to for testing.

efl_ui_focus_manager_calc: keep focus history arround

this now keeps items arround even if a explicit other widget was
focused. This is usefull if we have a few logical items on the focus
stack and you remove them.

efl_ui_focus_manager_calc: remove a logical item that caused a redirect

so we dont try to focus a logical item.

efl_ui_focus_layer: rely on _manager_in_chain_set

so we dont have to set the direct by hand anymore

elm_widget: dont use the inner var for searching the top widget

you think there is only one elm_parent behind the elementary property
parent? Oh no! Elm_Notify and Elm_Popup have theire own parent fields,
and those fields are not returned when calleing elm_widget_parent_get,
they are also not given to the elm_widget implementation of
elm_widget_top_get, so a call to elm_widget_top_get stalls at elm.notify
or elm.popup.

fix T6389

edje_cc: Not reset inherit_script during group inherit

If group inherits after setting "inherit_script: 1;", inherit_script
is overwritten by the value of parent group. However, inherit_script
indicates user wants to inherit script in this context, it should not
be initialized as false.

edje_cc: Fix successive script inheritance

eina: warning removed from eina_debug_timer

Reviewers: stanluk, lukasz.stanislawski

Subscribers: cedric, jpeg

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

eet: removed warning from eet_image

Reviewers: stanluk, lukasz.stanislawski

Subscribers: cedric, jpeg

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

elementary: removed warning from efl_ui_win

Reviewers: stanluk, lukasz.stanislawski

Subscribers: cedric, jpeg

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

eolian: Removed warning of unused var from eo_parser

Reviewers: stanluk, lukasz.stanislawski

Subscribers: cedric, jpeg

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

eina: Fix matrix header guard name

Summary:
eina_matrix.h includes matrix2, matrix3, and matrix4, so the header
guards shouldn't specify MATRIX3.

Reviewers: cedric

Subscribers: jpeg

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

drm ee: Allow more than 4 buffers

Multi-head is hitting corner cases where there are lots of locked buffers
and it looks like right now 5 is the magic number that makes the problem
go away.

Make it possible to set 5 or more (via env var) for testing, make a macro
for MAX_BUFFERS instead of just a number.

win: make _elm_win_opaque_update check that it has a wayland window

This function will destroy non-wayland engine data, so it should
make sure it's actually operating on a wayland window.

Originally the sd->wl.win test was sufficient, but now wl.win is
present on non-wl windows to facilitate cut and paste, so we need
to check more thoroughly.

ecore-wl2: Destroy buffer before we exit the buffer_test function

Small patch to destroy our test buffer before we exit the
_ecore_wl2_buffer_test function so that we do not leak here.

@fix

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

elementary: removed warning from elm_widget

Reviewers: stanluk, lukasz.stanislawski, bu5hm4n

Reviewed By: bu5hm4n

Subscribers: cedric, jpeg

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

elput: Remove unreachable code

As we are returning the evdev above, this block is unreachable so
remove it.

Fixes Coverity CID1382857

@fix

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

eolian: include all test eo files in build (for distcheck)

cxx: Fix slider example (bad downcast)

slider: Implement Efl.Content

radio: Implement Efl.Content

elm: Fix minor warning (return void)

cxx: Disable spinner example (broken)

A new API for spinner is coming soon. This example code is now broken
since my change on Efl.Content vs. Efl.Container.

button: Implement Efl.Content set/get/unset

This fixes make check (cxx compilation).

tests: Add EO check box in elementary_test

If checked, it will list only the EO test cases.
Also add some stupid icons, and try to make the test case names more
uniform.

Test cases marked as EO are not necessarily 100% EO code, but it would
be good to make them so.

image: Return file+key even for edje objects

slider: Remove bad defines from EO file

win: Make wm_available_rotations simpler for EO

The problem with the API is that it uses an array of ints, which is not
well defined in EO. We could set an array of pointers to int, but that
would be super awkward to use.

I believe the original API has been slightly over-engineered as it was
passing an array of available rotations when in reality only 4 rotations
could be supported (0, 90, 180, 270). It seems to me that the day
arbitrary rotation needs to be allowed, another API would be required
(maybe with a range, or a single bool flag to allow anything).

I have not modified the internal code, which still uses an array (as
ecore evas uses that).

Mote: ec464939d9b8e4daa removed wm_rotation_supported_get(), as well as
the preferred rotation from EO, but they could easily be added back if
needed.

Note 2: I couldn't test as desktop E doesn't support WM rotations.

Ref T5322

efl: Split Efl.Container and Efl.Content

When I first implemented the Efl.Container interface I made a mistake of
mixing "single slot" content API's with "multiple children" content
API's.  This should fix that, by separating API's that are for a single
part and those that deal with a list of children.

  Efl.Content: Single slot. This will be used a lot by efl_part()
objects, and for the default content of widgets (eg. the window
content).

  Efl.Container: Multiple children. Used by lists, boxes, layouts
(edje/elm), etc...

I didn't see any class that implemented both interfaces (note: Layout
implements Container and Button implements Content, so technically
Button implements both through inheritance).

For now the eo_prefix is not changed in Efl.Container. I wonder if it
should be reset (to efl_container) or not. This would only affect the C
API.

Ref T5328

efl_ui_focus_manager: emit callbacks after stack change

fix T6389

efl_ui_focus_manager_sub: remove event callback array when destructing

i forgot about that and this leads to a segfault in enlightenment, the
object then segfaults when the parent manager emits a event and then the
code tries to access the private data of a dead object.

eldbus: "fix" make check

As Cedric told me, eldbus_suite now fails on this line, where somehow
the properties array contains 2 elements, instead of 0 as expected. It
seems that the change is not related to EFL, but a new package on our
systems.

With d-feet (a dbus inspection tool), I can see two properties under:
 org.freedesktop.DBus
   /org/freedesktop/DBus or /
     org.freedesktop.DBus
       Properties
         Features
         Interfaces

Has anyone a better clue what's happening?

evas: fix typos in canvas/evas_object_textblock

Reviewers: lukasz.stanislawski, stanluk, jpeg

Reviewed By: lukasz.stanislawski

Subscribers: cedric, jpeg

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

elm: add "activate" action to ctxpopup items

Summary: Change-Id: I0974c3bec02a05d3d46d7cc0fd86572e32248a3b

Subscribers: jenkins, cedric, jpeg

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

elm: overload access children getter

Subscribers: jenkins, cedric, jpeg

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

eina: Indicate in and out parameters for eina_matrix

Subscribers: cedric, jpeg

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

elementary: fix typos

Reviewers: stanluk, lukasz.stanislawski

Subscribers: cedric, jpeg

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

elm_list: use internal focus set instead of public focus set

elm_object_item_focus_set ensures that the list also gets focus, thus calling
that in _elm_list_elm_widget_on_focus_update would result in a infinite
call recursion, if setting the focus fails (for example when the object
is not visible yet, [see enlightenment for that]).

This fixes a freeze if you open lunchers config.

wayland_shm: Don't use a fixed number of buffers

We no longer allocate 3 buffers at startup, we now allocate only as needed.

Trimming the queue will come later, as there are some situations where we
might need 3 buffers and later drop down to 2 (when on a hardware plane)

Most clients will only ever need 2 buffers, so this is a reasonable RAM
savings.

ecore_wl2: Properly handle buffer test failure during buffer_init

The buffer manager is already cleaned up at this point, don't try to
do it again.

eina: properly initialize EINA_ERROR_VALUE_FAILED.

eina: properly initialize EINA_ERROR_NOT_MAIN_LOOP.

eina: properly initialize EINA_ERROR_MAGIC_FAILED.

eina: fail with an Eina_Error on copy failure.

efl_ui_focus_manager_sub: do not specify a interface as first argument

this leads in eo to the meaning that the manager.sub is giving
implementations to that interface, but leaving the ->func with NULL,
which leads on some maschines to the assumation that this is
pure_virtual, which means not composition objects will be queried, on
other maschines this will work, since there other inherits will
overwrite this entry and set ->src to NULL.

While i still dont understand why this works on some maschines and does
not on others, this is is now fixed.

elm_widget: simplify code

there is now a utility function