ecore_wl2: fix ecore_wl2_buffer_wl_buffer_get

It was currently only used internally and had the side effect of
creating a new buffer instead of just returning the existing one.

Now it's useful to external callers, as it only returns the existing
wl_buffer and has no freaky side effects.

ecore_wl2: Add surface_window_get api

One of the last remaining blockers for compiling dmabuf support as a
plug-in.

ecore_wl2: Add API to remove surface managers

Someone might want to do this I guess.

ecore_wl2: Add a way to register new surface managers

And use it for the existing dmabuf surface manager.

ecore_wl2: rename _surface_flush for consistency

It was originally thought that this could be common code for multiple
back-ends, but that doesn't really make sense now, so rename it to match
the other dmabuf functions.

ecore_wl2: Move private data allocation into back-end code

The backend should allocate its own private data and return it instead
of a bool.

This assumes all back-ends will need some manner of private data, which
is certanly true for the one back-end we provide.

ecore_wl2: Move backend specific members into a private data struct

buffers and current buffers are really specific to the dmabuf backend,
another plug-in might not need them.

ecore_wl2: Move surface size update into generic code

This didn't have to be in backend code in the first place.

ecore_wl2: Add API surface_buffer_create

The specific surface code only needs these generic surface bits to pass
to buffer_create, so make a helper function for that instead of queries
for w, h, and alpha.

ecore_wl2: Remove force from surface reconfigure

It no longer has any users or need to exist.

wayland_shm: remove dirty bit entirely

Now that we're not using it for session recovery we can strip it out
entirely.

wayland_shm: Use surface flush instead of setting dirty

The dirty bit was a dirty hack to let session recovery force reconfigures
on startup.

Now that we have a surface flush we can achieve the same thing by just
discarding all buffers immediately.

ecore_wl2: Move smarts from surface_map to buffer_map

This simplifies the surface code so that it no longer needs access to
buffer structure members.

ecore_wl2: Add buffer_fit API and use it from surface code

This tries to resize the buffer's useable area to fit the specified size -
this is possible if the stride of the buffer is larger than the current
width.

ecore_wl2: Add buffer age inc and use it in surface code

Technically not necessary, since we could do this with get and set, but
it's a little more convenient to the user this way.

ecore_wl2: Add buffer_age_set and use it in surface code

ecore_wl2: Add buffer_age_get API and use it in surface code

ecore_wl2: Add API for setting buffer busy status and use it

Since surfaces will be managed by plug-ins, we need to make sure they
don't rely on internal structures.

ecore_wl2: Add API for querying buffer lock status and use it

Since surfaces will be managed by plug-ins, we need to make sure they
don't rely on internal structures.

ecore_wl2: Make the surface interface a pointer

ecore_wl2: Move surface manager interface into public API

So people can write plug-ins for this stuff.

ecore_wl2: Call check function through a pointer

We'll need this in the function array when we make surface managers
modular.

ecore_wl2: Move dmabuf surface specific tests into their own function

Refactoring to allow non-dmabuf surface managers.

ecore_wl2: Make session recovery more robust

Clear out the window callback when doing session recovery, and
make sure we have a valid on if we get a double commit.

This should stop a session recovery crash, and fix a small leak per
recovery.

edje_cc: simplify thread tracking for exiting on signal

edje_cc: make -thread option

this option was broken due to immediately triggering EMFILE errors
during thread initialization and then attempting to exit while threads
were still ongoing

some work is still needed to make this an improvement over the current
default of not using threads

@fix

quicklaunch: return child pid from elm_quicklaunch_fork() on success

efl_ui_focus_manager_calc: prepare everything before border elem eval

otherwise we might miss the elements that are registered on prepare.

efl_ui_focus_manager_calc: check earlier for LOGICAL

this safes us performance

elm_gen****: do not call realize on items

better rely on the adapter and wait for realization so the adapter is
wether created or not even created but used with content. This fixes
item content focus, crashes at startup and a freeze in genlist test!

pyolian: improve a bit the 2 example templates

ecore: prune existing thread_join callbacks during ecore_fork_reset

permitting these to persist and trigger at a later point will cause
the process to deadlock while attempting to join non-existent threads

@fix

efl_mono: fix DLL instalation path on efl-mono.pc.in

Destination of libefl_mono.dll is OS dependent, this was not translated
to efl-mono.pc.in. This commit fix this issue.

eolian_cxx: Fix "unused parameter" warnings

The Klass(Klass &&other) one was causing "synthesized method ... first
required here' warnings on Windows gcc-4.8.

examples: Remove duplicated build code for mono examples.

tests: Check return value in tests.

GCC was complaining about ignored return values from realpath, chdir,
etc.

eldbus: Uncomment tests

These were commented out during the promises changes but seems to be
working now.

efl_mono: Remove unneeded code from tests

eolian_mono: Add underlying pointer to ToString

It'll make easier to track where an object is coming from.

eolian_mono: Fix event argument for strings

eolian_mono: Fix visibility of generated events

Protected members do not make sense in sealed classes like the Concrete
ones.

Also document the placeholder field in extern structures.

mono: Fix AM_TESTS_ENVIRONMENT

The export commands were messing up with the elm_suite, causing the
layout do fail to be loaded.

ecore_con: missed compiler warning.

ecore_con: restore ability for negative port on local connection.

Thanks make check.

ecore-con pass port into err print

ecore_con - give error output if port num is invalid

elm: make elm_object_text_get return markup info as well.

This commit solves following issue

https://phab.enlightenment.org/T6642

If I set object text as below
elm_object_text_set(btn, "Some<br>text");
then elm_object_text_get(btn) returns "Some text" not "Some<br>text".

win: Fix legacy win type name

Window constructor is hijacked by the finalize method, which messes up
the normal order of operations. As a result, the legacy type name for a
window object was "elm_widget" rather than "elm_win".

This fixes a crash in E (as it checks the legacy type name to verify an
object's type).

See D5748

eo: only call costly function _eo_op_desc_name_get when in debug mode.

efl-wl: don't leak xkb mmaps when running under x11

ref a32735e9a7bc537430d155a4af853d498dda2009

pyolian: improve the doc generation

efl-wl: create and maintain xserver-based keymap and keyboard states under x11

when run in a non-wayland environment, it's necessary to do some extra work
in order to guarantee that the keymap remains the same in the compositor as
in the xserver and to also guarantee that modifier states are accurately
applied even when the compositor is not actively focused

fix T6631

efl-wl: use image border to optimize surface rendering with 1 opaque rect

efl-wl: unset kbd mods changed flag after sending modifiers

@fix

efl-wl: fix no-op of setting keyboard enter on already-entered surface

@fix

ecore-x: add some useful xkb function wrappers

@feature

ecore-x: re-add implementation of ecore_x_connection_get()

this symbol has been missing since the removal of xcb

@fix

ecore-x: filter XkbNewKeyboardNotifyEvent before emitting ecore-x event

this is only relevant to watchers of the ecore-x event if there is a keycode
change

@fix

ecore-x: add more info to Ecore_X_Event_Xkb event

this event corresponds to xkb state changes which are useful when working
with xkb instances

@feature

efl-wl: add functionality for directly passing through xkb objects

when used in enlightenment, it's necessary to be able to directly use
the compositor's keyboard state to ensure consistency when keypress events
are not propagated

@feature

efl-wl: add docs for smart callbacks

elm - fix makefile to install missing header for elm_code

eolian_gen: remove reference to elm_entry because it is not installed

Summary:
elm_entry.eo is not installed to system directory like
         "/usr/local/share/eolian/include/elementary-1", and
         eolian_gen tries to refer elm_entry.eo which results in failure.

Test Plan:
Create any eo class file which inherits Efl.Ui.Layout.
           eolian_gen eo_file.eo
           observe eolian_gen finishes

Reviewers: jpeg, herdsman, woohyun, Jaehyun_Cho, cedric

Subscribers: conr2d, id213sin, JongminLee

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

efl_ui_focus_layer: avoid a call to NULL

Summary:
efl_ui_focus_layer_enable_set(obj, EINA_FALSE) can be called before
registered_manager assigned

Test Plan:
1. EINA_LOG_LEVELS=eo:2 elementary_test -to menu
2. terminate the elemetary_test
3. check that there is no focus_manager warning about a call to NULL

Reviewers: bu5hm4n

Subscribers: cedric, jpeg

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

efl_ui_legacy: add new interface to indicate legacy widget

Summary:
For now, how to check whether a widget is legacy or not
is to check flags in private data or static flag, which is set
during elm_legacy_add.
If Efl.Ui.Legacy interface is added, it can be easilly checked
by efl_isa(obj, EFL_UI_LEGACY_INTERFACE)

Reviewers: woohyun, jpeg, cedric, Jaehyun_Cho

Subscribers: conr2d, cedric, jpeg

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

efl_ui_win: Avoid a call to NULL

Summary:
parent_get and smart_parent_get are called in parent_widget_get
this also remove the duplicated code

Reviewers: jpeg

Subscribers: cedric

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

eina: Make return doxygen consistent

Summary:
Ensure @return defines error returns consistently.  In several cases the
errors were explained in the body but not mentioned in the @return docs.

Drop redundant return value documentation for clarity.  Some routines
were defining the return values both in @return and in the doxygen body.

Subscribers: cedric, jpeg

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

pyolian test: just rename the main eolian state

efl_ui_focus_manager: listen to deletion for regular and logical

otherwise we might exit with a load of errors

elm_genlist: prepare the item when its set

so the border element is set accordingly on the parent manager which
then can be picked up for focus relations.

ref T6453

efl_ui_focus_composition: only rely on manager

for the case of a custom manager this resulted in wrong behaviour.

elput: Fix null pointer dereference

Coverity reports this as a null pointer dereference, so check for
'seat' existence before comparing with strcmp

Fixes CID1385369

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

elementary config: Fix to use ELEMENTARY_BASE_DIR for configure path

Summary:
"--with-elementary-base-dir" option was ignored by recent patches on elm_config.
The macro is being used in elm_theme. It should be syncronized.
The default value of the macro is ".elementary".

@fix

Test Plan: N/A

Reviewers: raster, cedric, jpeg

Reviewed By: jpeg

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

ecore-x: add more null checks for functions

@fix

ecore-x: perform internal shutdown on io error if callback is set

avoid crashing in callback when referencing dead display or handling events

@fix

ecore-wl2: correctly translate spacebar keyname into key events

fix T6620

@fix

ecore_drm2: fix seat matching when checking for a device with a null seat

Summary: Check to make sure that the seat from the matching device is still null.

Test Plan: Make sure that devices are not being rejected while on seat1. This depends on another patch

Reviewers: ManMower, devilhorns

Reviewed By: devilhorns

Subscribers: ManMower, devilhorns, cedric, jpeg

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

ecore_drm2: Only use a virtual terminal on seat0

Summary: Only seat0 supports virtual terminals.

Test Plan: Make sure enlightenment does not error out on non-seat0 due to a virtual terminal error. Enlightenment will not start yet on seat1 with this alone

Reviewers: ManMower, devilhorns

Reviewed By: devilhorns

Subscribers: ManMower, devilhorns, cedric, jpeg

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

genlist: Avoid a call to NULL

elementary_test -to genlist
-> LOTS of warnings, when creating the genlist, as the items aren't
ready yet. This just avoids the WRN as recalc will happen later anyway.

efl_selection_manager: fix compilation on osx after rename

In commit c91ae98fd3ecf532036b16499ffd521a2e91e2a3 data was renamed but
the ifdefed osx part forgotten to update.

ecore_evas: Check for null ptr deref

Summary:
This patch fixes a tentative crash owing to ptr dereference.
Unlike ecore_evas_object_cursor_set and ecore_evas_object_cursor_device_set,
ecore_evas_cursor_set uses Ecore_Evas *ee before calling internal function which
internally checks null ptr dereference of Ecore_Evas *ee.

Test Plan: Executes test suite

Reviewers: cedric, raster, stefan, Jaehyun_Cho

Reviewed By: Jaehyun_Cho

Subscribers: jpeg

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

Change-Id: Id24aa10e044dc63c09b29aa0d29ddf52cd3f3e80

fix break of hoversels

this fixes T6627

elementary: fix compilation of efl_selection_manager on windows

Test Plan: compilation

Reviewers: cedric

Subscribers: jpeg

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

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

wayland: Reduce number of unneccessary display flushes

b48781aa6c85c72f620c57cebff96580ddd67899 fixed multiple bugs where the
display wasn't flushed correctly, however it was a little overzealous.

Some of the flushes were added after calls that only updated internal
state, some in internal functions in which the caller was already going
to flush, and some were after wayland protocol calls that are double
buffered anyway and won't do anything until a following commit.

Also, I've removes at least one long standing flush where the recently
added flush is in a better location than the original.

wayland_shm: Clean up duplicate prototypes

These were supposed to be removed when I moved the associated functions
into ecore_wl2.

widget: Ooops! Fix focus break in previous patch.

Sorry... Weird that elementary_test worked fine. I might have badly
tested.

widget: Move on_focus_update to Focus.Object

Following @taxi2se's recommendation. This is indeed a focus method, and
Widget already inherits from Focus.Object.

Ping @bu5hm4n who probably wants to adapt this further.

Ref T5363

widget: Remove item from on_focus_update

It's legacy stuff, and always NULL.

Revert "efl_ui_win : resolve ABI break"

This reverts commit 6319f09a18feced8c5df429df5cf9b66e77ff7fb.

widget: Use enum for orientation_mode

This was proposed by Dave on the ML, I think it makes sense. Right now
the enum is just like the boolean, feature-wise, but it makes more sense
semantically (mode is not a bool) and allows for future extension (eg.
only apply orientation update for landscape vs. portrait modes).

There is absolutely zero testing for this in our existing codebase. Yay.

cxx: Fix a trivial warning

elementary: simplify Efl.Ui.Text.Factory test.

elementary: simplify Efl.Ui.Text.Factory.Emoticons.

Canvas image: add Efl.Canvas.Text.Factory + use in Ui.Text

This interface has a simple 'create' method to create Efl.Canvas.Object
given a key.
This is used higher-up in Ui.Text in the next commit.

Ui text: add ability to set item factories

Added API to set an item factory object.
This is similar to the previous item providers (that worked with
callbacks).

You instantiate a factory object and set it on the Ui.Text object.
Each factory implements the "create" method from
Efl.Canvas.Text.Item_Factory.

This also includes 3 public factories (Image, Emoticon and Fallback):
  - Image factory: creates images from added entries (key strings)
  - Emoticon factory: creates emoticons by querying the theme
  - Fallback: creates image, then falls back to emoticon

If no factory is set, then the fallback (internal) factory is used.

See the added "Ui.text Item Factory" test in elementary_test for an
example of usage.

@feature

Edje: remove item provider from eo api

elm_win: move wayland display flush to after method calls

ref b48781aa6c85c72f620c57cebff96580ddd67899

eina: Clarify that Eina_Matrix_Type can describe any matrix

Summary:
The Eina_Matrix_Type enum is returned by eina_matrix4_type_get and
eina_matrix2_type_get; it is not Matrix3-specific.  Update doxygen
accordingly.

Reviewers: devilhorns

Subscribers: cedric, jpeg

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

eina: Cleanup spelling/grammar/punctuation for Eina List

Summary:
This fixes some typos and misspellings, massages grammar in a few
places, tidys up a little whitespace, and fixes incorrect docs in a spot
or two.

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

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

eolian: fix redefined function checks

For one, the hash has to be populated once per inheritance tree
and the skipping on already-validated funcs was interfering with
that as the function might have been validated in another
inheritance tree already.

Also, if a class appeared multiple times in an inheritance tree,
as is common with e.g. Efl.Object, it would get added into the
hash the first time and then checked against the second time,
which would result in a strange error message about the
function being redefined in its own class.

So now we prevent both cases from happening.

cnp: Rename selection data to content

This solves an issue with C# bindings where a field of a struct can not
have the same name as the struct itself. Go figure why.