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.

layout: Add NULL check for ELM_NEW memory allocation.

clock: Add safety check for NULL pointer from localtime() func.

Summary:
localtime() can return NULL when there is an error.
@fix

Reviewers: raster, woohyun, Hermet, JackDanielZ

Subscribers: JackDanielZ

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

cnp: Add safety check for NULL pointer from malloc().

Summary:
It could make crash when malloc is failed.
@fix

Reviewers: raster, woohyun, Hermet, JackDanielZ, thiepha

Subscribers: JackDanielZ

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

cnp: Fix memory leak issue.

Summary: Call efreet_uri_free() API to free Efreet_Uri structure.

Test Plan: None.

Reviewers: raster, cedric, woohyun, Hermet, thiepha, JackDanielZ

Subscribers: JackDanielZ

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

calendar: + null check.

clendar: + null check

gmtime can return NULL vaule.
if so strftime will cause crash.

this patch just prevent that potential situation.

elm_interface_scrollable : Fix logic about setting EVAS_EVENT_FLAG_ON_HOLD

Summary: Even if scroller is not scrolled, EVAS_EVENT_FLAG_ON_HOLD is set. So clicked event is canceled in Edje.

Test Plan:
On the button in the vertical scroller, click button and move mouse horizontal.
Mouse up on the button, so now button is not clicked

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

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

DnD: fix memory leak

DnD/X11: Fix invalid access to memory

It seems X doesn't send the data with the termination.

DnD/X11: rename notifiers

As the notify functions don't notify anymore, they have to be renamed to
reflect their real job, i.e data preparers before calling user
functions.

DnD/X11: refactor the notifiers

Each data notifier was preparing data and then invoking the user callbacks
for DnD or selection protocols.
Most of the notifiers code was handling the user callbacks invocation
and was the same for all of them.

This patch simplifies the notifiers by moving the user callbacks
invocation into a common place for all the types, i.e into the notify
dispatcher.
The notifiers prototype has been changed to suit the code modifications.

DnD/X11: remove unused function

fileselector: add NULL check for memory allocation

entry: add dnd image handling

When image is dropped into entry, we should insert image to it.
This patch adds image inserting in dropping.

@fix

Cnp/X11: correct object parameter in datacb

The object parameter in datacb must be the selection request object,
not the selection owner object.
This patch corrects that.

Test case: open two windows with entry, copy text from one entry in
first window, paste to entry in other window.

@fix

Cnp/X11: move entry-only code in elm_cnp to entry

There is code in elm_cnp.c which is used to paste data to entry widget only.
This can cause error if widget is not entry.
This patch removes that code in elm_cnp.c and adds datacb to entry
to insert content to entry.

@fix

Dnd/X11: add dnd support for image, uri types

There is no dnd callbacks for image, uri types.
This patch adds dnd callbacks for those types.

@fix

cnp: Add safety check for NULL pointer from strdup().

Summary:
It could make crash when strdup() is failed.
CID - 329115
@fix

Reviewers: raster, thiepha, woohyun, Hermet

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

cnp: Add safety check for NULL pointer from calloc() for anim_icon.

Summary:
It could be failed to allocate memory for Anim_Icon.
CID - 337351
@fix

Reviewers: raster, thiepha, woohyun, Hermet

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

cnp: Add safety check for NULL pointer from _elm_util_text_to_mkup().

Summary:
_elm_util_text_to_mkup() func could return NULL when malloc is failed internally.
CID - 398441
@fix

Reviewers: raster, thiepha, woohyun, Hermet

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

elm_win: fix keygrab bug to use wrong keygrab mode

The code set worng ecore_x_keygrab mode according keygrab mode.
and changed some codes for readablity.

eo: migrate all event docs to new syntax

comp effects: add vertical visibility effect

eo: migrate enum docs to new format

widget: Add null check

Summary: Adding NULL checking when allocate memory.

Reviewers: Hermet, seoz, cedric

Reviewed By: cedric

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

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

transit: add NULL check for memory allocation.

Summary: Adding Null checking when allocate memory.

Reviewers: Hermet, seoz

Subscribers: cedric

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

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

Updating italian translation

list: add atspi SELECTABLE state to elm_list items.

[layout] support mirrored set for layout which is using elm_layout_file_set()

Summary: mirroed mode does not work, if layout uses elm_layout_file_set().

Test Plan:
the following is test code.
[test.edc]
collections {

   group {
      name: "layout/test";

      parts {

         part {
            name: "bg";
            type: RECT;
               description {
               state: "default" 0.0;
               color: 255 255 0 100;
            }
         }

         part {
            name: "test.rect";
            type: RECT;
               description {
               state: "default" 0.0;
               color: 255 0 0 255;
               rel1.to: bg;
               rel1.relative: 0.2 0.1;
               rel2.to: bg;
               rel2.relative: 0.5 0.2;
            }
         }

      } /* parts */
   } /* group */
} /* collections */

[test.c]
//Compile with:
//gcc -g test.c -o test `pkg-config --cflags --libs elementary`

#include <Elementary.h>
#include <Ecore_X.h>

static void
_bt_click(void *data, Evas_Object *obj, void *event_info)
{
   Eina_Bool mirrored;
   Evas_Object *layout;

   layout = data;

   mirrored = elm_config_mirrored_get();
   mirrored = !mirrored;
   printf("mirred: %d\n", mirrored);
   elm_config_mirrored_set(mirrored);
}

EAPI_MAIN int
elm_main(int argc, char **argv)
{
   Evas_Object *win, *box, *layout, *bt, *check;
   char buf[PATH_MAX];

   elm_app_info_set(elm_main, "elementary", "./test.edj");
   elm_policy_set(ELM_POLICY_QUIT, ELM_POLICY_QUIT_LAST_WINDOW_CLOSED);

   win = elm_win_add(NULL, "Layout", ELM_WIN_BASIC);
   elm_win_autodel_set(win, EINA_TRUE);

   box = elm_box_add(win);
   evas_object_size_hint_weight_set(box, EVAS_HINT_EXPAND, EVAS_HINT_EXPAND);
   elm_win_resize_object_add(win, box);
   evas_object_show(box);

   // Adding layout and filling it with widgets
   layout = elm_layout_add(win);
   evas_object_size_hint_weight_set(layout, EVAS_HINT_EXPAND, EVAS_HINT_EXPAND);
   evas_object_size_hint_align_set(layout, EVAS_HINT_FILL, EVAS_HINT_FILL);
   snprintf(buf, sizeof(buf), "./test.edj");
   elm_layout_file_set(layout, buf, "layout/test");
   elm_box_pack_end(box, layout);
   evas_object_show(layout);

   bt = elm_button_add(win);
   elm_object_text_set(bt, "mirrored");
   evas_object_size_hint_weight_set(bt, EVAS_HINT_EXPAND, EVAS_HINT_EXPAND);
   evas_object_size_hint_align_set(bt, EVAS_HINT_FILL, EVAS_HINT_FILL);
   evas_object_smart_callback_add(bt, "clicked", _bt_click, layout);
   elm_box_pack_end(box, bt);
   evas_object_show(bt);

   check = elm_check_add(win);
   elm_object_text_set(check, "test");
   evas_object_size_hint_weight_set(check, EVAS_HINT_EXPAND, EVAS_HINT_EXPAND);
   evas_object_size_hint_align_set(check, EVAS_HINT_FILL, EVAS_HINT_FILL);
   elm_box_pack_end(box, check);
   evas_object_show(check);

   evas_object_resize(win, 500, 500);
   evas_object_show(win);

   elm_run();
   elm_shutdown();

   return 0;
}
ELM_MAIN()

Reviewers: seoz, raster, tasn, Hermet

Subscribers: seoz, cedric

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

fix list item mode change on elm list

@fix

elm list - theme - make compress moder work again

someone.. somewhere along the way.. likely a long time ago.. broke
compress mode. likely the addition of the ellipsis value as an
ellipsis position or something related to that broke compatibility,
as this used to work. be that as it may... finding out just where is
virtually impossible, so fix it the simpler way - add elipsis values

pager16: add signals for toggling content visibility

shelf: use spacer for content sizing, toggle content visibility

shelf: use constant rects for eventing

test_win_keygrab: Better use strncpy when copying into fixed size buffer.

We should never copy more than PATH_MAX.

CID: 1298016

elm theme - toolbar - adjust padding for the toolbar to not look so bad

therw was always a small 2 pixel padding inside the toolbar. it just
looks bad for toolbar so expand out to boundary of toolbar

atspi: fix popup role and signal emission

tooltip: fix tooltip multi line text issue

Summary:
This is fixed by removing wrap=word from tooltip style.

Resolves: T2034

@fix

Test Plan:
1. elementary_test -> tooltip
2. click "unlock tooltip visibility"
3. check tooltip text. full text is not visible.

Reviewers: seoz, raster, Hermet, cedric, tasn

Reviewed By: tasn

Subscribers: seoz

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

entry: improve at-spi2 accessibility support

* return guide text instead of content
* mask passwords
* set proper at-spi roles when entry enables password mode.

atspi: fix signal name

interface scrollerable: modify the loop_v condition in _elm_scroll_momentum_animator() function

Summary:
change the 'sid->loop_v' condition to '!sid->loop_v' to delete momentum animator
when the vertical scroller reached the end of the content

@fix

Reviewers: raster, Hermet, woohyun

Subscribers: CHAN

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

Gengrid: Fix typo in API Documentation.

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

Reviewers: raster, Hermet

Reviewed By: Hermet

Subscribers: sachin.dev, singh.amitesh

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

Genlist: Fix typo in API Documentation

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

Reviewers: raster, Hermet

Reviewed By: Hermet

Subscribers: sachin.dev, singh.amitesh

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

Elm: Add context,open signal in elm_entry.

Summary:
When context,open signal is sent from elm_entry before showing context
menu, dynamic addition of context menu items to entry becomes easier.

@feature

Test Plan:
Run elementary_test
Launch Entry Scrolled test
Right Click on Multiline Markup entry

Reviewers: tasn

Reviewed By: tasn

Subscribers: subodh6129, poornima.srinivasan

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

examples: Replace setenv() with elm_config_accel_preference_set().

Replace environment value set with elm_config_accel_preference_set.

examples: Replace preferred engine "opengl_x11" with accel preference "3d".

Replace preferred engine "opengl_x11" with accel preference "3d".

Focus: add and fix comments

missed from prvious commit

elm_focus: added new focus move policy and elm_object_focus_move_policy_set/get

New focus move policy, ELM_FOCUS_MOVE_POLICY_KEY_ONLY, is added.
If you set this policy as base focus move policy, objects cannot steal
focus by using mouse click or mouse in. Only keyboard input (such as Left,
Right, Up, Down, ...) can make focus be moved.
Additaionally, an object can have its own focus move policy by using
elm_object_focus_move_policy_set API.

@feature

Elm widget: Fix some Eolian warnings.

Revert "Spinner: Add changed callback call when spinner value set."

This reverts commit 8278dff6a742c391f7c04c3de1033574d818dbb6.

This is not good for scenarios and will break the compatibility.

Thanks to David for reporting.

tooltip: fix positioning when tooltip is affixed to a window object

@fix

glview: preserved changed state when altering render mode

fixes the case where glview is marked changed and then the render policy
is modified prior to the render callback executing

@fix

Adjust according to Eo changes.

gengrid/genlist: decrement item counter before triggering item del callback

@fix

Gengrid: fix mirroring bug in gengrid widget.

Summary:
Below bugs are exist in gengrid widget when use horizontal mode and mirrored set.

1. Gengrid item is placed wrong position when mirrored set.
   Current calulation for mirroring in _item_place only consider pan object positions.
   if widget is mirrored, item should be placed opposited position so object x position must be
   consider in mirroring calculation also.

2. Gengrid scroll(pan) minimum size is returned wrong value when mirroed set.
   As the result of 1's calculation present oposite position of items, so align also reversed when mirrored.
   but current gengrid didn't change align x so minimum size is return wrong value
   and scroller expanded wrong direction.

@fix

Test Plan:
1. run elementary_test and set mirroring On
2. run Gengrid2 in elementary_test
3. see how items are placed by push append button repeatly.
4. see scroller shows correct position and items.
5. change usr/bin/test_gengrid.c to set another align value on gengrid and test again.

Reviewers: raster, seoz, Hermet, jaehwan

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

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

Spinner: Add changed callback call when spinner value set.

Summary:
There is a code to compare value is changed.
1256: if (sd->val == val) return;

Even value changed the value changed callback didn't called before.

Reviewers: woohyun, shilpasingh, Hermet

Reviewed By: shilpasingh, Hermet

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

genlist: do a proper clean up in clear().

Summary: Signed-off-by: Amitesh Singh <amitesh.sh@samsung.com>

Reviewers: raster, Hermet

Subscribers: seoz, sachin.dev

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

clipper: removed.

we've discussed and concluded it's unncessary.