elm_colorselector: avoid calling functions with null objects

Summary:
a number of calls during construction resulted in null object errors here

fix T7036

Depends on D6323

Reviewers: bu5hm4n, Hermet, woohyun, devilhorns

Reviewed By: bu5hm4n

Subscribers: cedric, #committers

Tags: #efl

Maniphest Tasks: T7036

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

elm_colorselector: avoid passing null to efl functions

Summary:
this check already existed in the function but was not applied to the
entire function

ref T7030
Depends on D6322

Reviewers: bu5hm4n, Hermet, woohyun, devilhorns

Reviewed By: bu5hm4n

Subscribers: cedric, #committers

Tags: #efl

Maniphest Tasks: T7030

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

evas: switch to EFL_EVENT_INVALIDATE for event grabber sub-object deletion

Summary:
this requires the parent of the deleted object to be reachable, so use the
invalidate callback as this is the last point during which it's possible to
retrieve a parent object

ref T7030

Depends on D6320

Reviewers: bu5hm4n, Hermet, woohyun, devilhorns

Reviewed By: bu5hm4n

Subscribers: cedric, #committers

Tags: #efl

Maniphest Tasks: T7030

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

efl_ui_win: check frame_obj existence before operating on it in pre_render

Summary:
this object will not exist for fake win objects, e.g., in enlightenment

fix T7033
Depends on D6319

Reviewers: bu5hm4n, Hermet, woohyun, devilhorns

Reviewed By: bu5hm4n

Subscribers: cedric, #committers

Tags: #efl

Maniphest Tasks: T7033

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

efl_ui_win: do not attempt to focus a null object on focus-in

Summary:
ref T7033
Depends on D6318

Reviewers: bu5hm4n, Hermet, woohyun, devilhorns

Reviewed By: bu5hm4n

Subscribers: cedric, #committers

Tags: #efl

Maniphest Tasks: T7033

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

eo: make the debug build print leaked errors

Summary: Depends on D6328

Reviewers: devilhorns, zmike

Reviewed By: zmike

Subscribers: cedric, #committers, zmike

Tags: #efl

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

eo: get rid of eo_id

Summary:
build with this disabled is just not working as the thread model of eo
falls apart. The threadmodel is required to have eo_id, as it decoded
the thread information in the eo_id, which is not working without eo_id.

This also fixes the testcases that have never been executed due to the
fact of the missing HAVE_EO_ID

fix T6610

Depends on D6327

Reviewers: devilhorns, zmike

Reviewed By: zmike

Subscribers: cedric, #committers, zmike

Tags: #efl

Maniphest Tasks: T6610

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

eo: only call efl_parent_set when parent_id is available

Summary:
this is no functinal change, we safe the calling to efl_parent_set.
Nothing else, the base implementation is initializated with NULL as
parent, and efl_parent_set with NULL would have returned in the first
few lines.

fix T7032

Reviewers: devilhorns, zmike

Reviewed By: zmike

Subscribers: cedric, #committers, zmike

Tags: #efl

Maniphest Tasks: T7032

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

tests: set monitoring interval for eio tests to 0.05s

Summary:
cpu usage is not an issue during test runs, we want the tests to finish
as fast as possible in all cases regardless of the cost. this resolves
the last remaining issue with eio tests where sometimes the fallback
monitor would trigger and the test suite would hang for 60+ seconds

fix T6915
Depends on D6248

Reviewers: stefan_schmidt, bu5hm4n, ManMower, devilhorns

Reviewed By: bu5hm4n, ManMower

Subscribers: bu5hm4n, cedric, #committers

Tags: #efl

Maniphest Tasks: T6915

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

tests: use fixtures for eio tests

Summary:
remove a lot of duplicated code and make this test suite read more like
all the other suites

fix T6813
Depends on D6247

Reviewers: stefan_schmidt, bu5hm4n, ManMower, devilhorns

Reviewed By: bu5hm4n, ManMower

Subscribers: Hermet, bu5hm4n, cedric, #committers

Tags: #efl

Maniphest Tasks: T6813

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

eio: add method for setting the global file polling rate for fallback monitors

Summary:
the default value for the fallback poll monitor timer interval is 60.0 seconds,
which is not useful for all cases, such as CI, where we don't care about cpu
usage and just want things to process as fast as possible at all times

this enables setting the interval to any value, ensuring that any existing
timers are modified to use that value immediately

@feature

Reviewers: stefan_schmidt, bu5hm4n, raster, devilhorns, ManMower

Reviewed By: bu5hm4n, ManMower

Subscribers: ManMower, raster, bu5hm4n, cedric, #committers

Tags: #efl

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

eina_threadqueue: add locking for variable used between threads

Summary:
the 'first' member of this struct is used simultaneously across
threads and can have conflicting read/write operations occurring at this
time

as int operations are not guaranteed to be atomic, ensure that we are
using atomic operations or locking as necessary

@fix

Depends on D6299

Reviewers: ManMower, devilhorns

Reviewed By: ManMower

Subscribers: cedric, #committers

Tags: #efl

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

eina_threadqueue: simplify/unify some locking code

Summary:
this moves lock calls outside #ifdef blocks to make the code more
readable

no functional changes
Depends on D6298

Reviewers: ManMower, devilhorns

Reviewed By: ManMower

Subscribers: cedric, #committers

Tags: #efl

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

eina_threadqueue: fix spinlock destruction by using spinlock api

Summary:
this would deadlock in some cases

@fix
Depends on D6297

Reviewers: ManMower, devilhorns

Reviewed By: ManMower

Subscribers: cedric, #committers

Tags: #efl

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

eina: replace memsets in thread debugging lock create/free with manual zeroing

Summary:
memset overwrites the thread value, triggering errors when running tools like
helgrind

attempting an operation on an invalid thread will cause errors naturally,
so zeroing the rest of the struct and ignoring the thread member is fine

Reviewers: ManMower, devilhorns

Reviewed By: ManMower

Subscribers: cedric, #committers

Tags: #efl

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

emotion_generic: clean up event handlers on fork failure

Summary:
this case would guarantee a crash if the module was ever unloaded
and the event handlers were called, and it leaks otherwise

@fix

Reviewers: ManMower, devilhorns, bu5hm4n

Reviewed By: ManMower, bu5hm4n

Subscribers: cedric, #committers

Tags: #efl

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

efl_mono: Fix call to System.Threading.Thread.

Summary: It was resolving to efl.Thread instead.

Reviewers: devilhorns, felipealmeida, zmike

Reviewed By: zmike

Subscribers: cedric, #committers, zmike

Tags: #efl

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

theme: rename non-API actionslider part to avoid confusion when reading

Summary:
no functional changes
Depends on D6211

Reviewers: cedric

Reviewed By: cedric

Subscribers: #committers

Tags: #efl

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

elm_dayselector: fix improperly named API theme parts

Summary:
API parts require namespacing, these parts have been namespaced with
compatibility code added to handle legacy naming
Depends on D6210

Reviewers: cedric

Reviewed By: cedric

Subscribers: #committers

Tags: #efl

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

theme: fix part namespacing in multibuttonentry theme

Summary:
this was released with improperly namespaced parts which must be maintained
for future releases, but the namespacing can be corrected for future
releases while adding aliasing to preserve compatibility
Depends on D6208

Reviewers: cedric

Reviewed By: cedric

Subscribers: #committers

Tags: #efl

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

theme: change part in elm photo group from swallow to spacer

Summary:
this appears to be a remnant of the time before edje had spacer parts
and other part types were randomly used instead

there is no library reference to this part and it is not namespaced so
there is no reason to leave it as a (confusing) swallow
Depends on D6040

Reviewers: cedric, Hermet

Reviewed By: Hermet

Subscribers: Hermet, #committers

Tags: #efl

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

theme: use elm namespace for spinner button signal source

Summary:
ref D6036

Depends on D6037

Reviewers: cedric

Reviewed By: cedric

Subscribers: #committers

Tags: #efl

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

theme: rename 'sizer.content' parts to 'sizer_content'

Summary: non-api parts should avoid using '.' in the name to avoid confusion

Reviewers: cedric

Reviewed By: cedric

Subscribers: #committers

Tags: #efl

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

ecore-audio: make this a Efl_Object

sorry for the late fixup, the Ecore_Audio_Object is documented as "the
audio object" however, it is not, internally it is used as struct, thus
gdb gets confused.

ecore_audio: add back function that is stable

this function was removed, but should not have been removed.

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

Revert "elementary: fix wrong signal emission"

This reverts commit 1b245787fe1fc5da41b6ed77dd67cb022900afa6.

This is a workaround patch, even occurs a regression bug that
breaks widget signal emission logic. (Happened in Enventor toolbar)

I reviewed this code seriously and found out
ui_layout sub object unset logic has been changed.
Obviously that breaks the elm compatibility.

When sub-object of layout is removed, it tries to remove sub-object from
the layout internal list. Problem is, some widgets sends internal signals
when sub-object is removed(i.e "icon,hidden") , but layout returns the
valid object even though sub-object unset is called prior to signal,
means, "icon,visible" not "icon,hidden" emitted.

This logic obvisouly changed from the previous efl version.
And we need to fix that logic first.

See _efl_ui_button_legacy_efl_ui_widget_widget_sub_object_del()
to check this issue.

1. button: sub_object_del()
2. layout: sub_object_del() => sub object must be removed.
3. button: signal emit() => for updating states
4. layout: content_get() => returns valid object?????! (Issue)

evas map: fix a typo.

evas vg: add warning msgs for users.

Since usage has been changed under the beta version,
Some users may keep the old-usage that breaks the vg behavior now.

For their information, vg prints messages in case of breaks.

elm_toolbar: fix icon sizing issue

icon size * scale - this causes the issue.
this is not wrong,
but Tizen UX did not consider scale to icon size.
so, this is removed for compatibility.

@tizen_fix

Change-Id: Ib71ec1998767988bc7ad39299c3282132f8a0fd5
Signed-off-by: Bowon Ryu <bowon.ryu@samsung.com>

elm_slider: do not clip indicator to slider

By calling evas_object_smart_member_add(), indicator is clipped to slider.
To keep backward compatibility, indicator should not be clipped to slider.

Change-Id: I2028b5edb8b6ce911d83771480ad18f9bf803a41

ecore: fix event flushing to not erase+leak events

stealing the message data here prevents events which aren't being flushed from
ever being usable again and is unnecessary since the free callback will be
automatically called during the destructor

ref 5dd52fd09b7d79c70b3134423a87aa6400a2d994

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

evas_cpu: Avoid SIGILL in evas startup on x86

Summary:
To determine if a system supports SIMD instructions, the cpuid facility
should be used.  However, for 15+ years EFL has been trapping SIGILL,
then attempting to execute these intstructions.

Continuing after SIGILL is explicitly undefined behaviour and can never
safely be relied upon - it is possible the CPU will respond to the
unknown instruction in an upredictable way and the program will not
continue correctly.  Even if it hasn't caused problems before, there's
no reason to believe a processor released in the future won't behave
differently.

Lately we've had a couple of bug tickets where SIGILL appears to cause
problems at a system level as well, but there seems little point in
chasing those problems down as we shouldn't even be doing this in the
first place.

ref T6711
ref T6989

We still rely on SIGILL in a few configurations where eina_cpu doesn't
know how to query features properly (powerpc, sparc, and non linux
ARM configurations).  Hopefully someone with expertise on those
platforms can follow up and we can remove this entirely.

Note: MMX2 appears to not really be a thing, and is instead provided by
both 3DNow! and SSE.  We already conflate it with SSE in other parts of
evas, so I've just used SSE here to test for its presence.
Depends on D6313

Reviewers: devilhorns, zmike

Reviewed By: zmike

Subscribers: cedric, #committers, zmike

Tags: #efl

Maniphest Tasks: T6989, T6711

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

evas_cpu: Refactor checks that use eina_cpu_features_get

Summary: Minor code simplification.

Reviewers: devilhorns, zmike

Reviewed By: zmike

Subscribers: cedric, #committers, zmike

Tags: #efl

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

eo: Fix internal reference count wrongly 0 when constructor returns a different object

The class's Eo constructor can return a different object, which makes
the efl_add return that object instead. However, a bug was not
initializing the internal reference count when a different object was
returned.

Revert "eina debug - fix segv if an app is compiled with -pg for profiling"

This reverts commit 8343de6ce3163f27257d386fb212d1eacf78ba7d.

This borke mac os build, CI is spamming on irc channels and addtional
tests are failing. I would give this back to the drawingboard.

ref T7029

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

ecore_con: fix possible segfault

with -O3 i get a segfault in this test. (with gcc and clang)
It looks like the compilers are doing constant folding because we
declared the struct as constant, however, due to using the address of
the fild the keyword const is simply not true, thus we are getting a
segfault because the address returned by &desc is not correct anymore,
due to the fact the constants are rolled out.

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

tests: wrap critical errors to disable aborting in eo tests

ref T7002

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

tests: disable eo test timeouts when not forked

this makes debugging much easier

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

tests: add macros to selectively disable aborting on critical log messages

in many cases, a test will intentionally trigger an error to verify that it
is handled correctly. when the test is manually run with EINA_LOG_ABORT set,
this may cause the test to abort, preventing further debugging. by wrapping
intentional cases where critical errors are triggered, debugging tests
becomes easier

ref T7002

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

emotion - emotion_test - dont limit fps to 30 as this doesn't help tests

so i noticed it was jerky... to move/resize the video obj. it's
because it set fps to 30... well that was silly. didn't help chasing a
ghost, so remove this to avoid chasing hosts.

eina debug - fix segv if an app is compiled with -pg for profiling

gprof uses SIGPROF ... and this causes all sorts of bad things with
the eina debug profiling too. so - use SIGRT instead ... fixes T7028

ecore_imf: Remove security_code in autofill type

Change-Id: Ice2c20358f37ecd047b6012b8a9a619b37e211df
Signed-off-by: Jihoon Kim <jihoon48.kim@samsung.com>

ecore_imf: Remove security_code in autofill type

Change-Id: I7221d1bbe311da5523151c889e71585785eead26
Signed-off-by: Jihoon Kim <jihoon48.kim@samsung.com>

elm_slider: keep backward compatibility of Tizen 3.0

This commit sets indicator hidden by default for backward compatibility.
This commit restores overwritten code of
6dae707407894f16a9bd87b62cbc087695f0e868

Change-Id: I6df50a8bdfa82c442c6e6d61c319c9a512c14d66

Revert "eocre_wl2_window: unset zxdg_surface zxdg_toplevel when hide"

This reverts commit 520034511855d67b5da9d5e63b179788d581f784.

Change-Id: I94564c39f5e8c50dcd32365fdd06fe0b33d468f6

ecore - handle G_IO_ERR conditions for net sockets

Glib integration with using of select() syscall doesn't properly
propagates G_IO_ERR condition for network sockets. Problem relies in
_ecore_glib_context_poll_to() where rewriting filedescriptor events to
GPollFD structures reside.

This fixes T5725

@fix

efl_debugd: don't force unlinking UNIX socket before binding

If two daemons are launched one after the other, we would like the
second one to exit directly. The problem is that if the UNIX socket is
unlinked before the binding, the second daemon will succeed to create this
socket, stoling it from the first daemon, and exit because the network
socket is not bindable.

It results in the first daemon to continue running and accepting
connections from the debuggers (network connection) but ignoring the
applications connection (local UNIX connection).

tests: disable toggling clouseau_enabled option in elm tests

Summary:
this option triggers codepaths which require and interface with an
external project not in the EFL tree, so it should not be tested in
the efl tree

also depending on the version of clouseau used, this causes the test
suite to fail 100% of the time

fix T6985
ref T7023

Reviewers: devilhorns, ManMower

Reviewed By: ManMower

Subscribers: cedric, #committers

Tags: #efl

Maniphest Tasks: T6985, T7023

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

eet: handle decode failures as errors in variant decoding

Summary:
all other null returns of _eet_data_descriptor_decode() are treated as
fatal errors when decoding, and failure to do so in this case guarantees
errors later due to incomplete decoding

@fix

ref T5379
Depends on D6293

Reviewers: devilhorns, ManMower

Reviewed By: ManMower

Subscribers: cedric, #committers

Tags: #efl

Maniphest Tasks: T5379

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

eet: print errors any time an error occurs while decoding

Summary:
catching these errors was super annoying since the codebase is mostly
comprised of goto statements, so this should make future debugging easier
Depends on D6292

Reviewers: devilhorns, ManMower

Reviewed By: ManMower

Subscribers: cedric, #committers

Tags: #efl

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

eet: add #if 0 around macro which will cause compile errors if used

Summary:
this macro wraps a function which is also inside an #if 0 block,
so it's best not to leave it where someone might try to use it

no functional changes

Reviewers: devilhorns, ManMower

Reviewed By: ManMower

Subscribers: cedric, #committers

Tags: #efl

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

eina_lock: remove safety checks for main thread

Summary:
these effectively prevent locks from being used to synchronize operations
between (non-main) threads, which restricts their usefulness and also
prevents our own unit tests from passing

fix T7004

Depends on D6267

Reviewers: devilhorns, ManMower

Reviewed By: ManMower

Subscribers: cedric, #committers

Tags: #efl

Maniphest Tasks: T7004, T1984

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

eina: disable abort on shutdown when triggered by system monitoring thread

Summary:
this lock/thread never gets destroyed, so it should not trigger failure cases
when that is always the case

@fix

Depends on D6266

Reviewers: stefan_schmidt, ManMower, devilhorns

Reviewed By: ManMower

Subscribers: cedric, #committers

Tags: #efl

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

eina_lock: remove locks from eina tracking list on free while thread debugging

Summary:
this prevents invalid memory access during subsequent inlist operations

@fix

sq
Depends on D6265

Reviewers: stefan_schmidt, ManMower, devilhorns

Reviewed By: ManMower

Subscribers: cedric, #committers

Tags: #efl

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

eina_lock: modify mutex debugging to ignore recursive locks

Summary:
recursive locks are locked multiple times in the same thread, leading to
failure with the tracking infrastructure which was written prior to
the addition of recursive locks and assumed that locks would only be
taken exactly once

given that this debug info is meant to handle non-recursive locks,
it makes little sense to include them in the handling

@fix

Depends on D6264

Reviewers: stefan_schmidt, ManMower, devilhorns

Reviewed By: ManMower

Subscribers: cedric, #committers

Tags: #efl

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

ecore_wl2: purge surface buffers in semi_free

Summary:
This fixes a session recovery bug with software render.

An attempt to re-use a buffer in a new wayland connection resulted
in another disconnect and broken rendering.
Depends on D6281

Reviewers: devilhorns, zmike

Reviewed By: zmike

Subscribers: cedric, #committers, zmike

Tags: #efl

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

ecore_wl2: Add purge to ecore_wl2_window_surface_flush

Summary:
It's convenient to be able to pass this through this api too.

@betabreak
Depends on D6280

Reviewers: devilhorns, zmike

Reviewed By: zmike

Subscribers: cedric, #committers, zmike

Tags: #efl

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

ecore_wl2: Handle purging flush in dmabuf module

Summary:
Allow destroying all buffers, even attached ones, in the dmabuf surface
module.
Depends on D6279

Reviewers: devilhorns, zmike

Reviewed By: zmike

Subscribers: cedric, #committers, zmike

Tags: #efl

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

ecore_wl2: Add a purge option to surface flush

Summary:
We need to be able to forcibly destroy all surface buffers to make
session recovery work safely for software rendering.

@betabreak
Depends on D6278

Reviewers: devilhorns, zmike

Reviewed By: zmike

Subscribers: cedric, #committers, zmike

Tags: #efl

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

ecore_wl2: Set the parent window's surface to NULL on surface destroy

Summary:
Prevents potential use after free.
Depends on D6277

Reviewers: devilhorns, zmike

Reviewed By: zmike

Subscribers: cedric, #committers, zmike

Tags: #efl

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

edje_cc: temporarily disable aborting compile when namespace validation fails

Summary:
there are far too many issues here for me to fix them all, the authors of
these failures should be responsible for helping to clean this up

revert this patch once all issues are resolved

ref T6966
Depends on D6042

Reviewers: cedric, Hermet, stephenmhouston, devilhorns

Reviewed By: stephenmhouston

Subscribers: #committers

Tags: #efl

Maniphest Tasks: T6966

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

build: enable namespace verification for default theme

Summary:
this will trigger a build failure if someone modifies the theme in such
a way that namespacing is not used correctly, saving some time when
reviewing larger patches which have many theme changes

ref T6911
Depends on D6036

Reviewers: cedric

Reviewed By: cedric

Subscribers: #committers

Tags: #efl

Maniphest Tasks: T6911

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

edje_cc: add option for verifying namespace usage

Summary:
when -N is passed, parts and program signals will be checked for
traditional namespacing based on the group name, causing compile failure
if inconsistency is detected

@feature

ref T6911

Reviewers: cedric, devilhorns

Reviewed By: cedric

Subscribers: #committers

Tags: #efl

Maniphest Tasks: T6911

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

eina_cow: rework debug profile safety checks for recursive writes

Summary:
recursive writes are not inherently bad, so long as the pointer is
consistently re-set (handled automatically by macros), and they are nearly
unavoidable in some places such as eo/evas internals

issues may arise in a specific corner case of recursive writes when a pointer
has been hashed for garbage collection, so adjust the checks to watch for this
specific case instead of crashing on every case

fix T7005

Reviewers: devilhorns

Reviewed By: devilhorns

Subscribers: cedric, #committers

Tags: #efl

Maniphest Tasks: T7005

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

Revert "evas: handle event COW modification when COW is already active"

Summary:
This reverts commit cd6b890c7322ec67d3c41281bda3b22d4a481775.

this resolves complaints from a safety check in the debug profile,
but it seems more likely that the safety check is bogus and should be
removed rather than making this code more complex

Reviewers: devilhorns

Reviewed By: devilhorns

Subscribers: cedric, #committers

Tags: #efl

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

eina_lock: do not set PTHREAD_MUTEX_ERRORCHECK on recursive locks

Summary:
a mutex can only have one type, so setting this type onto a recursive mutex
will unset the recursive attribute and cause deadlocks

ref T1984
@fix

Reviewers: stefan_schmidt, ManMower, devilhorns

Reviewed By: devilhorns

Subscribers: cedric, #committers

Tags: #efl

Maniphest Tasks: T1984

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

efreet: make efreet_menu_async_get() deprecated.

Marking deprecated. Func will warn and instantly return on
call.

Clouseau: use the config flag only for the old Clouseau

elm: remove access_type set API

The api was initially designed to skip objects in accessibility
hierarchy, however it was later replaced with usage of
REDUNDANT_OBJECT role on accessibility object.

Remove this api as it is unused, it obfuscates code even more
and may cause other problems.

Change-Id: I3202ee73ccf545e79d509d3b00569db896d4f975

ecore_wl2: change not to render after hide.

show->hide-> configure_cb state case, efl client deals the code like show state.
but efl client already changed into invisible state.
so menual_render_set cdoe should not run.
we will change manual render set code to draw_block code.

Change-Id: Id83a995567dc4c4da7a54366bf6ece97e16d2429
Signed-off-by: Jiyoun Park <jy0703.park@samsung.com>

eocre_wl2_window: unset zxdg_surface zxdg_toplevel when hide

ecore_wl2_window_show creates shell surface and its role
eocre_wl2_window_hide destroys them in pair
so, display server know explicit show /hide condition
as a result server would not map a client which attach buffer in background
if client destroyed its shell surface.

@tizen_fix

Change-Id: Idd5a671b69497936c59455b46eaeb1131565b784

evas: make objects again delete correctly

Summary:
what here was done was fundamentally wrong, deleting the pd->object
field of a evas object after a efl_del / evas_object_del is completly
wrong. evas object lifetimes are controller with eo_manual_free, this
means, they are still alive, even after you called evas_object_del on
them. removing pd->object results in eo_menual_free calls to NULL
objects and leaking the object carrying the private data. Overall,
breaking this pd->object field and unsetting it is a very bad idea, as
its the only way that evas cleansup the object correctly.
This brings down the number of ui related leaked objects on shutdowns to
0. (YEY :))
This also fixes weird error messages on app shutdown.

fixes T6964

Reviewers: devilhorns, zmike

Reviewed By: zmike

Subscribers: cedric, #committers, zmike

Tags: #efl

Maniphest Tasks: T6964

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

efreet: mark efreet_menu_async_parse() deprecated.

Marking deprecated, with warnning and instant return.
Disabling test code also (for now). Awaiting a fresh
implementation.

@fix T585

evas vg: prevent a corner-case crash.

tbh, current vg interfaces a little bit bad... here is one scenario to this
stupid case.

efl_parent_set() and evas_object_vg_root_node_set() both do re-parent
 job. They could be conflicted if user calls both apis in either way.

efl_parent_set(root_node, NULL); but Vg Object still keeps the root node
which is just a dangling pointer that occurs a crash while rendering.

eina: Turn stringshare printfs into EINA_LOG_DBG

Summary:
eina_share_common contained extremely verbose printfs which rendered
the debug profile (./configure --with-profile=debug) almost unusable.
They have been turned into EINA_LOG_DBG in the eina_stringshare log domain
(when related to stringshares) and without domain when related to other
eina_share facilities.

Also, cleaned up some printfs which have been commented out for 8 years.

Fixes T7006

Test Plan:
The printfs are regular eina logs and can be filtered out by regular means.
The debug profile output is far more usable now.

Reviewers: zmike, devilhorns

Subscribers: cedric, #committers

Tags: #efl

Maniphest Tasks: T7006

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

evas vg: code refactoring.

No logic changes.

I know this is not a good case in alpha freeze.
but code was too dirty, need more polishing.

examples: fix build on FreeBSD.

Missing header for AF_INET/6. Another small
difference between BSD and Linux et al.

evas vg: fix another memory leak

@fix

elm_datetime, efl_ui_clock : Add check 'legacy widget' for layout signal emission

Summary:
08a11d, Signal name is changed to efl ui signal name
legacy widget should use elm
This commit resolves T6928

#efl, #regression

Test Plan: elementary_test -to datetime

Reviewers: JackDanielZ, Jaehyun_Cho, Hermet, YOhoho, devilhorns, zmike

Reviewed By: zmike

Subscribers: YOhoho, cedric, Hermet, Jaehyun_Cho, #committers, JackDanielZ, zmike

Tags: #efl

Maniphest Tasks: T6928

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

evas example: removed unnecessary calls for vg shapes.

efl interface: remove weired implementation.

This color interpolation should not a part of efl_gfx_shape.

Color should be interpolated by Vg.Node which implements Efl.Gfx.Color.

evas vg: fix broken morphing(interpolation)

efl_gfx_path itself took care of efl_gfx_shape data but its heirarchy was
conceptually wrong. Even efl_gfx_shape is mixing the efl_gfx_path...

Damend design...

Some of derived classes of efl_gfx_path (i.e. Vg.Node and Vg.Container) are
none of the Path acutally. They are just mixing Path's interpolation interface.

So, Here patch changes VG.Node to stop calling the super's interpolate method
and Vg.Shape to call both super -gfx_shape and vg_node- interpolate method.

@fix T6996

Revert "elm_entry: add elm_entry_prediction_hint_hash_set/del API"

This reverts commit 432f5de7d3537b9a17806124b4bf76c1754f29c7.

Remove highlight after disabling screen-reader

Change-Id: I0363ee5c64682d8b6ca6994f74a5dfdaf4c4fd97

elm_spinner: Fix typo

Change-Id: Ife0fd94116220b0f91e64d6ba23e9f42a024c00b

[a11y] Add null check before calling highlight_hide

While navigating through homescreen segfaults appeared.

Change-Id: I7d1cb807e4e02781501c00a3a644cc8a64cd84fe

[EvasGL] fix the crash issue about egl_fence_sync

fix the crash issue about evas gl and egl_fence_sync
After upgrading efl 1.20, native_bind_cb function interface was changed.
To support, native struct has outbuf.

Change-Id: I2ffc10671d716ee478f0589bd2309efb62aed913

evasgl: add EVAS_GL_EGL_SYNC_ON feature

Change-Id: Ie852064783b295a1a8ea5d4b3752f61784328f81
Signed-off-by: Sunghyun kim <scholb.kim@samsung.com>

efl_ui_win: keep default geometry compatibility

The efl_ui_win default geometry had been 0,0,1,1 till Tizen 4.0.
The efl_ui_win needs to keep backward compatibility of default geometry.
Because the MM API, player_set_display returns if the window size is 0x0.
So mycontents-video-player-tv could not play video file.

@tizen_only

Change-Id: I531b9950f2d52fab18464812701d3a5b2b3e5c35

elm_entry: add elm_entry_prediction_hint_hash_set/del API

elm_entry_prediction_hint_hash_set API sets the prediction hint data at the specified key, and
elm_entry_prediction_hint_hash_del API is for deleting the prediction hint data identified by a key.

@feature

Change-Id: I613d1a923f96ce261a7b809b749a97e68acc94ff
Signed-off-by: Jihoon Kim <jihoon48.kim@samsung.com>

evas : Modified tbm_queue to use one per surface.

Change-Id: I7b148980a61ce1c06b3aea1a2626315270f0aa9c

evas : fixed bug for EVAS_NATIVE_SURFACE_TBM

  [NativeTBMSurface] Seperated by colorspace.

    To create RGBA_Image for NativeTBMSurface, the related code is
seperated by tbm format.
    So, Getting format internal function is added and seperated the
code.

    [NativeTBMSurface] Add _evas_native_tbm_init/shutdown

    Add _evas_native_tbm_inti/shutdown to evas_native_tbm.c
    In addition, one more patch is added to fix stride mismatch
    and allocate image data for converting from yuv to rgb.
    This patchset are  oriented by JP's and Minkyung patch
    ---------------------------------------------------------------------------------
    Author: Jean-Philippe Andre <jp.andre@samsung.com>  2016-03-09
16:18:34
    Committer: Jean-Philippe Andre <jp.andre@samsung.com>  2016-03-15
11:11:59
        Evas engines: Add internal functions for native images
    ----------------------------------------------------------------------------------
    Author: Minkyoung Kim <mer.kim@samsung.com>  2016-03-31 15:55:15
    Committer: Jean-Philippe Andre <jp.andre@samsung.com>  2016-04-01
12:09:06
        evas : remove native.func.data variable and data argument of
native calblacks.

Change-Id: I9517c4c802f1fbb9799eb920a33359dcfacea7ec

evas : add ecore_evas_tbm_surface_get and ecore_evas_tbm_stride_get api for ecore_evas_tbm

Change-Id: Ifac62b4b20a5804d147a53731f06931be3891b0b

evas_gl: remove GL_VERSION wrapper

because some drivers return a context of later version not specified version.

Change-Id: I4fc4567ccdefd63f422812d1f7461d13e1e547b3

ecore_wl2: fix bug for sending configure event by client

The position(x,y) value of Ecore_Wl2_Event_Window_Configure event was wrong
while sending ECORE_WL2_EVENT_WINDOW_CONFIGURE event in _ecore_wl2_window_configure_send_by_client.

@tizen_fix

Change-Id: Ic377fd8846b42b07faa63d0cda0f285033570fbd

ecore_evas: support to rotate window which does not have shell surface

There was a bug that the window which doesn't create shell surface such as ime keyboard didn't rotate.
This patch fixes it.

@tizen_fix

Change-Id: I7f7304ae07f43e071ab8427ca606fecedf4fb846

evas: Remove unnecessary rotation when saving

Summary:
At some point this code stopped being necessary. The image data is already
rotated and width and height match it, so no need to rotate it again
(which produced warnings and incorrect behaviour)

Fixes T5841

Test Plan: This fixed examples/evas_images{2,4,5} which were failing before.

Reviewers: bu5hm4n, zmike, devilhorns

Reviewed By: zmike

Subscribers: cedric, #committers

Tags: #efl

Maniphest Tasks: T5841

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

elementary: Add missing EINA_UNUSED for unused function parameter

evas: handle event COW modification when COW is already active

Summary:
enabling write on a COW which is already writing is a user error,
which can occur during object invalidate due to reuse of a helper function.
by adding an extra param containing the COW data, this scenario can be avoided

fix T7005

Reviewers: bu5hm4n, segfaultxavi, devilhorns

Subscribers: cedric, #committers

Tags: #efl

Maniphest Tasks: T7005

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

eina_barrier: handle PTHREAD_BARRIER_SERIAL_THREAD return from barrier_wait

Summary:
this is a valid return code which indicates success and is passed randomly to
one of the callers

@fix
Depends on D6267

Reviewers: stefan_schmidt, ManMower, devilhorns

Reviewed By: devilhorns

Subscribers: cedric, #committers

Tags: #efl

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