ecore_evas: Check for null ptr deref

This patch fixes a tentative crash owing to ptr deference.
Unlike ecore_evas_object_cursor_set and ecore_evas_object_cursor_device_set,
ecore_evas_cursor_set uses Ecore_Evas *ee before calling internal function which
internally checks null ptr dereference of Ecore_Evas *ee.

Change-Id: I555fc1e5c4c1c0ff3e6418996b7e144eb42f0b7f

ecore_anim: Check for null ptr deref

This patch fixes a tentative crash owing to null ptr dereference.

Change-Id: I2c7e618e319aff5323247d332b33738398875257

elementary: Add missed type for legacy.

Summary:
We've been missed type of class name.

Add legacy class name to maintain compatibility.

Reviewers: jpeg, woohyun, Jaehyun_Cho

Subscribers: cedric, jpeg

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

Change-Id: I27cec817c0032fa068904213ae45c4f3fb0d490f

Revert "[EXTN_GL] Add gl-backend in ecore_evas_extn"

This reverts commit f755812ca929c0cd71dfa4071f9e8c4f9bdd2ab6.

ctxpopup: fix build warning

@tizen_fix

Change-Id: I2c6f67b25cf07e2a01208683cea7154a54413bae
Signed-off-by: JinYong Park <j4939.park@samsung.com>

selection: remove elm_cnp.c

elm_cnp.c had been removed in upstream, but it is not removed in tizen.
It is a bug in migration.

Change-Id: I63cde8d1efa0b2b68c9366a615ab1c3321f2c12d

elementary: replace all HAVE_ELEMENTARY_WAYLAND to HAVE_ELEMENTARY_WL2

The WAYLAND macro was replaced with WL2 from upstream EFL.
The WAYLAND macro was used in Tizen only code. It should be replcaed with WL2, too.

@tizen_fix

edje, elementary: fix warning messages from Tizen only code

It fixes warning messages from the following widgets of elemenatry.
- elc_ctxpopup
- elc_popup
- elm_atspi_bridge
- elm_conform
- elm_widget

The authors who wrote the Tizen only code must revisit the widgets
to fix additional warning messages.
For next Tizen migration (6.0 or next?),
It must be considered as sepereated patches by seperated authors.

@tizen_fix

Revert "elementary: Add missed type for legacy."

This reverts commit 2e71f63b2e28ff6282454e71f971116a67156aa8.

Change-Id: I0cff4d48d409a95ac68d8f524ca51dee81529e8f

update Makefile.am of elementary-tizen for the consistency with elementary

Change-Id: Ib74b1d0ebb003a88b38ca93f24eedbeb49b24782

interface_scrollable: Remove redundant code block

wanted-updates logics is paired working with wanted-set logic.
but there are coordinates mirroring logics on the both side.
It is useless to reverse value which is reversed.

@tizen_fix

Change-Id: If3909559def5280677116b2a00036875b54cbbd7
Signed-off-by: Wonki Kim <wonki_.kim@samsung.com>

interface_scrollable: update pan position once pan size is changed

Becase bar is closely related with pan and content,
we have to update bar position and size on some event
such as position or size of pan is changed or size of content is
changed.

However bar related things are not updated,
even if pan changed callback is invoked by pan related changing.

This behavior makes invalid drag value and wrong drag value makes
scroller had a bad position.

@tizen_fix

Change-Id: I7f013fd9a05a7f139548c4e05dd1dfc088af8b4e
Signed-off-by: Wonki Kim <wonki_.kim@samsung.com>

interface_scrollable: Add "content,resize" callback to keep backward compat

There was a "content,resize" callback before tizen 3.0.
Even if it was a tizen only code block, there could be a backward compat issue
without the callback.

This commit emits the callback after whole procedure for content resizing.finished.

@tizen_feature

Change-Id: I0eccf8ebc8c704e1403056c0078bd6625d612265
Signed-off-by: Wonki Kim <wonki_.kim@samsung.com>

interface_scrollable: Fix scroller to keep its page when changing lang

When mirrored mode is changed, scroller goes to the logical current page
by reverting the current wanted x position

However, that reverting logic doesn't take place when lang changed from rtl to ltr
This commit modifies mirrored_set api to keep its current page even if lang changed

@tizen_fix

Change-Id: I1fc75c796bac8644d1325b88b293ce09809e3025
Signed-off-by: Wonki Kim <wonki_.kim@samsung.com>

interface_scrollable: Modify logics to make scroller stable on tizen

This patch is for a tizen feature that makes scoller stable on tizen

@tizen_feature

Change-Id: I672ecf5f39c61facd1445a174d5061ddece96d16
Signed-off-by: Wonki Kim <wonki_.kim@samsung.com>

[genlist] enable item align feature for wearable profile

Change-Id: I76fc335ea8c33391f91190cbeadf9ee29de2ec96
Signed-off-by: SangHyeon Lee <sh10233.lee@samsung.com>

Add overscroll effect

Change-Id: I064f44166ee0ebcf8c7cd06a0fa389946f41531e
Signed-off-by: Wonki Kim <wonki_.kim@samsung.com>

elm_plug: Add plug message feature

Change-Id: I453905c5b46311c38eb8ab0972f654ac820c5671
Signed-off-by: Wonki Kim <wonki_.kim@samsung.com>

elementary: Add missed type for legacy.

Summary:
We've been missed type of class name.

Add legacy class name to maintain compatibility.

Reviewers: jpeg, woohyun, Jaehyun_Cho

Subscribers: cedric, jpeg

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

Change-Id: I69146f9427b55fd7dd91b30bee4017cb13fd36c2

elm_genlist : Synchronize accessibility code(2)

   add children_get function to get contents of item.
   +
   ------------------------------------------------------
   [ATSPI][elm_genlist] Remove ", header" from the atspi name.

      Remove the code to add ", header" to name when genlist's item is group type.
      "header" read out screen reader side.
   ------------------------------------------------------

Change-Id: I6669c72a49a3967831796e39142ce03fd25cd9cb

Revert "elm_genlist : Synchronize accessibility code"

This reverts commit 90f49b488631efd317c6f8981b1163a836f5bfe2.

Change-Id: I8c664efdb4849855329f9b7ca046041ab5c5dd7f

[EXTN_GL] Add gl-backend in ecore_evas_extn

- it is base on tizen-remote-surface
- it can be enabled using environment value
  (ex) ECORE_EVAS_EXTN_SOCKET_ENGINE=wayland_egl

Change-Id: I621dd669025061973a77e9685e2b1c86bf4ddffa

efl_ui_win: add window configuring feature on WL

Change-Id: Ifb354a0f3b31fa748fbac9ef08395108e6106963

elm_genlist : Synchronize accessibility code

   Remove code related to name_get for old ux
   and add children_get function to get contents of item.

Change-Id: I109efb44f781c9ebd6171eb666daa49590387f1f

mark deprecated APIs with EINA_DEPRECATED

    ========================================================
    mark deprecated APIs with EINA_DEPRECATED

Signed-Off-By: Jee-Yong Um <jc9.um@samsung.com>
    Change-Id: I1a68481e9fcfaaa361ffc71a127aef477c318020

    Original commit: 46a5b2ef0eaa665ae6ea1deb25c29d9f15428dc8
    =========================================================

Change-Id: Ie303613f29122518175641049aceb8b1edafbcd2

slider: fix slider for tizen 3.0 GUI guide, elm_prefs: add @internal

    =====================================================
    slider: fix slider for tizen 3.0 GUI guide.

    - deleted unnecessary vector objects.
    - change slider indicator initial value.

Signed-off-by: Hosang Kim <hosang12.kim@samsung.com>
    Change-Id: Ib05f483e2d3f362294bed5ab280327bd27022d76

    Original commit: 07fb895f70cc65d4ec3681fdc12fe9f739cf76f5
    =========================================================
    elm_prefs: add @internal to defgroup not to show in document

Signed-Off-By: Jee-Yong Um <jc9.um@samsung.com>
    Change-Id: Id68f0880657284cfbc8a80ccfafd7793690dabf0

    Original commit: c616c666a14f326cdb193b3ec6a3cab2fcc27382
    =========================================================

Change-Id: I8fd1e776ee3305d4c67e856b362e177ead714470

elementary_tizen genlist: apply Tizen's color_class feature

@tizen_feature

elementary entry: Do not send anchor clicked when long pressed

@tizen_fix

elm_win: fix precreated window logic.

    =====================================================================
    elm_win: fix precreated window logic.

    Even though window's type is different, window precreation should be
    supported.

    @tizen_fix
Signed-off-by: Hosang Kim <hosang12.kim@samsung.com>
    Change-Id: I9b020971f3ce9885540dfd02e5b03165186b59e8
    =====================================================================

Original commit: 6c0c2d78a8e514536884008ba74f767d1c8ea1ea

Change-Id: I3c5f2810a03b77d74f0282ee8328e6e5e61cf3be

copypaste: add copypasteUI module to elementary

This migrates the copypasteUI.

Change-Id: Ic85b221e76595136d411b18e59a6a17c06319518

efl_ui_clock: Disabled Mirroring.

This is following tizen UX.

Change-Id: Iee0033605ad2f28d3442b5d39690424e7830dbc7

index: Add tizen only feature to support mobile UX.

@tizen_feature

Change-Id: I0010bb09abf13845bd8da86f93ad979f213054de

calendar: Disabled Mirror feature.

This is not tizen UX.

Change-Id: I38ff97002e129dc6dbf240cfba761bb1136e4b0e

Datetime: Prevent null pointer reference.

When _elm_datetime_items_get() calling with only time format after datetime creation.
The location has been swapped. so some of sorted_field index address gets a invalid
address. then it makes a crash.

@tizen_fix

Change-Id: I8e073237793d6af46a726be6cd51827f14c15661

elementary entry/widget: Fix entry size/cursor/region calculation for Tizen UX

@tizen_fix

entry: Fix elm_entry_entry_get to return utf8 plain text when password mode is enabled

It was Tizen Only feature in Tizen 2.3.

@tizen_fix

entry: Show/Hide guide text immediately

@tizen_fix

elm_calendar: Add Tizen Only features.

Using ICU, Add focused UI

Change-Id: I5de70fe4b7625eb792abaa9b96ac52070121cd94

efl_ui_multibuttonentry: Add tizen only features.

Change-Id: Ie21710f46f2c5faff3cb8d0306eb404f5f26f3db

efl_ui_clock(elm_datetime): Add tizen only features.

Change-Id: Ia989437f9ec17dbc48671aa5d251c0685b0ed42f

elm_spinner: Add Tizen_Only features.

It's for supporting Tizen UX, and compatibility.

Change-Id: Ide9913926b6204c944618a791d032d40ca702734

elementary label: add missing code for style user push/pop/peek patch

@tizen_fix

elementary layout: add internal APIs for handling a part's alignment

It is added for C# requirements.

@tizen_feature

elementary: Apply paragraph direction according to locale

It checks the direction of current system locale by
getting internally predefined keyword "default:LTR" and "default:RTL".

@tizen_feature

elementary: fix EFL_UI_WIDGET_EXTRA_OPS redefined issue caused by color class patches

@tizen_fix

atspi: remove build warning for efl_access interface

Change-Id: Ie3ba859053d0bfe42e7327ce9c014cd720512622

elementary: Override Paragraph Direction APIs

@tizen_feature

Default theme: remove wrong targets from program.

    Some of styles contain STATE_SET programs, that try to switch part into
    non existents description. For desctop application, such as Enventor and
    Eflete this make launch and work slow.
    This commit fix this situation by removing wrong targets or add missed
    states into target parts.

    Change-Id: Ia9f51afaa096cadff363d7ea28d40059b5be04a2
Signed-off-by: Mykyta Biliavskyi <m.biliavskyi@samsung.com>
Original commit: c214b769d8d83bd222447f39e150fdcbed5fac11

Change-Id: I2971c3c80c51b7949f69622601a427d3a6d8d1ab

ctxpopup: fix unintentional item style change

When ctxpopup set style, show, set horizontal, change item's text,
item change style to smae with ctxpopup although it has its own style.

@tizen_fix

original patch : https://review.tizen.org/gerrit/#/c/166589/

Change-Id: I8bdafd1adf34ca3b621925065144dbfb921b01a7
Signed-off-by: JinYong Park <j4939.park@samsung.com>

ctxpopup: add animation signal emit after sizing eval.

In some case, sizing eval calculate new ctxpopup direction,
but animation signal doesn't be emitted.
For example, move ctxpopup after show is called.

@tizen_fix

original patch : https://review.tizen.org/gerrit/#/c/166635/

Change-Id: I6b65a035ab8be6b84e70cd013ee20d2d360ff932
Signed-off-by: JinYong Park <j4939.park@samsung.com>

ctxpopup: remove deleted api, change old api to new api

deleted api like focus_cycle produce error, so wrap it in comments
old api like eo_add, atspi produce error too, so it change to new api

Change-Id: I7d20c107e10ab65b09b95037dfac1a95cfff32c5
Signed-off-by: JinYong Park <j4939.park@samsung.com>

elm_tooltip: set theme group to NULL

Summary:
Even though elm_tooltip does not support EO API, for now,
Efl.Ui.Widget will support elm_tooltip-like feature.
This change group name compatible with new group name policy.

Reviewers: jpeg, woohyun, herdsman, zmike

Subscribers: cedric

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

elm_datetime: use legacy elm_button

Summary:
elm_datetime and Efl.Ui.Clock uses same module: clock_input_ctxpopup.
The module creates internal field object.
Use legay button for field object until Efl.Clock and datetime module
policy is fixed.

Reviewers: jpeg, CHAN, woohyun

Subscribers: cedric

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

panes: Expose parts in EO file

efl_animation: fix typo

Test Plan: make

Reviewers: Jaehyun_Cho

Subscribers: cedric, jpeg

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

selection: delete callbacks when the manager is deleted

If the selection manager is deleted before the drop objects
are deleted, we need to remove callbacks in its destructor.

Thank @herdsman for reporting this.

efl_ui_focus_manager: fix infinit loop in enlightenment wizard

we unsetted and setted focus_manager once too often, so we ended up in a
infinite loop. This should be fixed now. Sorry Derek!

efl_ui_focus: merge efl_ui_focus_user and efl_ui_focus_object into one mixin

Summary:
focus_user and focus_object are similar classes. by merging them into
one mixin, we can maintain consistency.

Test Plan: make check

Reviewers: bu5hm4n

Subscribers: cedric, Jaehyun_Cho, woohyun, jpeg

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

Pyolian: fix and tests for recent changes

q66: please leave the pyolian updating work to me,
it's harder for me to fix wrong stuff instead of just
implement myself ;)

eina: make updating rectangle cache thread safe.

eolian: do not require unit when stringifying types

As it is no longer necessary to pass unit when evaluating exprs,
it is not necessary to pass it here either. Convert all the APIs
to the new style and update all instances in our tree.

eolian: do not require unit for expr eval APIs

The necessary information is now stored directly in the expr
during database validation.

Also enable expr validation for params.

eolian: cache enum expr in the database

eolian: expose _eval_type to internals

examples: Show off usage of "background" part

See bg_cxx_example_01

widget: Expose "background" part (for bindings)

cxx: Remove elm_widget.h from Elementary.hh

It's internal.

selection: Fix two remaining warnings

animation: Fix some warnings including a real one

Real issue here:

.../efl_animation_group.c:120:40: warning: for loop has empty body [-Wempty-body]
   EINA_LIST_FREE(pd->animations, anim);

efl_playable: split Efl.Player interface to Efl.Playable interface

Summary:
Efl.Player interface simply provides play functions,
but another interface which indicates Efl.Player will play is also
needed.

Test Plan: Run elementary_test->Efl.Animation tests

Reviewers: woohyun, conr2d, Jaehyun_Cho, jpeg, cedric

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

Efl.Ui.Progressbar: Add Efl.Ui.Translatable/Markable too.

it was missed in previous patch.

Efl.Ui.Progressbar: Add missing Efl.Text/Content

eo: Fix make check

The cache had a generation count, removing this check breaks eo suite.
EO is designed to tolerate init/shutdown cycles (as long as
dlopen/dlclose isn't involved).

Note: the ugly goto are useless as GCC -O2 understands EINA_(UN)LIKELY
and deals with it as expected (just look at the asm produced).

See 34d9f2070696027199a56cb621c0526ea1430e8f

eo: Remove unused goto label

edje: Add beta event when an invalid part is used

This can be used for debugging, by listening to
  EFL_LAYOUT_EVENT_PART_INVALID

Or just use
  EINA_LOG_LEVELS=edje:2
if you just need the logs.

panes: Fix invalid part name in layout recalc

code widget: Fix order of operations

super functions, when creating stuff, should be called first.

edje: add invalid part handler

ecore/edje/elm: Fix a few WRN from calls to NULL

This fixes some of the warnings generated by calling functions on NULL
objects. One of the main remaining points is to avoid unwanted warnings
on non-existing parts.

Ref T6326

eo: Warn on calls to NULL object

This makes eo print a WRN message in case a function is called on NULL.
efl_del is an exception to this rule (implemented in a hackish way, I
admit). I don't know any language or object model where using a null
object doesn't result in an exception or crash, except EO. In any case,
calls to null are invalid.

The next commits will resolve most warnings for EFL.

Ref T6326

evas: Remove stupid warning

edje: Add property "part_exist"

This allows to safely verify if a part exists, without triggering any
potential call to NULL object, or even requiring the efl_part() handle
to be created.

This is perfectly equivalent to edje_object_part_exists(), but
implemented by both edje object and elm layout.

selection: Disable debug logs

selection's namespacing still requires some love

elm genlist test - don't overflow test time nanoseconds if system slow

with a long enough test or slow enough machine these do overflow so
force everything to unsigned long long's for genlist test performance
measuring.

eina: remove usless newline

Summary:
ecore_evas: remove debug

eina: unregister log level when done with

Fixes a constant memory leak.

eina: introduce EINA_HOT and EINA_COLD

These attributes respectivelly expand to __attribute__ ((hot)) and
__attribute__ ((cold)) when available. They allow to mark functions are
being hot/cold (frequently used or not) as well as to qualify labels
within a function (likely/unlikely branches).

eo: speed-up generated calls by removing call cache

The call cache needed to by thread-local, to avoid concurrency issues.
Problem with TLS is that is adds an extra overhead, which appears to be
greater than the optimization the cache provides.

Op is naturally atomic, because it is an unsigned integer. As such, it
cannot be tempered with while another thread is reading it. When
entering the generated function, the first operation done is reading
'op'. If we have concurrency, we will have access sequences returning
either EFL_NOOP or a VALID op, because 'op' is not set until the very
end of the function, when everything has been computed. As such, we now
use the 'op' atomic integer to instore a lock-free/wait-free mechanism,
which allows to drop the TLS nature of the cache, speeding up the access
to the cache, and therefore making functions execute faster.

We don't test anymore the generation count. This can be put as a
limitation. If means that if you call efl_object_shutdown() and
re-initialize it later with different data, opcodes will be invalid.
I am not sure there is any usecase for this to ever happen.
We could move all the caches in a dedicated section, that can be
overwritten after a call to efl_object_shutdown(), but I am not sure it
will be very portable.

Benchmark: mean over 3 executions of
   ELM_TEST_AUTOBOUNCE=100 time elementary_test -to genlist

```
                     BEFORE               AFTER
------------------------------------------------------------
time (ns)            11114111647.0        9147676220.0
frames               2872.3333333333335   2904.6666666666665
time per frame (ns)  3869364.6666666665   3149535.3333333335
user time (s)        11.096666666666666   9.22
cpu (%)              22.666666666666668   18.333333333333332
```

Ref T6580

Reviewers: raster, cedric

Subscribers: cedric, jpeg

Maniphest Tasks: T6580

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

selection_dnd: fix warnings in clang

Fix warnings about type convert and vars init in clang and remove comments.

elm_panel: add exception handling in _drawer_close()

this fixes a bug with scrollable panel not being blocked
when it is closed. scroll is blocked in _anim_stop_cb()
which is called after elm_interface_scrollable_region_bring_in().
but if panel content is already at the target position, _anim_stop_cb()
is not called. so there is a need to check content's position and
handle the exceptional case.

Revert "elm_panel: add exception handling in _drawer_close()"

This reverts commit 2acfc3e381dc4fc87e3249b0f63f1ba5e0d1c2de.

elm_panel: add exception handling in _drawer_close()

this fixes a bug with scrollable panel not being blocked
when it is closed. scroll is blocked in _anim_stop_cb(),
which is called after elm_interface_scrollable_region_bring_in().
but if panel content is already at the target position, _anim_stop_cb()
is not called. so there is a need to check content's position and
handle the exceptional case.

elm_panel: prevent redundant callback function addition or deletion

there is a need to check if callback functions already exist or not
before adding or deleting them, because they are added or deleted
at two points:
in _elm_panel_scrollable_set() and _elm_panel_elm_widget_disable().

ecore: forgotten EFL_MAIN_EX.

elementary dnd: fix tautology

elc_popup: reorder the sub objects free sequence

Summary:
when the popup is deleted, some EVAS_CALLBACK_DEL callback functions
try to use already freed objects.
reorder free sequence to prevent it.

Test Plan:
1. elementary_test -to popup
2. check 'Enable popup scroll'
3. open several popup test and click Close button.
4. check that there are no error message

Reviewers: Jaehyun_Cho, bu5hm4n

Reviewed By: Jaehyun_Cho

Subscribers: cedric, jpeg, herb

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

examples: efl_ui_list: remove unused variables

examples: calendar_cxx: comment out unused variables

I normally would simply remove them, but they belong to some more
commented out code below. We should either remove both or get this
fixed.

examples: cxx: avoid tick in warning message

The compiler is not to happy about having this tick in the warning
message. Switch to the more formal can not and be done with it.

menu_cxx_example_01.cc:3:26: warning: missing terminating ' character

ecore_file_monitor: replace EINA_LIST_FOREACH to EINA_LIST_FOREACH_SAFE

If ecore_file_monitor_del is called inside the file monitor callback function,
eina_list found from monitor_hash would be freed. (You can check this inside
eina_hash_list_remove.)
Then, EINA_LIST_FOREACH makes one more for loop with invalid eina_list pointer.

EINA_LIST_FOREACH_SAFE can prevent from this problem.

Ecore_Win32: move monitor window initialization to earlier in init func

Summary: This monitor window is just used to receive events when mutiple
monitors are available. it should not be managed by the ecore loop
(creation and destruction events), so initting it earlier means
ecore_win32 attaches less memory/overhead to it as it's just being
used for notifications for devices.

Test Plan: DrMemory to check used memory

Reviewers: cedric

Subscribers: jpeg

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

Pyolian: add tests for recently added funcs

Correct module name for the interface context