elm_entry: use Efl.File in Elm_Entry

Summary:
Also using the second optional string parameter of efl_file_set as the
text format specifier.
Added string constants to represent the possible values.

Address the issue:
- Elm entry and the file interface clash on file_set.
Specified in the page:
https://phab.enlightenment.org/w/efl_interfaces/

Reviewers: felipealmeida, tasn

Reviewed By: tasn

Subscribers: thiepha, raster

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

win: Add autohide and ELM_POLICY_QUIT_LAST_WINDOW_HIDDEN

Summary:
Add the "autohide" property to elm_win. This property, when set to
EINA_TRUE, automatically hides the window upon a "delete,request" signal.

Create ELM_POLICY_QUIT_LAST_WINDOW_HIDDEN, a new quit policy that
automatically exit from the elm_run loop when all windows are hidden.
It is an alternative to autodel to conciliates the memory management
framework of Eo with any other memory management model the program may
be using (e.g. RAII principles of C++).

Create the auxiliary function "_elm_win_policy_quit_triggered" to check
triggering of quit policies.
The check in "smart_hide" is now necessary, since
ELM_POLICY_QUIT_LAST_WINDOW_HIDDEN can be triggered when a window
is hidden.

Create the auxiliary function "_elm_win_flush_cache_and_exit" to avoid
code repetition for exiting the elm_run loop.

Make a small update on documentation in elm_win.h to mention the new
autohide property.

@feature

Reviewers: raster, felipealmeida, cedric, tasn

Reviewed By: felipealmeida

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

genlist: broadcast atspi expanded state changes

layout: Fix eo API error while running "Genlist 7" in elementary_test.

In case of naviframe, resize_obj is not edje.

Steps to reproduce:
1. elementary_test
2. Open "Genlist 7" sample.
3. Switch focus between windows ("elementary_test" <---> "Genlist 7").

Eo Errors:
ERR<8720>:eo lib/eo/eo.c:678 _eo_call_resolve() in lib/edje/edje_object.eo.c:430: func 'edje_obj_message_signal_process' (733) could not be resolved for class 'Elm_Layout'.

@fix
Resolves: T2534

test_focus: Focus should to move layout's entry

Summary:
While going through the test_focus(focus 2-> give focus to layout "entry") code found
focus should move to layout's entry instead of entry outside the layout.
Just change the entry object in callback ;)

Test Plan: elementry_test-->focus2-->give focus to layout "entry".

Reviewers: cedric, Hermet

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

Elm_Interface_Scrollable: Improvement in looping behavior

Summary:
x and y position variable assumes values ranging from min to max.
cw, the content width is larger than max in x axis. Beacause of this
the scroller is not able to loop in rigth/bottom direction. For looping
comparision min and max values should be used instead of content width.

@fix

Signed-off-by: Umesh Tanwar <umesh.tanwar@samsung.com>
Reviewers: raster, seoz, cedric, Hermet

Reviewed By: Hermet

Subscribers: eagleeye, SanghyeonLee, singh.amitesh, sachin.dev

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

elm_datetime: Fix datetime ctxpopup resize issue

Summary:
In elm_datetime, ctxpopup does not resize

@fix

Test Plan:
1) Change the scale value in elementary_config to 2.0 (so that a better size change in the ctxpopup's can be observed)
2) Open elementary_test and then the datetime
3) Click on the first datetime field and observe the ctxpopup's arrow position and then close it
4) Click on the same datetime field again, now observe that the ctxpopup's arrow has concided with the diskselector as it wasn't resized properly.

Reviewers: raster, Hermet, shilpasingh, subodh6129, seoz

Reviewed By: shilpasingh, subodh6129

Subscribers: rajeshps, govi, poornima.srinivasan

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

multibuttonentry: Delete item size min set when first box resize time.

Summary:
The mbe items has to get own min size to calc each items locate, box size etc...
but when first time box resize it will ignored.
It makes item wrong size issue when mbe located at some part of layout in runtime.

Reviewers: Hermet, seoz, woohyun

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

atspi: enable atspi also in quicklaunch infrastructure

Genlist: use correct variable during realize

VIEW(it) is not set when the function is called. view should be used
instead.

@fix

atspi: fix missing Window signals on AT-SPI2 initialization

Patch adds connected/disconnected elm_atspi_bridge events to avoid
races between window getting focus and AT-SPI2 bus initialization.
When  bridge gets connected windows will reemit all "Created",
"Activated" and "Deactivated" events with regard to window focus.

Set file permissions back to 644

Few files permissions have been modified by mistake in the past patches.

atspi: add org.a11y.atspi.Application interface to root object

atspi: add missing header

atspi: refactor atspi_bridge to eo object

This will allow to reuse eo events infrastructure
in next patches.

elm_flip: porting evas smart callbacks to eo

elm_photo: porting evas smart callbacks to eo

elm_dayselector: porting evas smart callbacks to eo

elc_player: porting evas smart callbacks to eo

elc_popup: porting evas smart callbacks to eo

elm_box: porting evas smart callbacks to eo

Replacing  calls of evas_object_smart_callback_add/del/call
with eo_callback_add/del/call.
This will reduce the use of legacy events and callbacks lists in
smart_object which in turn will improve performance and reduce memory
use.

slider: call elm_widget_theme_get instead of elm_object_theme_get.

This reduces one function call and make code more consistent
between widgets.

Revert "theme: return false if elm_object_style_set() failed to set requested style."

This reverts commit 76004dfbec84664e253babc5bf576398a5901395.

We need to change other code also to accommodate this change.
_elm_theme_set should return an enum which tells what failed.
enum {
THEME_APPLY_FAILED,
THEME_DEFAULT_SUCCESS.
THEME_APPLY_SUCCESS
};

Based on that, we decide what needs to be done.
The above code will break the layout theme if incorrect theme are passed.
It should be backported to Elm 1.14.

@fix

Wayland cnp: Fix const qualifier discarding following eolian const fixes.

Replace C names by Eolian namespaced names in .eo files

Reviewers: tasn, felipealmeida, raster, cedric

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

elm_config: Add feature to control transition scale for edje

Summary:
If the transition scale flag is enable in edje,
this value affect the transition scale for edje.
@feature

Test Plan: test elementary_config with enventor tool

Reviewers: raster, Hermet

Subscribers: raster

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

test_focus_policy: Enhance object focus policy test.

- Add more descriptions in the test.
- Set the focus to the first button automatically.

layout: Add more description for layout's internal smart data members.

elm: Add missing const for getters.

interface_scrollable: Add more description to animators.

It is surprising that there is no such a comment on a scrollerable
interface considering the complexity of scroller.

elm: Remove shadow variable declarations.

edje_externals: Fix wrong free.

_signal is used after free. _signal will be freed soon any way.
This was introduced in 3da3082.

This fixes coverity CID 1308263.

test_win_keygrab: Fix buffer not null terminated error.

This fixes coverity CID 1308264.

elementary: Add rudimentary support for Elm Wayland apps to set border icon

Summary: This is a patch to fix T2453 where elm apps in weston/E did
not display the icon on the border. This is just preliminary support
because it only tries to find the efreet desktop file based on
_elm_appname. This support Could be extended to allow efreet to search
based on window title, etc, etc.

@fix

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

elm_interface_scrollable: fix wrong mirrored calculation

Summary:
This Patch is regarding D2553, but funtionally independent patch.
D2557 is also related with this patch, so to test working fine,
need to install those all three patches.

1. _elm_interface_scrollable_content_region_show store wx
   into scroll_interface_data without coverting mirrored_x.
   fix to store wx to mirror-converted x if is_mirrored.

2. _elm_scroll_x_mirrored_get return mirroed_x after compare zero not minx.
   so current mirrored_x couldn't be less then 0.
   fix to check min not 0 for set return value.

3.  _elm_scroll_x_mirrored_get calculation need to consider start position of scroller.
some scroller(e.g. gengrid) start from non-zero position, so calculation need to change
like below.
ret = (cw - (x + w - min));

@fix

Test Plan: You can test this after merge D2553 patch and D2557 and test gengrid2.

Reviewers: raster, Hermet, seoz, jaehwan, tanwar.umesh07, cedric

Subscribers: tanwar.umesh07, Jaehyun, anand.km, eagleeye, singh.amitesh

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

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

elm_gengrid: fix an overlap issue of item and group item on focus.

Summary: Issue: Gengrid item comes over group item on click.

Test Plan:
Run elementary test.
Open GenGrid Group. Click on any item which slightly overlap with group item.
Item comes over group item.

Reviewers: raster, cedric

Reviewed By: cedric

Subscribers: rajeshps, govi, poornima.srinivasan, shilpasingh

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

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

elm_gengrid: fix the bug that gengrid item index(position) is updated wrong value in item_update

Summary:
elm_gengrid_item_update perform item_unrealize/item_realize/item_place itself.
But when gengrid items are added and not calculated yet,
caller want to get item's index after execute elm_gengrid_item_update,
return value is wrong because item_place update it's position in not-updated item's x and y.

to prevent this,
if item position is already updated,
do not updates position in item place.

additionally,
I fixed gengrid item prepend wrong posiiton update.
gengrid item position start from 1 not 0, so item_position_update also must be started from 1.

@fix

Test Plan:
To test this scenario,
need to fix src/bin/test_gengrid.c first.

1. add below codes in bottom of _after_bt_clicked(line 703).

```
      Elm_Object_Item *it = elm_gengrid_item_next_get(id->item);
      printf("before update efl item[%p], index [%d]\n",id->item,elm_gengrid_item_index_get(it));
      elm_gengrid_item_update(it);
      printf("after update efl item[%p], index [%d]\n",id->item,elm_gengrid_item_index_get(it));
```

2. execute elementary_test and go to gengrid2 test.

3. add 3 items and select 2nd item then add new item by insert after.

4. you can see index is changed wrong value after item_update executed.

after add fallback code in item_update,
index is returned same value even after item_update execusion.

Reviewers: raster, Hermet, seoz, jaehwan

Subscribers: anand.km, eagleeye, singh.amitesh

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

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

elm_list & elm_genlist & elm_gengrid: fix the behavior of using the mouse.

Summary:
Elementary widgets have different behavior of using the mouse, they need to be unified.
- swipe : without checking on hold, just checking drag state.
- longpress : after longpress, _item_unhighlight(), _item_unselect() are called.
- select : when the mouse pointer leaves item area, _item_unhighlight(), _item_unselect() are called.

First, apply this commit https://phab.enlightenment.org/D2550

Test Plan: elementary_test -> list

Reviewers: seoz, Hermet, CHAN, woohyun, kimcinoo, jaehwan, SanghyeonLee, cedric

Reviewed By: cedric

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

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

gengrid: add unhighlight callback

Summary:
item_unselect check it->select status, so when item highlighted but not selected,
item highlight is remaining even after unselect_cb is called.

And item_select and item_highlight are separated, but item_unselect and item_unhighlight are mixed.
so divide them, too.

This patch will solve upon problem.

Reviewers: seoz, woohyun, Hermet, CHAN, raster, SanghyeonLee, cedric

Reviewed By: cedric

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

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

gengrid: add item cache in gengrid for efficient realize/unrealize operation.

Summary:
gengrid realize/unrealize items each smart calculation if their posiiton is updated.
This concept will be occurred useless view object creation and deletion.
So instead of delete items in unrealize function, push item cache into cache list,
and pop item cache from cache list in realize function to reuse item view object
more efficiently.
the cache concept is already implemented in genlist widget and this patch is improved
item cache of genlist cache.
@feature

```
NOTICE: This patch is written under the D2561 Patch which add elm_gengrid_item_fields_update.
        So it must submitted after D2561.
```

Test Plan:
I've tested below three cases,
   1. tested ith this patch in elementary_test and checked all gengrid features are working properly.
   2. tested with changing CACHE_MAX to another value which is set 40 in default
       and checked all gengrid features are working properly.
   3. tested with set it->item->nocache true, and checked cache is never generated
       and checked all gengrid features are working properly.

Reviewers: raster, Hermet, seoz, jaehwan, singh.amitesh

Subscribers: eagleeye, singh.amitesh, Jaehyun

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

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

elm_spinner: add feature in spinner to change values while dragging relative to the speed of dragging

Summary:
Currently the values in spinner change while dragging only based on
the amount of pixels dragged, this patch will enable Spinner to change value
based on the speed of dragging so that it will be more user friendly. This
will help users to alter drag values by big amounts if dragged in a good speed.

Test Plan: test_spinner.c in elementary_test

Reviewers: raster, prince.dubey, shilpasingh, cedric

Reviewed By: cedric

Subscribers: poornima.srinivasan, rajeshps, govi

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

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

genlist/list/win: Revert focus highlight animation when looping is enable

Summary:
  -
  Revert "focus: Added internal widget APIs of focus highlight object of elm window."

    This reverts commit 3f98d71830163a154762f2d16301d720c781b7c5.

  - Revert "List: Focus highlight when loop is enable"
This reverts commit 57ad32a900e71f83441bf7246671b8f16a8ec909.

  - Revert "genlist: Focus highlight when loop is enable"

    This reverts commit 91282a22cba66ee5e7799018d3abe8957fb1b1fa.

Reviewers: seoz, raster, jpeg, SanghyeonLee

Subscribers: anand.km, seoz

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

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

calendar: show days of prev/next month.

Summary:
Show days of previous and next month on the blank area of calendar.
@feature

Test Plan: See calendar widget on elementary_test.

Reviewers: raster, seoz, woohyun, cedric

Reviewed By: cedric

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

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

elm_colorselector: pixel-picker accuracy fix

Summary:
Looking at the base array of pixels,
the real middle of taken picture
is actually number 8 from array of 17 pixels (which starts from 0).

@fix

Reviewers: reutskiy.v.v, raster, NikaWhite, cedric

Reviewed By: NikaWhite, cedric

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

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

focus: fix typo in API Documentation.

Summary: Signed-off-by: Vaibhav Gupta <g.vaibhav1@samsung.com>

Reviewers: raster, Hermet, cedric

Reviewed By: cedric

Subscribers: sachin.dev, singh.amitesh

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

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

elm: constify Eolian methods correctly because of recent Eolian changes

cnp: remove loss callback when object is deleted

Summary:
SEG_FAULT happens when the object which has selection is deleted,
and new selection is done at another object.
Reason: loss_cb is not removed when the object which has selection is deleted.
When new selection is set for new object, the loss_cb is called for deleted
object. As result, SEG_FAULT happens.

Test Plan:
Run elementary test, open Entry, do selection,
close Entry window, open Entry one more time, do selection.

Reviewers: JackDanielZ

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

config: read flush.cfg when elm_config_all_flush is called.

When elm_config_all_flush is called, another processes should get
the config of current process instead of saved base.cfg.
so current process save the config to flush.cfg and another processes
read it when it is changed.

@fix

Scroller : Remove white-spaces in header file.

Summary: Signed-off-by: Umesh Tanwar <umesh.tanwar@samsung.com>

Reviewers: cedric, raster, seoz, Hermet

Subscribers: sachin.dev, singh.amitesh

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

Scroller: Remove white-spaces in scroller.edc file

Summary: Signed-off-by: Umesh Tanwar <umesh.tanwar@samsung.com>

Reviewers: cedric, raster, seoz

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

Genlist: Test case added in Genlist Focus example.

Summary:
Added test case to enable/disable mouse wheel. Mouse Wheel is
used to scroll the genlist up/down.

Signed-off-by: Umesh Tanwar <umesh.tanwar@samsung.com>
Test Plan: elementary_test -> Genlist Focus -> Wheel Disabled

Reviewers: raster, Hermet, cedric

Subscribers: sachin.dev, singh.amitesh, SanghyeonLee, eagleeye

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

X11/CnP: remove X11 invocations from elm_entry

Code specific to X11 should not be part of elm_entry and can be easily
replaced with elm_cnp APIs.

@fix T2183

CnP: improve loss callback

- Reset loss cb after invocation as it is no more needed
- When selection is set, previous owner loss cb is invoked only
if the new owner and the previous one are different.

config: fix the flush file path.

@fix

DnD: remove text/uri handling

This type is not a standard type and can be handled as a text/uri-list
of one uri.

@fix

DnD: remove from elm_cnp code specific to elm_entry

The tag insertion has to be done only from elm_entry callback. elm_cnp
is not supposed to know what will be done with the data.

Tests/DnD: fix uri parsing

When no \n is present at the end of the uris list, the function doesn't
return the uri.

widget: Add a case that elm_layout is an item view object.

Summary:
The internal API "_elm_widget_item_highlight_in_theme()" look like for list item highlight.
but it can use in another widget which has item view as elm_layout.
In that case, ERR log will be printed.

Reviewers: singh.amitesh, Hermet, woohyun

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

list: --printf

examples: fix changes in Evas.Canvas3D namespace.

elm_genlist: code refractoring of _item_process

Summary:
When item_process is success, then we want to call _item_process_post.
Old code was confusing before.

Reviewers: raster, Hermet, SanghyeonLee

Subscribers: seoz, sachin.dev

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

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

elm_genlist: reset timer to NULL on cancel only.

Summary: @fix

Reviewers: raster, seoz, SanghyeonLee

Subscribers: sachin.dev, seoz

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

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

elm_scroller: add wheel_disable_set API

Summary:
This API can be used by widgets like Genlist/Gengrid, which implements
Elm_Interface_Scrollable interface.Mouse wheel is used to scroll up and
down on widget.

@feature

Test Plan: elementary_test -> Scroller -> Wheel Disable

Reviewers: raster, Hermet, cedric, SanghyeonLee, singh.amitesh

Subscribers: JackDanielZ, herdsman, cedric, sachin.dev

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

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

elm_colorselector: fix horiz/vertical padding to be affected by base_scale in edc

Summary:
elm_colorselector get its padding size from data in edc.
However, when calculating the padding size, it does not consider
base_scale of the style from where it gets padding size.
In this commit, the padding size from edc is divided by its
base_scale so that desirable size can be calculated.

@fix

Reviewers: Hermet, woohyun

Subscribers: woohyun, Hermet

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

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

elm_genlist: fix highlighted items overlap with group item during scrolling and focus when intersect.

Summary: Highlighted genlist item overlap with group item during scrolling and focus when itersect.

Test Plan:
Run elementary_test.
   open Genlist Group.
   select one item and try to cross the group item via scrolling.
   Highlighted item comes over group item.

Reviewers: Hermet, raster, cedric

Reviewed By: cedric

Subscribers: rajeshps, govi, poornima.srinivasan, shilpasingh

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

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

genlist: fix Issue "In Tree effect feature, items floates when number of item crosses item block size".

Summary: "In Tree effect feature, during expansion of tree, when total number of items crosses block count, items start floating".

Test Plan: Run elementary_test, open Genlist Tree, keep expanding items untill it crosses number of count 32 (Block count). Resize the window, floting items can be seen.

Reviewers: raster, cedric

Subscribers: shilpasingh, poornima.srinivasan, govi, rajeshps

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

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

elm_entry: modify to not allocate Eina_Rectangle

Summary: modify to not allocate Eina_Rectangle and use it directly on the stack with EINA_RECTANGLE_SET

Reviewers: raster, woohyun, cedric

Reviewed By: cedric

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

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

genlist: fix resize of items when added after elm_genlist_clear().

Summary:
This fixes following issue.

1. Add genlist items
2. Clear genlist by elm_genlist_clear()
3. Append items.
   Genlist items are shrinked. This patch fixes that issue.

@fix
Partially resolves: T2367

Reviewers: Hermet, raster, cedric, raoulh, SanghyeonLee

Subscribers: sachin.dev, seoz

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

update po's

DnD: fix usage of TARGETS

- TARGETS notifier is specific to CnP and should not be called for DnD
- On DnD (when drag starts) and on CnP, it doesn't make sense that the TARGETS
type are sent to the potential target. Only basic types should be sent.

elm confiug - remove now useless x11 property handling - donr by files

follow on from b1c5de0b9aa67b9d5cc4722e9462830beb3af37b,
aa7f859da10ce007a7b945a8703e70161c96146d

colorselector: fix missing style warning

Test Plan:
1. open "elementary_test colorselector"
2. click "all" button

Reviewers: cedric, seoz, Hermet, raster

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

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

gengrid: remove duplicated focus state recovering.

Summary:
gengrid focus state recovering in item_realize is duplicated.
remove unnecessary lines.

you can find same signal emit codes under line 1018.

Test Plan: test elemetary test gengrid focus highight show correctly after code merged.

Reviewers: raster, Hermet, seoz, jaehwan, cedric

Reviewed By: cedric

Subscribers: singh.amitesh, eagleeye

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

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

colorselector: add safety check for NULL pointer from ecore_x_image_new().

Summary:
ecore_x_image_new() API can return NULL.
@fix

Reviewers: raster, woohyun, Hermet, cedric

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

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

notify: fix prevent issue, dereference null return value

Summary: Fix prevent issue: Dereference null return value

Test Plan: Use static analysis tool such as prevent

Reviewers: raster, woohyun, jaehwan, Hermet

Subscribers: seoz

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

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

dayselector : add safety check for NULL pointer from _item_find()

Summary:
It could make crash when _item_find() is failed.
CID - 404129
CID - 402467
CID - 398714
CID - 390012

@fix

Reviewers: raster, thiepha, woohyun, Hermet

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

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

elm_icon_external: cleanup ERR msg in error case

Summary:
Setting the group without checking it exists causes a errormessage, this
is pretty annoying for people which want to set a standart icon. And
give a better errormessage if the standart setting failed.

Reviewers: raster, cedric

Reviewed By: cedric

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

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

atspi_bridge: fix memory leak issue

Summary:
Call eldbus_message_unref to free Eldbus_Message structure.
And delete unused Eldbus_Message variable.
CID-386474
CID-391869
CID-394549

Reviewers: raster, woohyun, CHAN, Hermet, seoz, lukasz.stanislawski, cedric

Reviewed By: cedric

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

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

test_gengrid: remove EINA_UNUSED for a used argument

Summary: Signed-off-by: Vaibhav Gupta <g.vaibhav1@samsung.com>

Reviewers: raster, Hermet, cedric

Reviewed By: cedric

Subscribers: sachin.dev, singh.amitesh

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

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

elm_gen_common: fix typo in Struct variable documentation

Reviewers: Hermet, cedric

Reviewed By: cedric

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

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

edje_externals: add safety check for NULL pointer

Summary: Dereferencing null pointers in external_signal()

Reviewers: bdilly, raster, woohyun, cedric

Subscribers: cedric

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

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

app_server: add null check for dbus_state_set/elm_app_server_views_eet_new

Reviewers: Hermet, woohyun, raster, cedric

Reviewed By: cedric

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

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

elm_image: Solve clash on the meaning of "animated"

Summary:
Both Elm_Image and its base interface Efl.Image have a property named
"animated".
The meaning for Efl.Image is whether or not the image has animation.
On the other hand, Elm_Image use it to set whether an image, which supports
animation, is to animate itself or not.
Thus Elm_Image have another property (animated_available) to tell whether the
image have animation or not, making it even more confusing.

Renamed the "animated" property of Elm_Image to "animated_enable" to make
the intended meaning clearer.
Also replaced the "animated_available" property with the "animated" property
from the Efl.Image interface, since both refers to the same thing, in order
to reduce the API.

Added implementations of the legacy methods for backward compatibility.

Address the issue:
- Elm image and the image interface clash because of the meaning of "animated".
Specified in the page:
https://phab.enlightenment.org/w/efl_interfaces/

Reviewers: felipealmeida, tasn

Subscribers: raster

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

config: fix comment

config: make a config directory, if it is not.

create the config directory to register flush file to eio_monitor.

fix more elm config file pointer access

again - follow up from changes to config code to fix seg

elm config files - don't write to file if file handle is null

fixes seg with new file only config for elm

win: fix bug about engine value.

When application does not have any value of engine,
this app should be launched with elm_config's value.

Signed-Off-By: Wonsik Jung <sidein@samsung.com>

win: do not use deferred ecore evas deletion for FAKE wins

these ees will be deleted externally anyway

@fix

win: only trigger del trap if win type is not FAKE

@fix

theme: migrate all e users of darken_rounded_square to new darken_square image

nearly everything in e is square except for this, which was a holdover from the
b&w theme. the new image looks more coherent with the rest of the square look

image created by me, licensing to match rest of theme images

@feature

notification: set fixed on app icon to eliminate edje calc error

entry: emit changed,user signal when text is pasted or dropped

Summary:
When text is pasted or dropped into entry, the "changed,user" signal
should be emitted, not "changed" signal.
Currently, the "changed" signal is emitted.
This patch fixes this issue.

@fix

Reviewers: herdsman, tasn

Subscribers: tasn

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

elm_config: remove X dependency.

Summary:
In wayland, elm_config_all_flush doesn't work.
It has X dependency totally.
It uses ecore_file_monitor instead of X property.

Reviewers: Hermet, devilhorns, raster, cedric

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

comp effects: add tooltip visibility effect

comp effects: increase size of vertical visibility effect to include shadows

elm_flip: small error handle fix

if we cannot create slice, we had better stop to change slice

index: Add NULL check for malloc memory allocation.

layout: Add NULL check for calloc memory allocation.

layout: Fix to pass correct argument for edje_object_part_unswallow.