Revert "elm_photocam: Limiting image reload in zoom in/out"

Summary:
After discussion with DaveMDS, D2402 is a wrong fix and must be reverted.
This reverts commit a7978f931a4b67820b6b65758ce6f362c1ebec70.

Reviewers: cedric, DaveMDS

Subscribers: jpeg

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

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

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

Fix scrollbar position issue when content is resized.

Summary:
When content is resized, scrollbar's position is not updated.
Add bar position update after bar size is changed.

Test Plan: elementary_test -> scroller3

Reviewers: seoz, jaehwan, woohyun, cedric, raster, kimcinoo, Hermet

Reviewed By: Hermet

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

elm_toolbar: fix memory leaks

Summary:
incorrect evas_object_box_children_get usage: returned list must be freed

@fix

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

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

elm_multibuttonentry: fix label position and size defects

Summary:
1.) Fixing defective label position.
Whenever user create multibuttonentry and set some text into label,
label will be shown at zero point (0,0) of MultiButtonEntry parent.
But then, after clicking on any button of MBE object, label position forced to
be recalculated and changes it's position.

2.) Fixing wrong size when label is not used.
When label is not being used, buttons shift by few pixels,
and it looks like if label is actually there, which is wrong.

@fix

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

Reviewed By: Hermet

Subscribers: NikaWhite

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

theme: fix typo, thanks arrowdodger.

spinner: Fix entry not inactive even gets an enter key cb.

Summary:
 The entry inactive signal emitted when finish spinner value set.
but the active signal emitted after that. so it makes weird view state.

 The function 'key_action_toggle()' called twice. (for entry, spinner)
event propagate called this func twice and its make this issue.

This patch makes to ignore toggle event when entry is not visible.
T2262

Test Plan:
Run elementary_test, execute spinner widget sample.

Click the first spinner.
Input something on entry.
Press enter key.
Check the issue.

Reviewers: raster, Hermet

Subscribers: kuuko

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

test_naviframe_complex.c: naviframe prev button auto pushed set as false.

Summary:
There is a invisible prev button. but it can get a focus and delete the current naviframe item.
Its not intend to added here i think, so i add a function that not add prev button automatically.

Test Plan:
Run elementary_test -> Naviframe Complex
Move focus use arrow key button and input enter key on prev button.
Click the View list, check the ERR msgs.

Reviewers: Jaehyun, Hermet

Reviewed By: Hermet

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

elm_multibuttonentry: change theme_set API to provide full widget theming

Summary:
While setting theme currently changes style and theme to all button and
whole multibuttonentry object, it doesn't change few other parts of this object.
Part are, for examples, "guidetext", "label" and "closedbutton".

Fixing this sad mistake leads to be able to create different styles for such
wonderful widget.

@fix

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

Reviewed By: Hermet

Subscribers: NikaWhite

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

tests/elm_test_image: Give a proper path to test images

../../ does not always work. VPATH builds like distcheck have a different
dir layout.

ref T2028

Test gengrid: use the gengrid enum, and not the genlist one.

They are probably in sync, but this is just wrong.
Especially if people copy/paste this.

Spinner: use fabs for floating point absolute value.

Gesture layer: fabs->abs for int parameters.

Clear more correct clang warnings.

elm_photocam: Limiting image reload in zoom in/out

Summary:
Reloading image in zoom in/out only if its previous orientation was changed
@fix
Signed-off-by: kabeer khan <kabeer.khan@samsung.com>
Reviewers: jpeg, raster, cedric

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

Genlist: Fix for memory leak

Summary:
Fix memory leak in
API _elm_genlist_item_elm_interface_atspi_accessible_name_get()
added free after _elm_util_mkup_to_text.

@fix

Reviewers: raster, singh.amitesh, Hermet

Reviewed By: Hermet

Subscribers: sachin.dev, singh.amitesh

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

elm_transit: Add missing "EAPI" prefix for elm_transit_smooth_get()

Summary: Add "EAPI" prefix for elm_transit_smooth_get() in elm_transit.h

Reviewers: Hermet

Reviewed By: Hermet

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

entry: Fix elm_entry_input_panel_layout_get API to return ELM_INPUT_PANEL_LAYOUT_INVALID when it fails.

Summary:
Even if the given Evas_Object is NULL, API returns ELM_INPUT_PANEL_LAYOUT_NORMAL.
But, ELM_INPUT_PANEL_LAYOUT_INVALID seems proper in that case.

Test Plan:
Call the following API with NULL.
   elm_entry_input_panel_layout_get(NULL);

Reviewers: woohyun, Hermet, eunue

Reviewed By: eunue

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

elm_photocam: Fixed T2337, zoom in/out rotated photocam image

Summary:
Currently zoom in/out rotated or flipped image was causing lost of orientation
and aspect of image is also changed. Fixed that by reloading image on pan_obj on zoom
@fix
Signed-off-by: kabeer khan <kabeer.khan@samsung.com>
Reviewers: DaveMDS, jpeg, cedric, raster

Subscribers: jpeg

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

interface scroll: handle wanted x coordinates in case of mirroring.

Summary:
In case of mirror enabled, wanted x should come from _elm_scroll_x_mirrored_get().
This defect is visible in case of some fancy effects on scroller (map).

@Fix

Reviewers: raster, Hermet

Subscribers: herdsman, sachin.dev, kimcinoo, eagleeye, seoz

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

colorclass: move header behind EFL_BETA_API_SUPPORT

may end up iterating a bit more on this, don't want to be stuck with api

elm_segment_control: mark default return value in eolian API

elm_segment_control_item_index_get should return -1 when input parameter is NULL.

@fix

elm: add safety check

when insert a new item before/after an item, the relative item should not be NULL.
this patch fixes naviframe, gengrid, genlist, list, and toolbar.

@fix

multibuttonentry: This fixes item,clicked cb broken issue.

Summary: This fixes T2334.

Test Plan: elementary_test -> multibuttonentry

Reviewers: raster, Hermet

Reviewed By: Hermet

Subscribers: DaveMDS, seoz, sachin.dev

Maniphest Tasks: T2334

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

doc: improve ambigious doc descriptions.

even descriptions weren't matched each other.

intf scroller: Added wheel_disable_set/get API descriptions.

Reviewers: herdsman, raster, Hermet

Reviewed By: Hermet

Subscribers: seoz

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

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

Elm win: Add a warning to wl_id_get func.

As discussed on IRC, users of this API should be warned, as it's
almost always a bad idea.

Revert "checkbox: Adding third state (Indeterminate) support in checkbox"

This reverts commit d0507e117acd5bbfdecf4c43f1d338c00802af44.

oops this patch wasn't intended.

elm_clock: fix theme apply

Summary:
theme change was delayed untill some other change (i.e. mode change, clock update)
@fix

Test Plan:
1. create new style "test" for elm_clock
2. add this style to theme
3. compare results of following modifications of src/examples/clock_example.c:

```
   /* am/pm */
   ck = elm_clock_add(win);
   elm_object_style_set(ck, "test"); //style is applied before ap_pm set
   elm_clock_show_am_pm_set(ck, EINA_TRUE);
```
and
```
   /* am/pm */
   ck = elm_clock_add(win);
   elm_clock_show_am_pm_set(ck, EINA_TRUE);
   elm_object_style_set(ck, "test"); //style is applied after ap_pm set
```
ExR: style is applied in both cases
Result without this patch: style is applied only in first case

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

Reviewed By: Hermet

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

checkbox: Adding third state (Indeterminate) support in checkbox

Summary:
Checkbox should support a third state "indeterminate" along with "Checked" and "Unchecked"
This third state is a state of checkbox which is shown when checkbox is neither Checked nor Unchecked.

- Added this new feature on the basis of a boolean variable's value.
- By default this boolean variable is disabled and checkbox will treat like old way.
- While adding this, I kept in mind, that applications which are already using checkbox, should not be affected, so I used 0=False=Unchecked, 1=True=Checked, and 2=Indeterminate
- Also added an example check_example_o2.c, which is using checkbox with both ways, using boolean, and using enum.
- Now also values can be set using boolean values, but it will give a type casting warning. As a boolean doen't support third state, so I used an enum int like.

- Added APIs to enable disable third state mode. elm_check_three_state_mode_set(check_obj, bool_val), and elm_check_three_state_mode_get(check_obj)
- Modified old APIs which were setting or getting states of checkbox.
- Added a state in theme of checkbox, with third state image.

Reviewers: seoz, raster, Sergeant_Whitespace, Hermet

Subscribers: Hermet, Sergeant_Whitespace, sachin.dev

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

elm_slider: fix theme apply

Summary:
using theme of the object instead of default for subparts.
@fix

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

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

multibuttonentry: update longpress signal case.

Since the signal name was changed, usage should be also updated.

multibuttonentry: changed longpress callback name.

contextually, the longpress triggered by item.

so the name should be item,longpressed

thank Davide Andreoli for reporting.

win: add note for (mis)users of elm_win_window_id_get()

this function has a very small use case. if you are outside of this use case,
use a different function.

Revert "elementary: Fix elm_win_window_id_get for wayland"

This reverts commit 257452f46d344b64d611aa555a8aacdd509a492e.

Have to revert this as it breaks E compositor functionality wrt pixmap id.

elementary: Fix elm_win_window_id_get for wayland

Summary: This fixed the elm_win_window_id_get function when running in
wayland to return the ecore_wl_window_id rather than the surface id.
This change is necessary so that anyone calling this function can use
the return value in ecore_wayland function calls.

NB: Thanks to Hermet for pointing this out ! :)

@fix

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

elm_color_class: Avoid resource leak

We would leak ecc here when we return so we need to free it before.

CID: 1294249

mailmap: update q66

atspi: add missing signals array sentiel.

@fix

build: Make sure we remove check-results.xml during cleanup

This file gets created durign a make check run. We need to remove it manually
as it is not tracked otherwise.

elm - fix repeat event fix - thanks dave

this fixes 2d64175722db124b1e4be6c735503f29e4453015 which was a fix to
elm image always repeating - which it shouldn't

toolbar: fix conditional compiler warning in horizontal_set

Add missing file_get legacy definition for Video and Layout

was missed from commit:
e5cda41c889f98b6f3f5f63b909f595a7a656cc0

radio: update doc

elm_glview: make_current(NULL, NULL) before delete

The surface and context should not be current when deleted.

Evas GL would work just fine but since it deletes the surface
and context immediately upon request, those should not be
current.

This is a slightly different behaviour from EGL.

elm_gengrid: mark cursor_engine_only in data and update state when realized

@fix

spinner: fix entry style in non 'button_layout' mode

Summary:
entry should receive style in both modes

@fix

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

Reviewed By: reutskiy.v.v, Sergeant_Whitespace

Subscribers: Sergeant_Whitespace, reutskiy.v.v

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

colorclass: only perform remote actions if current editor is remote

should fix case of editing local + remote simultaneously

colorclass: add reset signal for correctly managing remote colorclass resets

previously this would just perform teh reset locally, which was not intended

colorclass: flush config after saving

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

flipselector: Remove has some nbsp's and random whitespaces

Flipselector has some random whitespaces that do not belong.
Additionally there are a few hidden non-blocking spaces that prevent
edje_cc to compile the default.edc on arm. Appearantly on x86/amd64 this
works normally.

This patch works around T2285  while the issue should remain open as the
real issue, edje_cc treating nbsp as a whitespace on arm, is not fixed
through this patch.

ref T2285

Signed-off-by: Olliver Schinagl <o.schinagl@ultimaker.com>

elm_image: Do not open file again if already opening (async)

examples: Use new efl apis for size_set and visibility_set.

evas_obj_size_set -> efl_gfx_size_set
evas_obj_visibility_set -> efl_gfx_visible_set

elm_image: return EINA_FALSE if the data source is NULL in elm_image_memfile_set

@fix

elm_suite: Add test for async file set

elm_image: Fix potential race conditions in async mode

Without any locking or thread-safe mechanism, the previous implementation
would have failed as soon as too many file_set() happened on the same
object. Indeed, file_set() can happen while the async open thread is
running. I shouldn't have blindly listened to Cedric :P

test_naviframe: Fix content image file names.

@fix

[elm_panel] Fix mirroring issue.

Summary:
Issue: Mirroring of elm_panel not working correctly.
Solution: Handle mirroring appropriately for animations.

@fix

Test Plan:
Case: 1
1. Open elementary_test
2. Mirror on
3. Open scrollable panel, click on toggle button.
Case: 2
1. Keep the scroller panel opened
2. Turn off Mirror
Case: 3
1. Turn on mirror
2. Check direction button in non-scrollable panel.

Reviewers: shilpasingh, raster, Hermet, Princekrdubey

Reviewed By: shilpasingh

Subscribers: poornima.srinivasan, subodh6129

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

list/genlist/gengrid: Fix default return value in eolian APIs.

Summary:
Some APIs need to return specific values for their usage.
For Examples, elm_xx_select_mode_get function need to return
ELM_OBJECT_SELECT_MODE_MAX when invalid input comes.
**Fixed API list :**
elm_list_mode_get
elm_list_multi_select_mode_get
elm_genlist_mode_get
elm_genlist_select_mode_get
elm_genlist_multi_select_mode_get
elm_genlist_item_type_get
elm_genlist_ttem_index_get
elm_genlist_item_select_mode_get
elm_gengrid_select_mode_get
elm_gengrid_multi_select_mode_get
elm_gengrid_item_index_get

Test Plan: Call API with NULL parameter and check return value.

Reviewers: Hermet, seoz, jaehwan, raster, eunue

Subscribers: id213sin

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

Delete whitespaces for header and eo files.

Summary: Delete whitespaces header and eo files, otherwise add a line.

Reviewers: Hermet, seoz, JackDanielZ

Reviewed By: JackDanielZ

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

elm - undeprecate preferred engine config api - still neeed it for cases

undeprecate this api as to be frank - you still need it if you want a
buffer enigne for back-end processing. doesn't break api or abi.

elm_genlist: pass eo_it to eo_do_ret func, not VIEW(it)

@fix

build: Allow to define path for eldbus-codegen binary

Since the latest elm colorclass changes we need eldbus-codegen to generate
some files. Not all setups have this in their normal $PATH so let allow these
setups to set the correct path during configure. This allows our Jenkins setup
as well as others to work again.

elm_object:  Add since tag for new elm_object_access_info_get()

elm image - stop repeating events by default - wrong

why have elm image repeat events always (and unable to stop it)? this
is neither useful nor convenient. it's just painful. it makes images
not behave like prety much every object. they already are set up to
capture events - then do it normally.

@fix

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

gesture_layer: Fix unused value assignment

Fixes CID1261280

colorselector,scroller: Check return value of elm_layout_theme_set

Fixes:
- CID1273968
- CID1291831

elm_test: (GLView) reset accel_preference after win_add

This might even make the application run with two different engines.

elm_test: GLView: Add depth to the window

Without depth, glview / evas gl basically fall back to indirect rendering.

Also change bg color from yellowish to brown (and use PREMULTIPLIED
colors, not random values).

elm_glview: Call resize func whenever the surface is updated

This will allow apps to set the proper viewport, even if the
actual surface geometry didn't change.

This fixes a rare situation where the mode is changed from indirect
to direct (eg. in elm_test glview).

elm_glview: Fix usage of direct rendering

You can't have direct rendering and ALWAYS render mode. It just
didn't make sense.

Also fix the test case.

edc: use new empty description inherit statements in comp theme

coverage/testing++

edc: fix button theme syntax error

colorclass: move test translate+list callbacks to main

enable remote editing of all colorclasses in elm_test

colorclass: sort colorclass list by colorclass name

colorclass: block colorselector dismissal when clicking inside csel area

colorclass: fix test window initial resize geom

colorclass: only show one fader at a time

showing both faders obscures the app too much to be useful

colorclass: slightly expand colorselector bg to improve clicking area

colorclass: add clip for color type groups, move inset inside edje area

colorclass: add focus glow to color type selector

edc: add descriptions for all listed color classes

Merge branch 'devs/stanluk/eo_object_items_atspi'

Make elm object items atspi-aware objects. Implement atspi support
for elm_genlist, elm_gengrid and elm_toolbar items.

atspi: changed del event broadcast. Fix genlist & gengrid tests

atspi: remove redundant signature check.

atspi: update genlist item Name property when text becomes realized.

atspi: fix mem leaks spotted in valgrind.

atspi: override default atspi parent of genlist item content

atspi: delay registration of atspi object until its shown

atspi: allow to override default Elementary objects hierarchy

atspi: remove deleted atspi header from elm_interfaces.

atspi: add 'checked' state for elm_radio

atspi: add 'editable' to elm_entry atspi states

atspi: allow AT-SPI clients to filter-out key events

widget: use ATSPI protected functions.

atspi: add text tags filtering

gengrid: remove debug

atspi: improva child add/del broadcasting.

This commit enables generic broadcasting of changes in widget hierarcies.
Previously this kind of broadcasting had to be implemented inside widget.
This solution is using elm_widget sub_obj_add/sub_obj_del functions.