evas event handling2 - fix incorrect object reporting

this fixes a new bug brought up by
9f0fd66ab818d212fa88faef316ac17625f1a2f5 which fixes event reporting
etc. etc. .. this fixes T4017

@fix

elc_combobox: add/remove item doesn't resize container with genlist

recalculation of item count is getting only on a first load of combobox.
whenever user add more items, size of container with items doesn't getting more
place (provoking scroller to appear).
but its event more annoying when user delete items, then there are lots of
free spaces left that looks ridiculus

@fix

notify: change return value of "theme_apply" function with inner _theme_apply

Summary:
notify's theme_apply function return value with eo_do_super(... elm_object_widget_theme_apply)
but notify's super class is widget, widget_theme_apply return always ELM_THEME_APPLY_SUCCESS.
so, notify's theme_apply function always return ELM_THEME_APPLY_SUCCESS, if it couldn't apply style correctly.

notify apply style via _notify_theme_apply function, so that function must decide theme_apply function's return value.

Reviewers: herb, singh.amitesh, Hermet, cedric, raster, jpeg

Reviewed By: jpeg

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

evas: Fix crash (?) in evas events

Reported by Shuhrat Dehkanov on the ML.
See f0fd66ab818d212fa88faef316ac17625f1a2f5.

Note: I didn't have a crash myself.

efl: Remove del_intercept before calling eo_del

In class destructor. Still not sure if we should do this
or just set the pointer to NULL.

Ping @TAsn

eo: Fix crash during eo_shutdown

I was getting a crash in eo_shutdown, inside
_efl_event_pointer_class_destructor as I was calling eo_del
from there. But the parent class was already destroyed.

Assuming class IDs can only go up, and child classes are only
instanciated after all their parents, it is safer to call the
class destructors in reverse order.

Obviously, still pretty sure eo_del() in a class_destructor
is not a good idea...

tests: ecore_audio: disable stalling audio_obj_pulse test

This test is stalling. Locally as well as on Jenkins. I tried to bisect it
without any luck. Even running it from the 1.17 release it does no longer work
so i guess it is some change coming from a pulse update on my system. I have
version 7.1 here. As we have no-one working actively on ecore_audio I disable
the test here and we can track the problem on T4018.

tests: ecore_audio: remove dead code

These test cases do not exist and the code is commented out for a long time.

tests: ecore_audio: move _finished_cb behind PULSE guard

This callback is only used from the pulse object test.

efreet - fix mime file path messing with short paths like /

this fixes T4015

@fix

[efl_ui_image] fix documentation

evas event handling - fix incorrect object reporting

this fixes event reporting for mouse in/out/move/down/up due to evas
just totally getting tese objects wrong.

this fixes T3718

@fix

Evas GL: Fix warning about invalid type (GLES 3.1)

elementary/fileselector: Fix legacy path set/get of Fileselector.Entry

 elementary config: Use SI units.

 efl ui frame: Update the file name for translation purposes.

 This unbreaks build.

 See 53035f23e6d4d0cfc93356c1776d8a2a58d18f91.

elc_combobox: fix theme and style set for combobox

@fix

efl - threads and signals - make efl work on "insane" os's with signals

so. on linux signals are delivered to the main process thread/loop.
thats' where signal handlers are set up and always run. this is sane.
it's predicatble. but of course this is not the same in bsd land.
there "just send the signal to any old thread and call the signal
handler there" seems to tbe the order of the day. this explains why
wer are losing sigchld signals in edje_cc - it's heavily threaded and
bsd is just randombly picking a thread to call it on.

this fixes that. in theory. i hope. i can't test, but putting it in to
share

@fix

eo: use ref types in eo_base

efl - fix lots of little init/shutdown pairs that are wrong

i've fixed almost all the eina init/shutdown pairs to do the right
thing now... except one (ecore_shutdown) with comment inline where
eo_shutdown is not called. if this is called we are in crash land.
this needs further inspection.

eina btlog - add -n option to not use color escapes

EvasGL: Add support for Evas GL 3.1

Summary:
2 wrapper functions are added.
(glFramebufferParameteri, glGetFramebufferParameteriv)

update gl2ext.h(generated 20160209) in evas_gl.h

Test Plan: OpenGL ES Conformance Test

Reviewers: jpeg

Subscribers: spacegrapher, wonsik, cedric, jpeg

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

edje: Remove shadow variable

elm_frame: updated the eo prefix and documentation

Summary: No idea what else can be done with this widget.

Reviewers: jpeg, cedric, woohyun

Subscribers: raster, cedric, jpeg

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

spinner: spinner entry reactivate when focus come again.

Summary:
When user edit spinner value on entry.
The user want to back on entry to edit spinner value even swiching view(focus out and focus in).

I considered a couple of exception case ( case that entry should not reactivate.)
1. User click spinner button to give focus.
2. User give focus to spinner's object using key action.

Test Plan:
Add sample in elementary_test.
Edit spinner value.
Gives focus to other window.
Back to spinner view.
See the action.

Reviewers: Hermet, woohyun, cedric, jpeg

Subscribers: cedric, jpeg

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

test: Fix strange black rectangle in a test case

elementary_test -to "Entry on Page Scroll"

Click on "Click this and close popup"

A weird rectangle would be temporarily visible while the
popup is fading in. The rectangle was the block_events
rectangle. By default, elm_popup blocks events, and its
parent must be the whole blocked input area. This fixes the
parent in the example.

I think the block_events region should be fading in and out
as well, because it looks terrible right now.

Note: this changes the position of the popup, but makes
the whole "block_events" thing actually work (avoiding
other render artifacts).

theme: Add fade-in/out for block regions (popup & notify)

Those are a bit faster than the content transitions. I believe the
popup transition time should be reduced. 500ms is way too long for
such animations.

elm: test_glview - allow window to close

evas: Fix rare issue with recursive proxy src invisible

Scenario:
  smart {
    text
    proxy -> text, src_invisible
  }
  proxy -> smart

What we should see:
  smart {
    (blank)
    proxy -> text
  }
  proxy -> {
    (blank)
    proxy -> text
  }

What we saw:
  smart {
    (blank)
    proxy -> text
  }
  proxy -> {
    text
    proxy -> text
  }

Solution:
Check in evas render, when we're inside a proxy render, and the
proxy src_invisible flag is on (evas_object_source_visible_set(0),
that we're rendering the object itself to its proxy surface. If not,
it means we're rendering another proxy surface, ie. a parent smart
object's proxy surface.

Still loving evas render.

Fixes T4006.

@fix

ecore: Fix obvious crash with legacy timers

ecore_timer_del() checks a flag "inside_call" that can be
set before calling the timer cb... but it was never reset
to 0. So, all legacy timers would keep on ticking forever
and ever, until they return CANCEL.

Anyway, I find the distinction between eo_del and
ecore_timer_del very troubling. eo_del() should work
on a legacy timer. Ping @cedric. Maybe override eo_del()?

Fixes T3898

edje - reduce memory footprint by rearranging structs and types

this should cut some memory used by edje by using smaller types like
shorts instead of ints where we just dont need a full int range and
short will do, and re-ordering in memory data soit packs better when
accoutning for alignment

examples: Fix typo in filename

Elementary: cnp - check on NULL data in elm_drag_start

Summary:
check on NULL add in wayland elm_drag_start API
@fix

Test Plan: Call elm_grag_start with data = NULL under wayland

Reviewers: zmike, NikaWhite, jpeg

Reviewed By: NikaWhite, jpeg

Subscribers: bu5hm4n, myoungwoon, cedric, jpeg

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

elm/gengrid: Fix null pointer dereferences.

CID: 1357210 and 1357209.

evas/textblock: Mark unused variable.

ecore-wl2: Fix error handling for bad fd

If we end up failing to get the fd from ecore_main_fd_handler_fd_get,
then we should just bail out of this function and try again later.

Fixes Coverity CID1357213

@fix

Signed-off-by: Chris Michael <cpmichael@osg.samsung.com>

elementary: Grab all key presses in elm_code for now

Working around issues where new key bindings were breaking the editor

elementary: add test for edje external combobox

elementary: release genlist item classes when done

edje_external: add combobox external

The combobox widget can now be used from edje externals.
It accepts the "guide" parameter, which is of type string.

elementary: document elm_table parameters

elementary: Update connman theme for all security

Provide the appropriate lock icon for less common security types.
@fix
also allow the icons to grow on scaled displays

Ui text: port changed_cb from entry

Canvas text: simplify efl_text_get

Ui text: move defer call to a function

Make it easier to call/debug this.

Ui text: fix leak in selection_changed

Canvas text: fix leak in annotation_insert

Canvas text: fix leak in efl_text_set

Ui text: fix invalid call to image api

Ui text: send signals to cursors explicitly

The edje code used to register the cursor objects as "edje sub-objects" of the
edje entry object. This is not edje API, so will send these signals explicitly
to the objects.

Canvas text cursor: fix bug after code port

Fixes T4005.
Also, unified repeated code and added a test for multiple cursors.

evas update buf - dont ref and unref outbug as an evas image - it's not

we need calls to ref/unref them from engines, but atm it's ok because
they dont get deleted until the flush is done and not used after that...

@fix

elm layout - add and delete signal cbs from the same consistent obj

elm layout signal handling was all over the place. using 3 different
ways of adding or deleteing signals from the object. it uses either
obj directly, eo_super(obj) or wd->resize_obj. come on. be consistent.
so using wd->resize_obj worked before and now works properly with
sgnal cbs PROPELY deleted unlike before.

@fix

edje signal cb - dont delete already deleted signals

let's assume you registered a callback twice with the same ptr data
and same func and same sig/src... if you delete it once you're ok.
delete twice... and you re-delete the first one (just makging it for
delete_me). this fixes this corner case

@fix

ecore-wl2: Fix uninitialized variable

Coverity reports that len is used un-init here, so let's give it a
value at declaration time

Fixes Coverity CID1357214

@fix

Signed-off-by: Chris Michael <cpmichael@osg.samsung.com>

eolian: cleanup type serialization logic

gengrid: fix focus issue when prev/next item is disabled

Summary:
After setting focus to disabled item, focus does not work properly.
It is necessary to consider disabled state of each item
when selecting next focused item.

Signed-off-by: Jehun Lim <jehun.lim@samsung.com>
Test Plan:
1. elementary_test -to 'gengrid disabled item focus'
2. check 'item select on focus disable'
3. move focus to disabled item
4. observe focus and compare before this patch

Reviewers: cedric

Subscribers: minkyu, cedric, jpeg

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

tests: Disable profile_exists as it fails on jenkins

theme: return enum from elm_widget_style_set instead of bool

Summary:
if trying to apply incorrect theme, widget apply default theme and return TRUE.
so there is no way to check it really apply correct theme.
To resolve this problem, _elm_theme_set return three type enum

* related history : 4ca3ef45146e05908a13d1010909abeba9693ad6
* elm_object_style_set is public api, so I didn't change it.
* typedef name [ Theme_Apply ] is temporarily, please suggest better one.

@fix

Reviewers: singh.amitesh, herb, Hermet, cedric, jpeg, raster

Subscribers: cedric, jpeg

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

elm: fix compilation

"
make[1]: *** No rule to make target 'efl_ui_text.h', needed by 'all-am'.  Stop.
"

edje_pick: load just used images.

If the edj is made by partial groups, the images that groups use
should be included.

@fix

wayland_shm: free dmabuf resources on fallback

Add destructors for the buffer managers, and clean up properly when
falling back.

Fixes Coverity CID1357163 and CID1357162

elc_hoversel: just little fix of expand/noitem behavior

when hoversel has no item at all and use clicks on it, then it goes into
state called "expanded".
and so then, no matter how many items user would try to add, hoversel won't work
anymore.

@fix

examples: update various gitignore files for new example binaries

These have been forgotten to be added recently.

atspi_image: improve existing documentation

Mixin and property descriptions.

atspi_widget_action: add mixin description

elm_scrollable: add mixin description

atspi_component: improve existing documentation

Ad more descriptions, consolidate property descriptions and fix smaller
problems.

atspi_action: improve existing documentation

Fix typos, add parameter descriptions and consolidate property describtions.

atspi_accessible: improve existing documentation

Make sure we describe whole properties and not getters and setters only. Adding
some more docs while being there.

eolian: add support for static and terminated arrays

Adds two new type types, STATIC_ARRAY and TERMINATED_ARRAY. Static arrays are
only allowed as struct members right now - they translate to regular C static
arrays (allowing them elsewhere wouldn't be good, as C isn't very good at
working with the size information). Terminated arrays are basically sequences
of data terminated at the end. The base type of static arrays can be any type
that is not marked ref (explicit ref may get allowed later). The base type of
terminated arrays has the same restriction plus that it has to be either
implicitly reference type (i.e. translating to pointer in C), integer type
or a character. In case of ref types, the terminator is NULL. In case of
integer types, the terminator is a zero. In case of character types, the
terminator is also a zero (null terminator like C strings).

@feature

ecore-wl2: Fix issue where hash would not get destroyed on error

Coverity reports that we could not reach the eina_hash_free here on
erorr condition. This was due to an erroneous return NULL above the if
(hash_create) check.

Fixes Coverity CID1357171

@fix

Signed-off-by: Chris Michael <cpmichael@osg.samsung.com>

ecore-drm2: Don't leak return from _output_name_get function

Coverity reports that we end up leaking the return from
_output_name_get as the storage is never freed.

Fixes Coverity CID1357161

@fix

Signed-off-by: Chris Michael <cpmichael@osg.samsung.com>

ecore-wl2: Fix issue of passing negative number to close() and read()

We should be checking the return value of ecore_main_fd_handler_fd_get
calls as they can return a negative number...which cannot be passed to
the close() or read() functions.

Fixes Coverity CID1357152 and CID1357153

@fix

Signed-off-by: Chris Michael <cpmichael@osg.samsung.com>

ecore-wl2: Check the return value of ecore_main_fd_handler_fd_get

This patch fixes an issue where ecore_main_fd_handler_fd_get could be
returning a negative number and passing that to close() which cannot
accept negative numbers.

Fixes Coverity CID1357152

@fix

Signed-off-by: Chris Michael <cpmichael@osg.samsung.com>

ecore-evas-wayland: Fix issue of passing wrong values to resize

Coverity detected that the same code was being passed to resize
regardless of canvas orientation. This patch fixes the issue by
passing the proper values to window_resize.

Fixes Coverity CID1357150

@fix

Signed-off-by: Chris Michael <cpmichael@osg.samsung.com>

ecore-wl2: Fix derefence after null check

'window' here could potentially be null so we should be checking that
it is valid before trying to assign event window

Fixes Coverity CID1357148

@fix

Signed-off-by: Chris Michael <cpmichael@osg.samsung.com>

elementary: Fix explicit null dereference

This patch fixes an issue reported by coverity where 'type' variable
could be null and passing null to strcmp is not good ;)

Fixes Coverity CID1357147

@fix

Signed-off-by: Chris Michael <cpmichael@osg.samsung.com>

evas-wayland-shm: Remove logically dead code

Coverity reports that this bpp check is actuall dead code due to the
fact that bpp can never be < 0.

Fixes Coverity CID1357144

@fix

Signed-off-by: Chris Michael <cpmichael@osg.samsung.com>

elput: Check return of read function call

If we fail to read any bytes from the fd, then we should be returning
a failure here.

Fixes Coverity CID1357143

@fix

Signed-off-by: Chris Michael <cpmichael@osg.samsung.com>

ecore-evas: Fix error handling issues in wayland_shm engine

This patch fixes 2 Coverity issues where engine_info_set was being
called, but the return was never being checked.

Fixes Coverity CID1357141 and CID1357142

@fix

Signed-off-by: Chris Michael <cpmichael@osg.samsung.com>

ecore-input: Check return fo ecore_main_fd_handler_fd_get

This patch fixes 2 coverity Error handling issues.
ecore_main_fd_handler_fd_get can return a negative fd on error so we
should be checking that return before trying to make use of the fd.

Fixes Coverity CID1356632 and CID1356624

@fix

Signed-off-by: Chris Michael <cpmichael@osg.samsung.com>

edje: fix return type for part_text_cursor_content_get

Another spot that should've stayed "char *" and not "string". Also added the
missing 'own'.
Found due to a warning after ee36fdc2183a3de9569cac5ae74c0fb30d3d76db.

ector: add basic description to renderer mixins

ector: further document gl buffer base class

efl: gfx_shape: add some more documentation and improve existing one

efl: gfx_shape: document property itself and not the getters and setters

edje_edit: remove useless pointer grab as it conflicts with elementary combobox

Summary:
So currently having an issue related to clip_to function from
EdjeEdit API. I am not sure about how and why combobox use pointer grabbing
(just started to use it) but when setting NULL into clip_to of edje edit it
suddenly block entire screen away from mouse. Only when point is moved out of
window and back, only then it will be able to access widgets again.

Take a look at video and maybe try example to reproduce problem

I am not sure if deleting those functions from edje_edit is actually right
decision. But it will fix the problem of course.

Test Plan: See attached video and examples in attached files

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

Subscribers: jpeg

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

Revert "elm: Fix wrong use of char instead of string and void out parameter"

This reverts commit 61f52a26fb5a512e77d3141e398182107b1cf752.

This fixes the reported break on T4000.

slider: fix edje warning for vertical style

Follow-up of D4117

slider: fix edje warning

Summary:
ERR<18878>:edje lib/edje/edje_calc.c:2767 _edje_part_recalc_single() file /usr/local/share/elementary/themes/default.edj, group elm/slider/horizontal/default has a non-fixed part 'elm.swallow.icon'. You should add 'fixed: 1 0'. But in order to optimize the edje calc, we add it automatically.
ERR<18878>:edje lib/edje/edje_calc.c:2767 _edje_part_recalc_single() file /usr/local/share/elementary/themes/default.edj, group elm/slider/horizontal/default has a non-fixed part 'elm.swallow.end'. You should add 'fixed: 1 0'. But in order to optimize the edje calc, we add it automatically.

Test Plan: Launch an app with slider, open the slider and observe the console output

Reviewers: cedric, singh.amitesh, jpeg

Subscribers: seoz, minkyu

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

elm_widget: Fix mouse wheel event propagation

The wrong type was propagated. Luckily it didn't crash.

Thanks @Hermet for the report.

edje: Lower ERR to INF for message about non fixed parts

This message really is just informational at best, since it's
not an error, and we even implemented an optimization.

It should be done at compile (edje_cc) time though.
See comments on D3801

evas object: Move display_mode to Efl.Gfx.Size.Hint

win: Remove font & image cache functions

Those actually belong to elm_config, or rather Efl.Config:
  efl_config_int_set("cache_image_cache_size", 42);
  efl_config_int_set("cache_font_cache_size", 1337);

layout: Add NULL check in _elm_layout_text_set function

When the parameter 'text' is NULL in elm_layout_text_set function,
the sub object data with the same part name is removed
from the layout's sub object list and the function returns immediately.
However, if the text part doesn't exist in the list,
a new sub object data is appended to the list even though the text is NULL.
This patch adds NULL check so the function can return in such cases.

photocam: add missing EOLIAN prefix

ecore-wl2: Fix formatting

NB: No functional changes

Signed-off-by: Chris Michael <cpmichael@osg.samsung.com>

elementary: Fix erroneous usage of EINA_UNUSED

'data' parameter is actually used in this function, so remove mis-use
of EINA_UNUSED in function

@fix

Signed-off-by: Chris Michael <cpmichael@osg.samsung.com>

elementary: Fix over-zealous optimization attempt

This is a fix for some over-zealous optimization attempt. Basically,
we cannot optimize out multiple calls to ecore_wl2_window_input_get as
the window can change in the for loop.

@fix

Signed-off-by: Chris Michael <cpmichael@osg.samsung.com>