ecore-wl2: Prevent duplicated mouse up events

If you touch downs and ups about all of fingers,
that is sufficient condition to call ecore_wl_input_ungrab().
And then one mouse up event is generated from that function.
So if you set a callback about ECORE_EVENT_MOUSE_BUTTON_UP,
you can watch two mouse up events are occurred.

This is a port of the patch for Ecore_Wayland library (deprecated) to
the newer Ecore_Wl2 library.

ref D4112

@fix

Signed-off-by: Chris Michael <cpmichael@osg.samsung.com>

ecore-wayland: Prevent duplicated mouse up events.

Summary:
Prevent duplicated mouse up events generated from input_ungrab.
         If you touch downs and ups about all of fingers,
         that is sufficient condition to call ecore_wl_input_ungrab().
         And then one mouse up event is generated from that function.
         So if you set a callback about ECORE_EVENT_MOUSE_BUTTON_UP,
         you can watch two mouse up events are occurred.

Test Plan: Touch and release. And watch mouse up events.

Reviewers: raster, zmike, devilhorns

Reviewed By: devilhorns

Subscribers: ohduna, input.hacker, cedric, jpeg

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

eo: Fix composite objects destruction

This walks the list and removes objects from it. Not
using EINA_LIST_FREE as the actual list_remove is done
inside eo_composite_detach.

evas: Use proper type for stringshares

evas: Fix double stringshare_del

A stringshare for font "style" was not properly duplicated,
resulting in lots of invalid stringshare pointers.

Ping @id213sin (commit 2ce33e73a7d48d4b54a075c07f12c0162).

tests: elm: make cnt unsigned for correct compare

Not really somethign we would hit here but still better to change this.

tests/elementary/elm_test_config.c: In function ‘elm_config_profiles’:
tests/elementary/elm_test_config.c:206:12: warning: comparison between signed and unsigned integer
expressions [-Wsign-compare]
         fail_if(cnt != eina_list_count(list));

tests: elm: remove wrong const for Eina_List

We are assigning somethign new to this list so no chance for const here.

tests/elementary/elm_test_config.c: In function ‘_eina_list_string_has’:
tests/elementary/elm_test_config.c:174:11: warning: assignment discards ‘const’ qualifier from
pointer target type [-Wdiscarded-qualifiers]
    EINA_LIST_FOREACH(list, l, s)

popup: remove max_sc_w because it's duplicate job

Summary:
max_sc_w decide popup's "content" area size when popup scrollable is TRUE.
but in popup's edc, "elm/popup/content/XXXX" has min size when scrollable is TRUE. (and it is content area layout)
max_sc_w has fixed value, so I think deciding by edc is better than fixed code.

Test Plan:
1. elementary_test -to popup
2. check "Enable popup scroll"
3. click any list item.
4. observe popup and compare before this patch.

Reviewers: singh.amitesh, herb, Hermet, raster

Subscribers: cedric, jpeg

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

evas: add pattern "style" when evas query fonts via fontconfig

Summary:
Some fonts can have weird style and weight value.
If a font has a style name as "medium" and a weight value as "semi-bold",
Evas can't load the font using "font=Somefont:style=Medium".
It only can be load with "font=Somefont:style=SemiBold" or
"font=Somefont:weight=SemiBold".
On the other hand, it could be loaded when I tested the following commands.
fc-match -s ":family=Somefont:style=Medium" or
fc-match -s ":family=Somefont:weight=SemiBold"

Evas also should load font based on font's style name.
@fix

Test Plan: N/A

Reviewers: tasn, herdsman, cedric, woohyun, raster

Reviewed By: raster

Subscribers: Blackmole, z-wony, jpeg

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

elementary/layout: changed to Evas_Object * from Elm_Layout * in legacy header.

if EFL_EO_API_SUPPORT is not defined in user side, this will break the build.
because Elm_Layout type is defined in eo header file.

Just keep using Evas_Object * in legacy yet.

efl: Remove eo struct Efl.Gfx.Color

So far this was protected behind ifdef EO_API_SUPPORT. It also
was not used internally. Dropping this before the release, since
we will soon have a (hopefully) better solution to handle various
color representations.

edje: refactor edje color class API

Summary:
refactor setter and getter of edje color class and
move edje_color_class_set/get() into Edje.Object as class functions

Reviewers: cedric, jpeg

Reviewed By: jpeg

Subscribers: kimcinoo

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

Note by jpeg:
The EO APIs introduced here will be revisited in upcoming patches,
in particular:
1. avoid non-premultiplied colors
2. move to a common color class interface

elm_atspi_bridge: add bounds-changed signal sending code.

Summary:
Add bounds-changed signal, this signal can be sent on geometry change
of any evas object when atspi mode is enabled.

Test Plan:
connect to "object:bounds-changed" signal on screen-reader.
when signal BoundsChanged is emitted from elm using elm_interface_atspi_accessible_bounds_changed_signal_emit, the signal should be received
and we should be able to access geometry.

Signed-off-by: Shilpa Singh <shilpa.singh@samsung.com>
Reviewers: cedric, lukasz.stanislawski, kimcinoo

Subscribers: govi, rajeshps, cedric, jpeg

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

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

elementary/layout: attach edje object API with eo compositing

Test Plan: make check

Reviewers: Hermet, jpeg, cedric

Subscribers: slotus.lee

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

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

label: set sliding only if label size is valid

Summary:
When call slide_go before label is resized, setting the sliding is a meaningless.
This patch is for reducing redundant operations.

Signed-off-by: Minkyu Kang <mk7.kang@samsung.com>
Signed-off-by: Deepjyoti Dutta <deepjyoti.d@samsung.com>
Test Plan: elementary_test -to "label slide"

Reviewers: cedric, Hermet

Subscribers: seoz, jpeg

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

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

ecore_wl2: Fix dnd breakage when focus leaves window

Summary:
If dnd in the same window is activated, the focus goes away from the
ecore_wl2 window to the dnd window, after that focus.pointer is NULL.
After focus.pointer is NULL ev->win of all the events will be 0 which
breaks dnd-motions, drop and end.

With prev_pointer beeing the last focused window, we can simply set this
window as event window. After that dnd with jesus works perfectly.

@fix

Reviewers: devilhorns

Subscribers: cedric, jpeg

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

Ui text: make elm test editable

Canvas text: fix format annotations deletion

Ui text: defer decorations to one job

Also, remove callbacks on the deletion.

Ui text: port item support from Entry

This uses the annotations API to retrieve items and their geometries, instead of
the old format API.

Ui text: add cursor_new method

The cursor object is meant to be instantiated in the following manner:
   eo_add(EFL_CANVAS_TEXT_CURSOR_CLASS, text_obj,
      efl_canvas_text_cursor_text_object_set(eo_self, text_obj));

This can't be done directly on the Ui Text object, so it has to be with an
method call, sadly.

@feature

evas: Fix bad clipping with proxy

evas-images2 example was broken due to excessive clipping.
This is one more issue with the cached clip geometry.

By default proxy_src_clip is true. The doc (that I wrote)
mentions that it means both objects (image proxy & source)
share the same clipper. So, this does not mean that the proxy
is clipped to the geometry of the source (this can be useful
in some cases, when replacing an object with a proxy to apply
some kind of effect on it... but this could be rare).

Thanks Amitesh for the report.

@fix

ecore: Replace TABs with spaces

elm_main: Remove useless shadowing variable

elm test: Remove unused variable

Evas events: Implement support for hold event

Fix 'maybe used uninitialized' warning

Summary:
    bin/edje/edje_cc_out.c: In function ‘data_write_images’:
    bin/edje/edje_cc_out.c:1164:7: warning: ‘f’ may be used uninitialized in this function [-Wmaybe-uninitialized]
        if (!f) return EINA_FALSE;
           ^
    bin/edje/edje_cc_out.c:1154:15: note: ‘f’ was declared here
        Eina_File *f;
                   ^

Reviewers: cedric, jpeg

Reviewed By: jpeg

Subscribers: seoz, jpeg

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

evas: Make shaders regen an opt-in behaviour

While the shaders automatic regeneration is very useful when
working on shaders, it has been the source of too many
build breaks.

So I'm making this regeneration an opt-in.

You work on shaders?
Set the env var EFL_SHD_REGEN=1

Revert "evas: Fix compilation when using different build directory"

This reverts commit 3c13ef14e3511611cabf184ff0b83c264b3e620d.

This introduced new build breaks, especially with in-tree builds.
In particular, DIR was not defined. See T3975.

I'll make the shaders regen an opt-in rather than on by default.

elm_entry: restrict focus region to inside of entry object area

Summary:
Focus region must be located in entry object.
Therefore if it get out of entry,
 it just returns last cursor position that can be shown.

@fix

Test Plan:
1. elementary_test "Entry on Page Scroll"
2. click 2nd btn and close popup
3. page should not be scrolled

Reviewers: raster, herdsman, id213sin, woohyun, tasn, cedric

Subscribers: jpeg

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

evas object: Hide "del" event

In EO world, we should stick to EO_EVENT_DEL.

elementary: add test for external icons

The ExtIcon elementary test shows how edje external icons
can be used.

edje_external: fix external icon handling

External icons were handled with a global variable, which reason
to exist seem quite weird. It seems to me it was used to retain
some states of the actual parameters to get them later without having
access to the real parameters.

I don't really get how this would have worked with more than one
icon. When I tried to animate an external icon, with two states,
there were leaks and memory corruption... Now both are gone.

@fix

eina: fix typos in eina_rectangle doc

elementary: include Ecore_Cocoa.h normally

Guards are not necessary anymore.

ecore_cocoa: release Ecore_Cocoa as stable

ecore_cocoa: mark clipboard API as BETA

Also, Since the API is beta, don't make is available to Doxygen.

elm: Always connect to wl when enforced

This is a followup on ca55a920d9a5f6a4274e40fcf901a89e622c3007.

This fixes cnp/dnd for applications started outside of the weston
session. (There are still more problems with dnd and cnp)

edje/example: add a missing closing bracket in svg.

evas: Fix compilation when using different build directory

elementary: return an exit code at exit with the new EFL_MAIN macro.

ecore: add the possibility to return an exit code when leaving the main loop.

elementary: introduce EFL_MAIN and rely on Eo_Event to call the initialisation function.

FIXME: Find a way to return an exit value. Maybe on efl_loop_quit ?

Revert "elm_entry: fix duplicate text set"

This reverts commit 0959e87d999f424341acbc62a5934ad56134f99b.

This broke the entry behavior that introduced enventor critical issue also.
It looks like a timing issue, but i didn't deep into further.

Please check a simple test case.
https://phab.enlightenment.org/F38614

Before/after applying your patch, the result is totally different.

evas: do not unref if there is no clipper

check if there is a clipper set before accessing it. This fixes crashes
of the drm engine.

ecore_wl2_input: Fix key keyname pairs

before key and keyname could have been filled invalid, for example the
character "ö" from the german layout produced "Odiaeresis" instead of ö.

This commit also documents and refactors _ecore_wl2_input_key_translate.
I added the documentation for later use, it was hard to find out why
this was done.
The function also drops the input now if invalid control codes would
have been generated, this is equivalent to the behaviour in x. So
applications cannot get confused by codes which dont exist.
It also doesnt check if there is actual output by checking the highest
bytes anymore, it just tries to create a utf8 character. This fixes
arabic greek and korean keyboard input on wl.

If this breaks the input for some user, feel free to contact me.

This fixes T3073.

evas canvas: fix a proxy+map rendering issue.

This fix is wrt the non-updation of the proxy object.
If a source of a proxy object has mapped objects among its children,
Rendering frame won't be correct while they were keeping changing.

The reason is, evas_object_smart_changed_get() couldn't figure those changes
out correctly because of cached-clip visibility.

Cached-clip of smart member can't be used for map surface updation,
because cached-clip includes the visiblity status out of the smart object.

So, loose the criteria to make it rendering smart members in map.

config: Prevent accidental deletion of the config object

config: Test profile APIs

win: Attach config API with eo compositing

config: Add support for profiles and save

Most of these functions actually shouldn't be used by
standard applications. Some are hidden behind @protected.

flush_all() and reload() have not been implemented, on
purpose (save() should flush, and reload is automatic).

config: Implement support for audio_mute

Instead of being a key "audio_mute" + another key "channel"
I've simply implemented a number of different keys:
audio_mute (= all)
audio_mute_all
audio_mute_effect
...

config: Simplify Efl.Config API, make C helpers only

This removes the mixin functions and uses static inline functions
instead. Much cleaner, looks the same to C devs.

config: Add support for some enums

elm: EO-ify some elm_config enums

Elm_Focus_Autoscroll_Mode
Elm_Softcursor_Mode
Elm_Slider_Indicator_Visible_Mode (rename only)
Elm_Focus_Move_Policy (rename only)

This is for Efl.Config API.

efl: Introduce general Efl.Config API

This is to port elm_config to EO APIs.

The current implementation relies on the legacy API, by
simply forwarding calls.

The new API is simply efl_config_set("config_name", value)
where value is an Eina_Value (aka. generic_value).
The C interface proposes a few helpers like config_int_set,
config_double_set, etc...

Unfortunately at the moment, not all config options are
supported, as some rely on more complex types:
- lists
- color class and multiple arguments
- unset functions
- enums

Profiles are also not handled at this point.

@feature

evas: (Try to) fix shaders gen without breaking everything

So... adding the .x files to BUILT_SOURCES means they would
get deleted during clean. This was bad, but turned to be awful
when this made the check for writability fail.

See:
8a273158788d630e56b4dc14cdd44f0eaae6b7b5
2b8912462e0ec81efb8e30a63383b811f52e16f5

This commit reverts:
0b69356f1bbceab3261835bc2d4632f8f4669206

Notes:
Please revert if this breaks the build for anyone.
I'm pretty sure the use of top_srcdir is not recommended the
way I've done it.

I hate autofoo.

svg/example: upload the correct svg file

Reviewers: cedric, jpeg

Reviewed By: jpeg

Subscribers: cedric, jpeg

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

edje: fix the order of applying transformation when creating vg tree.

Reviewers: cedric, jpeg

Reviewed By: jpeg

Subscribers: cedric, jpeg

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

evas vg: don't apply transformation to the origin point

Summary: Which will apply translate property twice once to shape and once to origin.

Reviewers: cedric, jpeg

Reviewed By: jpeg

Subscribers: cedric, jpeg

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

evas/canvas: rewrite a previous patch, ed444b698b63e0086812062abb9e3065360eb0b2

Reviwed further, I found out the origin shouldn't be accumulated with
previous clippers origin. I forgot this function is recursive
function.

Revert "evas: Fix distcheck by disabling shaders generation"

this breaks the simplest configure + make ... so obviously it's broken

po: catchup with file rename to efl_ui_video in POTFILES.in

In commit 549e274acb69131b5fb7dccc1659372cef7a6572 this file was renamed but
POTFILES.in not updated.

wayland_shm: Make dmabuf opt-in

Some weird bugs and performance issues around these days, and I
keep chasing them in dmabuf first.  Removing it from the equation for
now.

ecore: fix Windows build by properly defining EAPI in all case for internal API user.

elementary: fix test of internal widget to redefine unaccessible variable.

edje: handle named color in svg.

Reviewers: jpeg, cedric

Reviewed By: jpeg, cedric

Subscribers: conr2d, cedric, jpeg

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

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

canvas/vg: fix the reference issue in evas_vg objects.

Reviewers: jpeg, cedric

Reviewed By: cedric

Subscribers: cedric, jpeg

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

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

edje/svg: add few error message reagrding parsing and storing of svg file.

Reviewers: cedric, jpeg

Reviewed By: jpeg

Subscribers: cedric, jpeg

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

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

edje: replace 4000x4000 min size limitarion with loop count restriction.

Summary:
4000x4000 min size limitation was added to prevent infinite
calculation loop. But, it can ruin calculation of some proper Edjes.
Normally, properly generated Edje runs the calculation loop below 10 times.
So, "255" could be proper limitation for calculation loop.
It also make better performance for broken Edje.

Test Plan: N/A

Reviewers: raster, Hermet, woohyun, cedric

Reviewed By: cedric

Subscribers: jpeg, z-wony, Blackmole

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

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

elm_index: include horizontal index to omit calculation

Summary:
- when index is set to horizontal by elm_index_horizontal_set()
  there is no logic about omit calculation. However, when
  omit is enabled/disabled by elm_index_omit_enabled_set(),
  horizontal index is ignored by
  if (sd->orientation == EFL_ORIENT_HORIZONTAL) return;
- This can cause inconsistency when order of calling those APIs
  is changed. And also, that "horizontal index cannot be omitted" is not documented
  and does not make sense.
- This commit enables omit for horizontal index, but further
  change is needed because current logic of getting min_w for
  dummy edje object is not accurate when text is set.

Reviewers: cedric, jpeg, Hermet, conr2d

Reviewed By: conr2d

Subscribers: eunue, conr2d, cedric, jpeg

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

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

genlist: scroll to item when calc is done

Summary:
If elm_genlist_item_bring_in() is called immediately after appending
item, genlist will not scroll to the last one but stop in the middle.
This patch blocks genlist from scrolling to not calculated item.

Test Plan:
1. Download F31560 and uncompress the file
2. make
3. ./test

Reviewers: SanghyeonLee

Subscribers: cedric, DaveMDS, jpeg

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

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

elementary/scroller: fix to skip focus for unfocusable scroller

Summary:
There is a issue about passing the focus to unfocusable scroller.
To prevent the issue, check the status of focusable object and child object
before passing the focus.

Test Plan:
{F38439}

I changed the test file.
The problem occured as below.

1. click button
2. show popup
3. click cancel button in popup
4. lost the focus

Reviewers: Hermet, raster, cedric

Reviewed By: cedric

Subscribers: minkyu, sju27, cedric, jpeg

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

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

evas: avoid calculation bounding for camera and light objects

Summary:
Including camera and light to calculation aabb can lead to confusing in detemination
bounding of the whole scene (root node).
@fix

Reviewers: cedric, Hermet, raster

Subscribers: jpeg

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

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

efl_ui_video: convert elm_video to efl_ui_video

Test Plan: elementary_test -to video

Reviewers: singh.amitesh, raster, cedric, jpeg

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

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

evas canvas: fix broken proxy clipping.

Origin of clipper's clipper won't be transformed to derivative space.
So it needs to transform the coordinate additionally.

evas canvas: code refactoring.

Compare integers clearly.

examples: emotion: make sure all callbacks are actually setup

In commit 3340745c65ef8b8361dc6eff5e746f7ce1104378 the callback array got
removed letting all callback functions around as unused. Somehow the setup
of all these callbacks have gone missing over various refactors.

evas: Fix distcheck by disabling shaders generation

make distcheck is verified by checking the env var "top_distdir"
This is not a great solution but should work.

svg/example: fixed typo in  svg file name

Reviewers: cedric, jpeg

Reviewed By: jpeg

Subscribers: cedric, jpeg

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

svg/loader : Added parsing support for use and def node.

Reviewers: cedric, jpeg

Reviewed By: jpeg

Subscribers: cedric, jpeg

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

win: Add property accel_preference

It supports the same syntax as the config value, but
overrides it. This provides a more convenient way to select
the acceleration for a single window.

Its get() value should be what was effective when the window was
created. So it can depend on the config or a previous call
to set().

@feature

examples: elementary: remove unused variable

If we are not using something we should not keep it in the code base.
Git history has it all.

examples: rename eio_job to efl_io_manager, finish a job half done

This rename was missing in the examples. Running make examples or even a
simple grep shows this. I expect higher b0rking skills by now. :)

edje_text: fix the fit calculation for different text size

Fixed T3569

@fix

elm code: check return value from elm_layout_theme_set

CID: 1356991

svg_loader: Fixed the parsing of style attribute in g node as well as inheriting style attribute from parent.

Reviewers: jpeg, cedric

Reviewed By: cedric

Subscribers: cedric, jpeg

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

elm_entry: fix duplicate text set

Summary:
When user set text to entry, "changed" smart callback is invoked 2 times.
Because entry set to ""(empty string) and append text internally.
But, in a recent, set flag is added for _entry_text_append().
So, empty string set is does not needed before appending.

@fix

Test Plan:
- elm_entry_entry_set(entry, "text") : 2 times invoked (bug)
- elm_object_part_text_set(entry, NULL, "text") : 2 times invoked (bug)
- elm_object_part_text_set(entry, "elm.text", "text") : 1 times invoked

Reviewers: herdsman, tasn, id213sin, woohyun

Reviewed By: woohyun

Subscribers: cedric, jpeg

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

ecore: fix arguments to use Eo native array type.

eio: register one Efl_Io_Manager by default on the main loop at init time.

eio: rename eio_job to efl_io_manager.

There is still a lot of work to do in this API to make it efficient and
stable. Will try to get at it before 1.18 release.

ecore: add test for registered provider.

ecore: add ability for the main loop to provide other class of object.

docs: update gendoc.lua to latest eolian API

elm win: only set wl cursors on mouse in to the window area

in the case of csd regions, canvas mouse in != window mouse in

ref T3819

elm win: set input regions for wl windows again

this seems to have been lost at some point and was breaking input

ref T3819

Ui text: Add an editable variant (tiny wrapper).

As part of this commit, also add a way to change whether the widget is
editable or not.

Canvas text: Add missing cursor changed callbacks.

In some cases we were changing the cursors but not emitting callbacks.
This should fix that.

Canvas text: Don't call callbacks while cursors are in inconsistent states.

The issue is that in some cases we were calling user code (callbacks)
when some of the nodes were referencing already deleted text nodes. This
caused invalid memory access. This commit delays the callback calling
until after all of the cursors got into a consistent legal state.

elc_combobox.c: apply style to inner parts of combobox like entry and genlist

this will allow to make more custom styles and themes for combobox

@fix