ecore_drm2: Merge ecore_drm2_device_free and _close

There's no reason to have them be separate, all callers must do them
back to back, so let's just combine them.

ecore_drm2: Merge ecore_drm2_device_find and ecore_drm2_device_open

There's never a reason to find a device and not open it, so all callers
end up calling these two functions back to back.

ecore_drm2: Remove connector bitfield

This is unreliable - there's no reason to expect these connector ids
will be low enough to sensibly store in a bit field.

ecore_drm2: remove unused crtc bitfield

We'll need something like this when multi-head works, but it can't
be implemented this way anyway.  There's no guarantee that crtc
ids will be low enough to fit sensibly in a bitfield.

gl_drm: Remove some stale comments

We do planes now, so this comment isn't helpful anymore.

ecore_drm2: Pass Ecore_Drm2_Device instead of fd to most functions

Intended to simplify the upcoming commit that merges device find and
device open into a single function that returns a device.

The fd is something callers shouldn't really need to get their hands on,
right now there are still a few places where it's needed, but those will
be gone soon too.

elc_naviframe: Fix to handle item deletion right after item push

If an item is deleted right after the item is newly pushed, the push
transition should not happen for both the new item and the current item.

elm: Prevent style/theme set after finalize (EO)

This is only for EO obviously. style_set and theme_set should only be
called when the object is being created, not after. On-the-fly style
changes are complex to handle and in most cases it should be easy to
simply repopulate the object after creating a new one with a new style.

There are only a few cases where style_set is used long after creation
of an object, like changing how a label slides, or in the evas 3d map
examples. Menu seems to change the hover style a lot, so rewriting it in
pure EO would need some extra work, maybe.

Ref T5307
Ref T5363

elm: Fix modules due to a missing EAPI

I recently added an undef EAPI which wasn't in fact the best idea ever.
The EAPI needs to remain defined as is for elementary modules and
edje_externals.

Ping @vtorri

See ad6e3ce3df422751cb1c4524f1ea0b16ce897410

elm: Cleanup theme and style set functions

This makes efl_ui_layout_theme_set() return a Theme_Apply error
code. The type is now public as Efl.Ui.Theme.Apply.

Ref T5329
Ref T5363

layout: Remove unused import in EO file

Ref T5363

Fix recent openjpeg detection for Jpeg2000 on Debian

ecore glib support - if we are not the main ctx owner loop to acquire

the code didn't try acquire every loop if we were not, so it wouldnt
wake up if someone stole context (and then gave it up). found by roy!

@fix

test_gesture_layer3: add null checking after memory allocation

test_gesture_layer2: add null checking after memory allocation

test_gesture_layer: add null checking after memory allocation

elm_prefs_data: add null checking after memory allocation

elm_gesture_layer_extra_gestures: add null checking after memory allocation

elm_gesture_layer: add null checking after memory allocation

elm_dbus_menu: add null checking after memory allocation

elm_color_class: add null checking after memory allocation

elm: Add missing undef EAPI to Elementary.h

This may solve issues on Windows where EAPI can be either dllimport or
dllexport. Anyway keeping EAPI defined is invalid and inconsistent.

Also, change an invalid EO_BETA_API define tag to EFL_BETA_API_SUPPORT
and EFL_EO_API_SUPPORT.

Thanks @vtorri for the report!

@fix

widget: Remove scrollable_children from EO

It's an internal only function, which IMHO was not required and an
overhead over the simple loop.

Ref T5363

widget: Clean up doc about disabled

Ref T5363

widget: Remove sub_obj arg to resize_object_set

This removes an argument that was false only for a single widget:
naviframe. Hopefully this logic is now simpler, even though it involves
a small hack within naviframe itself.

Ref T5363

naviframe: Override resize_object_set

This is a first step into allowing me to removing the sub_obj argument.
Make elm_widget resize_object a protected property.

See also D265

Ref T5363

eo: Add a space in documentation

This separates the brief and the content.

widget: Move signal functions to internal only

elm_widget APIs are internal only, even if they may be EAPI. The EAPI
here is for modules.

I believe this API is only required for layout objects, so simply should
remain in the layout class. Then you can do efl_isa() and call the
functions or accept error messages if you want to be more careless.

Ref T5363

examples: Correct the comments for key handling

Subscribers: cedric, jpeg

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

examples: Fix sp. exemple_data

Subscribers: cedric, jpeg

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

ecore_fb_li: add error handling code related with calloc

elementary cursor: remove a warning message from Wayland supports

Summary:
"_cursors_count" static variable is not used when Elementary is built
for Wayland. It should be removed if els_cursor doesn't use it.
@fix

Test Plan: N/A

Reviewers: raster, zmike, ManMower

Tags: #efl

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

ecore_audio_pulse_ml: Check memory allocation fail

ecore-evas-wayland: Add handler for Iconify State Change event

Small patch to add a handler for catching Window Iconify State Change
events

'#divergence'

@feature

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

ecore-wl2: Add support for Window Iconify State Change

This patch adds support for the Window Iconify State Change event
structure and the ecore event type to support it.

'#divergence'

@feature

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

test_gesture_layer3: fix typo

test_gesture_layer: fix typo

elm_panel: use widget data for getting geometry

_elm_panel_efl_gfx_size_set() ends up calling
_elm_panel_elm_layout_sizing_eval() to adjust the layout
according to the updated width and height.
however, evas_object_geometr_get() doesn't return the updated values.
in fact, it is not necessary to call any API since the values are
stored as widget data in _elm_widget_efl_gfx_size_set().

examples: Fix grammar - 'forcely' is not a word

Subscribers: cedric, jpeg

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

elm: Continue elm_layout renaming

This renames a few macros.

Ref T5315

elm: Rename elm_layout to Efl.Ui.Layout

Some names have not been changed, hopefully making a distinction
between legacy APIs and internal code (elm_layout_blah) and valid EO
usages.

This means many internal functions are still elm_layout_ as their
sole purpose is to support the legacy API.

Ref T5315

Evas Jpeg2000 loader: port it to openjpeg 2.*

Test Plan: files in https://github.com/uclouvain/openjpeg-data/tree/master/baseline/nonregression

Reviewers: jpeg, raster, cedric

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

eo: Fix memory leak with overrides

See 0834511067a99faafc2c59f0aabcd58dab1cbc0b
And b7840d9177b32f480087bd7f062cc66ab2583485

Ref T5580

layout: Implement missing edje APIs

This makes elm_layout implement:
 - efl_canvas_layout_signal_message_send
 - efl_canvas_layout_signal_process

This only transfers the calls from the elm widget to the internal
edje object.

PS: message_send is quite ugly in C...

Ref T5315

@feature

elm: Move elm_layout_sizing_eval to legacy

elm_layout_sizing_eval() marks an object as requiring recalc.

Unfortunately, it's been massively abused by various widgets into
actually doing the calc, or the min calc. So we end up with one API
that has 3 different definitions depending on the widget type:
1. Mark as requiring recalc (correct, respects doc, elm_layout)
2. Calculate min size and other size hints
3. Actually do some geometry modification

I believe we need to clarify these 3 requirements into 3 very clear
and specific APIs in elementary. Right now we have similar functions
in evas for 1 (evas_object_smart_changed) and 3 (smart_calculate).
But their exact definition also isn't necessarily what we want for
elementary.

Another clear problem is that layout_eval does not do any calculation
(in theory), so the "eval" word is a bit of a stretch here.

Once we're sure about the exact API we want, we can add this back to
EO and make it work across our EO widgets. For now let's just keep
the legacy API, and its EO overrides, as is.

Ref T5315

eo: temporary prevent segfault.

In some case, detected during eo test suite, the vtable does fail to
be fully assigned, but it is still being assigned as the new vtable.
Of course when later destroying it, it has already been freed. Leading
to a double free.

elm_win: set shadow geometry based on borderless state

@fix

elm_win: simplify/fix recalc logic when changing csd

forcing a full eval here is unnecessary and broken since such an eval could
either change geometry in unexpected ways or fail to accurately change
the underlying canvas geometry

@fix

ecore-evas wayland: remove move() function

this is conceptually wrong to include in a wayland engine

Canvas text: fix re-add of existing style to list

Styles were appended regardless their key already existing.
Now it's fixed.
Also, improved readability of 'style_set'.

ecore-evas wayland: clamp max size hints to 0

evas wayland: always create gl context when non-existent

evas image: add checks for ENDT existence during image free

this is null e.g., during enlightenment restart and causes crashes, probably
should be removed after underlying cause is resolved

@jpeg

evas_object_image: formatting

this was unreadable at parts due to mixed tabs/spaces

evas wayland: call makecurrent before querying gl attrs

fix T5845

Allow to override build date

Summary:
Allow to override build date
for distribution packages that use autoreconf
to recreate the configure script

See https://reproducible-builds.org/ for why this is good
and https://reproducible-builds.org/specs/source-date-epoch/
for the definition of this variable.

Note: This patch will work with GNU date. If BSD date support is
important, a more complex patch needs to be done.

http://rb.zq1.de/compare.factory-20170713/efl-compare.out
also shows other diffs in .edj files that still remain

@fix

T5495

Reviewers: cedric

Subscribers: stefan_schmidt, jayji, netstar, jpeg

Maniphest Tasks: T5495

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

edje: make the generated default program name reproducible over builds

For programs without specific names edje_cc generated default names in
the form of program_$MEMORY_ADDRESS. That worked well enough for keeping
the names unique, but it causes problems if one wants to have these files
being binary reproducible due to different memory layouts, compilers,
etc. Simply using a counter as unique part should work well enough for
our use case and help people who want to verify builds.

Thanks a lot to Bernhard M. Wiedemann for review and testing.

Fixes T5113
Ref T5495

examples: fix build break after efl_ui_panes change

In commit 40945d985967e58d6e6c00ecfd6292a8d67b83b5 it was renamed but
the examples not updated.

efl_ui_focus_manager: free iterator when done

efl_ui_focus_manager: fix leak when setting order

We should free the list when we set it again.

Revert "efl: terrible kludge so avoid termination crash on osx"

This reverts commit 9368eedd353a88dd31ba50c71433f81a2978adb5.

The release is out so we can revert this bandaid again. In the hope to
find the real culprit and solution before the next release.

elm: rename elm panes to Efl.Ui.Panes

ecore evas wayland - add explicit falltrhough tags for warning removal

edje - add embryo echo command for debugging

finally add one...

@feature

edje crash with run program references - fix

i found a crash today where a heme could cause a crash if it just did
the right things. the run program was freed while still being
accessed. so add some ref counting to keep it alive until references
go to 0. and add soem refs while we store it in lists.

@fix

efl-wl: no-op the re-setting of minmax/aspect handling

efl-wl: add handling for external protocol interfaces

@feature

ecore-evas-wayland: rework window hint calcs to be more accurate

this is still pretty broken for some directions, client-side aspect fml

efl-wl: set minmax for test app

efl-wl: support min/max hinting

@feature

efl-wl: watch/update aspect hints in test

elm_test: set window aspect hint for icon transparent test

elm_win: propagate window aspect hint in wayland

wayland: add efl-hints protocol for setting aspect on surfaces

@feature

theme: clip start gadget arrow to intended clipper

make activate visual state function as intended

@fix

efl-wl: set cursor surface roles

ensure cursor surfaces without buffers will be mapped if a buffer is
attached

@fix

efl-wl: add handling for commit during canvas render

prevent release of pre-existing buffers during a render cycle

@fix

ecore-evas-wayland: update size when size hints are changed

ensure hinted size is used

@fix

efl-wl: do shm pool ref/unref for bm safety

@fix

wayland: fix min/max size setting

also don't set arbitrary hints on init

@fix

efl-wl: block connection attempts from unmanaged clients

I guess posting an oom error is security

@fix

ecore-wl2: sync aux hints after window creation

ensure hints are available when needed

@fix

ecore-wl2: don't send null set_parent requests for every window

this is redundant

@fix

ecore-wl2: make _ecore_wl2_display_sync_get() local instead of static

ecore-wl2: simplify server pid detection

ecore-wl2: pass display to aux hint event_adds

fixes display refcounting

ref 6bb56b3f5651fab85b1a0a1a1a8040f4e488c799

@fix

elm_test: reset win stack static vars when window is closed

ecore_drm2: Fix linker problems

Accidentally used functions in the library directly instead of through
the sym_* dlsym looked-up variants.

Why this only caused problems in some installations may still be worth
investigating - we may be pulling in libdrm at link time from some
other library?

efl_wl: fix typo in file name to make sure xml file lands in dist

In commit 6bb56b3f5651fab85b1a0a1a1a8040f4e488c799 a new xml wayland
protocol file was added. Due to a typo in its name in EXTRA_DIST it
never made it into the tarball breaking the wayland build.

edje: implement edje.player interface functions

fixes T5560

layout: fix "elm,state,text,hidden" signal never be emitted

Summary:
 In _elm_layout_text_set function, text_signal_emit is called.
 But in that case, check text whether it is null or not null before call signal_emit.
 So "text" is not null always, and text_signal_emit's parameter "visible" is also always EINA_TRUE.

Reviewers: Jaehyun_Cho, cedric, jpeg

Reviewed By: Jaehyun_Cho

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

layout: Fix compilation break

Oooops! This broke external apps!

layout: Implement data_get from edje

This also implements size_min and size_max get.

Ref T5315

edje: Move size_min/max and data to an interface

The new interface is:
  Efl.Canvas.Layout_Group

I'm not fan of the name.

Ref T5315

elm: Remove custom layout signal APIs

This merges them with the now standard interface:
  Efl.Canvas.Layout_Signal

Some wrapping work was required for legacy API which
takes no user_data in del() but instead returns it. The
new EO function, while harder to use, is more correct
(you can't delete the invalid callback by accident, and
this follows EO events design).

Another crazy wrapping was done in entry/text in order
to add the callbacks to 2 objects instead of just one,
and still return the user data.

As for Naviframe and Popup, those two widgets override
signal_emit to forward the call to another object than
the resize object, but not callback_add/del. So they
are definitely broken.

Ref T5315

edje: Move signal APIs to an interface

This interface will be used by elm_layout as well.

Ref T5315

layout: Use only legacy "elm_layout_sizing_eval"

This removes all uses of elm_obj_layout_sizing_eval.

Ref T5315

layout: Improve doc for theme_set

Ref T5315

layout: Move Part_Alias struct to legacy

Part aliases need to be handled differently in EO.

Ref T5315

layout: Move edje_get to legacy only

Here's the reasoning:
 1. We will expose as many edje APIs as possible (and meaningful)
    through the elm layout class.
 2. Access to internal objects is usually risky, as it allows apps
    to bypass EFL in some ways, leading to potentially undefined
    behaviours.
 3. If the need arises we can still add a similar API back to EO,
    later.

Back to #1, it seems that the need for edje_get() was mostly to
call manual sizing functions, or the missing message_send(). I will
make sure these are accessible from the layout itself.

Ref T5315