ecore-wl2: handle case where session recovery listener triggers on dead surface

@fix

ecore-wl2: add function for checking if a client connection sync is done

@feature

ecore-wl2: add some accessors for useful Ecore_Wl2_Input struct members

@feature

ecore-wl2: split data offers into selection and drag

these are distinct objects and can coexist simultaneously; a drag
source should not overwrite an existing selection source

@fix

ecore-wl2: store names for inputs and add api for retrieving name

@feature

ecore-wl2: add checks for a valid display object in ecore_wl2_input fns

no display = invalid object, so this needs checking

ecore-evas: account for framespace when processing mouse movement

in some cases (e.g., the mouse-out emission from the ee wl engine),
this was calculating wrong coordinates, which would lead to a broken
chain of events like:
  object mouse out -> object mouse in -> object mouse out
which would severely break some apps

@fix

ecore-evas: add function for unsetting a specific seat's cursor

@feature

ecore: allow creation of fd handlers with no flags

it's possible to set flags to 0 with another function, so allowing
creation with 0 makes the api more consistent without breaking the
documented behavior

elm_entry: request both text and markup when pasting

setting this to only markup means that it's impossible to paste into
an elm_entry from a plaintext app under wayland

@fix

elm_cnp: don't explicitly set wl window type when starting drag

this is no longer necessary

evas: ensure even no-op renders emit RENDER_PRE

ref 67fae7aa0fdc9d778e8db88fc49bc149576994d2

tests: win_dialog: remove unused variable

efl ui win -> add test for centering + fix center of window before show

we haven't gotten replies yet on what our position or size should be,
so we should store them so centering works before show but after
resizing is evaluated (that also fixed by forcing an eval).

@fix

efreet: Fix failure to save list data to output

Summary:
This fixes a typo in the fix 55676b33, which introduced an invalid early
return from the save_list function, preventing it from outputing the
list data to the file.

@fix CID1375005, CID1375004

Reviewers: jpeg

Reviewed By: jpeg

Subscribers: stefan_schmidt, cedric, jpeg

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

evas gl: Fix typo and revert hack (GLES 3.1)

Omg... Thanks Daekwang Ryu for pointing me to my error. I remember
struggling a lot with this OpenGL API and libGLdispatch (glvnd) when
in fact this was all just a typo in the code.

GLES 3.1 and the upcoming 3.2 support need a proper test case...

See c68a40987404375b36460b6a7f5ccceb45444b57

@fix

elm_entry: Cursor disappears on calling select function in an empty entry.

Summary:
Issue: Selection cannot happen on an empty entry, if selection functions
are called on empty entry, cursor is hidden, even if entry is focused.
Soln: Check for whether entry is empty or not before proceeding with selection.

Test Plan:
1. Call select_all, select_region_set on a focused entry
2. You can observe cursor cannot be seen.

Reviewers: jpeg

Reviewed By: jpeg

Subscribers: cedric, jpeg

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

popup: move "visible_set(FALSE)" before calling dismissed callback

Summary:
 After hiding animation is finished, "dismissed" callback will be called.
 It means popup's visible state is FALSE.
 But if evas_object_show(popup) is called in it's custom dimssed callback function,
 popup's visible state is TRUE, so evas_object_show is canceled.
 So visible_set(FALSE) is must done before call dimissed callback.

@fix

Reviewers: Hermet, herb, jpeg, cedric, singh.amitesh, raster

Reviewed By: jpeg

Subscribers: woohyun

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

popup: enable to set last_button_number to 0

Summary:
If last added button is removed, any statements to set last_button_number in for loop doesn't be executed.
So last button number has previous value, not 0.
Because of that, action area is not removed, but that area has zero height, so it doesn't be shown.

@fix

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

Reviewed By: jpeg

Subscribers: woohyun

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

evas: Revise documentation for evas_async_events*()

Summary:
Straightens up grammar and simplifies phrasing to make the functions
purposes clearer.

Subscribers: cedric, jpeg

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

examples/evas: Fix weird use of word synchrony in comment

Summary: 'keep in sync' is more familiar programmer jargon.

Subscribers: cedric, jpeg

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

efl_ui_focus_manager: Fix misspellings in comments

Subscribers: cedric, jpeg

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

evas: Fix two whitespace errors

Summary:
Removes extraneous newline, and fixes the indentation of the while
statement's sub-clause (which looked like a code error at first
glance).

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

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

evas textblock: fix dereferenced memory after free issue for ellipsis

Summary:
_layout_ellipsis_item_new() function deallocates "ellip_ti" and
allocates memory for new one. The local "ellip_ti" pointer should be
updated.
@fix

Test Plan: N/A

Reviewers: raster, cedric, herdsman, jpeg, woohyun

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

evas map: Rename raw_coord to coord_absolute

This is more in line with the other "absolute" APIs.

elm_test: Add EO version of "Flip Page" test

This is a copy of the "Flip Page" map usage example that relies
on the new set of APIs for EO. This was used to test the API
and show its usage.

The calculation being done in absolute values, this does not
really exploit the new API, but instead proves that it is on
par feature-wise.

The performance is worse than with legacy, because of extra list
walkings, map calculations, small struct allocations and eo calls.
This fixes the shadow of the page which was broken with the legacy
API (as color_get did not recalc the map).

A better implementation can probably be done without having
to rely so much on absolute coordinates.

evas map: Reach feature parity with legacy API

I've done this by translating "Flip Page" to this new set of
EO APIs. In particular, absolute coordinates need to be used
in some calls, and the map needs to be calculated between
get and set operations.

This required an adjustment of the raw_coord API as the flip_page
code does some math after reading and then writing to the map.
Same for color. Those two properties now act like commands (ie.
like the other gfx map functions).

This also introduces a duplicate set of APIs to handle absolute
coordinates. Other solutions included:

 - Use an enum to specify the type of coordinates (but then the
   unit of cx,cy varies between non-unit relative position and
   pixel position. Also this adds an extra argument to all those
   function calls.

 - Pass a special value (an empty eo object) as the argument for
   pivot. Same remark about the unit as above. This way was
   deemed too confusing because of the weird object.

Those two options have been discarded after asking the opinion
of a few developers I could reach.

@feature

evas map: Introduce new API for maps (Efl.Gfx.Map)

This implements an entirely new API model for Evas Map by relying
on high-level transformations on the object rather than an external
Evas_Map structure that needs to be constantly updated manually.

The implementation relies on Evas_Map.

To rotate an object all you need to do now is
  efl_gfx_map_rotate(obj, 45.0, NULL, 0.5, 0.5);

Or with a C++ syntax:
  obj.rotate(45.0, NULL, 0.5, 0.5);

Or even simply (with default arguments):
  obj.rotate(45.0);

The map transformation functions are:
 - rotate
 - rotate_3d
 - rotate_quat
 - zoom
 - translate (new!)
 - perspective_3d
 - lightning_3d

@feature

evas map: Replace EO APIs for populate with reset

Manual points population will eventually be useless as the
map API will become more like a transformation API, where
the current object geometry doesn't matter as much as which
transformation is applied to it.

evas map: Move Efl.Gfx.Map mixin to its own C file

ecore_evas: Use proper enum values in axis events

Somehow the evas enum values were used instead of the ecore ones.
But I believe this is just a cosmetic change as their integral
values was the same.

evas examples: Fix invalid assumption about proxy

In the map examples, the map image UV size was based on the image
source geometry, rather than the image geometry itself.

In the example, this affects how the glass is mirrored. Before this
patch, the reflection is a single line stretched. EFL 1.18 and 1.19
seem to have the same issue, while 1.17 simply fails to show any
reflection. 1.16 fails miserably and the entire window is black.

If the original code was correct, then I believe that map and/or
proxy rendering have been modified in a way that affects the meaning
of those image UV parameters. But this seems like the regression (if
it is one) is in fact quite old.

@fix

edje: Fix shadow variable warning

mapbuf: Make point_color a property

container: Remove method part_name_get

Remove this for now, as it may not apply to all contents.
Note that this was a new API in EO.

efreet: free the allocated buffer before returning

This fixes the commit 169a08c03a6a7270e185bda85d4ab9afd9063c8e (efreetd:
BSD optimizations). Coverity rightly pointed out six different leaks of
various buffers on error paths.

CID: 1374949 1374950 1374951 1374952 1374953 1374954

gl_drm: Fix rendering problems with dmabuf

Somehow this long standing bug wasn't obvious until wayland 1.13.0 made
some additions to an opaque structure.

This changed the frequency that new buffers came to us with the exact
same pointer value of a buffer that had just been freed.

This shortcut in eng_image_native_set has always been wrong - we need to
proceed to the end to make sure we pick up new dmabuf attributes.

efreetd: BSD optimizations.

Summary: Related to ticket T5475.

Reviewers: raster, cedric

Subscribers: jpeg, cedric, raster

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

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

examples/evas: Improve linespacing consistency

Summary:
Also fixes a handful of obvious indentation irregularities,
including some reformatting of some printf() multi-line indents that
commit a71b770b did not properly adjust.

No functional code changes.

Reviewers: cedric

Subscribers: jpeg

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

ecore_evas: Improve linespacing consistency

Summary:
Adds a line between each function so it's more obvious what doc goes
with what API routine.  Reorganize the doxygen elements so they're
consistently ordered and spaced.

No code or documentation changes; mostly just whitespace.

Reviewers: cedric

Subscribers: jpeg

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

evas: adjust pointer mode set to work in headless scenario.

With the wayland backend, it is posible to have no seat connected
until later. This would lead to calling evas_object_pointer_mode_set
and fail without returning a boolean it was impossible to detect it
did fail. This patch correct it.

evas: make sure that even when nothing is rendered there is a pair of RENDER_PRE and RENDER_POST events generated.

ecore_evas: fix link issue on macOS

696ed3e2e8f97de482e98db34997ad1848da985d introduced a build failure on
macOS. _ecore_evas_subregister is being used in a foreign code module,
but it was not exported. Enforcing EAPI gives this symbol enough
visibility to be used outside of ecore_evas.

ecore_input: suppress warning (unused parameter)

eina: fix for escapable charachters not getting escaped if it comes after '\t' or '\n'

Summary:
Escaping is not happening whenever any escapable characters is coming after
'\t' or '\n'. It will also fix invalid read of 1 byte which happens for string where
last charachter is '\t' or '\n' like "eina\t".

Test Plan:
Take a string like "eina\t ". Observe space which is followed by tab is not getting
escaped.

Signed-off-by: Prasoon Singh <prasoon.16@samsung.com>
Reviewers: shilpasingh, rajeshps, govi, cedric

Reviewed By: shilpasingh

Subscribers: cedric, jpeg

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

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

ecore_x: add function to show and hide X cursor.

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

ecore_x: handle DPMS state for a more accurate idling evaluation.

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

ecore-evas-wayland: Remove unused variable

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

ecore-drm2: Add start of refcounting for FB objects

Small patch which starts to implement refcounting on framebuffer
objects. This will be needed so that we do not free FB objects while
they are on the screen.

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

ecore_drm2: Move test for atomic_req into atomic flip path

We ended up doing an occasional atomic ioctl when we shouldn't.
This should be a harmless failure, but it's also a pointless one.

ecore_drm2: Fix dmabuf fb destroy

I missed the fact that it'd run through the existing non-gbm path on
destroy, resulting in a (probably harmless) bad ioctl

examples/evas: streamline the intro description statement for examples

Summary:
For people browing through the examples, having the opening statement be
concise and consistent will help them more quickly find what they're
looking for.

Signed-off-by: Bryce Harrington <bryce@osg.samsung.com>
Test Plan:
Some of the examples had identical opening statements (e.g. the image
object examples).  I've tried to give each a unique description defining
what they are demonstrating, but you may want to doublecheck I got these
correct.  Of particular note, to me evas-images5.c looks like just a
fixup to evas-images4.c, so I'm not sure what makes these two distinct.

Subscribers: cedric, jpeg

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

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

evas: fix typos in Evas Text docco

Subscribers: cedric, jpeg

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

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

ecore_evas: doc experimental comp_sync API

Summary: (And indicate it's likely to be made private in EFL 2)

Reviewers: stefan_schmidt

Subscribers: stefan_schmidt, cedric, jpeg

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

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

elm_theme: check whether object is valid before calling edje_object_mmap_set

Summary:
In Issue scenario when _elm_theme_set was called,object was already deleted by app causing
unnecessary addition of the style in style_not_found list.
Check for object validity, if object is NULL return theme apply failed.

Test Plan: Make object pointer NULL before _elm_theme_set is called.

Reviewers: cedric, shilpasingh

Subscribers: rajeshps, govi, cedric, jpeg

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

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

examples: use printf instead of fprintf(stdout, ...)

Summary:
Applies same change as e8355c93 for evas, to the remaining examples.
This uses the shell command-line:

    src/examples/evas$ grep -sr 'fprintf(stdout' . | cut -d: -f1 \
        | uniq | xargs sed -i "s/fprintf(stdout/printf(/"

Note that use of the "fprintf(stdout" construct can generate warnings
when -Wformat-security is enabled, if the fprintf statement has no
format arguments, so in addition to the stylistic simplification this
also helps quell those spurious warnings.

Subscribers: cedric, jpeg

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

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

Ecore_Evas (Win32): add geometry_get() method and fix dpi_get(), using the new API in Ecore_Win32.

This fixes fullscreen feature in Elm on Windows as the geometry of the desktop
was not known.

In case of multiple displays, the desktop, where the window is displayed, is used for fullscreen.

@fix

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

Ecore_Win32: add API to retrieve the geometry and dpi of plugged displays

@feature

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

ecore_evas: refactor win32 backend to use generic render infrastructure.

ecore_evas: refactor wayland backend to use generic render infrastructure.

ecore_evas: refactor sdl backend to use generic render infrastructure.

ecore_evas: refactor psl1ght backend to use generic render infrastructure.

ecore_evas: refactor fb backend to reuse generic render infrastructure.

ecore_evas: refactor extn backend to use generic rendering infrastructure.

ecore_evas: refactor drm backend to use generic render infrastructure.

cocoa: refactor cocoa backend to use generic render infrastructure.

ecore_evas: refactor ews backend to use generic infrastructure for rendering.

ecore_evas: first step into refactoring ews backend.

elm_cnp: Fix typo

Summary:
Removes extraneous 'or' in debug message due to change from commit
27375735.

Reviewers: bu5hm4n

Reviewed By: bu5hm4n

Subscribers: bu5hm4n, cedric, jpeg

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

edje: null checking the return value of a function eina_mempool_malloc

Summary:
The return value of the function eina_mempool_malloc was dereferenced without checking. I added the checking code similar to the other codes.
@fix

Reviewers: raster, cedric, jpeg, herdsman, woohyun, stefan_schmidt

Subscribers: stefan_schmidt

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

evas: Fix can't open tiff file on loader

Summary:
Evas can't open tiff file because of no implement in client read api.
I wrote codes simply for open.

Test Plan: self

Reviewers: jpeg, cedric, jypark

Subscribers: stefan_schmidt

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

docs: Fix broken edje tutorial (swallow 2)

elm_code: Move the 'default' font name and size into the theme.

This can still be overridden by the standard method calls

eolian: help and version shall not cause an error exit status

eolian_gen called with --help or --version is a valid action. It shall
terminates with the 0 exit status.

ecore_evas: first step into refactoring x backend.

ecore_evas: first step into refactoring wayland backend.

ecore_evas: first step into refactoring win32 backend.

ecore_evas: first step into refactoring sdl backend.

ecore_evas: first step into refactoring psl1ght backend.

ecore_evas: first step into refactoring fb backend.

ecore_evas: first step into refactoring extn backend.

ecore_evas: first step into refactoring drm backend.

ecore_evas: first step into refactoring cocoa backend.

ecore_evas: use generic function for rendering ecore_evas buffer backend.

ecore_evas: add infrastructure to properly track sub ecore_evas.

ecore_evas: add logic to allow partial refactoring of backends.

ecore_evas: introduce a generic rendering function.

ecore_evas: adjust buffer engine with nicer behavior from evas.

evas: always pair RENDER_PRE and RENDER_POST.

evas: don't double check if the canvas has changed.

elm_cnp: fixup behaviour

On touch devices there is the normal gesture to touch on the screen and
hold until the drag operation started.

For users of a mouse there is the gesture of just click and drag the
mouse away.

This commit changes the behaviour of the start based on the device that
sent the event

ecore_drm2: Add checks for using a plane after death

We keep planes on the plane list to ensure a released plane is removed
from display - however this means that if a caller starts messing with
a plane after release, that it could potentially reposition a plane it
doesn't own anymore.

Use EINA_SAFETY macros to prevent this.

ecore_drm2: Replace plane state release flag with plane dead flag

The release flag is actually less useful than the existing in_use flag
for determining if a plane is unused.  If a new plane is assigned before
the next flip cleans up released planes, then it can point to a released
plane state, and both it and the previous user will be freed on the next
commit, leaking a plane.

Putting the flag in the plane structure fixes this while still allowing us
to keep released planes around to ensure a recently released plane is
cleared from atomic state.

ecore_drm2: Call flip test in plane release

If we don't do a flip test, the atomic state isn't updated.  This fixes
a potential problem where the last operation in state preparation is
a release - the following commit wouldn't include state from the release.

ecore_drm2: Fix flip test error handling

We need to free the atomic req if commit fails, so let's merge these
failure paths and simplify code a bit.

ecore_drm2: Require x and y value in plane assign

Saves us a flip test, and gaurantees that we're always testing with a
reasonable x, y and not something leftover from a previous assignment.

ecore-drm2: Release any marked planes during atomic commit

This patch fixes plane_state values during atomic flip test for any
planes marked for release. When the fb_flip actually completes, we
will remove the marked plane(s) from the output.

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

ecore-drm2: Add 'release' flag for planes

As we cannot immediately remove a plane from an output, due to needing
an atomic commit to actually remove the plane from screen, we can use
a 'release' flag to indicate that a given plane needs removal from the
screen during our next atomic commit.

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

ecore-drm2: Don't remove plane from output list on release

As we need to be able to commit a new plane state for any released
planes, we should not be removing them from the output list just yet.

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

ecore-drm2: Fix formatting

NB: No functional changes

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