elm: Fix make check (change in atspi role of MBE)

See da5ba14664b71f19afc3359f36ff11442012d19a
Ref D4925

ecore_evas: Document the drm and gl drm window constructors

Reviewers: cedric

Subscribers: jpeg

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

evas: Disable calls to evas_device_class_set

After finalize, the type of an input device can not be changed.
So, deprecate the function and prevent anything from happening
unless we're inside efl_add().

Ref T5515

widget: Make cursor_add/del internal

This API is meant to be used internally to elm, not from outside.
The public API only exposes part_cursor_set on layout, and a series
of cursor EAPIs.

ector: document ector_glsym_set

Reviewers: cedric

Reviewed By: cedric

Subscribers: cedric, jpeg

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

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

ecore_evas: document client leader routines

Subscribers: cedric, jpeg

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

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

edje: add efl_version property

Summary:
efl_version will be used to figure out which version of EFL lib
is used to develop a edje file.
If there is no property of efl_version, it will be 1.19 as default.
This feature is needed to fix T5548.

Example:
  efl_version: 1 20

@feature

Test Plan: N/A

Reviewers: cedric, jpeg, herdsman, zmike, raster

Reviewed By: cedric

Maniphest Tasks: T5548

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

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

evas: forgotten unecessary initialization.

evas: use Eina_List instead of Eina_CList for event grabber.

Eina_Clist can actually change the pointer in the cell next bypassing
the CoW infrastructure leading to trouble. Considering the case here,
using the optimization of Eina_Clist is not necessary and if performance
issue arise, can be fixed by using a dichotomic search when removing
data. I don't think it is necessary to add this complexity without
a real life case.

evas: allow list for _evas_event_object_list_in_get utility function.

elm_code: Fix make check

The tests were using some code that became somewhat invalid, adjusting tests

evas: actually free the cow allocated memory.

Revert "Revert "evas: put events related pointer into a cow to reduce evas_object fat.""

This reverts commit f9d3219c1739d7abea16fecd92fcc0981ddf801c.

evas xpm loader - remove tabs and clean up formatting

just non-code changes here...

evas xpm loader - remove the rgb txt file db loading and compile in

so modern systems seem to have abandoned rgb.txt files. this leads to
us breaking the loading of xpm files tha use color names ... i added
the rgbt.txt from vim but that didn't seem to help... odd... so to just
stop adding path after path to load... ship our own. we could ship the
file... but then we'd still have to load and parse it... every time we
look up a color. so i munged the data with awk and now we compile it
in. it should consume the same space the rgb.txt does in the shared lib
binary. if not read it shouldn't be paged in. it should end up cheaper
than our floaing of the file and mmaping it when xpm module is
loaded/initted... so either way more efficient, uses a little less ram
(12306 bytes vs 17780 for the rgb.txt) ... and bonus - dont have
an extenral out-of-code data blob to find, manage install etc...

this means we should load xpms with colornames correctly again on
systems without an rgb.txt provided by x11 ... which seems to be the
common case now. :(

@fix

test suite - evas - fix xpm ref image - it was badly broken...

since rgb.txt has seemingly disappeared from systems we didnt parse
colornames (missing the colorname db entirely) and the test image was
generated using a broken missing rgb.txt thus colors were wrong. a
recent fix made evas find colornames again...

tests: Remove invalid call to elm_run

Nothing is supposed to happen here. Just exit after running
the checks. Don't wait for something to happen until eternity...

tests: Fix eina_thread_queue test case 5 & 6

ecore_test_ecore_thread_eina_thread_queue_t6 failed often for me.
eina_thread_queue_wait() was returning NULL.

I believe this is because the test case ended abruptly without
waiting for the threads to finish. Indeed, both threads tried
hard to reach 10000 messages but it didn't make sense for them
both to reach this value, only one would end there.

This patch adds an exit message sent by thread 1 to the two other
threads, and all threads are waited upon using a single semaphore.

Note: This also renames some functions to match their test case
number.

@raster

win: Use efl_data_scope_safe_get instead of macros

This removes macros that don't make the code easier to read
and aren't actually as safe as this safe data_get.

evas: Use efl_data_scope_safe_get in evas events

This removes a few NULL + efl_isa safety checks.

eo: Add API efl_data_scope_safe_get

This is a safe version of efl_data_scope_get, meaning that it will
return NULL if the object is not of the required type, or if there
is no data for that class, or if the given class was used as an
interface (and isn't a mixin).

@feature

scroller: Remove propagate_events to avoid name collision

elm_radio, elc_multibuttonentry, elm_colorselector: "checked" signal sent

Summary:
Radio, MBE items, color selector items have ELM_ATSPI_ROLE_RADIO_BUTTON role, when one item
is selected the other items are automatically unselected, so a notification has to be sent in atspi mode
to read the status accordingly

Test Plan:
In ATSPI mode when radio, mbe item, color selector item is selected
status has to notified.

Reviewers: kimcinoo, cedric

Subscribers: govi, rajeshps, jpeg

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

evas: Fix resize & move intercepts

Thanks @zmike for the report.
This in therory should restore behaviour from 1.18,
where the intercept function is called even if the geometry
isn't changed.

Test scenario, in E:
 1. remove bryces, shelves on a screen, or mark all as autohide
 2. open chromium or any CSD application as unmaximized
 3. maximize the window
 4. make it fullscreen (eg. with chromium: F11)

See 8a9f0bd603aaaf3abd3ca9a.

Ref T4749
Fixes T5573

@fix

widget: Implement mirrored from Efl.Ui.Base

Ref T5363

elm config: add more NULL check

Add NULL check when we cannot allocate memory.

ecore-wl2: Group window functions together in header file

This patch just moves window related functions together in the
Ecore_Wl2_Window_Group

NB: No functional changes

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

ecore-evas-wayland: Add support for setting auxiliary hints

This patch adds function pointers to the Wayland Interface that can be
used for supporting auxiliary hints on a given window

@feature

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

ecore-evas: Fix return value in ecore_evas_aux_hint_val_set function

Small patch to fix misleading return value when
ecore_evas_aux_hint_val_set fails. In the above code, we will return
EINA_TRUE already if the aux_hints_set works, so this return value at
the bottom of the function should be EINA_FALSE.

@fix

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

ecore-evas: Support auxiliary hints for ecore-evas-wayland

Small patch to support using Ecore_Evas_Interface_Wayland in order to
add support for setting auxiliary hints on a surface

@feature

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

ecore-wl2: Free auxiliary window hints when we destroy a window

@feature

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

ecore-wl2: Add API to support deleting auxiliary window hints

@feature

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

ecore-wl2: Add API function to change auxiliary window hint

@feature

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

ecore-wl2: Add API to support adding auxiliary window hints

@feature

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

ecore-wl2: Add API function to get list of supported auxiliary hints

@feature

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

ecore-wl2: Add event and structure for auxiliary hints

@feature

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

ecore-evas-wayland: Add aux_hint functions to Wayland Interface

@feature

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

Canvas text cursor: add NULL checks for some functions

Passing a NULL cursor is permitted. Forgot to add to two functions
during port from EO.
Fixes some crashes.

Efl.Ui.Slider: implement Efl.Ui.Progress.progress_min_max

and move elm_slider_min_max_set/get as legacy APIs

ref T5361

intf: Efl.Ui.Progress: Add progress_min_max property

We need this for slider and progressbar widgets

ref T5361

Revert "evas: put events related pointer into a cow to reduce evas_object fat."

This reverts commit f69686ba4023c7ddd83ea38000e8fa0c61818365.

this causes major crashes in e every time you move and resize a
window. i cant even debug it properly because i cant move or resize
windows to get terminals up to debug it... this is bad... so until a
fix is found better to go back to working...

evas: put events related pointer into a cow to reduce evas_object fat.

elementary: fix recursive content group calculation in genlist

Summary:
There are several problem is left on recursize content calc.
previously genlist only calculate size of layout class,
but after recursive content group calculation patch,
layout couldn't get proper size because sizing eval is
not pre-processed.

Test Plan: elementary test working fine.

Reviewers: jpeg, cedric, raster, conr2d

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

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

evas: cleanup use of Eina_Cow during orientation set.

evas: fix a stride value after calling evas_object_image_orient_set API

Summary: Before the value of stride is calculated, the weight and height of the rotated image should be set.

Test Plan: {F1409510}

Reviewers: jpeg

Subscribers: cedric, jpeg

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

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

elementary: fix fallthrough warning.

edje: fix unitialized warning.

edje text: add a missing return macro

Summary:
RETURN_* macro does unref before exiting functions.
Must be called after using PROXY_DATA_GET macro.
@fix

Test Plan:
1. Run terminology
2. Open Setting
3. Check error messages when you change something from Setting.

Reviewers: herdsman, raster

Subscribers: cedric, jpeg

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

evas textblock: fix evas test suite build errors

Summary:
Fix build errors caused by recent patches for Text cursor.
@fix

Test Plan: run "make check"

Reviewers: herdsman, jpeg, raster

Subscribers: cedric

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

Text cursor: fix macOS build.

On macOS build failed with
lib/elementary/efl_ui_internal_text_interactive.c:801:50: error: too few
arguments to function call, expected 2, have 1

evas: Fix use of invalid EO object

During a window destruction, the input devices are deleted and
removed from the internal devices list. Since 83d18617b4375
basic devices are children of a seat, which means that the seat
deletion was triggering the devices deletion as well. As a
consequence, we were walking an invalid list of devices.

I'm not sure we should have a list of children in the device
data and I'm also not sure we should have a list of devices in
evas, instead of just the list of seats. It's a bit messy right
now.

Ref T5540

elm: rename elm slider to Efl.Ui.Slider

ref T5361

elm slider: indicator_visible_mode - move API document at one place.

elm slider: remove *not* required set/get{} from eo file

elm hoversel: Clean up sizing_eval function

elm hoversel: Delay sizing_eval to prevent too frequent recalculations

efl.canvas: Add "seat" (by id) and "device" (by name) properties

See d47197e65b59406e0f25c5a4db56e68da61bec30

@feature

evas: Rename device Class to Type, remove Sub_Class

1. The word "class" is a pain point with many languages where
   it's a keyword. Type is a little better. Also, the property
   was already named "device_type" and not "device_class".

2. Remove Efl.Input.Device.Sub_Class
   It's not used inside EFL upstream codebase, and unlikely to
   be used anywhere else (even in Tizen).

Hopefully no one used the Efl_ enum types. So far only the Evas_
types should be in used.

Ref T5540

evas: Remove device "parent", "name" and "description" properties

Those are now merged with Efl.Object parent, name and comment.
The reasoning is that only seats can be parent devices; And name
and description are not only name clashes but also not extremely
useful anyway.

Tested with VNC.

Fixes T5540

ecore_evas: Document ecore_evas Wayland operations

Subscribers: cedric, jpeg

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

evil: Fix edc scripts compilation on windows

Fixes T5410

Summary: should fix the error T5410

Reviewers: vtorri, t.naumenko, cedric, raster, an.kroitor, rimmed, NikaWhite, FurryMyad

Subscribers: artem.popov, cedric, jpeg

Maniphest Tasks: T5410

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

edje: Remove bool return from scale_set

It was always returning true. There is little point in returning
a bool here, an invalid scale value (eg. <= 0) wouuld lead to a
state where scale_get() != scale_set() and that's about it.

widget: Implement scale from Efl.Ui.Base

edje: Move base_scale to Efl.Ui.Base

This API is used by elementary widgets like:
  edje_object_base_scale_get(elm_layout_edje_get(ly));

This means elm_layout in fact should also expose it directly.

Ref T5315

elua - fix build for luajit2.1.0-beta3+

it seems luajit (and lua) broke api again... removed a #define from
their headers from 5.1->5.2 of lua (and seemingly luajit2.1 too). :(

this should fix T2728

@fix

elm test:image.zoomable: fix the compilation warnings

this fixes below compile warnings.

"
 warning: format ‘%s’ expects argument of type ‘char *’,
 but argument 3 has type ‘void *’ [-Wformat=]
 printf("Successfully set the edje file: %s, group: %s\n",
        file, eina_list_nth(grps, 0));
"

Efl text: shorten naming of methods

eo_prefix are set to "efl_text".
Also, "Efl.Text.Format" is shortened to now include the "_format"
prefix.
"Efl.Text.Font" keeps the "_font" prefix, for better readability.

Canvas text: disable legacy_newline by default

Gfx filters: set multiline to Ui.Text widget

By default, multiline is disabled, so needs to enable this.

Ui text: set some default font and size

This should later be setup through the theme.
One option will be to use global text classes.
Until the above is finalized, at least set this font and size to make
setup of the widget a bit easier.

gfx_filter: add efl_style_gfx_filter

Also, rename the format string to 'gfx_filterstr' to match the other
strings.

Update the filter test to use this property.

gfx_filter: null-out freed pointer

Text: remove leftover printfs

Elementary: bring back disabled tests

Canvas layout: port 'part_text' to Efl.Text.*

The following API is now supported with efl_part:

  - Efl.Text.text { set; get; }
  - Efl.Text.Cursor.cursor { get; }
  - Efl.Text.Cursor.cursor_paragraph_first;
  - Efl.Text.Cursor.cursor_paragraph_last;
  - Efl.Text.Cursor.cursor_position { set; get; }
  - Efl.Text.Cursor.cursor_coord_set;
  - Efl.Text.Cursor.cursor_line_char_first;
  - Efl.Text.Cursor.cursor_line_char_last;
  - Efl.Text.Cursor.cursor_char_next;
  - Efl.Text.Cursor.cursor_char_prev;
  - Efl.Text.Cursor.cursor_line_jump_by;
  - Efl.Text.Cursor.cursor_copy;
  - Efl.Text.Cursor.cursor_content { get; }
  - Efl.Text.Cursor.cursor_geometry { get; }
  - Efl.Text.Cursor.cursor_text_insert;

Many of the 'part_text' functionality was moved to legacy, too.
See the edje_object.eo to see which ones are still supported.

Text: enhance cursor_get to ask for a "get type"

Canvas layout: port part_text to efl_part

You now use the following:

  efl_text_set(efl_part(edje_obj, "part"), "text");
  const char *text = efl_text_get(efl_part(edje_obj, "part"));

The former method of edje_object_part_text_set/get is now legacy.

Also, adjusted 'tests/emotion/emotion_test_main-eo.c' with
this change.

Text: add annotate interface

Ui text: fix test style applying after cursor port

Ui text: fix test inserting object

Ui text: fix text initialization

It uses elm_layout's, where it shouldn't (it's legacy).
Also, it's not needed to to that there.

Text cursor: add interface and implement in Canvas.Text

Ui text: fix leak of selection on destruction

Ui text: initialize potentially unset variables

Ui text: free iterator once done

Ui text: fix leak in decorations

Canvas text cursor: move to Efl.Canvas.Text namespace

Originally it was its own object.
There are some valid claims that there is no justification for it to
remain an object.
Furthermore, it's apparent that it added little benefit: changes of
each cursors, in practice, triggered a query for all objects of the
same textblock. There wasn't real advantage to have a finer resolution
of controlling the cursors with their own events.

This ports back a lot of code, and changes a lot of other code in the
higher-up widgets, such as Efl.Ui.Text and co.

The usage was replaces from:
  efl_canvas_text_cursor_char_next(cur_obj)
to
  efl_canvas_text_cursor_char_next(text_obj, cur_obj)
that is, it is an operations on the TEXT OBJECT, rather than on the
(now removed) cursor object.

So, one less efl object to worry about now.
Hopefully, the port went smooth.

Eina Debug: include Evil for realpath support

Revert "efl_debugd: don't compile this on macOS"

This reverts commit e9f727044b38b1e1897dc28161d355f0d18a4ecb.

This is not needed anymore as efl_debugd is now portable.

Revert "efl_debugd: only compile this on Linux."

This reverts commit eafe0c74e13f1a379c219e03f754f8d47e237e27.

It is not needed anymore as the daemon is now portable.

Eina Debug: rewrite efl_debugd for portability

The previous version of the daemon was using functions specific to
Linux, such as epoll...

The daemon communication part has been rewritten to use Ecore
functionalities.

Sorry for the inconvenience guys

Eina Debug: fix a bug resulting in registering opcodes twice

The opcodes registration request is sent directly in case the connection
is already made. Otherwise, the request is waiting for the connection to
be made by the dedicated thread (not the main loop).
That's why the request can be sent by the two different threads at the
same time, leading to send it twice. It means a callback for an opcode
would be invoked twice everytime a request with this opcode is received.

This patch fixes it by checking if the request has already been sent.

elm_inwin: set the redirect manager on the correct object

otherwise the redirect is never set on the window which does not work

efl_ui_focus_manager: enhance documentation of the semantics of that
function

elm_naviframe: this is not handling focus

the widgets that are used to compose the naviframe are handling it, not
the naviframe itself.

efl_ui_focus_manager: use focus geometry for the shortest distance

elm_toolbar: do not focus a separator

elm_code: Fix possible crash when deleting a selection

If there was a multiline selection that ended at the end of a line this could crash
@fix

Revert "Revert "Edje calc: Fix textblock size calculation logic""

This reverts commit 0392cd3c489ee1c69dc768b2c3e8ac01ba1854c1.

oops this was supposed to just be local

theme: add signal to e border theme to set NOGRAB on titlebar part

for compatibility reasons this can only be changed in a signal callback
in the default theme.

all themes should now use NOGRAB for parts which can be used to trigger
window_move signal bindings

ref T5552