edje_cc: fix wrong comment about lazEDC norequired.

Summary: - norequired; stands for required: 0

Reviewers: conr2d, jpeg

Reviewed By: jpeg

Subscribers: cedric

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

elm: Move calls to efl_destructor(efl_super()) to the end

When destroying any object, its parent class destructor should
be called after the subclass destructor has been called. Only
some extremely limited work may be done after the super call.

This commit makes sure that all efl_destructor() overrides in
elementary are doing operations in the right order.

Also, remove a return void.

elm_widget: only emit events when there is really a changed manager

A manager change to NULL is to be used as "this object is not registered
anymore" call, which will happen quite regular during shutdown.

elm_widget: call the events after eval outself with new manager/parent

otherwise we might end up with a node_get error, since the widget where
the event is called on is still registered in the wrong manager

elm_widget: only set focus.manager if we are really registering

otherwise we might set focus.manager even if we are not registered

elm_widget: give feedback if registration was successfull or not

with this we can whipe out the focus.manager field in elm.widget so for
the case that something goes wrong we only get the error message where
actually something went wrong, and not the whole bunch of follow ups
where the code assumes its registered but it isnt.

elput: Fix multiple open/close of drm devices

When I added the code to probe drm devices to ensure they're
modeset capable (ref 414d406b3b442216543cdaef112787696ae09898)
I didn't realize elput didn't allow us to open and close more than
one drm device at startup without blowing up libinput.

This is a somewhat dirty hack to rough that in.

The problem is that open/close the device during startup will
result in an async "gone" callback from logind, which then kicks
off an input shutdown.  We need to try harder to only do that
shutdown when it makes sense.

ecore-wl2: handle null offer drags more effectively

this is valid and refers to an offer with no types; a leave event
with no enter is a protocol error, however

fix T5770

@fix

Revert "ecore wl2 - dnd - handle NULL drags... shouldnt happen but does"

This reverts commit 85e585846600742a7f395751816f348b4152269f.

please do not randomly add null checks for the purpose of quickly closing tickets.

ecore wl2 - dnd - handle NULL drags... shouldnt happen but does

this should fix T5770

@fix

efl net server udp - report erro on alloc failure

unwind nicely and complain

ecore con proxy helper - complain when realloc fails

so you know why things might be failing... complain about out of
memory errors.

ecore exe - pisix - handle malloc fails

handle malloc fails better
@fix

ecore win32 exe handling - check realloc and malloc returns

handle out of memory errors better
@fix

ecore_exe - handle realloc failures by complaining and rolling back

@fix

embryo_cc - be sensible about realloc returns so analysers are happy

assingto tmp var then assign to real one after a check to analysers
dont complain.

efreetd cache create - fix reallocs to bail on out of memory cleanly

edje_cc - make svg loader (still beta) handle out of memory nicely

print ERR and abort.

edje convert - make realloc fatal and complain about it

cleaner error handling

edje_cc out: fix div by 0 correctly

elm_code: Fix crash with long lines

Also fixes issue where widget would sometimes blank when scrolling
@fix

edje_cc out - fix theoretical div by 0

fixes CID 1371125

edje_cc out ... fix theoretical div by zero...

if files size was 0... which would basically make a lot fail before...
it could be a divide by zero. fix and just assume minimum size of 1.

fix CID 1371127

efl ui grid - handle invalid 0x0 fully if the grid size fails to do it

if efl_pack_grid_size_set is overridden and somehow allows 0x0 to
still happen then handle this caase and skip the layout.

fix CID 1374317

edje_cc out ... check more iui list element ptrs to help coverity

fix CID 1374482 ... and others i hope.

edje_cc out - double check list element ptr to make coverity happy

fix CID 1374483

evas seat focus add - handle other case where layer may be null

fix CID 1377522

eina debug timer - handle fail wakeup write with error complaint

fix CID 1377528

add comment about poll high/medium/low implementation of events

not brilliant as it stands. :(

ecore main - fix division to remain a double/float as expected

1/60 != 1.0/60.0 ... the first is an integer which is 0... the second
is 0.016666666 .... a float/double. fix CID 1377532

eina debug - don't try and collect stats on more than a million threads

sanity check... just in case.

elm cnp - check if selbuf is null or not and skip copying if null

this should fix CID 1377536

evas callbacks - check if obj->events is valid because it's cow alloced

it COULD be null and we do have other checks for it being not-null and
we dont check if the alloc fails...  so check. this SHOULD fix
CID 1377547

eina debug - add comments to cpu debug using locks to pause sysmon

this is intended so leave comments saying so. coverity doesn't like it
with CID 1377549

eina debug - check return value of syscall for errors

this should fix CID 1377550

efl canvas image - dbug ingo get - dont get if no root dbg info supplied

if root dbg info is null we'll leak and not have anything useful
stored somewhere for fetching anyway.

this should fix CID 1365656

evas module load add comments pointing out this is intentional ptr loss

efreetd cache - fix leak of strbuf when error conditions abort save

fix CID 1374951

efreetd cache - handle queue case leaking strbuf

fix CID 1374954

ecore_audio headers conflict with app have ifdefs

Ecore_Audio.h had #ifdef HAVE_PULSE ... and that is just so wrong as
this is something an ap[p may or may not set in its config.h ... but
this certainly hase no place in our public headers. the api's there
should always be there... symbols always... just imtplementation may
be empty ... though ecore_audio doesnt build at all without pulse...
so it's moot.

so fix this build issue i found when fixing e build issues with meson
changes etc.

@fix

efl_wl - also add xkbcommon as it's actually used

ecore_drm2: Ensure device we find can mode set

Some systems have dri devices that can't mode set, and if they're first in
the directory they'll get picked by our code and things fall apart later.

So, open the potential device and ensure it has basic functionality before
selecting it.

This is a little inefficient as it gets the device via elput twice before
it can be used - this will be addressed later as the changes are a little
more invasive to optimize.

ecore_drm2: Open elput manager before drm device

We're going to need to access the device as part of the find process,
so we need a manager.

hover: Return valid slot for main_menu_submenu

Fixes T5760

ecore-evas-wayland: destroy frame callback on hide

ensure cursor surfaces can render again after being hidden

 #TheDisappointer

@fix

eldbus: print full filename when codegen write fails

wayland: stop creating 1x1 surfaces on init

fix T5226

 #IGot99TicketsBut1x1AintOne

elm_win: instantiate csd frame after creating pointer surface

csd attempts to create sw cursors if the pointer surface doesn't exist,
so create the pointer surface before the csd to avoid duplicate cursors

 #TheDisappointer

elm_win: do not set invalid geometries on the window object

clamping to 1x1 is not valid, neither is trying to always position at 0,0

 #IGot99TicketsBut1x1AintOne

elm_win: hide pointer surface on mouse out

this should be unmapped so edje stops trying to render to it, though
at some point it could be optimized to not hide and just reuse the buffer
when showing again

 #IGot99TicketsBut1x1AintOne

elm_win: show pointer canvas after setting cursor

setting cursor calls resize, which is what sets the surface size

 #IGot99TicketsBut1x1AintOne

evas sw x11: formatting

gl_drm: half fix eglGetPlatformDisplayEXT usage

Just because the #define is present doesn't mean the extension is, so we're
BAILing on egl completely on some systems for no good reason at all. (saw
this on an SGX stack)

This is still wrong.  I don't want to try too hard until after the upcoming
release, though.

We should actually be testing for the presence of client extensions before
attempting to do any of this.  It's entirely possible that a gl stack will
return bogus functions for these from eglGetProcAddress

efreet - dont complain if INTERNAL x extension hash of desktop is NULL

asking for an x- field from a desktop shouldnt result in null safety
checks IF the x hash is null - it's fine if it's null - that field
just doesn't eixst. if other fields were in the x hash but the desired
one wasn't it'd not complain, so why complain if there is just no x
hash?

evas xpm loader - don't error print for files that cant be mmaped

no point complaining because we can't mmap... it'd be an unloadable
file.

efl_wl - add wayland-server pkgconfig dep for headers from wayland

efreet - remove noisy debug printfs

Evas textblock: add definition to 'cursor_equal'

This is a follow-up to c4ea25cbdbfb027d5119880ddcf0bf5ba6a5720b to fix
legacy API.
The definition was missing, so adding it.

eldbus: codegen: free allocated strbuf in error path

fname would have been leaked if we failed to get a file_handler.
Also make sure we always create a new fname strbuf even without
a output_dir present to fix a potential unbalance between creating
and freeing.

CID: 1378112

release: Update NEWS and bump version for 1.20.0-beta3 release

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.