test_focus3: Correct wrong data for smart callback.

focus: Fixed focus issue that the highlight was not visible on first
click.

Summary:
Focus highlight was not visible on first click on a widget with focus_highlight enabled on the window.
This happens because on first click, the "elm,action,focus,show" signal was emitted before setting focus highlight edje object as theme object of elm window.
In this patch, moved the  _elm_win_focus_highlight_visible_set() call after focus highlight edje object set as theme object of window.
This fixes T774.

Test Plan:
elementary_test->focus then click any widget. Focus won't appear for the first time.

Reviewers: seoz, raster, woohyun

Maniphest Tasks: T774

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

naviframe - don't access item after del

test_hoversel: Change the hoversel button text when an hoversel item is
selected.

This reveals hoversel bug on mobile profile that changes the size of
text parts on continuous item change.

spinner: Uncomment ctype.h inclusion because isspace is used.

It looks like ctype.h is included in some other headers by luck but it
is recommended to explicitly include the necessary header.
Confirmed by glima as he commented this out.

spinner: Internal code clean up for spinner.

- Used macro for constant.
- Changed timer name to be more descriptive.
- Added descriptions about timers.

theme - win shadow - stack under all so pulsing icon is not BELOW the shadow.

naviframe: Use elm_widget_item_del() when you want to delete the item
data.

It will call del_pre_func and free the item data.
Special thanks to glima for the report.

[naviframe] Fix to enable item deletion in pop_cb

Summary: Fix to enable item deletion in the registered pop_cb function

Reviewers: Hermet

Reviewed By: Hermet

CC: seoz
Differential Revision: https://phab.enlightenment.org/D487

[naviframe] do not size evaluate during deletion

Summary: Author: Shinwoo Kim <cinoo.kim@samsung.com>  2012-12-11 20:13:39

Reviewers: Hermet

CC: seoz
Differential Revision: https://phab.enlightenment.org/D485

[naviframe] Fix to hide the previous view by emitting signal

Summary:
Previously, previous view was hidden by evas_object_hide.
         However, this method is no longer correct because size calculation of naviframe causes evas_object_show.
         Now, previous view is hidden by emitting signal to naviframe theme and clipper hides the previous view.

Reviewers: Hermet

Reviewed By: Hermet

CC: seoz
Differential Revision: https://phab.enlightenment.org/D484

fileselector: Make fileselector focusable.  Ok on Enter key and Cancel on Escape key.

Now fileselector accepts the key event on Enter and Escape.

fileselector_button: Added missing description about focus signals.

popup: Replace focus_direction_go with focus_cycle.

theme update to handle rename on/off in efm

shoudl fix T799

elm_widget: Fixed some codes about focus custom chain.

- When appending an object to the focus custom chain,
  event callback about deletion should be added.
- When a widget is deleted, its focus custom chain should be
  managed correctly.

ctxpopup: Fixed typo and flourished comment.

elm filesel - scale button in filesel button.

elm filesel - fix theme layout for button+entry

inwin - the "greying out" fade looks just wrong - make it black.

fix some typos in comment and description

test_conform_indicator: Refactor internal code of conform_indicator test
code.

- removed unnecessary casting.
- removed unnecessary internal variables.
- removed wrong window resize object addition.
- changed parent parameter correctly.
- fixed wrong grammar in command line message.
- fixed resource leak.
- enhanced comments.
- removed unnecessary size hint align set.
- removed unnecessary parameter passing.

config: Added more description about elm_config_indicator_service_get().

entry: Fixed to emit "*enabled" signal to scr_edje when it is not disabled.

Summary: This fixes coverity issue CID 1162751.

Reviewers: seoz, woohyun, tasn

Reviewed By: seoz

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

elm: Removed unnecessary null checks.

As all efl public free apis  and free function get null as valid parameter,
we do not need to check null. I also used ELM_SAFE_FREE if necessary.
After this job code got cleaner.

This is a continue of a commit e3372339.

elm: Removed unnecessary null check for eina_hash_free and
elm_widget_stringlist_free.

elm: Removed unnecessary null check for free.

elm: Removed unnecessary null check for eina_list_free.

elm: Removed unnecessary null check for ecore_event_handler_del.

elm: Removed unnecessary null check for evas_object_del.

elm: Removed unnecessary null check for eina_stringshare_del.

elm: Removed unnecessary null check for ecore_idler_del, ecore_idle_enterer_del.

test, examples fileselector_entry: Changed the usage of deprecated APIs
to recommended APIs.

fileselector_entry: Moved declarations of deprecated APIs to
elm_deprecated.h and marked them as EINA_DEPRECATED in c code.

test, examples fileselector_button: Changed the usage of deprecated APIs to recommended
APIs.

fileselector_button: Moved declarations of deprecated APIs to
elm_deprecated.h and marked them as EINA_DEPRECATED in c code.

url: Added underscore(_) to elm internal functions, elm_url_xxx.

- _elm_url_download, _elm_url_cancel, _elm_url_get
- other internal functions in elm_priv.h also have same prefix.

elm: Removed unnecessary null check for ecore_timer_del,
 ecore_animator_del, and ecore_job_del.

As all efl public free apis get null as valid parameter, we do not need
to check null. I also removed some null check for other free apis which
were right next to timer/animator/job del. After this job code got
cleaner.

Introduce fileselector interface and applied to elm_fileselector.

Summary:
fileselector_button and fileselector_entry should have almost same interfaces with fileselector.
Previously , we exposed them individually such as elm_fileselector_is_save_set().

Instead of adding the same APIs three times, we can reuse one fileselector's API using Eo interface.

This patch introduced fileselector interface and applied it to elm_fileselector.

Reviewers: seoz, raster

Reviewed By: raster

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

spinner: Create an internal entry when it is really needed.

Do not create the internal entry on smart add. Create that when it is
needed with editable mode.

spinner: Toggle entry on Enter and space key.

entry: NULL should be set to the correct position of tmpbuf.

If edje_object_part_text_get gives different text from original one,
length of text can be changed. So the last position of tmpbuf should
be calculated again refering to the current text to be returned.

toolbar: Do not check ev->string for Return key and react on space key.

space key works as same as Return key.

elm: Fixed formatting about EINA_LOG stuff.

test_focus: Call window object show after its content creation.

This is a successor commit of 64fd5a825b24591aedb9cac695d03f5b4d867442.
Window show call triggers all smart objects' smart calculation and in case of
X11, XMapWindow() is called directly.
Also moved resize of window accordingly.

widget: doxygen fix

all widgets: change Eo API for "elm_widget_theme()" to "theme_apply"

    There are elm_widget_theme/theme_set/theme_get functions.
    In Eolian these functions will be described as "theme" method and
    "theme" property. There is clash here.
    So add suffix "_apply" to Eo API for "elm_widget_theme".

[Gengrid] Refactoring gengrid item show reigion calculation

Summary:
Gengrid _item_show_region function calculate gengrid x, y moving postion with scroll to type.
Belows are summaries of this refactoring works.

1. Refactoring recalculation of scroll type.

2. Bug fix :
   - when x or y is 0, y or x value is set as default value, it->position-1.
   - when x or y is less then 1, calculation is not correct.
   - make calculate correct position of y when even though scroll direction is horiziontal
     (when virtical, x is also calculation correct position).

3.  change smart data variable s_type to scroll_to_type to make unity on gen widget.
     (genlist smart data variable name is scroll_to_type);

Reviewers: seoz, Hermet, raster

Reviewed By: raster

CC: bluezery
Differential Revision: https://phab.enlightenment.org/D474

PackageKit theme: pass make distcheck

Also the image used by the edc need to be included in the tarball

prefs init should probably clean up after itself when it fucks up

prefs should CRI on add/file_set if init has failed

theme: Add packagekit edc to fix make distcheck

Adding it to EXTRA_DIST makes ure it gets packaged in the tarball
and is available for a plain tarball build.

conform: Fixed to get a geometry from *focus_region_get for show_region_job.

Summary:
The geometry of elm_widget_show_region_get is only updated by elm_widget_show_region_set.
If the geometry of *show_region_get is not updated, the result of show_region_job would be incorrect.
So, we need to get a proper geometry using *focus_region_get for focused object.

Reviewers: seoz, woohyun, Hermet, raster, tasn

Reviewed By: raster

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

entry: Add smart activate for accessbility.

Summary:
In TTS mode, when the entry is highlighten and double tap to the entry,
the entry will be focused. But "clicked" smart callback is not called, input panel is not shown.
So I we need to call the callback and show input panel in smart activate.

Reviewers: seoz, Hermet, woohyun, tasn, jaehwan, raster

Reviewed By: raster

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

entry: If the scrollable entry is set disable, send "*,disabled" signal to scr_edje too.

Summary:
Default theme of entry does not have any disabled state in "elm/scroller/entry/*" styles.
But EFL developer can make any custom style for entry and scroller and it can has disabled state.
So _elm_entry_smart_disable has to emit "*disabled" signal to scr_edje.

Reviewers: woohyun, tasn, seoz, Hermet, raster

Reviewed By: raster

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

theme for the packagekit module IN

fileselector_button, fileselector_entry: Updated FOO_inwin_mode_set() documentation.

fileselector_button initialize inwin mode using the configuration.
Fixed wrong information about default value in API documentation.

In addition, fixed test_fileselector_button.cpp not to show wrong information at initial time.

theme - ibar - emit show doen signal for ibar menu

SHUT UP ELM PREFS!!!!!!!!!!

seriously, this is and has been bullshit since it was added. I don't need an error message every time I do ANYTHING just because some module has never been found.

tweak e menu shadow offsets

T793

flipselector: Remove unused variable of Elm_Flipselector_Item

The data for callback is stored in Widget's data variable.
So, Elm_Flipselector_Item's data variable is not needed.

test_flipselector: Fixed that buttons use wrong flipselector which is not intended.

Since e2db48abe, disabled flipselector was passed button callbacks.
So, buttons just print warnings instead of the intended behavior.

datetime: Delay elm_layout_theme_set until memebers are initialized in _smart_add

There are some warnings in elm_object_disabled_set() because elm_layout_theme_set()
calls _elm_datetime_smart_disable() before creating file->item_obj.
This patch refactors _smart_add to initialize file->item_obj before calling
elm_layout_theme_set().

Fix unknown command warning in generating doxygen

elementary/src/lib/elm_access.h:238: warning: Found unknown command `\type'
elementary/src/lib/elm_bg_legacy.h:98: warning: Found unknown command `\r'
elementary/src/lib/elm_bg_legacy.h:98: warning: Found unknown command `\g'
elementary/src/lib/elm_object.h:124: warning: Found unknown command `\content'
elementary/src/lib/elm_image_legacy.h:82: warning: Found unknown command `\group'
elementary/src/lib/elm_image_legacy.h:56: warning: Found unknown command `\group'

panel: Guarantee the integrity of parameter

- Check whether ret is NULL.
- Guarantee hidden variable is only EINA_TRUE or EINA_FALSE.

panel: Check whether sd->content is NULL in _panel_toggle

sd->content can be NULL when elm_panel_hidden_set is called.

panel: Delay elm_layout_theme_set until memebers are initialized in _smart_add

elm_layout_theme_set can call some methods like _elm_panel_smart_theme and there are some warnings
when these methods manipulate some member variabls which are not initialized yet.
It's because elm_layout_theme_set is called too early in _smart_add.

Spotted while digging warnings after rELM4c3e23de9e46

elm_widget_xxx.h: Adopt EINA_UNLIKELY and fix indentation in FOO_DATA_GET macros.

win: Check null before calling elm_widget_highlight_in_theme_get(). It
can be null.

'target' can be null when the focus in is just called.

test: Cleaned up elementary_test internal code.

- removed unnecessary/unused macro.
- fixed formatting.
- added prefix '_' for internal function.

fileselector_entry: Removed _mirrored_set(), one line local function

Small refactoring.
It is unnecessary one line function which just calls elm_widget_mirrored_set.

fileselector_entry: Delay elm_layout_theme_set until memebers are initialized in _smart_add

Summary:
elm_layout_theme_set can call some methods like _smart_disable and there are some warnings
when these methods manipulate some member variabls which are not initialized yet.
It's because elm_layout_theme_set is called too early in _smart_add.

Spotted while digging warnings after rELM4c3e23de9e46

Reviewers: seoz, raster

Reviewed By: seoz

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

pager16 - match same theme look for popup as pager orig

e19 - deskmirror - use proper scaling for mini frames

ChangeLog/NEWS: Removed unnecessary ChangeLog and NEWS entries.

We do not modify them at every commit after 1.8 release. These are
changed by stefan when it does release.

elementary/ctxpopup - no need to use RESTACK callbacks anymore.

Now bg is smart member of the internal object.
let it have a behavior as the member of the smart object.
don't need to care it additionally.

test_photocam: Support zoom in and out with more condition checks.

Now zoom in and out work sanely.

ctxpopup: set smart member relationship for objects in ctxpopup

Summary:
to support mouse_down/up/move callbacks of ctxpopup,
all objects in ctxpopup should be in smart relationship with ctxpopup.
Also, with this patch, bg object and arrow object don't have to be called with
evas_object_show or evas_object_hide when ctxpopup is shown or hidden

Reviewers: Hermet, seoz, raster, thiepha

Reviewed By: seoz

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

photocam: Updated elm_photocam_zoom_set() API documentation.

elementary - don't use __UNUSED__ anymore

[naviframe] Replace naviframe title visible APIs to naviframe title enabled APIs

Summary:
To support naviframe title transition effect, naviframe title enabled APIs are applied.
         To remove the duplicate functionalities, naviframe title visible APIs are deprecated.

Reviewers: Hermet

Reviewed By: Hermet

CC: seoz, raster
Differential Revision: https://phab.enlightenment.org/D426

test_glview: Refactor glview test layout.

- Display button just vertically.
- Changed button text more descriptive.
- Create glview related buttons only when glview is created correctly.

test_glview_simple: Close the window instead of exiting the program on
close button.

e menus now have their own shadows

fix comp theme fast popups to still have shadows

change theme animation for window frame icon urgency to be more in line with other e animations

a number of e animations are "zoom" related. bouncing is clearly identified as being OSX, so let's stay away from that.

feature: add PIN desklock theme

currently just reuses syscon buttons since they're identical in function

feature: iconify/uniconify animations for windows

feature: pager16 module theme

readd deskmirror

this file seems to have been magically and mysteriously lost during the recent theme merge

redo comp theme to match e19 signals

* shadows should be uniformly moved to whatever thing they're supposed to be on (WIP)

* trans-clipper was a failure idea, this is now done in C

* signals renamed to be more consistent

* groups renamed to comp/frame/*

* comp/border/* groups preserved for compat

cnp - dont access null parent obj in getting xwin

test_ctxpopup: Internal refactoring of ctxpopup test codes.

- Handled the deletion of image object correctly. Do not overuse/misuse
evas_object_data, it will make code less readable.
- Cleaned up button click callback. sharing one function for different
buttons is not a good idea in some cases. Just split the callbacks.
- Removed unnecessary evas_object_data_set().

win: Added more description to elm_win_floating_mode_set() API.

elm widget - skip null list entires in focus list

toolbar: Check whether icon is null before sending signal

There are warnings when icon is null since rELM4c3e23de9e46.

fileselector: Fix that folder only mode shows only files instead of only folders

Issues: T756

It's regression of rELM4693736bb028

elm: Build in an alphabetical order.

po: Updated po files.

It's actually annoying that po is changed whenever I do clean build.

scroller: Fixed wheel event callback

Summary:
scroller is moved page by page if scroller's paging is enable

1. default is move to vertical.
2. move to horizontal with Shift key

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

Reviewed By: seoz

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