elementary/perf:remove unnecessary elm_widget_is() check

As we always make sure the parent is a widget class we don't have to check anymore.

Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de>
Reviewed-by: Hermet Park <<hermetpark@gmail.com>>
Differential Revision: https://phab.enlightenment.org/D11985

elm_image: remove the spaces and keep the indentation for elm_image_file_set()

Summary: there are wrong indentation in the elm_image_file_set, so removed the tabs

Reviewers: Jaehyun_Cho

Reviewed By: Jaehyun_Cho

Subscribers: cedric, #reviewers, #committers

Tags: #efl

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

Change-Id: I797d0861fff02a8b49aacc748cec9c7238926441

elm_image: keep backword compatibility for elm_image_file_set API when setting url file set twice

Summary:
when trying to set file using url path twice, the second api call's return value is EINA_FALSE
since image obj has already has same file path. After applying Efl.File interface, the behavior is changed compared to before.
both of the return values should be EINA_TRUE.
@fix

ex)
Eina_Bool ret1, ret2;
ret1 = elm_image_file_set(image, "http://sameurl/image.jpg", NULL);
ret2 = elm_image_file_set(image, "http://sameurl/image.jpg", NULL);
ret1 and ret2 should be EINA_TURE

Test Plan:
1. call elm_image_file_set api with same url path
2. see the return value

Reviewers: Hermet, kimcinoo, jsuya

Reviewed By: Hermet

Subscribers: bu5hm4n, cedric, #reviewers, #committers

Tags: #efl

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

Change-Id: If64d8d58424b559ea07dba14efb36996484c2fd5

efl.spec: enable WebP Animation Feature

libwebp-1.1.0 library is included in the binary after tizen_unified_20200528.2
WebP Animation can be enabeld now.

Change-Id: I20b17049f7caca332a1af7ceb5c4e32e252bdaa5

modules: eina: chained_pool: check for NULL before passing it to deref

Make sure we check if pool->first_fill is NULL before passing it onwards
where it gets dereferenced. In case this happens return NULL instead of
garbage.

CID: 1293520

Reviewed-by: Christopher Michael <devilhorns@comcast.net>
Differential Revision: https://phab.enlightenment.org/D11980

Revert "ci: travis: disable the webp loader on CI builds for now"

This reverts commit 29c491b977543e9ef86147604ab3a297ece18327.

We went back to a much older version which should be available on all
our CI test systems. Safe to enable again.

evas_object_table: fix not to use hint_fill on elm_table

Summary:
The hint_fill logic in evas_object_table was designed to be used for
Efl.Ui.Table not for elm_table.
Since Efl.Ui.Table does not use evas_object_table logic, hint_fill logic
in evas_object_table is removed.

Reviewers: YOhoho

Reviewed By: YOhoho

Subscribers: cedric, #reviewers, #committers

Tags: #efl

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

Change-Id: Ide4fe092d9eeee4120596bae3dc3cb2eab2a1da5

efl : Fix build warnings.

Change-Id: I0777c22ad7784beee94646cf17420e41b0e798e8

Efl.Canvas.Vg.Object: Fix wrong render area

Summary:
When determining the size of the ector buffer,
it was used the smaller of the object's geometry or path_bounds.
However, because of that, path worked as absolute coordinates.
path_bounds should be relative to geometry of object.

Test Plan:
{F3900444}
original
{F3900445}

before
+ evas_object_geometry_set(vg, 100, 100, 200, 200);
{F3900447}

after
+ evas_object_geometry_set(vg, 100, 100, 200, 200);
{F3900448}

Reviewers: Hermet, kimcinoo, herb, smohanty

Reviewed By: Hermet

Subscribers: cedric, #reviewers, #committers

Tags: #efl

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

Change-Id: Id190aeb88735be0b9cbb5f60879c6a8a63eb33e7

doxygen: remove unsupported links to preview images

Change-Id: I936462469e6713268fb8a41e0d10f77b4c78a424

tests: edje: check result of evas_engine_info_set()

Make sure we check the result of evas_engine_info_set and only continue
if it succeeds.

CID: 1401077

Reviewed-by: Christopher Michael <devilhorns@comcast.net>
Differential Revision: https://phab.enlightenment.org/D11978

modules: gl_x11: ensure we check the result of  evas_eglMakeCurrent()

In case we fail make sure we error out to allow better debugging.
Also fix some surrounding indent.

CID: 1402693, 1402670

Reviewed-by: Christopher Michael <devilhorns@comcast.net>
Differential Revision: https://phab.enlightenment.org/D11976

elm_atspi_bridge: call _elm_win_atspi only once

Change-Id: I843c37646f60dfad985db66ee49838a68c239c25

efl_ui_win_socket: fix socket_proxy release

socket_proxy was created with efl_add method, so use efl_del
instead of efl_unref for object release.

Change-Id: I81a470f1acc8d803aa1146dd0d6b8a1663c797ec

elm_atspi_bridge: fix memory leak

Fix memory leak on error path.

Change-Id: I314f200ec20f2f0810d15ed1b6ae9fec1597e775

webp - 0.5.0 is actually new enough

webp anim apis came out in 0.5.0 not 1.1.0 :)

evas/engines/gl_generic: fix byte order after glReadPixels on BE

This fixes the screenshot tool in Enlightenment on big endian
systems besides other things.

meson_options: really fix the typo

Sigh, check your patch, even if it appears to be tiny.

meson_options: fix typo in option description

modules: vnc: allow building for FreeBSD with a different header path

Original patch provided by phyb user CeDeROM.

Fixes T8713

build: decrease the minimal needed meson verion to 0.50

When we jumped from 0.47 to 0.52 to fix problems some users have been
seeing when building we might jumped a bit to far. We want a good
balance between stable and fast building but also mature enough to be in
distro releases and widely available.

We will try to settle for a middle-ground with 0.50 here. If that causes
problems for you building please let us know instantly.

efl_ui_win: reuse bg which was created for version check

Summary: This is just for not creating bg object twice.

Reviewers: bu5hm4n, zmike, YOhoho

Reviewed By: YOhoho

Subscribers: YOhoho, cedric, #reviewers, #committers

Tags: #efl

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

Change-Id: I548a3f12a728d90212d47c110b2d1904b3d472b7

ui image: fix a missing else case.

it sholud be turned on enabled status.

it's side issue by 0826b4916efb0bce20a353cece56e84d05fb406c

Change-Id: I84b06594fda8694f865c74db4bdabfda505665c3

efl_ui_image: fixed elm_image_preload_disabled_set api is not working when it is called before file set

Summary:
when elm_image_preload_disabled_set(img, EINA_TRUE) is called before file set.
The image preload is not canceled, because the function will be returned since the image is not loaded.
so preload disabled api is fixed to work even if it is called before file set.
@fix

Reviewers: Hermet, kimcinoo, jsuya

Reviewed By: Hermet

Subscribers: cedric, #reviewers, #committers

Tags: #efl

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

Change-Id: I72e559314dbe494bbcf1dddf91f309ff8772780c

tests: elm: check return of efl_config_bool_set

Make sure we fail the test, unless this settign succeed.

CID: 1403891

Reviewed-by: Christopher Michael <devilhorns@comcast.net>
Differential Revision: https://phab.enlightenment.org/D11968

modules: only build gl_drm evas engine when using es-egl

This module does not work with desktop opengl contexts,
and needs EGL to work.

edje_cc_handlers: update documentation of "fixed" to be clearer

Summary:
Since "fixed" does not affect the group size by the part's min size, the
documentation is updated to be clearer.

Reviewers: woohyun, raster

Reviewed By: woohyun

Subscribers: cedric, #reviewers, #committers

Tags: #efl

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

elm_calendar: Do not pass NULL to function.

Summary:
T7076

legacy calendar inc/dec button has auto repeat feature.
for that if user click the button very quickly, we delete internal timer than it will pass to function.

to prevent this, need to check NULL pointer.

Reviewers: devilhorns, Hermet

Reviewed By: Hermet

Subscribers: cedric, #reviewers, #committers

Tags: #efl

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

ecore: use standard LC_ALL instead of __LC_ALL in systemd module

The former is a POSIX name, the latter is non-standard. I don't
know why the latter was used, considering glibc literally has
just #define LC_ALL __LC_ALL, but change it and unbreak build
on musl and other systems.

eina: only enable EINA_LOG_BACKTRACE when backtrace API is present

This prevents build breakage on platforms that either don't have
backtrace() or don't have the appropriate library for it installed.

evas glengine: fix code sync after migration.

Introduced a wrong result after upstream migration,
since we have tizen only code diff....

Change-Id: Ia4bac99a244a63a1d54c63dd93aea80c5a0bfce0

efl - build again - test suite break in using undefined ERR macro

tests: eina: make sure we always check on eina_matrixsparse_data_idx_get return

It has been checked in all other places, but has been forgotten in these
two.

CID: 1401081

Reviewed-by: Christopher Michael <devilhorns@comcast.net>
Differential Revision: https://phab.enlightenment.org/D11960

tests: eo: handle error return of eina_value_int_convert()

CID: 1400962

Reviewed-by: Christopher Michael <devilhorns@comcast.net>
Differential Revision: https://phab.enlightenment.org/D11958

tests: eina: also handle failure cases when cleaning up created files/folders

Somethign fishy is going on if we can not delete the files and folders
we created.

CID: 1400986

Reviewed-by: Christopher Michael <devilhorns@comcast.net>
Differential Revision: https://phab.enlightenment.org/D11955

ecore_drm: check return of drmModeSetCrtc()

Report if we fail to reset the Crtc during output free. Only print an
error and keep going with the rest of the output free.

CID: 1402668

Reviewed-by: Christopher Michael <devilhorns@comcast.net>
Differential Revision: https://phab.enlightenment.org/D11965

tests/ecore_wl2: Fix unchecked return value

Coverity reports that we do not check the return of eglInitialize
here.

Fixes Coverity CID1412362

tests/ecore_wl2: Fix unchecked return value

Coverity reports eglChooseConfig is called without checking return
value here. This patch fixes the issue by checking the return value
and by also making _init_egl return a bool so that we can fail the
test if eglChooseConfig results in failure.

Fixes Coverity CID1412365

ecore_wl2_tests: Fix formatting

tests: eina: check return of eina_module_load and unload

These could fail as well. Check in testsuite.

CID: 1400961

Reviewed-by: Christopher Michael <devilhorns@comcast.net>
Differential Revision: https://phab.enlightenment.org/D11959

tests: evas: check fd being non -1 before using close()

eina_file_mkstemp would return -1 in an error case. Make sure we check
the return when clsoing here.

CID: 1400790

Reviewed-by: Christopher Michael <devilhorns@comcast.net>
Differential Revision: https://phab.enlightenment.org/D11957

Use extern after EAPI

Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org>
Differential Revision: https://phab.enlightenment.org/D11961

evas stack: ++safe code.

null check to prevent any crashes (just in case)

Change-Id: Ic77a2c785feba9be32eb1460439f4c3315087e07

ui textpath: updated doc.

Change-Id: I562734a64b5ad8f371c1952a996a80f54a05e0c7

efl_ui_textpath: add the efl_ui_textpath_text_user_style_set API for applying the user text style

Summary:
For the text style customizing, added the new API to apply user text style.
The following example is the user style sample.

DEFAULT='font_size=16 color=#F00 underline=on underline_color=#00ffff'";

{F3899541}

Test Plan:
1. run elementary_text (textpath)
2. toggle user style check box
3. see the result

Reviewers: Hermet, kimcinoo, jsuya, bu5hm4n

Reviewed By: Hermet

Subscribers: cedric, #reviewers, #committers

Tags: #efl

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

Change-Id: Ia3a182aebadcad24383c3021e358d4e40e5e9515

mono: update window construction

they simply missed parameters here

Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org>
Differential Revision: https://phab.enlightenment.org/D11954

ci: travis: disable the webp loader on CI builds for now

With the bump of the needed libwebp version to 1.1.0 we have have our
Debian and Ubuntu builds failing with a to old version. Disable by
default in CI for now and check back later.

efreet - handle runtime relocation right with default XDG_DATA_DIRS

XDG_DATA_DIRS was only set up to a default including where efl was
installed prefix-wise as the compiled-=in prefix, not runtime
determined prefix. it shouldn't actually affect most people except
those making use of this.

@fix

eina - prefix - use smaller buffers for building strings

use less stack space but no features as buffers are big enough for all
content used or alloca'd now.

eina - abstratc content - smaller stack buffer for limited size string

no need for a 4k buffer when 128 bytes will be plenty - short string.

efl_ui_win: declare constructors not optional

they cannot be optional, or in later languages types will collide.

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

cxx: Fix uses of intrinsic eolian binbuf type

Add special binbuf and Eina_Strbuf conversions

Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de>
Differential Revision: https://phab.enlightenment.org/D11950

eo-files: convert all Eina.Strbuf & Eina.Binbuf

these structs should not be used, but rather the builtin types that
exist for it.

Reviewed-by: Felipe Magno de Almeida <felipe@expertisesolutions.com.br>
Differential Revision: https://phab.enlightenment.org/D11742

dotnet: Ignore Efl.Object.parent as constructor

In C# we already have Efl.Object.parent as an implicit constructor. Ignore it if it is marked as a constructor.

Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de>
Differential Revision: https://phab.enlightenment.org/D11951

eo: make the parent a optional constructor

this is needed in order to sanely bind parent calls to some objejcts,
but not all.

However, from the canvas structure up we *need* a parent, so we can
fetch the evas from it. So declare it there a none optional

Reviewed-by: Felipe Magno de Almeida <felipe@expertisesolutions.com.br>
Differential Revision: https://phab.enlightenment.org/D11734

Revert "ci: disable wepb loader until we have a change to update the mingw cross env"

This reverts commit 63b1334e2c98f0c0f08801ce77d66f372e7abb30.

No longer needed as we have updated the mingw image to have 1.1.0
available.

evas_vg_cache: Don't caching vg file if value_provider is applied.

Summary:
value_provider can change the value of the property received from VG file.
When a file is cached, the changed properties are applied to all other objects using the same file.
So. If value provider is applied, evas_vg_cache is not caching vg file.

Test Plan: N/A

Reviewers: Hermet, herb, kimcinoo

Reviewed By: Hermet

Subscribers: cedric, #reviewers, #committers

Tags: #efl

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

evas_common_font: release reallocated glyphs bitmaps data

Reviewers: woohyun, smohanty

Reviewed By: woohyun

Subscribers: cedric, #reviewers, #committers

Tags: #efl

Maniphest Tasks: T8743

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

doxygen docs: Fix API reference group to Ecore_Eo and eina_vector

Change-Id: Ic20831f0db719f935dbf4e7421c70f7c4cdbd5cb

elementary/example: Fix memory leak in gengrid test

Reviewers: Hermet, SanghyeonLee

Reviewed By: Hermet, SanghyeonLee

Subscribers: cedric, #reviewers, #committers

Tags: #efl

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

ecore_ipc: Fix memory leak

Summary:
As delete_me flag was not updated in the delete callback it dosen't
call ecore_ipc_client_del() on the client hence the leak.

Reviewers: Hermet, raster, bu5hm4n, CHAN

Reviewed By: Hermet, CHAN

Subscribers: CHAN, cedric, #reviewers, #committers

Tags: #efl

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

canvas gl: remove out-dated compiler keywords

exactness: record and print test time on success

Having the runtime of a test is interesting for debugging and
optimization. Printing them only on success for now.

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

ci: travis: add more fold and timing markers in some forgotten places

In teh initial patch we forgot some interesting places we also want to
fold and get timing info on. Dependency installation, cross native build
step, coverity tools download and codecov run.

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

ci: simplify ccache setup handling

We no longer need the special case for release ready. This was needed
for the way autotools setup directories, which we no longer use. (We are
not having a config.log anymore)

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

ci: disable wepb loader until we have a change to update the mingw cross env

We now need webp 1.1.0 and our current mingw cross env only has 1.0.2.
Disable to keep CI working while we update.

webp: add version check in configuration time

Reviewers: Hermet, herb, bu5hm4n, stefan_schmidt

Reviewed By: stefan_schmidt

Subscribers: stefan_schmidt, cedric, #reviewers, #committers

Tags: #efl

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

eldbus: fix the data size issue.

Summary:
dbus message requires the void* value for the message data,
the returned value of dbus_message_iter_get_basic() can be mismatched by to the architure,
This could bring the returned va_argues memory corrupted among the series of values.

We can use the defined type for exceptions.

Reviewers: herb, kimcinoo, jsuya, raster

Subscribers: cedric, #reviewers, #committers

Tags: #efl

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

evas gl - tune gl engine to use a bit less cpu overhead

we have used a fair bit of cpu to avoid gpu overhead and this tunes
the balance back a bit to throwing more at the gpu and less at the cpu
by reducing the number of pipes and max vertex counts per pipe by
default and only resetting pipe state vars if adding the first pipe
member.

@opt

Change-Id: Id102255c532ec7d116ca9a4d45f18ad69cde3829

ui scroller: fix broken scroller loop

image is streched since proxy fill/region size is mismatched

this is a regression bug introduced by 76b89b8e49f4b16ad490c04134a9bdddea27950f

Change-Id: I1eb4de61e6a5e080a2f350de30cb76d1db575429

ci: drop lua from default binding set as elua is off by default

build: disable elua by default, plus nicer detection

Elua is now disabled by default. There are some other changes:

1) Elua scripts are only installed if Elua is enabled
2) Lua bindings are only installed if Elua is enabled
3) Elua with interpreter is clearly experimental and will message

emotion: remove all left overs from removed backends

We removed them in the last release. Make sure we cleanup all the
references and dead code still belonging to it.

Extra note on the Emotion_Module enum removal. While it was exposed in
the public Emotion.h header it is never used anywhere in the code and
has no purpose, thus it can safely be removed.

Reviewed-by: Carsten Haitzler (Rasterman) <raster@rasterman.com>
Differential Revision: https://phab.enlightenment.org/D11941

build: do not publish surface .eo files

they are privat, they should never be installed.

Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org>
Differential Revision: https://phab.enlightenment.org/D11933

evas: correctly declare filter_internal as public API

I know this is internal. However it is public used in public .eo files.
Even if it is internal, and beta, we still need to expose the API in
order to have it correctly in the .so.

Reviewed-by: Stefan Schmidt <stefan@datenfreihafen.org>
Differential Revision: https://phab.enlightenment.org/D11932

edje: markup flag should be TRUE in legacy edje_object_part_text_set

Summary:
Through some APIs (elm_entry_editable_set, elm_entry_single_line_set, etc.)
located after edje_object_part_text_set(legacy)

If efl_ui_widget_theme_apply is called,
In edje_object_part_text_raw_generic_set, the legacy flag becomes FALSE.
And in this case, the logic works in the unintended direction
because the set_markup flag is FALSE.

Test Plan:
/*
gcc -o entry_example entry.c `pkg-config --cflags --libs elementary`
 */
#include <Elementary.h>

EAPI_MAIN int
elm_main(int argc EINA_UNUSED, char **argv EINA_UNUSED)
{
   Evas_Object *win, *en;

   elm_policy_set(ELM_POLICY_QUIT, ELM_POLICY_QUIT_LAST_WINDOW_CLOSED);

   win = elm_win_util_standard_add("entry-example", "test");
   elm_win_autodel_set(win, EINA_TRUE);
   en = elm_entry_add(win);
   evas_object_size_hint_weight_set(en, EVAS_HINT_EXPAND, EVAS_HINT_EXPAND);
   elm_object_part_text_set(en, "elm.guide", "<font_size=32 color=#FFF>TEST</font_size>");
   elm_entry_editable_set(en, EINA_FALSE);

   evas_object_show(en);

   elm_object_content_set(win, en);
   evas_object_resize(win, 300, 200);
   evas_object_show(win);

   elm_run();

   return 0;
}
ELM_MAIN()

Reviewers: woohyun, ali.alzyod

Subscribers: zmike, cedric, #reviewers, #committers

Tags: #efl

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

evas - genetic update regions smart merge - merge v adjacent regions

regions sometimes produce box sets like:
// +---+
// |   |
// +---+    +-------+
// |   |    |       |
// +---+    +-------+---------+
//          |                 |
//          |                 |
//          +-----------------+

so the upper-left 2 boxes can be merged into 1 and they have the same
x coords and are flush-aligned one above the other. that is what
this does - find these and merge them to have fewer rects

@fix

evas render - add debug to draw boxes around update regions

helps debug update regions to see if there is too much overdraw or too
many regions. i smelled too manhy and this showed it. i was right. fix
for too many regions coming next

tiler - add maxreg for newtiler - not used atm but matches current

atspi: return proxy object

Proxy objects were never returned by function because
they do not have x,y,w,h coordinates.

Change-Id: I237e63b6da3bd345f74e92c9e135bb9ff8d17599

elm_poup_eo.legacy.h: modify Elm_Popup_Group to Elm_Popup

Change-Id: I4a49df80c7a92ba388683cb7f96ecbb6666fdf24

need to upgrade meson. it looks like older ones have trouble

like adding endless: -lm -ldl -lm -ldl ....

elm test - animated icon - big win to have a bg and not garbage

build: move static out of none_static

Summary: these are static libs, they shall not be in the static list.

Reviewers: stefan_schmidt, jsuya

Reviewed By: stefan_schmidt, jsuya

Subscribers: cedric, #reviewers, #committers

Tags: #efl

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

Efl.Canvas.Vg.Object: Fix backup size

Summary:
Make the size to be backed up and the size to be compared equal.
and minor fix

Test Plan: N/A

Reviewers: Hermet, herb, kimcinoo

Reviewed By: Hermet

Subscribers: cedric, #reviewers, #committers

Tags: #efl

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

Efl.Gfx.Path: Modify boundary calculation.

Summary:
The point of the path is of type float.
The boundary must contain the coordinates of the point,
so min_x,y value must be round down and the max value round up.

Test Plan: N/A

Reviewers: Hermet, herb, kimcinoo

Reviewed By: Hermet

Subscribers: cedric, #reviewers, #committers

Tags: #efl

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

ci: log time in every fold start and end

Annotating the build log with timings will help us to understand better
where the time is spend during the builds and see areas to optimize.

Switching over to the "native" bash functions in Travis, so switching
all scripts to bash.

Signed-off-by: Stefan Schmidt <s.schmidt@samsung.com>
Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de>
Differential Revision: https://phab.enlightenment.org/D11913

ci: remove no longer used address sanitizer leak suppression file

We switched the asan build to ignore leaks for the internal build tools.

Signed-off-by: Stefan Schmidt <s.schmidt@samsung.com>
Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de>
Differential Revision: https://phab.enlightenment.org/D11922

ci: remove outdated README file

This is no longer up to date. Remove before it gives a wrong description
of things.

Signed-off-by: Stefan Schmidt <s.schmidt@samsung.com>
Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de>
Differential Revision: https://phab.enlightenment.org/D11921

ci: remove not in use script

We are not buildign the external examples. This script is no longer
used. We have an tiny internal app build test and building the in-tree
examples as well.

Signed-off-by: Stefan Schmidt <s.schmidt@samsung.com>
Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de>
Differential Revision: https://phab.enlightenment.org/D11920

Revert "elementary: build elementary_test additionally as shared object"

We now longer need this workaround. Exactness is now pre-loaded and we
can run it with any executable without this trick. Remove to simplify
our build.

This reverts commit 78ad088dd73a18fb4cfcfedcc18a6bf03e28080b.

Reviewed-by: Marcel Hollerbach <mail@marcel-hollerbach.de>
Differential Revision: https://phab.enlightenment.org/D11919

Efl.Canvas.Vg.Object: Prevent null pointer access

add null check

evas/engine: Fix memory leak in generic cache.

Summary:
During shutdown we used to call engine_image_free() which was causing some deadlock.
as we have evas_cache which takes care of freeing all the images we just have to
delete the generic cache without freeing the image during shutdown.

Reviewers: Hermet

Reviewed By: Hermet

Subscribers: cedric, #reviewers, #committers

Tags: #efl

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

Efl.Canvas.Vg.Object: Optimize Ector Surface Size

Summary:
The ector surface size was determined by the size of the vg object.
vg object is usually sized by the size of the container.
So, the ector surface is set unnecessarily large.
This patch sets the ector surface size to the path boundary.
And the path boundary refers to the stroke width and miterlimit.

Test Plan:
vector sample
{F3887634}
{F3887632}

[grey area is ector surface size]
{F3887633}

Reviewers: Hermet, kimcinoo, smohanty, herb

Reviewed By: Hermet

Subscribers: cedric, #reviewers, #committers

Tags: #efl

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

ecore: Fix Asan stack overflow warning

Summary:
As argument can be passed by register (depending on the compiler optimization)
when we take the adress of the nbytes and pass it to send() function which reades
4 bytes from it ASAN flags it as a stack overflow . So just assign the value to a
local variable to avoid the warning.

Reviewers: Hermet

Reviewed By: Hermet

Subscribers: cedric, #reviewers, #committers

Tags: #efl

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

build: add gl_common files to engine_src

@tizen_fix

Change-Id: I06eb654aed79e19c3e945e586ea87dd44e1c0a78

elementary_test: Added --autoclose option to elementary_test

Summary:
This option will enable writing tools which will automate running the
elementary test cases.

Reviewers: Hermet

Subscribers: cedric, #reviewers, #committers

Tags: #efl

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

Change-Id: I9c11fdaf44141e4610e63de363779021f382c251

elementary: Fix memory leak in elm_win

Change-Id: I1003d7e7abc31c0425e8c12553a20c77193cff01

elm_gengrid: resolve double free

Change-Id: I17789de33c088c6baeced2ea5758e6a34c37902c

elm_atspi: resolve migration build error

Change-Id: I175602b125bcfb1101011b37067b6b3cd1397576