win : Prohibiting auto-rendering, until elm_win is shown.

Summary:
Although elm_win is only created and evas_object_show with elm_win is not called,
evas_rendering works sometimes.
This rendering is not necessary. Because on one is shown and just back buffer is allocated.
This patch is the prohibiting auto-rendering, if elm_win is shown by calling evas_object_show()
It is effective for power consumption and reducing memory.

@fix

Test Plan:
1. elementary_test : checking all menus's working
2. modifed bg_example_02 : updated elm_bg's color by animator without evas_object_show(elm_win)

Reviewers: jpeg, jypark, raster

Reviewed By: raster

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

atspi: add missing SELECTED changed signals

@fix

elm_segment_control: fix invalid parameter

Summary:
Eo complained about an invalid type 'Edje_Object' passed to
elm_widget_sub_object_add(). It is the segment control itself
that should be the subject of the call to this function.

@fix

Test Plan: elementary_test: no more Eo errors

Reviewers: cedric, stefan_schmidt

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

genlist tree test - minor - show all callbacks called so people know

act as "docs" so people can see what is called when. minor just in elm
test.

autoscroll focus: pass the object region position relative to the scroller.

Summary:
The region position passed to region_show and region_bring_in used to be
relative to the object position, not the scroller.

This fixes T1686.

@fix

Reviewers: seoz

Maniphest Tasks: T1686

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

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

interface_scrollable: delete unnecessary smart callback call.

Summary:
When _elm_interface_scrollable_page_bring_in is called, page is not changed yet.
So "page,changed" smart callback is unnecessary.

Sometimes "page,changed" smart callback is called twice.
Because "drag,stop" and "anim,stop"is called it.
So I add updating current page code.

Test Plan:
elementary_test -> scroller

1. using wheel.
2. using "prev page" and "next page".
3. draging sceen

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

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

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

colorselector: picker spinner UI not proper issue fix.

Summary:
Picker spinner UI not proper, showing blank buttons issue fix.

@fix T2401

Test Plan:
1. Launch elementary_test
2. Select colorselector
3. Tap on picker
4. observe, spinner UI not proper.

Reviewers: cedric, subodh6129

Reviewed By: subodh6129

Maniphest Tasks: T2401

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

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

notify: animate on hide

There was all the necessary code, except that the default
theme did not implement properly the signal & required data.

There is still a problem though, as the "timeout" smart callback
will be called at the start of the animation, so it's not possible
for an app to hide() or del() at this point (that would get
rid of the animation). But there's no other callback after
the animation ends.

elementary_test -to Notify > "Bottom" illustrates this issue.

Revert "elm: Add eglfs backend support."

This reverts commit 0d6aedbf44e5c371bddfa2b6b61c9409ed2184e3.

This patch landed a little bit to early.

elm: Add eglfs backend support.

Remove #include for config.h from C++ public header

Removed #include for elementary_config.h if HAVE_CONFIG_H is
defined. Which causes errors with external projects. Instead,
defined the necessary macros to use the Eo API.

Hoversel: Fix dismiss callback

Summary: Dismiss callback was getting called on hover object instead of hoversel object.

Reviewers: raster, Hermet, alok25, mvsovani, conr2d, singh.amitesh

Reviewed By: singh.amitesh

Subscribers: sachin.dev

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

test_explode: use evas_object_clipees_has instead

fixes memory leak and better performance

widget: update child_can_focus flag on focusability change

Summary:
Previously child_can_focus flag could be only updated when
child is deleted from object's subobject list. This patch
additionally updates child_can_focus flag when focusability
is changed with elm_widget_focus_can_set function.

Patch solves child_can_focus issue in similar situations:

elm_icon_add(layout);
elm_object_content_set(layout, icon);
elm_widget_child_can_focus_get(layout); // returns EINA_TRUE

icon = elm_icon_add(win);
elm_object_content_set(layout, icon);
elm_widget_child_can_focus_get(layout); // returns EINA_FALSE

@fix

Reviewers: cedric, stefan_schmidt

Subscribers: seoz

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

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

multibuttonentry: fix internal entry be an orphan when editable set API called.

Summary:
When user called "elm_multibuttonetnry_editable_set()" API as false.
the internal entry will be unpacked from box.

Then called that API as true again.
the internal entry just show without box packed.

Also, editable set API should not work in MBE shrink mode.

Test Plan:
Alternately Call elm_multibuttonentry_ediable_set API as true, false.
Then check the internal entry state.

Reviewers: Hermet, cedric

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

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

multibuttonentry: when item is clicked/selected, select function not called issue fix.

Summary:
The select function set in item_append does not get called on item selection issue fix.
As per documentation, the function has to get called.

@fix

Test Plan: elementary_test is updated with the demo

Reviewers: navnbeet, Hermet, CHAN, cedric

Reviewed By: cedric

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

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

hoversel: remove scrollable_set/get() API

Summary:
Hoversel scrollability can be considered as default behavior of hoversel.
These APIs are not necessary any more.

Reviewers: cedric, DaveMDS

Subscribers: DaveMDS, cedric

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

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

test_gesture_layer: use bigger font size for entry

win: reemit atspi "focused" event on bridge connection

At-spi clients assume that object:state-changed:focused event is emitted
on application start. By the time atspi_bridge will establish
connection on at-spi bus all focused events are already emitted.
To fix this we reemit "focused" event in bridge connection callback.

This fixes a bug when Orca screen reader do not read first element after
application launch.

@fix

elm: Fix typos fun!

release: Update NEWS and bump version for 1.16.0-beta3 release

po: update po files

hoversel: expands scrollable hoversel as large as it can

Summary:
In D2063, "max_size" data item is added to limit the number of items
to show at a time when hoversel is expanded.
However, it limits the number of items too few, so makes scrollable
function useless.
This patch removes limitation of hoversel size with pixels,
but if it needs, developers can set the limitation yet.
(By theme customization)

Test Plan:
elementary_test "hoversel"
click the second hoversel

Reviewers: DaveMDS, cedric

Reviewed By: cedric

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

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

multibuttonentry: fix label packed many times in internal box.

Summary:
The Internal box can has a lot of label for child when user set text many times.
Fix this wrong implement. It has to get only one label for child.

@fix

Test Plan:
Call elm_object_text_set() many times for MBE.
Check the label got a wrong geometry.

Reviewers: Hermet, cedric

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

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

gesture_layer: fix long tap gesture has wrong timestamp.

Summary:
The timestamp has 0 value when n_long_tap_start callback called.
Then timestamp is same on each n_long_tap_move callback functions.

For fix this bug, update timestamp before called state_set function.
Also update timestamp in every timeout.

Test Plan:
Run test_gesture_layer2.c
Print timestamp in every n_long_tap_XXX callback.

Reviewers: tasn, cedric

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

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

elm_colorselector: fix color selection with spinners

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

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

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

elm_hoversel: remove unused internal functions

Summary:
remove unused internal functions
(that added for scrollability, but not used any more)

T2765

Reviewers: Hermet, cedric

Maniphest Tasks: T2765

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

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

Revert "elementary: Comment out (#if 0) unused functions"

This reverts commit 74a7661119d122ec52844ad58c0901db739949f1.

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

elm_hoversel: change API name (label_auto_changed -> auto_update)

Summary: elm_hoversel_label_auto_changed_set/get() API names are not clear.

Reviewers: Hermet, cedric

Reviewed By: cedric

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

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

elm_hoversel: change Elm_Hoversel_Data member name from scroll_enabled to scrollable

Summary:
elm_hoversel_scroll_enabled_set/get() API were changed to
elm_hoversel_scrollable_set/get().
However, the name of member of Elm_Hovsersel_Data related to those API
are not changed.
It can confuse developers later, so change it same to the name defined
in elm_hoversel.eo file.

Reviewers: Hermet, cedric

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

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

elm_index: Fix memory leak issue in realloc failure

Summary:
@Fix

Signed-off-by: Vivek Ellur <vivek.ellur@samsung.com>
Reviewers: cedric

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

dayselector: fix warning related to argument type

Summary:
@fix

Signed-off-by: Vivek Ellur <vivek.ellur@samsung.com>
Reviewers: cedric

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

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

Fix typo in docs

not "a castrated ram (a male sheep)"  :D

flip: use eo_isa for checking object types

image: use eo_isa for checking object types instead of string comparison

elm_cnp: DnD/X11: correct drag window position in rotation

Summary:
For various angles 90, 180, 270,  the calculation of dnd window position
is not proper causing window to be placed wrongly while dragging.
Signed-Off By: Kumar Navneet <k.navneet@samsung.com>

@fix

Test Plan:
Longpress and drag and drop in entry by placing device at various angles
90, 180 and 270.

Reviewers: woohyun, cedric, thiepha

Reviewed By: thiepha

Subscribers: navnbeet

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

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

build: simplify SUBDIR usage in examples

examples/sphere_hunter: fix another instance of floating point incorrectness

I missed this one yesterday. Also change it to use floating point values
instead of casting which does not make much sense for hard-coded values.

CID: 1327343

examples/sphere_hunter: remove unused function

After fixing the Coverity issue clang told me that this function is actually
not used at all.

elm_widget_prefs: no need to redefine a typedef we already have

Spotted by clang. This was already typedef'ed in elm_prefs_data.h

examples/sphere_hunter: initialize struct in all cases

If l == 0 we would never initialize vec3 struct members.

CID: 1327345

examples/sphere_hunter: make sure we do not loose fractional part of double.

When dividing with an int we would loose the fractional part. Better cast to
a double to make clear we want double precision here.

CID: 1327343, 1327344

examples/sphere_hunter: use coorect logic for if condition.

We want a value here which is between 0 and 10. We need a logical AND here. If
we use OR this condition could never be false.

CID: 1327357

examples/camera_light: free resources if we fail to allocate all and leave

We return here leaving maybe some other allocations around and leaking.

CID: 1327348, 1327349, 1327350

atspi: fix invalid shift operation.

Add additional check in case when Elm_Atspi_Role_Type enum will enlarge.

CID: 1325724

elm test - make automated test betetr by auto-exiting after 50 bounces

elm test - genlist test - add bounce option test for testing performance

for release we need to test performance - esp of evas and eo in real life usage
and scroling is just such one. this adds a simple automated scrolling
test to genlist test (hit the button or ELM_TEST_AUTOBOUNCE=1
elementary_test -to genlist) so you can get consistent input and benchmark info

toolbar: limit number of min restricted calc to exacly what we need.

This change is the simplest I could do, but I think it would be better
to move _item_theme_hook inside _item_new. This is a further attempt
to fix T2777.

elm_win: fix wrong comment.

Summary:
static const char SIG_FOCUS_OUT[] = "focus,out"; // deprecated. use "unfocused" instead.
static const char SIG_FOCUS_IN[] = "focus,in"; // deprecated. use "focused" instead.

@fix

Test Plan: N/A

Reviewers: seoz, cedric

Reviewed By: cedric

Subscribers: cedric

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

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

elc_file_selector_button: free allocated path name

Summary:
Currently path is strdup() when getting real path, but it is not freed
in some cases.

@fix

Reviewers: seoz, Hermet, woohyun, cedric

Reviewed By: cedric

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

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

elm_map: block longpressed event when scrolled, zoomed, or rotated

Summary:
Currently, longpressed event can be called when map is
scrolled, zoomed or rotated. This might cause unintentional
result, because mouse need to be down at lease once to
execute scroll, zoom, or rotate, and longpressed event will start
when those events takes long enough time.

Reviewers: seoz, Hermet, woohyun, cedric

Reviewed By: cedric

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

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

hoversel: rename label_auto_changed -> auto_update

Thanks to Davide Andreoli for his review and suggestion.

atspi: clean-up children-changed event emission.

Patch fixes bugs caused by difference between elm widget
and atspi objects hierarchy.

atspi: always update cache

libatspi implementation allows to call methods on objects which
are not added to client cache with "AddAccessible" at-spi signal.
Bridge should allow to perform such calls by updating its internal
cache as soons as it returns a reference to the dbus object to client.

release: Update NEWS and bump version for 1.16.0-beta2 release

atspi: fix typo

test_genlist: set focus on popup's button

toolbar: shutup compiler warning.

toolbar: delay evaluation of the toolbar size until it stabilize.

Elementary is not using much the calculate callback from Evas. This
create all sort of problem where it transform O(n) algorithm. Like
sizing a toolbar for next frame to O(n^n) by actually sizing the
toolbar every time an object is inserted.

T2777

toolbar: remove useless call to _sizing_eval.

_sizing_eval is very costly on toolbar as it recalc all edje part and sends
signal to all of them. _item_theme_hook was already calling _sizing_eval
doubling the cost of adding new item.

border: bring sparklebear theme up-to-date with current efl rendering

various fixes have occurred over the years to improve proxy rendering.
as a result, this theme, which is apparently the only other user of proxy
edje parts besides deskmirror, was broken.

@fix

deskmirror: unset proxy.source_clip for urgency effects

cfedba598c23844946c5af2e038870c7ca6befdc in efl fixed source clipping
for proxy objects, breaking badly-clipped edje proxy objects everywhere

@fix

example: genlist - intialize buffer array to 0

CID: 1327347

example: genlist - intialize buffer array to 0

CID: 1327346

example: glview - use delete to deallocate memory

CID: 1327362

example: glview - use delete to deallocate memory

CID: 1327363

elementary: Fix genlist example to not free data until After it has
been removed from the list

@fix CID1327342

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

elementary: Remove unused static variable

Summary: clang reports _list_focus_objects as being an unused static
variable, so remove

@fix

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

elementary: Remove unused static variable

Summary: clang reports SIG_THEME_CHANGED as being an unused static
variable, so remove

@fix

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

elementary: Remove unused variables

Summary: clang reports SIG_BLOCK_CLICKED and SIG_TIMEOUT as being
unused variables, so remove

@fix

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

elementary: Remove unused function

Summary: clang reports _icon_size_min_get as being an unused function,
so remove

@fix

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

elementary: Remove unused static varible

Summary: clang reports _smart_callbacks as being an unused static
variable, so remove

@fix

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

elementary: Remove unused variable

Summary: clang reports SIG_LANG_CHANGED as being unused here, so remove

@fix

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

elementary: Fix passing wrong parameter

Summary: clang reports passing EINA_FALSE here is being treated as an
Eo object, so pass NULL instead

@fix

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

elementary: Fix passing wrong parameter

Summary: clang reports passing EINA_FALSE here is being treated as an
Eo object, so pass NULL instead

@fix

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

elementary: Remove unused static variable

Summary: clang reports ELM_APP_SERVER_VIEW_PROPS_ENTRY as being an
unused variable, so remove.

@fix

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

elementary: Remove unused static variable

Summary: clang reports SIG_CLICKED as being unused so remove the
variable

@fix

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

elm_test: Fix bug in GLView test

Scenario:
  const char * accel = accel_get();
  accel_set("gl:depth");

Now accel may not be valid anymore since the original
Eina_Stringshare might have been destroyed. So there was
no point in saving the const char * pointer. It was not valid
after the call to set(). This is a bit confusing here. Returning
char* would be clear: the caller would own the data.

toolbar: do not change align for "noicon" toolbar items

too many rels and complexity to track this to a more coherent
message

probably broken since 1.0 since there are zero "noicon" toolbar
tests

@fix

fix T2782

image: remove unused variables

elm_image: let's not trigger confusing update add.

Elm_Image rely on Evas_Image ability to rotate content. There is no
pixels manipulation involved that require an update_add here. In fact
this cause a bug where the content would disapear due to evas trying
to get some pixels data that are not where it think they are.

elm_hoversel: fix to show dismiss animation

Summary:
In hover.edc, hoversel dismiss animations are defined,
but it doesn't work at all. (Hover is removed before animation)
This patch makes hoversel remove its hover after dismiss animation.

@fix

Test Plan: elementary_test hoversel

Reviewers: Hermet, cedric

Reviewed By: cedric

Subscribers: woohyun

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

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

elementary: Fix calling ecore_evas_wayland_move with improper values

Summary: As ecore_evas_wayland_move ends up updating the location of
the ecore_wl_window, we should be calling move with the proper
geometry as this ends up updating the internal ecore_wl_window
geometry (which in turn sends to xdg_surface_set_window_geometry).
This is part of a fix where xdg_surface_set_window_geometry was
getting sent inproper values.

@fix

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

release: Update NEWS and bump version for 1.16.0-beta1 release

Revert "gengrid : add all_contents_unset API in gengrid for reusing contents."

This reverts commit 1094fb77a0ee23cbe1e8b15783e406a8d93b7fd4.

Feature is freezed for new released version.
This patch must updated after release finished.

toolbar: Fix crash on more_item when shrink mode is set with no item.

Toolbar creates a more_item when shrink mode is
ELM_TOOLBAR_SHRINK_EXPAND.
If select mode is ELM_OBJECT_SELECT_MODE_ALWAYS and there is no item
appended, then toolbar sets a selection on a newly created item.
In this case, sd->more_item is referenced before the new more_item is
assigned to sd->more_item.

@fix

gengrid : add all_contents_unset API in gengrid for reusing contents.

Summary:
Add elm_gengrid_item_all_contents_unset API for reusing contents in application side.
User can unset all contents or re-use their contents for increase scrolling performance.
genlist already support this API.

@feature

Test Plan: Need to add test in elementary_test.

Reviewers: raster, singh.amitesh, cedric

Subscribers: woohyun, Hermet, seoz

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

elm cursor - fix hotspot tracking to actually work right

hotspot tracking was broken. this fixes it.

@fix

elm cusro config - move elm to by default look for cursors in theme

previous config ignored elm theme if ithas cursors. this was just
wrong, so switch default back to using theme and update configs
accordingly etc. this is houw it should have worked.

elm cursors - if min size is too small then force it to be sane

cursor if less than 8x8 then make it 8x8. avoidx 0 sized pointers.

@fix

tests: add new tests for AT-SPI beta API.

list: set atspi parent despite atspi mode.

Path fixes failing test case when atspi mode was off.

release: Update NEWS and bump version for 1.16.0-alpha1 release

po: update po files

tests: disable new atspi parent test for me as it breaks

Breaking distcheck for me and delaying the alpha1 tarballs. The API is still
marked as unstable but we might want to look into it during the beta's.

elementary: Comment out (#if 0) unused functions

Summary: Unsure whether these are left-over, or are going to be
implemented in the near future...but for now, #if 0 out these 2
functions as they are currently unused.

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

elm run - be explicit about elm run not nesting in docs

@feature

atspi: add new relationship append/remove/clear API

Allow to add some extra contextul information about accessibility objects
which can be used by Assistive Technology to provide better user experience.

API is still marked as beta.

@feature

atspi: support org.a11y.atspi.Collection interface

org.a11y.atspi.Collection interface allows to perform fast search
queries on accessibility objects tree.
Due to lack of specification of collection interface this implementation
base on gnome project implementation from https://git.gnome.org/browse/at-spi2-atk/.

@feature

Revert "Genlist: only get item data if needed."

This actually doesn't work for coverity, I guess we just need to mark it
as false-positive, as it will never be null.

This reverts commit 178e7e6ac9419742d1bbe583fd252fc6536e5649.