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

Eo: be a bit more detailed in the docs

this is completly not obvious. But now its documented.

efl_ui_focus_manager_calc: efl_wref_del deletes the wref

A happy thank you goes to the doc for leaving this trivial peace of
information undiscovered!

Followup commit will fix this

efl_ui_focus_manager:do not followup if we have not changed candidate

_followup_previous_direction should only be called if there was a real
change to the redirect. In was happening that we have not changed the
focus but called _followup_previous_direction, which lead to weird focus
changes.

elm_widget: remove focus set

calling this function was not working at all, so its gone now

elementary: stop using elm_widget_focus_set

elm_main: checking the redirect chain is not neccessery

the manager is in or has no focused element

elm_main: replace _elm_object_focus_set with efl_ui_focus_util_focus

efl_ui_focus_manager_calc: error if a element is not registered yet

efl_ui_focus_object: emit changed event after the flag gets set

otherwise new efl_ui_focus_object_focus_get calls could be confused.

efl_ui_focus_util: add a util class

the util class is for common code that is just added again and again.

elm_genlist: fix rare segfault

Conditions:
- style is "double_label"
- the is some content for items (i.e elm_label)
- elm_genlist_filter set was once called after genlist creation with
  NULL data
- label_get callback uses elm_genlist_item_prev_get on its current item
- at least one item is added as a sub-item
- ~2 blocks of items are added afterwards
- items are added quickly while holding 'enter' on an elm_button

@fix

Efl.Ui.Slider: add default indicator format to %0.2f

This was broken in legacy code as well.
Thanks to @taxi2se for reporting.

elm_test -> slider -> check vertical slider

@fix

Efl.Ui.Slider: replace default with on_drag in visible_mode enum.

Ref T6376

Efl.Ui.Slider: add visible_mode() as an indicator part API

Ref T6376

Efl.Ui.Slider: make indicator_visible_mode() legacy APIs

Ref T6376

Efl.Ui.Slider: make indicator_show_on_focus/indicator_show() legacy APIs

Ref T6376

tests/eldbus: Use different bus for cxx and C tests

I had eldbus_suite and eldbus_cxx_suite fail quite often for some
strange reason: name already in use... yeah but by who??? Turns out both
test cases used the same name, and when ran in parallel (make -j10
check) this would often fail.

tests: Add better debugging for eldbus failure

It keeps on failing for me, but quite inconsistently (i.e. sometimes,
not always). Annoying.

elm_index: Index item role modify

Summary:
Modified index item role to EFL_ACCESS_ROLE_RADIO_MENU_ITEM from EFL_ACCESS_ROLE_PUSH_BUTTON
as index item should maintain its current state.

Test Plan:
Query the role of index item from atspi client, ATSPI_ROLE_RADIO_MENU_ITEM role should
be returned.

Reviewers: kimcinoo

Reviewed By: kimcinoo

Subscribers: cedric, govi, rajeshps, jpeg

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

ecore_wl2: fix user pointer for dmabuf param listener

Proving once again that those that defeat type safety usually get what
they deserve.

ecore_wl2: Fix dmabuf test failure

We need to destroy the buffer manager on fail, since it will exist but
only be able to create buffers we can't actually use.

ecore_wl2: Don't try to close drm_fd if we never opened it

derp.

eina: fix docs for manage_read_only_new

win: Rename specific APIs to avoid clashes

This renames 3 basic APIs specific to Win:
 - name
 - type
 - role

Ref T5322

elm: return Efl.Access.Actions only if action_area is not set

Summary:
The accessibility clients should not have a possibility
to dismiss popup in case when action area is set.

Reviewers: jpeg

Subscribers: cedric, jpeg

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

elm_index: Send selected signal to atspi client

Summary: When Index item is selected, notify atspi clients.

Test Plan:
when atspi mode is enabled, item selection notification
should be received by atspi client

Reviewers: kimcinoo

Subscribers: cedric, govi, rajeshps, jpeg

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

elm: enhance access name for spinner

Summary: Try to take access name from internal entry object or text button first.

Subscribers: cedric, jpeg

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

elm: add Efl.Access.Value interface to progressbar

Subscribers: cedric, jpeg

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

elm: Define and rename internal atspi struct in EO

This thing is used by only 2 EO APIs that are marked as @beta. I wonder
if the @beta tag or the ptr() expression made it work for eolian,
because it simply wasn't defined in EO.

I'm renaming it just so that it's more consistent with the new names
used by atspi (and EO API in general).

elm: Rename elm_interface_atspi_widget_action mixin

Reviewers: jpeg

Subscribers: jenkins, cedric, jpeg

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

elementary: make sure that ecore shutdown match ecore init.

elementary: do not send a PAUSE event if not iconified.

ecore: should be the one triggering EFL_LOOP_EVENT_TERMINATE.

elementary: add an interface to enable/disable PAUSE/RESUME application state.

If you want to have EFL application behave in a more efficient way. Like dropping
memory usage or stopping unecessary network operation, this option should be on
and the application should listen on PAUSE/RESUME event on the main loop.

ecore_wl2: Don't open rendernode if we're not going to use it

We should only open this when actually testing dmabuf.  Otherwise we're
just wasting time and adding an opportunity to fail shm init over
unrelated issues.

ecore_wl2: Reset buffer_manager to NULL on failure

Calling this multiple times even after it fails the first time is a legit
thing now.  We'll be doing that when we want to test dmabuf at connection
start.

ecore_wl2: Send a dmabuf non-immediate create at startup

We use immediate mode dmabuf creation at runtime, but this can result in
clients being killed with no option to fallback if the buffers can't be
consumed by the compositor.

This test should catch when a system can allocate a dmabuf buffer and the
compositor claims to accept dmabuf, but the buffer can't actually be used
for whatever reason.  We'll then use wl_shm at runtime instead of dmabuf.