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.

ecore_wl2: factor out part of buffer creation

So we can use it to make a non-immediate mode buffer test.

ecore_wl2: Filter buffer types by compositor capabilities

It does us no good to be able to allocate dmabuf capable memory if the
compositor can't handle it.  This should fix failures on systems where
allocation is possible but the compositor doesn't advertise dmabuf.

ecore_wl2: Add a second sync for dmabuf formats

While we don't yet use these, we should, so let's make sure they're
available before we start making windows.

ecore_wl2: Refcount syncs at startup

There are some binds at startup that result in additional information
being sent, so we may need to call wl_display_sync() multiple times, and
only send the client a SYNC_DONE event when the final one completes.

ecore_wl2: Create xkb_context before compositor connection

We really need this to exist before potentially processing seat/keyboard
stuff in _ecore_wl2_display_connect()

wayland_shm/ecore_wl2: Move buffer allocation into ecore_wl2

This moves all the platform specific buffer allocation into ecore_wl2
instead of the engine.

Note that this makes an internal struct available in the header.  This
will be removed shortly.

wayland_shm: Move wl_buffer creation function further up file

It just looks nicer when it's here and doesn't require a prototype.

wayland_shm: Make the buffer_manager static

Should always have been, oops.

wayland_shm: Become unhoseable

Now that shm is handled through these paths the dmabuf_totally_hosed state
is nonsense.

wayland_shm: rename Surface_Type to Ecore_Wl2_Buffer_Type

wayland_shm: Remove surface_type from surface

This has become not only a write-only variable, but also incorrectly set.

Just remove it.

wayland_shm: Pass ecore_wl2_display instead of surface to buffer_create

The display was the only thing we used through the surface.  Pass just
the display to break the dependency between surface and buffer.

wayland_shm: Remove surface from buffer struct

All users are now removed so we no longer need this.

wayland_shm: Stop looking up alpha on surface

We've got it in the buffer now.

wayland_shm: Store alpha in buffer

Currently the buffer code looks up the alpha stats from the surface code.
This won't be possible when we move the buffer code into a library, so
prepare for it now.

wayland_shm: Make ecore_wl2_buffer_init return bool

Nothing uses the returned pointer, and nothing will.

wayland_shm: Rename types and functions

We're going to move this into ecore_wl2, so I'll start by renaming it all
here before moving large chunks with minimal change.

No functional changes.

wayland_shm: Remove extra display flush

Since we're using create_immed now we shouldn't need to force a flush here
anymore.

efl_ui_calendar: set focus object protected mode

i forgot a define here and was confused by the fact that we have two
calendar widgets, sorry for that.

efl_ui_focus_manager: unset the redirect chain once a redirect is set

this prevents the manager from having a redirect chain while not even
beeing used by the root manager

efl_ui_focus_manager_calc: correct handle focus when redirects are set

so we dont have two focused elements

efl_ui_focus_manager: ensure the manager is in focus when focus is set

this makes handling of focus easier in elementary

efl_ui_focus_object: document the bahaviour of the manager property

efl_ui_focus_object: make it protected

_this_ focus set should never be called by anyone other than the
efl.ui.focus package

elm_test: Fix a warning

CtxPopup: Add implementation for getting focused item

Summary:
Adding the focused_item_get implementation for ctxpopup widget for
getting the focused item

Signed-off-by: Godly T.Alias <godlytalias@yahoo.co.in>
Test Plan: elm_object_focused_item_get

Reviewers: cedric, raster, prince.dubey, shilpasingh, jpeg

Reviewed By: shilpasingh

Subscribers: Jaehyun_Cho, Blackmole, jpeg, rajeshps, shilpasingh

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

genlist: fix state corruption of content

When using reusable content, genlist preserves old object's state and is
expecting reusable_content_get callback to change all needed properties.
But there was an inconsistency: it was silently re-enabling the old content.

@fix

Ctxpopup: resizing ctxpopup after its content is removed

Summary:
When ctxpopup's content is removed,
ctxpopup doesn't recalculate its size, so empty space is remaind.
To fix this problem, ctxpopup recalculate its size when content is removed.
This patch is about T6327

@fix

Test Plan:
1. run elementary_test -to ctxpopup
2. select 10st item
3. click button in ctxpopup

Reviewers: jpeg, Jaehyun_Cho, zmike

Subscribers: cedric

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

efl_ui_win.c: send bounds changed signal on window rotation

Summary:
On window rotation send bounds changed signal to inform ATSPI client
about the orientation of window

Test Plan:
In ATSPI mode, when orientation of efl window is changed, ATSPI client should
get notification

Reviewers: kimcinoo

Subscribers: cedric, govi, rajeshps, jpeg

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

elementary: make sure efl_general.h detection work also when you are not using BETA interface.

elementary: protect from stall include.

This should be reverted before next release as this header was only
there during this release cycle.

elementary: provide network when doing ui.

elementary: only include Efl_Ui.h when BETA API is requested.

elementary: elm_widget actually rely on elm_object_item.

elementary: fix typos in header inclusion rules.

ecore: always declare EFL_MAIN not just for application using the BETA flag.

Revert "Revert series of patches from cedric (due to dependencies and conflicts)"

This reverts commit 2cea85db388d34337676466ef7f50c22e685c7d0.

Their was a typo that I made during cleanup of the patch before pushing that I didn't
notice broke some stuff. But also you may have an old efl_general.h in your elementary
directory that is now being picked instead of the one provided by the tree.

Revert series of patches from cedric (due to dependencies and conflicts)

Revert "elementary: currently double declare elm_init/shutdown."
This reverts commit 44bb0c18480f5094fcd0c8be93de87be5c1d78c5.
Revert "elementary: fix efl_ui_multibutton installed headers."
This reverts commit 32a213dc722be2bfec5fb2b513dd556cfb7e9f33.
Revert "elementary: introduce Efl_Ui.h."
This reverts commit df3d3f7334a79f1ab661b31787002f50af59b3f3.
Revert "ecore: do not display error message on cancel."
This reverts commit 99654b7cd29b418e0308be350c8d26208c0905a8.
Revert "efl: and don't forget to install the new dependencies."
This reverts commit 814ffb9b6bd50d498bfd98f4b8a75f1cad3cc0cf.
Revert "ecore: remove EFL_OBJECT_BETA as Efl_Core.h is for Efl new inerfaces."
This reverts commit 619d0f3cff023414feb8f2aaeebf468806c50b44.
Revert "ecore: move EAPI_MAIN from elementary to ecore."
This reverts commit e5d84da864214b9f772808040f22e1614889a25f.

as such commit e5d84da864214b9f772808040f22e1614889a25f starts the
breaking. enlightenment, terminologya and other apps can't compile
against that efl anymore. 619d0f3cff023414feb8f2aaeebf468806c50b44
then makes this even worse with even more header errors and undefined
types. on top of this df3d3f7334a79f1ab661b31787002f50af59b3f3 then
starts making elementary_test segfault when it runs. it wont even
start up.

asu such of these 7 commits in the first 4 (that are then relied on
later) 3 of these first 4 cause serious breakage. this simply is a
complete lack of testing changes, so i've rolled fl back to before
these things so it builds and works again and you can build against it.

PLEASE test these things. this looks ot me to be obviously a lack of
any testing... :(

elementary: currently double declare elm_init/shutdown.

elementary: fix efl_ui_multibutton installed headers.

elementary: introduce Efl_Ui.h.

ecore: do not display error message on cancel.

efl: and don't forget to install the new dependencies.

ecore: remove EFL_OBJECT_BETA as Efl_Core.h is for Efl new inerfaces.

ecore: move EAPI_MAIN from elementary to ecore.

elm_code: Add basic scope parsing to syntax highlighting.

This is currently displayed by showing scope in the widget gutter

eina: Improve API docs for eina_crc.h

Reviewers: cedric, ajwillia.ms

Reviewed By: ajwillia.ms

Subscribers: jpeg, segfaultxavi

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

eolian: scan system dir by default

The '-S' option lets you reverse that. But by default, most
people will want the prefix to be scanned for eo files.

elc_popup.c: accessible name_get interface added

Summary: accessible name_get interface added, popup does not have default text.
What should be read for the name information should be define by popup.

Test Plan:
In atspi mode, when an AT-client such as screen-reader queries, it should return
as "Alert title(if present)/text(if present)"

Reviewers: kimcinoo

Subscribers: cedric, jpeg, govi, rajeshps

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

eolian: Fix odd syntax (remove EINA_UNUSED)

cxx: Only give a single ref to unparented objects

On instantiation objects get either one or two refs:
 - with a parent, they will have 2 refs, one for C++, one for the
   parent.
 - without a parent, they get a single ref, the one for C++

This will break the existing C++ examples, which I will fix in later
patches.

Note that the window is a strange object which can be created with no
parent but internally reparents itself to an object it creates (oh so
ugly).

win: Move autodel to C only

Note: this is C only, not legacy only.
The problem is that bindings will hold a strong reference to the window,
which will then die "under the rug" if autodel is enabled. This then
leads to at least ERR if not crashes.

Note:
elm_policy needs to support autodel and quit on last del only for C
applications. Bindings will require some other mechanism that doesn't
break all assumptions wrt. references.

test calendar: Add test case for string format func.

Efl.Ui.Format: Add support for "struct tm" in string format.

test calendar: make format callback function better

Add comments too.