ecore-wl2: Add event handlers even if no data device manager exists

If we have no data device manager present, we would still like to
receive events when input devices get added or removed so these
handlers should be setup before we return from this function.

@fix

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

ecore-wl2: Rename internal function

As we are in the Ecore_Wl2 library, lets use a function name that
reflects this.

NB: No functional changes

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

Evas textblock: add null check in evas_textblock_cursor_free

examples: Flesh out the evas-object-manipulation example

Summary:
This is one of the basic examples that introduces Ecore-Evas, so needs
to be more detailed in commentary than other examples.  It also points
out the use of the legacy API and directs the reader to the Eo example
where appropriate.

Reviewers: cedric

Subscribers: jpeg

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

evas: Fix mouse event info in global events

When using the legacy API (and in fact also with the EO API) to
listen to mouse events (move, in, out...) on a window instead
of an actual evas object, some information was missing:
 - buttons (bitmask of pressed buttons)
 - prev.x/y (previous position)

This is because Evas had not handled the event yet at this
point, it was coming directly from ecore_evas with incomplete
information. This patch involves evas a little bit earlier, and
also fixes evas_events_legacy.c to have consistent values for
cur/prev canvas/ouput coordinates. See also 890a91785 and
484dae76e675318. Those commits were making the pointer coord
a seat-based property (instead of canvas-based) but the event
should already have those proper values before converting to
a legacy struct. This patch restores the meaning of the DUP
macros, as I observed 4 different coordinates from the app side
(instead of just 2: prev and cur).

Thanks to Andy for reporting the original issue on the ML!

Efl.Ui.Image: fix crash on passing NULL file in file_set()

img was crashing on elm_image_file_set(image, NULL, NULL).

fixes T5697

add missing license line for libefl to COPYING

LGPL will do - we never declared a license so take the most
restrictive one we have for this (LGPL).

default theme - add signals to pointer theme to fix glitch in x

in x11 the mouse pointer is separate to everything else on the screen,
and so when screensaver kicks in and we fade to/from black or we
suspend/resume and do the same... the mouse pointer stays annoyingly
visible and it just lookes like a bug. this allows that to be fixed by
allowing the pointer to be suspended or resumed... :)

@fix

Ecore_Evas win32 module: fix memory leak

Add ecore_win32_cursor_show() API and ecore_evas_win32_cursor_(un)set() functions

This fixes double mouse cursor in elm entries on Windows (for example)

@feature
@fix

elm_cnp: fix selections for wl windows and FAKE wins

don't use hardcoded seat names and attempt to use wl protocol for
FAKE wins

elm_cnp: use local cnp when elm_cnp owns the selection

no need to use protocol to send an app's own selection to itself

elm_entry: don't use window_id to determine type of window

there's a real function for this...

ref a173efd0a9892daca66bde76f7e90c66f39b9d23

elm_entry: don't update PRIMARY selection under wayland

this doesn't exist yet, and elm_cnp forces PRIMARY updates to overwrite
CLIPBOARD which is not good

@fix

elput: don't leak event pointers

elm_cnp: mark x11 selection as lost when selection is lost in event

do not pretend that a widget maintains the selection when notified that
the selection has been lost

@fix

elm_win: remove engine checks from window_id_get()

if these exist then just return the id

elm_win: remove engine checks from wl_window_get

if there's a win, return it, otherwise this returns null anyway

elm_win: manually create a wl2 window for FAKE wins under wayland

this enables elm_cnp to function using protocol

ecore-wl2: create surfaces once compositor global is bound

it's expected that surfaces are created when a window exists

@fix

ecore-wl2: don't leak display event pointers

ecore-wl2: add display object refs for windows

ensure the display lifetime exceeds the window lifetime

@fix

elm test: img zoomable - fix compiler warnings

efl/src/bin/elementary/test_photocam.c:779:19: note: initialize the variable '_zoom' to silence this warning

win: Avoid CRI message before finalized

Test case was Bg EO API in elm_test

popup: fix wrong signal emit when remove all button

Summary:
In visuals_set function,
check action_area is NULL, and signal emit action_area,visible/hidden
So, before call visuals_set, action_area should be deleted.

Test Plan: Actually, action area has not min size, so it couldn't be observed.

Reviewers: herb, singh.amitesh, jpeg, cedric, conr2d

Reviewed By: jpeg

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

elm test: fake events - use efl_content_set instead

elm test: bg - use efl_content_set instead

there is no efl_packed on window. it won't work.

elm test: img zoomable - change anim code sample to use eo apis only

This sample was added to show the img anim usage.
lets make this sample to use eo apis only.

Evas and Eolian : Freed string buffer

Summary:
Summary : String buffer returned by eina_strbuf_new() is not freed in some cases

@Fix

Signed-off-by: Uma Devika <u.bodapati@samsung.com>
Reviewers: cedric, tasn, jpeg, raster, singh.amitesh

Subscribers: tanwar.umesh07, yashu21985, cedric, jpeg

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

eina_file: fix unmap of unmapped region when Eina_File has copied data and not a map;

Summary:
This was causing problems on non-Linux architectures as eina_file_real_close unmapped not mapped data. Added a "copied" flag to Eina_File which is set on eina_file_virtualize (on copied data), and tested for when eina_file_real_close does the unmap. I'm surprised Linux allowed this. Certainly all of the BSDs crashed with the previous behaviour.

@fix T5479

Test Plan: Example inlcude Rage and Enlightenment Thumb on BSD systems which use eina_file_virtualize with emotion to obtain album artwork.

Reviewers: raster, cedric, jpeg

Reviewed By: jpeg

Maniphest Tasks: T5479

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

evas,edje,elm: Mark all legacy objects as such

eina - fix eina_file_virtualize to copy data to an aligned addr

so we copy data to an UNALINED memory address (just after whatever
string we packed on the end of the eina file struct header). this is
bad. especially for non-intel architectures. this forces a 16 byte
alignment which should cover us.

@fix

Theme: Follow gadget naming conventions.

elm_code: Fix number parsing to include legal separators

This also allows rust to use underscore!
@fix

elm_code: Fix rust syntax to include types

efl.ui.image: fix the old align behaviour in img.

After ab23a5309490d726fe, evas_object_size_hint_align_set()
has stopped working on elm img and that patch was perfectly right
to do so but we don't change the old behaviour.
This patch is an effort to make evas_object_size_hint_align_set()
to work with elm img again.
make sure not to mix both eo apis and legacy apis.

Thanks to dave for reporting and jpeg to show the right fix.

fixes T4635

Revert "elm test: image align - use image align api instead"

This reverts commit 5a0db7cc679c5d2d6888b457f7d13a36ebd0a100.

Reverting this since ab23a5309490d breaks the old working behaviour
on image align. subsequent patch in efl.ui.image brings back the old
behaviour.

refer T4635

elm test: image align - use image align api instead

refer T4635

Revert "elm test: image - use table to align image swallowed in layout"

This reverts commit a4e37b61f9a402d07aacb7855a3487515ab1c10b.

elm: Don't alias in efl_part()

Inside efl_part() we don't know whether we are dealing with a text
or content API, so we can't actually guess the proper alias.

The legacy API should have already dealt with aliasing at this point.
The EO API should not use those aliases.

elm: Forbid calls to efl_part(obj, NULL)

This in theory should only affect the EO API usage. In EO
we don't want efl_part() to be used for NULL part. In other
words, there is no "default" part in EO objects. Instead, those
functions like text_set or content_set should simply be
implemented by the object themselves.

The legacy API on the other hand will make sure that the
"part" argument is set to a non-NULL value before reaching
this point.

elm: Remove EOLIAN tags in some files

Some functions were marked as EOLIAN when in fact they were
used by the ELM_PART override APIs.

Elm layout: only allow legacy for using NULL text parts

This is a follow-up to a4b79fdbe16e1495c187dc4f9da99924cf51e539.

efl_part no longer supports NULL parts.
NULL text parts are now aliased in legacy code beforehand.

Signed-off-by: Jean-Philippe Andre <jp.andre@samsung.com>

edje: Fix some API issues (missing const)

win: Try to fix some resizing issues with main menus

Omg this was a pain... The code is complete spaghetti and the
fact that the main menu is now in the framespace doesn't help
either. With this patch and the previous one (about ecore_evas_x)
Edi should be able to open correctly at the proper size and with
a visible menu.

This patch introduces a lot of changes so it'll need testing
before we accept it fully.

Fixes T5482

ecore_evas_x: Track changes in framespace size

If the framespace size has changed and by accident (or in fact, by
design) the evas size + framespace size is equal to the size sent
by the X server, ecore_evas_x was skipping the resize event. This
patch adds a tracking of the framespace size so that we redraw the
canvas if it changed.

This will fix issues with the main menu (since it's in the framespace,
23 pixels tall with the default theme & scale).

Note that all this is partly because the ecore evas size is the size
without the framespace, so weird calculations are made during resize...

Ref T5482

win: Fix some sizing issues with main menu

This should fix issues when the main menu is wider than the
window content. This assumes that the menu is horizontal and
aligned with the client content.

The theme should probably handle this case better but right
now I am lost in the spaghetti resizes happening between
edje, elm, evas and X...

I am not very happy with this patch, and I think border.edc
needs some love in order to make it more robust and simpler
to read.

Ref T5482

evas: Fix a crash in textblock

I could not reproduce it but the case should not happen,
so I'm adding a safety check rather than a silent return.

The private data should be NULL only after evas_object_free()
or before construction...

Fixes T5656

evas: Add internal type documentation (event grabber)

elm test: image - use table to align image swallowed in layout

refer T4635

Revert "build: fix examples dist based on various configure flags"

This reverts commit ded5b1e764fdc97403bd829b02106ae8b9af10a4.

this broke build in some cases

Revert "build: enable examples build by default"

This reverts commit b58629bbbef3d811700b016eb9fa97e2af71e1a1.

not needed

ecore-x - add ecore x dpms force api

add api that allows us to force dpms on or off. needed to fix e
support for non-suspend new systems that don't do S3 ...

@feature

edje: support legacy Textblock calculation logic for 1.18 edje file

Summary:
From EFL 1.19, Edje Textblock calculation logic was fixed according to
Edje documents. But, it broke old edje files which ignored text.min
option for minimum width. Even if the old edje files were wrong,
we need to support them as discussed from T5548.

Also, this patch will change default efl_version to 1.18 from 1.19.
So, without efl_version property, edje file will run on the legacy logic.

Fixes T5548

Test Plan: Turn on/off presentation mode in Enlightenment.

Reviewers: herdsman, cedric, jpeg, zmike, raster

Subscribers: stefan_schmidt

Maniphest Tasks: T5548

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

Adjusted by @jpeg

layout: Fix default content part

This fixes call to:
  elm_layout_content_set(ly, NULL, obj);

This only affect this legacy API, not the EO interface.

Thanks Dave for the report!

Ref 59081043a892f6db74a1d557c7422c7a543c909d

Emotion: freed allocated string.

Summary:
GStreamer API gst_caps_to_string returns allocated memory which needs to be freed with g_free() after use.

@fix

Signed-off-by: Uma Devika <u.bodapati@samsung.com>
Reviewers: cedric, tasn, singh.amitesh, raster, jpeg

Subscribers: tanwar.umesh07, yashu21985, cedric, jpeg

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

elm_code: Add rust syntax

build: enable examples build by default

the previous method of forcing this to be enabled for dist builds caused
breaks when the original configure disabled examples, as the little-known
DISTCHECK_CONFIGURE_FLAGS variable would need to also be set to disable
examples even though the user would think they were disabled based on configure
output

build: fix examples dist based on various configure flags

this would previously break if:
* cxx bindings were disabled
* elua was disabled
* base configure disabled examples and dist build disabled examples
* base configure disabled examples and dist build enabled examples

it still breaks if:
* base configure disables examples and dist build enables examples

build: add efl_wl .x files to EXTRA_DIST2

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

elm_spinner: Entry hide when inc/dec button pressed.

Summary:
The entry cursor does not disappear when the user pressed(long press value change case) inc/dec button.

@fix

Test Plan: elementary_test -> spinner.

Reviewers: cedric, jpeg, woohyun

Subscribers: cedric, jpeg

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

drm: Fix compilation warnings

eina file: Fix rare crash in line iteration

I'm not sure about the rest of this code, so it's possible that
the index is increased even if it shouldn't. But I've observed
a crash at this line, apparently when reaching the end pointer.

Filter: check nullity before accessing structure member

@fix

evas: add @since for map_coords_get()

ref eaeaf8831c139880a5e179cdbab8873320a62476

evas: expose evas_map_coords_get.

efl-wl: add .pc file

Add a new test for elm_image align inside a swallow

This is broken since 1.18, more info on T4635

software_generic: Improve dmabuf validity tests

Now that upper layers set load errors this can be less gruesome.

Also, add some checks that were in E here where they belong.

gl_drm: Improve dmabuf validity test

Now that higher layers are setting load errors this can be less gruesome.

evas image: Set load error on native image set failure

evas_object_image_native_surface_set() has no return, so without
setting load error we can't tell if it failed.

fix T5418

theme: use FLOOR rounding for digital clock themes

the default rounding method allows for a value of 60 when setting the
seconds on a clock

fix T5639

efl_wl: a multiseat wayland compositor in an evas smart object

build when wayland support is enabled and provide two test/demo cases

beta api

@feature

Reviewed-By: Cedric BAIL <cedric@osg.samsung.com>

ecore_drm2: fix scanout_on callback firing too often

We need to increase the on scanout count for a buffer only when the
plane it's on makes its transition from pending to visible.

Previously it was firing for every refresh which would break refcounting
for any plane using surface that didn't change every frame.

Merge branch 'devs/bu5hm4n/gengrid_item_manager'

This merges a little refactor of gengrids keybindings code.
The refactor includes consistent item looping in horizontal and mirrored
mode. It also fixes the divergence between the focused and selected item
after a few operations

elm_gengrid: remove duplicated code

elm_gengrid: refactor duplicated code

this brings the simple selection move into a single spot

elm_gengrid: refactor single selection move

elm_gengrid: refactor focus movement

This removes duplicated code and brings it into a smaller way more
readable function

elm_gengrid: gengrid never sets the focus objects this is always NULL

elm_gengrid: refactor reorder mode

Its breaking the logic down into 3 little functions that could be reused
later. Overall this reduces the code duplication

elm_fileselector: correct the focus chain

elm: Always pass valid part name inside part APIs

This affects the legacy content_set/get/unset part APIs. This
should avoid some unwanted ERR messages in case an elm_object_
API is used on an elm widget that doesn't implement said API.

What this does is request the widget for the name of the default
part if NULL was passed in. Since some widgets are not elm_layout,
they have to override the API themselves, which is why I made it
an internal EO API (rather than a series of efl_isa()).

In theory, part should never be NULL when reaching the internal
implementation code in the widgets, at least for content.

In EO, efl_part(obj, NULL) should be invalid.

Ref T5629

elm_entry: use layout text/content aliases when text_set/get content_set/get/unset

Summary:
- elm_entry has elm.guide text part and it can be set by "guide".
- However when using text_aliases_get, this cannot be found.
- Add elm_obj_elm_layout_part_aliasing_eval() internal APIs to make entry
  use proper aliases.

Test Plan:
1. Run elementary test
           2. Observe search entry has guide text with "guide" part.
           3. Run Entry 8.
           4. Observe "elm.guide" part also works.
           5. Observe "icon" and "end" part works.

Reviewers: id213sin, herdsman, jpeg

Reviewed By: jpeg

Subscribers: conr2d, cedric, jpeg

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

widget: Avoid ERR in elm_object_content_ APIs

If content_part_{set,get,unset} are called on a widget that
is not a layout (or flip, and a few others), then some ERR
messages would be printed out.

We hide these ERR messages as all elm_object APIs should be
callable on an object for which elm_object_widget_check()
returns true.

Ref T5629

Efl.Observable: Free an iterator after use in observers_update()

Summary:
- In _efl_observable_observers_update() iterator is allocated but not freed.
- This frees the iterator.

@fix

Reviewers: conr2d, jpeg

Reviewed By: jpeg

Subscribers: cedric, jpeg

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

genlist: move to next focusable/selectable item when looping

this fixes a bug in genlist when scrolling is enabled and
items at top and bottom are disabled. Focus behaviour is not normal
in case up arrow is pressed when focus is at the top enabled item
or down key is pressed when focus is at bottom enabled item.

fixes T5576

drm: Implement hardware plane support

Adds the ability for drm to place native surfaces on harware planes.

ref 33b50238d2d54e77c23f42c00c6bfafc9a9a1080

software_generic: fix eng_image_native_get

This has been a stub for a long time, but now we have a native_set, so
we should have a native_get too.

This is required for hw plane usage on the software engine.

Elm layout: fix broken text part name resolving

This adds a few workaround to overcome the efl_part port.
Quite a few layouts got broken in some specific cases.

gl_drm: Check dmabuf attribute version before trusting attributes

We shouldn't do anything with a dmabuf structure until confirming the
version matches the one we know.

software_generic: Check dmabuf attribute version during set

We should check here instead of only at time of usage.

dmabuf: Make a define for the version of the dmabuf attribute struct

I should've done this in the first place, and the version checks are
starting to spread out a bit.

Evas textblock: fix legacy_newline in legacy API

Fixes 4fda417bc9e0c59f3a3645057279a29cc0057cd2 that disabled
legacy_newline for both legacy (textblock) and new (canvas text) api.
It should enable legacy_newline for legacy.

Evas textblock: re-apply paragraph position inference

For some reason, c39855a8ac6f86c2dd8b1856ed70f85e0e521f3e also seems to
address an issue with height calculations (although not mentioned in
the commit message).

It already went in v1.19, so by applying
b8beb6834b01f9688cb5e7aa177bacbc713318b9 I also reverted that behavior.
Seems like a correct one, so re-applying.

Fixes T5594

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

build: split of BUILT_SOURCES cleanup from CLEANFILES

This was another instance of the arguments list to long problem when the
file list of CLEANFILES got to long. During distclean the cleanup failed
and left tons of files sitting around. We rm these files in our own
local clean rule now.

po: update po files for alpha release

tests : eolain: make sure all the needed files for land in dist

All kind of test files for the function pointer test in eolian have been
missing in dist and thus failed distcheck.