elm_widget: do not check for the visibility

it seems like the visibility is evalulated from different parent
relations, Just because the box (that is a container of a element) is
not visible, does not mean that the element in it is also not visible,
this somehow comes back to the box beeing deleted, the children beeing
reparented in evas, but not in elm. This is for now just a quickfix,
since those calls are just for optimization.

This fixes a load of errors once the elm_test window is closed

@fix

eina_list: Ensure single init

This fixes a crash in make check when --profile=dev is explicitely
enabled. eina_list_init() is called by the standard eina_init() loop
and by eina_debug_init() as well.

Honestly I'm not sure why it doesn't crash for other people as
well...

The crash was in eolian_suite during the second eina_init (called
from eolian_init).

evas: Fix a crash in emotion with GL

Ooops! When mapping for writing we can't use the same code
path as when tofree is true. This restores the path for image
writing. This basically fixes rage with GL engine.

See also 45c8e5e983

slider: fix jumping handler bug.

Summary:
Actual slider value is not changed.
So when user release mouse or touch, slider value is returned
to the value by user set.

This is a partial revert of f41e2761608fff7b070d630a738

Fixes T5745

Test Plan: elementary_test -> slider

Reviewers: jpeg

Subscribers: cedric

Maniphest Tasks: T5745

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

evas: Restore flag pre_render_done in render_mapped

Some objects don't go through render_pre (unchanged, child
of mapped parents), even though they will go through
evas_render_mapped. Those were marked as pre_render_done
inside evas_render_mapped since it seemed to fix some issues
a long time ago.

Unfortunately, if those objects are changed their flag may
not be reset to false, which means they never go through
render_pre, leading to render issues.

I believe simply restoring the value of the pre_render_done
flag should be good enough. I don't know why it is set to
true inside evas_render_mapped but I also don't want to find
out :)

See 9ac13e4aec83ea6dadfa62e4253b1db4ffe167e8 (old)
See 87e5e70a9d888ce93ed807bb0d39d1738241ad9b (older)

Patch made with @jiin.moon

evas: Avoid infinite loop on evas shutdown

If an object fails to call efl_destructor() on all the parent
classes, then it may never be removed from the layer object
inlist and never would its data be deleted either (eo keeps
it safely alive).

Here's how to test: add a "return;" statement inside an object's
destructor (eg. the window class).

Fixes T5680

evas focus: Fix crash when shutting down with errors

Example test: add many evas_object_ref() to a window and click
on its X to close it.

elm: Fix a safety error message

Happens when shutting down E

evas gl: Rename secsym_ to eglsym_

eglDestroyImage has nothing to do with SEC...
This is a cosmetic change.

evas gl: Check EGL version before using dlsym

A recent commit broke texture_from_pixmap for NVIDIA EGL
(again), because eglCreateImage is a symbol in libEGL.so
but isn't in fact implemented by the driver.

That's because eglCreateImage() is exposed by libglvnd but
the underlying EGL implementation is NVIDIA and its version
is only 1.4, not 1.5 (where the API was introduced as core).

Instead of reverting the patch, it's better to cover our
bases properly and use dlsym() only if the version is right.

Note that GetProcAddress() may return garbage function
pointers for ALL functions as dynamic virtual functions may
be created on the fly by libglvnd. So it is absolutely
necessary to check the extension string as well.

See 0255f14dc2189c71776408b00307b8488bfa4dc5

gl_common: extsn is no longer unused

Also, we should throw an error when it's NULL, since it's important
for finding symbols for extended egl/gl.

gl_common: Prefer unextended eglCreateImage

eglCreateImage is objectively better than eglCreateImageKHR - it allows
attributes large enough to hold pointer values.  We should use it when
available and only use the older extension version as fallback.

Also, eglCreateImage is core EGL functionality so don't depend on extensions
to be present to use it.  Theoretically we should be testing for
EGL version >= 1.5 but it's probably safe not to.

gl_common: Remove fallback hack for wayland

This was only necessary due to bugs in the wayland_egl and gl_drm engine
that have been corrected.

Wayland has no bizarre requirements making this necessary.

gl_drm: Defer gl symbol lookups until they're possible

See previous commit for details.

wayland_egl: Defer gl symbol lookups until they're possible

Commit 2e6587a14b adds a gl extension string to glsym_evas_gl_symbols()
to prevent using functions that are provided by extensions that aren't
available in the current gl context.

However, we can't query the extension strings until after we create an egl
context.

Split the regular symbol lookup stuff from the gl symbol lookup stuff so
we can query at the appropriate time.

wayland_egl: de-wtf the eglGetProcAddress prototype

gl_drm: de-wtf the eglGetProcAddress prototype

This doesn't seem to serve any purpose other than rigorous proof that
typecasts can make literally anything compile.

gl_common: Stop looking for eglCreate/DestroyImageOES

These aren't a thing.  the GL_OES_EGL_image_base extension doesn't exist
and the GL_OES_EGL_image extension doesn't extend egl, because it's a gl
extension.

So let's stop being the only google match for these nonsense strings.

eo: Fix deadlock due to the shared mutex

Explicit lock / unlock of the shared mutex mixed with implicit
lock / unlock when accessing the internal Eo_Object data lead
to uncaught issues such as these.

This was found by trying to run E with gfx filters under eo_debug.

evas: Fix cmopilation warning

Oops I forgot to add that to the previous commit!

evas: Fix support for image_data_get on snapshot

evas_object_image_data_get() is a legacy API that I made work
with snapshot objects (evas_object_image_snapshot_set()). Some
changes in the engine broke the behaviour and this patch fixes
it.

When getting the pixels from an FBO, in read-only mode, we need
to create a temporary image (pixels surface) that contains the
copy of the pixels we get from glReadPixels. This image needs
to be deleted afterwards. It is thus stored by the image object
and freed upon _image_data_set() (good) or object deletion (bad).

FBO + read-write is not supported by this API (it is supported
through buffer_map as the filters had to use that).

Fixes T5754

evas engine gl: Add missing image unlock

This is a rare error path. Unlikely and not very important as
the lock/unlock doesn't actually perform any meaningful operation.

multibuttonentry: Fix item resource clean up logic.

Summary:
There are four cases where items are deleted.

1. Select item and push back button.
2. Delete "X" part of item.
3. Calling elm_object_item_del().
4. MBE deleted.

Previous logic is can't support for all cases.

below problems were there.

eina_list_remove() called twice.
list data free twice.

@fix

Test Plan: elementary_test -> multibuttonentry test.

Reviewers: jpeg, cedric

Subscribers: cedric, jpeg

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

emtoion gst1 - workaround gst break between 1.10 and 1.12

so emotion was using a much older (pre 1.0) buffer map mechanism for
getting video data. a new frame map api was added to gst in 1.0 but we
didnt use it. gst broke the old buffer direct mapping for some codecs
paths between 1.10 and 1.12. since we were using a very old method/api
this broke us. this also happens to fix some vaapi issues (except
where the buffer is simply never mappable ever). so this is kind of a
fix (updating us to as more modern api) and a workaround.

@fix

eo: clarify what extensions are.

elm_widget: do not register disabled logical nodes

this reduces the load of useless logical trees that

ref T5753

elm_widget: eval all children if the disable property has changed

since all children could have a new register state in the manager

fix T5753

elm_widget: disable is a property across the tree

if a widget is disabled its children are also meant to be disabled.

ref T5753

eldbus: support output dir in codegen

@feature

elm_test: Set label as word wrap

This exposes an elm_win sizing bug. To test (under X):
  ELM_DISABLE_EXTERNAL_MENU=1 elementary_test -to "Main Menu"

The layout is not completely wrong 100% of the time, but it is
exactly correct 0% of the time. Window sizing is our flying
spaghetti monster.

win: Fix main menu when not in framespace

On OSX the framespace and CSD (Client-Side Decorations) are not
supported at all... I am not able to test this case. This patch
restores the main menu functionality based on pre 1.19 themes,
where it was located inside win.edc (app content) and not in
border.edc (framespace).

Note that the initial size of a window may be wrong, eg as in
  elementary_test -to "Main Menu"

Fixes T5734 (hopefully!)

ecore_evas: make sure fn_post_render callback is called last.

Should fix T5595.

Text: move Annotation type to Efl.Text.Annotate namespace

Efl text: replace all usages of cursor handle to same type

Changes cursor handle name from 'Efl.Text.Cursor.Cursor_Data' to
'Efl.Text.Cursor.Cursor'.
Also, replace all usages of Efl_Canvas_Text_Cursor
with Efl_Text_Cursor_Cursor as the handle for the cursor.

Evas textblock: fix legacy header

Fix ABI changes with const qualifiers, as well as some missing docs.

Ui internal text interactive: handle signed cursor position

CID 1377551.

Ui text: don't send focus action to cursor on init

It made the cursor animation start on init, even if widget wasn't
focused.

Ui text: add more examples to test

Ui text: set default color (white)

Canvas text: re-add support for tabs

Somehow this was left out. Fixes tabs, so you can now use "\t" and
press the "tab" key.

Ui text: use new api instead of 'style' property in the test

OSX: fix version detection in build

eolian: always generate a class initializer

Because there might be hidden C implements defined, we need to
always generate the appropriate class initializer. This does not
hurt anything as the contents would have been called automatically
by Eo anyway.

@fix T5736

win: Amend previous commit about modifiers

The code was invalid, so making it valid now. Explicitely marking
the modifiers as unused, as mentionned in the documentation as
well.

See 9baa8752a9170e68e9430474f26cc1e672677d8e

eina debug bt sig handle - fix build on osx for clockid_t

fix #ifdefs so it should in theory build on osx 10.11.x

@fix

edje_signal: reduce member count when deleted edje_signal_callback is found

Summary:
 - when deleted callback is found _edje_signal_callback_move_last() is called
   in order to pack match array.
 - during _edje_signal_callback_move_last() index skips when another deleted
   callback is found, but does not reduce members_count.
 - this duplicates a remaining callback and calls the callback twice.

Test Plan:
1. add multiple edje_signal_callback by edje_object_signal_callback_add()
              which have the same source, signal, func but different data.
           2. delete first and last callback by
              edje_object_signal_callback_del/edje_object_signal_callback_del_full.
           3. emit edje_signal.
           4. observe one callback is called twice.

Reviewers: SanghyeonLee, conr2d, jpeg

Subscribers: cedric, jpeg

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

win: Fix ABI compatibility for grab modifiers

This fixes an ABI change when moving from unsigned long long
modifier mask to a simple enum. This is a fix for the release
of EFL 1.20.

ecore joystick: Use const char * in API, not slstr

The API eina_slstr_ is meant to look just like const
strings, wihch means here there could instead be an internal
struct holding the name, and the API would behave the same.

Note: This API is new in 1.20

Revert "elm_widget: separate elm_widget_item codes from elm_widget.c"

This reverts commit 4f8d20c8265385f5eb7412dc96327b273cadd0b6.
I'll apply this after finishing of release.
Thanks JP to give notification.

elm_widget: separate elm_widget_item codes from elm_widget.c

This will help to focus on creating efl_ui_widget class work.
And, we need to change all the Elm_Widget_Item related logics to
factory or something else.

evas-gl_cocoa: fix compilation

Booo for commit 2e6587a14b5e301e66f0ec5ef080eccb897b0fc5.

evas gl - fix compositing/native surface with egl/gles after glvnd

this fixes an issue that has cropped up in the past few months - only
nvidia drivers with egl/gles in x11... and compositing won't work
(native surface) and the introduction of libglvnd

it's a combination of libglvnd lying that it has symbols it can't
later find, new features to get core functions via procaddress that we
hadn't migrated to use AND use preferring core functions that libglvnd
will expose, so switching to KHR extensions by preference. we also
need to symmetrically use destroy image khr too...

oddly enough using procaddress purely for create/destroy image makes
wayland fail ... sofor now i'm taking advantage of the fact that
wayland has no extensions string passed in at the moment and still
doing dlsym... this is odd though.

@fix

efreet icon cache create bin - check realloc results and handle

@fix

efreet cache create binary - check realloc results and handle failure

@fix

eo lexer - handle malloc failure with longjmp

handles possible error case nicely

evas mesh save - use alloca for tmp strings instead of heap

this reduces code and means we dont have to handle NULL exceptions as
alloca always "works".

@fix

elm_code: add missing selection signals

build: set tests according to profile by default

ecore-evas: make the window_get apis return null without errors on failure

the normal usage of these is something like

if (!strcmp(engine, my_engine))
  win = window_get(ee);

which is a waste of effort since the window_get() functions all check
the engine interface internally

eina: make tiler test compare tilers which are not equal

eina: fix eina_tiler_equal to not always return true

ref e3f4f4457d1614912cd43fc9db3dd07a7a8fe911
broken by D996

@fix

eina: make tiler function params const where appropriate

stop throwing warnings when passing const params to functions which don't
modify the params

examples: elm_icon: remove no-op deprecated function call for icon lookup

The function is deprecated and actually a no-op already. Better make
sure we do not promote deprecated functions in our won examples.

tests: elm_popup: mark unused parameter as such to avoid warning

efl.ui.image.zoomable: mark setter of img_region property as eo api

thanks to stefan for reporting.

edje_cc: Add copy item for proxy part

Summary:
Although user use "inherit" in a state of proxy part,
the inherit function does not copy the source_visible and
the source_clip value of parent to the state.

Test Plan: {F1751132}sample

Reviewers: jpeg, cedric

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

@fix

ecore_evas: Perform a tick before a manual render if we're not ticking

If we turned off the ticking with manual_render_set then perform a
manual render, we need animators to update.

ref T5462

ecore_evas: Don't start custom tickers for manually rendered canvases

When a canvas is manually rendered the ticker is just a waste of cpu, and
worse - it can wake the drm back-end from dpms sleep, as the display needs
to be awake to generate vblanks.

We fire a DBG message when attempting to start an animator in this state
because it's frequently a bug that wastes battery life - (like E doing idle
cursor animations or clock updates while the display is off)

However, dpms off is not the only potential usage of manual render, so
another commit will follow shortly to fix the bug this commit introduces -
when using a backend with a custom ticker and doing manual render with
the display on, calling ecore_evas_manual_render() will not draw with
updated animator state.

Fix T5462
Again.
Really.

ecore_evas_drm: Provide a last_tick_get callback

Allow the drm engine to provide a tick timestamp for the previous vblank
interval.

ref T5462

ecore_evas: Add a function pointer for last_tick_get

Engines that provide their own tickers may need to be able to provide the
time of the last tick even if they weren't sending ticks to EFL at the
time.

This is a feature added during freeze as it's necessary to resolve a bug.

ref T5462

ecore_drm2: Add a query for the next vblank time

I guess this is a feature, and we're deep in freeze, but:

a) this is critical for fixing T5462 properly without any side effects.
b) ecore_drm2 is all beta api
c) this should only affect wayland users

ref T5462

ecore_evas: Refactor event catchers to call custom_tick_begin/end

Make the event catcher functions call the custom_tick_begin/end helpers
instead of directly calling the engine functions.

ecore_evas engines: Add some nulls to the func arrays

Purely cosmetic, but gets the comments back in sync with reality.

ecore_evas: protect against application holding a reference on the canvas while destroying the ecore_evas.

doc: Include two missing properties

elm: naviframe/prefs : do not include non-public header files

We are not shipping those (part of the private eo header files) and thus
we can not include those either. Simply removing them fixes the problem
I see with our ABI checker and it still passes al build tests I have.

elm: build: fix indentation of file lists

I was looking over this file for other problems but this jumped into my
eye.

eina: inarray: remove documentation pointer to non-existing functions

These do not longer exist in our current code base (did they ever?).

Thanks a lot to Vincent Torri for spotting this!

efl_ui_focus_manager: this whole thing is not just eo api its also beta

elementary: no need to provide the new legacy api

The api is beta AND only usable in eo, so no need to add those apis here

win: fix compiler warning

Thermal theme: Remove text from theme and quit wasting space.

ecore_anim: set the tick thread's fds CLOEXEC

If we don't set them CLOEXEC then they end up in the fd space of every
client Enlightenment launches, allowing any client to write to them
and mess with animator timings or freeze the compositor.

efl-wl: initialize variable

CID 1377573

build: efl_wl: make sure we have the two files  for efl_wl included

These two files coming in via EXTRA_DIST have not been included becuase
they have been living inside an ifdef which would only have them
included in a dist when configured with wayland enabled. We always want
to have these files in the dist though. Building for wayland out of a
tarball release works now again.

Problem reported by William L. Thomson Jr.

edje_edit: fix source generation

efl-wl: change x11 atom name var to normal char*

efl-wl: free atom name when sending selection

CID 1377542

efl-wl: free x11 pipe read buf on read fail

CID 1377520

efl-wl: return if pipe reading fails in x11 bridge

CID 1377526, 1377534

efl-wl: check returns in x11 selection request handler

CID 1377518

efl-wl: mmap byte for nul terminator in keymap string

CID 1377545

efl-wl: check return of ecore_main_fd_handler_fd_get

CID 1377541, 1377546, 1377519, 1377529, 1377543

efl_wl: check mkstemp return

CID 1377533

efl_wl: error and cleanup on CLOEXEC fail

CID 1377530, 1377531

efl_wl: Fix resource leak

Coverity reports that 'name' is leaked here, so free the return
ecore_x_atom_name_get.

Fixes CID1377537

@fix

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

elput: Fix resource leak

Coverity CID1377539 says that we leak the eldbus reply here when we
error out, so be sure to cleanup 'reply'

Fixes CID1377539

@fix

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

ecore: Fix coverity resource leak

Coverity CID1377544 reports that poller is leaked here if we error out
and return null. As poller is created with calloc, we should free it
before returning.

@fix

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

edje: give proper width to entry's cursor according to its theme

Summary:
evas_object_textblock_cursor_geometry_get() always return 0 as cursor's width
for BEFORE type cursor. It casued different cursor width when cursor_fg2 was shown.
The cursor_fg and cursor_fg2 must have same width according to its min width.

Also, the patch will enclose code for cursor_fg2 by cursor_fg code.
It is used when only there is cursor_fg.
@fix

Test Plan:
1. Give bigger width to entry's cursor by changing its theme.
2. See the width cursor_fg, cursor_fg2 when there is [LTR text + RTL text].

Reviewers: raster, herdsman, cedric, jpeg

Subscribers: stefan_schmidt

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

evas_smart: Fix evas_smart_callback_description_find

bsearch was returnign a cb** not a cb*

Fixes T4540