multibuttonentry: Delete duplicate running code.

Summary:
When layout clicked, "elm_widget_on_focus()" called with "entry_focus_in()" callback.
The "view update" and "focus set to entry" already call in those callbacks, so i deleted duplicate code and it also make focus issue when user click the entry to add item.
@fix

Test Plan:
Run elementary_test.

1. excute multibuttonentry.
2. click the entry area.
3. see the elm_widget_on_focus called twice.
   (layout callback function called view update -> elm_box_unpack called in view update ->
    entry lose focus -> on_focus_callback call again)

Reviewers: Hermet, seoz, Jaehyun_Cho, JackDanielZ

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

win: Add default return value at elm_win_type_get API according to API description.

Summary: The API must return ELM_WIN_UNKNOWN (-1) when the given object is improper.

Test Plan: None.

Reviewers: woohyun, JackDanielZ, seoz

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

test_label: Use meaningful test name for label test.

Elm_Entry: Removed empty line and added error message

Summary:
Resolved FIXME in utf8_markup_save by adding error messages
for fopen and fputs command

Signed-off-by: kabeer khan <kabeer.khan@samsung.com>
Reviewers: devilhorns, seoz

Reviewed By: seoz

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

NEWS: Clean up news file before release.

test_panes: Remove unnecessary align/weight set for panes contents.

edje_externals/elm_video: Fix bug where we override the rememberred position

This bug was there from the initial commit of elm_video. Classical
copy&andpaste bug.

CID: 1267207

panes: Fix elm_panes_fixed_set.

@fix

NEWS: Manually fix up a news entry

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

Icon: choose the fdo icon size using the evas obj geometry

Use the min between w and h of the evas object to
request the size in the icon theme. Fdo icon are always
square (1:1) so min seems correct.

Also improved the test to include a test that do
not relay on the size_hint_min of the icon.

Now in all the test I have done always the correct
fdo icon are used.

Revert "Icon: use size_hint_min instead of image_size to know the wanted size"

This reverts commit 75176a789981bd6b1ef0bfcc95793c896b5bde07.

Was not the correct solution, as min_size can be smaller than
the real wanted size. Need to find another solution.

Thanks TAsn to spot this

Icon: use size_hint_min instead of image_size to know the wanted size

This @fix fdo icons to load the correct image for the requested size.

Whitout this the small images (16x16) was never used, because once a big
(64x64 for example) image was loaded, that size was used as min.

See the Icon Standard test for a better explanation

Icon: no more reimplement sizing_eval

The code there was trying to reload the icon to have an
image that should match the requested size, this was totally
wrong, the calculation was always resulting in an image bigger
than the needed.

It was also useless as this logic is yet handled in the resize
callback.

Improve Standard Icon test

You can now select the size of the icons in the list and
also added a single icon that you can resize.

The size of the icons is important as icon themes usually
provide different images for different sizes.

This test now spot out some errors in elm_icon wrt icon
sizing, will fix soon

Add correct fdo name for the last added search icon

elm_image: Add missing semi-colon to terminate line.

No compiler warning for this because the macro was terminated with it.
It still looks wrong without it and does no harm.

elm_image: Fix memory leak in GETDAT macro.

Free data2 on error path.

CID: 1265609, 1265610, 1265611, 1265612

test_disable: Extend button array to fix memory corruption

We are accessing btn[3] in the code so we should make sure the array it big
enough for it.

CID: 1265613

transit: Fix read from pointer after free.

This fixes coverity CID 1265738.

test_win_state: Add elm_win_center() sample.

win centering: unbork!

not centering on an axis does not mean that it needs to be moved to the screen origin.

also clamp to screen x,y, not 0,0

transit: fix crash issue.

This fixes the side effect that introduced from 1e06309be42fd2b5d0900b16da0178215c111681

Since transit has supported adding a map effect on the previous map status,
transit tried to access object-state map pointer.

but theses map pointer won't be valid if the transit keeps the last state
because it doesn't back up any object-state data.

test_gengrid: Dynamically allocate item data.

Do not waste unnecessary memory.

test_colorselector: Clean up a little bit.

- Remove unnecessary data set.
- Use ELM_SCALE_SIZE on min set for a better scalability.

Yes this colorselector test needs a looooot of love.

colorselector: Remove unnecessary pointer redirection.

Elm_Colorselector_Data pointer is already there.

test_icon: Clean up icon standard sample.

- Do not set unnecessary min size for the icons. Just follow list's item
size naturally.
- Do not free unnecessary list which was already freed by
EINA_LIST_FREE.
- Do not set align for the window resize object which is not required to
be set.
- Follow conventional way of pointer declaration.

image now implements edje object size functions

      Edje.Object.size_min.get;
      Edje.Object.size_max.get;
      Edje.Object.size_min_calc;
      Edje.Object.calc_force;

fixes all image-in-edje sizing issues/errors

image animation functions now work as expected for edje images

win_example now exits fullscreen on any key press...always

fix T516

Elm_Entry: set handler style according to current widget style

Previously, entry set handlers' style as default when they are created
and changed style on theme_apply function. But in usual scenario, users create
entry and set style before handlers are created, so when they are created
the style is set as default regardless of the style users already set.
This patch fixes it.

@fix

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

Revert "genlist: added reorder feature on key events"

This reverts commit f99be3a1d6bdc63153598b592b5437c33da44bf0.

The API is not ready to be released. Please see the thread on the e-devel
mailing list under this subject. This can get another try in the 1.14
release cycle.

Conflicts:
src/lib/elm_genlist.c

genlist: fix incorrect grammar.

No allow attaching double definite articles.

Thanks vincent for reporting.

elm_transit: Fix crash issue caused by multiple transit effects.

Summary:
Fix crach issue caused by multiple transit effects on a single object.
         This patch prevents deleting obj_data while the obj_data is still used by other transit objects.
         This fix is related to commits "Support zoom keeping current map effect."(60214e6a172e42303fdfc69ef3d1996b25132242)
         and "Support rotation keeping current map effect."(1e06309be42fd2b5d0900b16da0178215c111681).
         @fix

Reviewers: Hermet

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

genlist: fix genlist multi-selected problem.

user request was ignored if they calls the genlist_item_selected_set(true)
to another item in an item selected callback. because genlist made
the selected item highlighted after user "selected" callback.

The behavior had been changed on my genlist refactoring,
so the behavior was actually just restored.

This issue resolves T2020

elm_transit: Fix to support zoom rate zero.

Summary:
Fix to support zoom rate zero by using evas_map_util_zoom() instead of evas_map_util_3d_perspective().
@fix

Reviewers: Hermet

Reviewed By: Hermet

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

widget: elm_win can be unfocused only when its ecore_evas loses focus.

configure: Fix typo in coverage option help text

tests: add coverage support.

And our current score is 19% functions coverage for Elementary.

tests: actually run the main loop properly to be able to enter in idle state.

Note that the iterate function will never trigger the idler that are registered
in Ecore. That's by definition. I changed the code to actually use the full
main loop and trigger the change detection on idle enterer. That should be enough
for Elementary as all idler should logically affect the visual aspect of something
at some point and exit idle.

Thanks marcel-hollerbach@t-online.de for helping me debug this issue.

Updating italian translation

genlist: code clean up

highlight item with one entry.

genlist: don't access item after removing it.

genlist: guard the item deletion problem.

this exceptional code prevents the deletion of the item
in the middle of the callback.

genlist: fix the access of the invalid items.

still, there were case item or genlist is cleared in the user callback.
don't access the cleared items if they are requested deletion.

genlist needs to also highlight items on keyboard select

this should probably just happen in _item_select() to avoid future incidents

ref T1986

Revert "genlist: fix a crash when deleting a item in selected callback."

This reverts commit c80b24baf815e35c160bf0dbe8ecae6ea247aef8.

this broke the selection callback completely and makes every genlist app unusable.

please, please, please, please, I'm seriously begging you hermet, do ANY amount of testing before making further genlist commits.

genlist: fix a crash when deleting a item in selected callback.

if user deletes the item in the callback,
current function(_item_select) would be corrupted.
just leave the callback call in the last of the function.

genilst: fix regression bug added in 0e7d521151ba93f3b5955caaf9b906e160ab6057

some logic of deleting item were duplicated.
this fixes fileselector crash issue.

Do not explicite include atspi headers.

Summary: fixes make check failures on atspi tests.

Reviewers: cedric

Subscribers: seoz

Maniphest Tasks: T2001

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

genlist: set missed item reference.

this missing would cause crash of the genlist when it's items are cleared.

genlist: fix a crash of genlist group tree example.

don't corrupt list pointer.
we can clear items more simply.

Thanks ya-kov for reporting.

widget: set missing elm_widget_item_magic

photocam: force reload of image on file_set.

Thanks, bug reported by thierry@substanciel.fr .

elm image - fix image rotation in non-common cases

this fixes d6f4040d0c62c220f2339d2a57126fde43e85e19 for the non-common
path that normalizes then applies new orientation

@fix

elementary: Fix dereference of null return value

Summary: This fixes Coverity CID1261370. strrchr can return NULL here,
so we need to check that 'p' is valid.

@fix

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

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

Elm win: Fix added internal parameter to the win constructor.

Internal hacks should not affect user facing API. This commit adds a new
API to do the internal hack. This is still bad, but it's much less than
what was there before.

This fixes the main issue introduced by commit:
63a5cdf0f8516c16a62233da0c2af0e10a13264b

Elm win: Fix elm_win_fake()'s name to follow conventions.

DnD: fix callbacks deletion for inline windows.

During deletion of a window, widgets considered as droppable targets
have to remove their DnD callbacks. To achieve this, elm_drop_target_del
is called from the DEL callback (destructor). This function has to
determine if X11 or Wayland is used. Since the parent is already unknown
at this stage, only checking the engine name can give this
information.
On a regular window, the engine name is related to the target display.

The problem happens when an inline window is used. The engine is a
buffer and no information is given regarding the target display.

The patch fixes it by checking the nature of the Ecore Evas parents. It
supports nested windows (inline inside inline... inside XWin).

@fix

elm_colorselector: Fixed to make palette_box have proper height

Summary:
palette_box had stretched height exceeding its items size.
Now it will have proper height fitting to own items.

Reviewers: Hermet, Jaehyun, raster

Reviewed By: raster

Subscribers: raster, Hermet

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

elm - fix build problem with examples - elocation api

this fixes build of location_example_01.c as per fix in D1863 but done
properly so people can't use elm  elocation unless they enable beta apis

Scroller : Fix elm_scroller_last_page_get API wrong return value

Summary:
When first page is last page, elm_scroller_last_page_get need to return 0.
         But if pagesize is bigger then current content height or width, value will be returned -1.
         So fix this cases to return 0.

Test Plan: If set elm_scroller_page_set bigger then current content size, then issue is occurred.

Reviewers: seoz, jaehwan, raster

Reviewed By: raster

Subscribers: eagleeye

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

elm - elm image orientation api fix - major bug in implementation

@fix

The elm image orient_set api looks like it is a state, and it is
documented to SET the orietnation, but the code actually DOEs that
orientation when you set it. so docs and api name didnt match
functionality at all. this was bad an an oversight. you didn't notice
until you set orientation multiple times to an image. this fixes the
implementation to be as documented - a state that is applied and
remains that way until set to something else. setting to the same
orientation has no efect as with color, size, etc. etc. in evas.

you could look on this as an api break, but then either the docs
change AND api name changes to match the code, or code changes to
match the docs. we don't keep bug compatibility in efl and in this
case i'm making the call to break the behavior of a  function as it
was broken already with respect to name and docs.

genlist: recommit 1879cc39258c63c24f0abcff56558b1f1b60167a with correction.

elm_table now supports alignment

fake win should not perform widget focus setting on canvas focus-in

Revert "genlist: use eina bool"

  This reverts commit 1879cc39258c63c24f0abcff56558b1f1b60167a.

  Hermet, please check this commit it makes Fileselector crash,
  looks like you inverted the values: 1 should be EINA_TRUE.
  I recommend you to use Exactness to run all the tests automatically.

genlist: remove unnecessary null check.

genlist: revert 6f177e5db01553276058235a81cbfcc9b2cd4e23

this broke decoration item text. reverted.

genlist: send decoration active signal to view item.

emit signal to view item to do something additional.

genlist: fix regression bug. (T1986)

newly added in 2955ebc5dd01fbfb5a3286b724c8a40915864ad8

ELM_WIN_FAKE types should allow focus callbacks on windows

entry now finds its window object more effectively for cnp init

+elm_win_fake(), ELM_WIN_FAKE type

this is a giant hack to provide widget parenting in e. do not use this function, do not use this type, do not even read the related documentation.

@feature

genlist: code refactoring.

clean up decorate_item_realize() function.

use macro and remove old debug code.
no actual logical change.

genlist: clean up genlist_smart_del() func.

removed unnecessary calls.

widget: remove redundant internal macros

we don't need these macros anymore since eo has introduced.

Do not call ecore_x_init() when not using X display.

Summary:
On some systems (e.g. OS X), both Apple's windows manager and X11 can cohabit,
therefore we need to check for the DISPLAY environment variable only when ELM_DISPLAY
is not specified.

Reviewers: cedric, devilhorns, seoz, raster

Reviewed By: raster

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

client frame client clip should be relative to client during clip, not border

ref T1930

genlist: if the item was selected, then skip item_select() procedure.

genlist: simple code clean up

no logic change.

genlist: use eina bool

genilst: descriptive comment++

genlist: remove internal callback usage.

there were multiple ports for item procedures.
one port is enough.

genilst: get rid of lazy deletion.

delete genlist item directly.
instead, remove the bunch of code that was related to lazy deletion.

we can success, if we clean up the resources and callback calls properly.
here i only found item focus callback that (may) required the lazy deletion.

theme: Add newly added icons to EXTRA_DIST to make sure they end up in the tarball

We need to list the icons here to let autotools know we want them int the release
tarballs.

Revert "win INLINE_IMAGE test now crashes as expected when trying to use it in a non-useless manner"

This reverts commit fd71874024d14e5374aef93eae7d4cfbc0ea4521.

As the commit message says, the test crashes now. The ticket 1973 is
related to that issue.
Now, I understand there could be a bug inside inline stuff but it doesn't
mean that you have to push it and screw up all the test. You could just
attach a patch to the ticket and that's all folks!

genlist: send signal to deco_all_view additionally.

genlist: +warning

genlist: make up lack of behaviors in item_disable

genlist: replace to internal call

elm_win: Fixed the issue of updating ecore_wl_window location

Summary:
@Fix

Signed-off-by: vivek <vivek.ellur@samsung.com>
Reviewers: devilhorns

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

Gengrid: fix gengrid reorder animation bug

Summary: When gengrid reordering by thumb scroll, some items disapear on screan.

Test Plan:
1. Enabled thumb scroll in elm config
           2. Launch gengrid test in elementary_test
           3. Check reorder mode enabled
           4. Try to move reordering item to upon or lower area by thumb scrolling,
              items disapear.

Reviewers: seoz, raster

Reviewed By: raster

Subscribers: bluezery

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

[spinner] use strncmp, not strcmp to support styles starting with vertical

Summary: Use strncpm, not strcmp to support styles starting with vertical

Test Plan: Define spinner style which has 'vertcial' as a prefix

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

Subscribers: seoz

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

genlist: +parent validation check

genlist: code refactoring.

there are 2 variables that has same purpose.
cut off one dependency.

elm_widget: elm_object_disabled_set API fix

Summary:
When elm_object_disabled_set API is called on any widget, All the
subobjects of the widget also has to be disabled issue fix.
@fix

Test Plan:
Call the API elm_object_disabled_set on any container widget like box,
layout etc:-, fix can be tested with test_box.c and test_layout.c test
codes.

Reviewers: Hermet, seoz, raster

Subscribers: govi, rajeshps

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

elm_image: Fixed to have exact size using elm_image_fill_outside_set()

Summary:
There was a problem that image is stretched over its size
when elm_image_fill_outside_set() is applied.
This patch fixed that.

@fix

Reviewers: Hermet

Subscribers: Hermet

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