ecore-wl2: Add API function to set if wm rotation is supported

Small patch to add an API function which can be called to set if
window manager rotation is supported.

@feature

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

ecore-drm2: Group API functions within their doxy group

NB: No functional changes

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

ecore-x: Minor formatting fix

NB: No functional changes

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

elm_fileselector_entry: this does not need focus

its composed by focusable widgets, no need to focus them by themself

elm_fileselector: this is not focusable

the widget is composed from focusable widgets the fileselector is not
focusable.

eolian: prepare to defer all dependency parses

Dependencies are now going to be parsed strictly after the main
file, to allow proper dependency cycle behavior in all places. For
now there is correct parsing of deferred deps but they're not
queued yet.

Efl.Ui.Image.Zoomable: make zoomable to work with edj file

image.zoomable can load edj files now.

usage:
efl_file_set(zoomable, "../somefile.edj", "mygroupname");

test:
elm_test -> photocam
click "open" btn
select an edj file, it would show first group in selected edj file.

@feature

elm_widget: add factory method to the widget

with this function you can trap the creation of the mangers, and change
the behaviour as you need it. For example returning a custom manager.

map: Make sure efl_gfx_map_reset triggers a render

Thanks again Jaehyun for reporting this!

Genlist: Calculate the min size of content before item realize

Summary:
Genlist item doesn't change its size when its content size is changed,
but its size is determined in realization.
Therefore, deferred calculations for content should be performed immediately
before swallowing it by genlist item.

Test Plan: make and run attached sample

Reviewers: jpeg, SanghyeonLee, cedric

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

elm map: Fix broken link to openstreetmap.org

Generated link to openstreetmap.org is broken because of the comma in
the last of URL.

elementary entry: update focus state when editable state is changed

Summary:
To recieve keyboard events, the entry_edje should have Evas focus.
But, if a non editable Entry widget takes focus, it can't recieve
keyboard events even if it becomes editable after taking focus.
So, elm_entry_editable_set() function should update Entry's focus state.
@fix

Test Plan:
The code of elementary_test - entry is modified to test this issue.
Please, check the issue with the following steps.

1. Run "elementary_test entry"
2. Click "Unfocus" button to make entry to "unfocused" state.
3. Click "Edit" button to make entry to non-editable mode.
4. Click "Focus" button to make entry to "focused" state.
5. Click "Edit" button to make entry to editable mode.
6. See a cursor is blinking in entry.
   => But, you can't edit text without this patch.

Reviewers: raster, herdsman, cedric, jpeg, woohyun

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

elm-test: ui.image.zoomable: make anim example to work with zoom

mousewheel can be used to zoom in and zoom out.

map: Fix efl.gfx.map render

Internally the call to map_reset was setting the count to 0
rendering the map data invalid and useless.

examples: Flesh out the evas-buffer-simple example

Summary:
This serves as an early example for new Evas programmers to introduce
the buffer engine as well as very basic canvas usage, so we're going
into a lot more detail for both of those than we'll do in subsequent
examples.

Reviewers: cedric

Subscribers: jpeg

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

ecore_evas: Document the X11 XShape input region APIs

Reviewers: cedric

Subscribers: jpeg

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

elput: Fix compilation warning (GCC)

Use GCC sematic {} instead of {0} as the array members are structs,
thus {0} isn't valid either.

map: add more checking on input and memory allocation

More checking on input parameter and memory allocation
when set number of map points.
Thanks jp.

Elementary: fix Elm.Layout Model Connect

Use "elm" has source to model_connect update
Disconnect part/property in model_connect

Ecore: fix Efl.Model.Item

sent property_changed signal in property_set

eina: fix implicit declaration of pthread_set_name_np in eina_debug.

Summary: Checks and includes necessary headers (FreeBSD et. al)

Reviewers: cedric

Subscribers: jpeg

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

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

eina: as we use string.h function in this inline function, we better include it.

Reported by @netstar, thanks.

ecore-evas-x: Remove blank line

NB: No functional changes

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

efl_debugd: only compile this on Linux.

Summary:
Well, the build is broken again on BSD and Windows.
efl_debugd is full of lots lovely Linuxisms. Just
don't compile.

This unbreaks the build for FreeBSD and others.

Reviewers: cedric, raster, jpeg

Reviewed By: jpeg

Subscribers: JackDanielZ, jpeg

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

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

ecore-drm2: Fix potential SIGFPE when adding output mode

If we receive bad crtc info from libdrm, then we could end up with a
SIGFPE here due to division by zero if info h/v total are not set.

@fix

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

ecore-drm2: Add API function to set pointer rotation

This patch adds a new API function which can be used to swap x & y
pointer axis and invert them according to rotation angle. Mouse input
events occur according to canvas coordinates so this can be used when
a canvas is rotated.

@feature

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

elput: Add API function to swap dx & dy axis from pointer motion event

Small patch which adds an API function that can be called to swap x
and y axis and invert them according to rotation angle.

@feature

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

elput: Minor formatting fixes

NB: No functional changes

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

elput: Minor formatting fix

NB: No functional changes

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

elput: Get touch event slots using slot not seat_slot

@fix

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

ecore-evas-wayland: Fix potential null dereference

Quick patch to fix a potential null derefence in render_flush_pre
callback.

@fix

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

evas-gl-drm: Fix uninitialized variable

Small patch to fix an uninitialized variable when using
eglGetConfigAttrib()

@fix

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

evas-gl-drm: Be sure to initialize ncfg variable

This could lead to an uninitialized variable or use of ncfg variable
without a value.

@fix

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

widget: Remove unnecessary define

Looks like a copy & paste went wrong :)
Thanks @bu5hm4n

edje - fix signal propagate crash

fixes segv when using boxes/tables and sub groups with signal
propagation working in _cb_signal_repeat() when name is NULL (not
found/there)

@fix

elm_test:ui.image.zoomable: Allow image click to enable/disable animation

its better to have animation start/stop on image click rather
than clicking a button.

Efl.Ui.Image.Zoomable: inherit elm.widget as class

this fixes a mistake done in cb1d53c835ef465
Thanks to jp for notifying it.

ref T5352

Efl.Ui.Image.Zoomable: implement efl.play.get/set

ref T5352

Efl.Ui.Image.Zoomable: implement efl.player.playable.get

ref T5352

edje: pass whole text to IMF in case of selection

Summary:
This patch partially reverts D2951.
_edje_entry_imf_retrieve_surrounding_cb() function has to pass
whole text nearby the entry's cursor. If IMF needs to ignore
selected text when the entry has selection, IMF can check
selected text by calling _edje_entry_imf_retrieve_selection_cb().
So, we don't need remove selected text before passing it to IMF.
@fix

Test Plan: N/A

Reviewers: woohyun, jihoon, subodh6129

Reviewed By: subodh6129

Subscribers: cedric, jpeg

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

efl.gfx.map: support for map points as multiples of 4

This patch adds support for map with more than 4 points.
However, we only support points with multiples of 4.
This also adds efl_gfx_map_point_count_set/get APIs to
set number of points.

@feature

evas_map: support map with number of points as multiples of 4

Currently, in evas map, we only support map with 4 points.
This patch adds support for map with number of points as
multiples of 4.

@feature

edje: Fix warnings introduced in recent fix

See b9b0177d5e0f2b3a03f77e04c64de534b2c786ac
See d3420cd35c692a808c0a626c8e54a0c85af3a4d4

elm_panel: set hidden property when panel is close/open

Summary:
 - elm_panel has a property named hidden which stores
   open/close status.
 - This is updated when:
    1. bring_in animation is done(anim_stop_cb).
    2. mouse_up on panel.
    3. API is called. (elm_panel_toggle, elm_panel_hidden_set)
 - In case 3, API changes hidden, and starts bring_in animation
   which will call anim_stop_cb() which will update hidden again.
 - If bring_in animation is canceled (eg: sizing_eval),
   anim_stop_cb will be called and calculate hidden status
   which will not guarantee updated hidden state by APIs.

Test Plan:
   1. Call any APIs which will call elm_layout_sizing_eval(panel)
      right after calling elm_panel_toggle()/elm_panel_hidden_set().
   2. Delete content of panel during "toggled" cb.

Reviewers: jpeg, eunue, cedric

Subscribers: conr2d, cedric, jpeg

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

Signed-off-by: Jean-Philippe Andre <jp.andre@samsung.com>

evas - font dir fonts.alias - limit alias parse length

a rouge fonts.alias file in a font dir/path could contain long strings
that buffers dont have space for, so limit length of string.

@fix

elm spinner - handle more digits for format string + handle invalid fmt

handles format strings like %0.234567f even tho before it was just %2f
- single digit after . ... not that it's useful handling more...

@fix

evas xpm loader - add more rgb.txt file locations...

2 more. /etc/X11/rgb.txt /usr/share/vim/vim80/rgb.txt  ...what i do notice
is that this file seems to have vanished from modern systems... so we'll have
lots of un-fun loading old xpm's with colornames if we cant figure out what
color names map to what colors...

edje_cc handle line too long for buffers

if a line from the cpp processor is too long (more than 4k) then abort
compilation as something is wrong asnd our buffers can't handle it
anyway.

@fix

elm_prefs_cc: handle line too long for buffers

if a line from the cpp processor is too long (more than 4k) then abort
compilation as something is wrong asnd our buffers can't handle it
anyway.

@fix

evas-gl-drm: Fix potential memory leak

If/When we exit _evas_outbuf_egl_setup function, we should be freeing
the allocated cfgs variable else we leak it.

@fix

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

ecore-wl2: Use output height when calculating dpi

When calculating output dpi, we should also be considering output
height in the calculation.

@fix

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

ecore-wl2: Add API function to set surface buffer transform

Small patch to add an API function which can be called to set a buffer
transformation on a given window.

@feature

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

ecore-wl2: Add API function to return an output transformation

Small patch to add a new API function that can be used to retrieve the
current transform value for a given output.

@feature

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

ecore-wl2: Add missing doxygen groups for documentation

@fix

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

ecore-wl2: Add API function to find an output for given window

Small patch which adds an API function that can be used to find the
output where a given window resides.

@feature

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

ecore-wl2: Add event for output transformation

Some servers may need to do special animations or other effects when
an output transformation happens, so raise an event in the output
geometry callback when this happens.

@feature

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

ecore-evas-wl: Don't check window visibility on raising

As we won't get a request from a server to raise a window which is not
visible anyway, this check is useless.

@fix

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

Efl.Ui.Image: fix efl.player.playable.get API

an edje obj is always playable. return TRUE always.

@fix

scroller : Call mirrored_set function after edje_obj is initialized

Summary:
Problem that scroller is not mirrored in RTL mode has been fixed in D4908.
However, I found a problem in init process.
I will revert D4908 commit(https://phab.enlightenment.org/D4908).

After edje_obj is initialized, mirrored_set is called.
When wanted_region_set is called, x-coordinate flip normally and acts as RTL.

Test Plan: scroller test on elementary_test.

Reviewers: raster, woohyun, SanghyeonLee, akanad, eagleeye, cedric, singh.amitesh, eunue

Subscribers: jpeg, cedric

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

evas: Set locks and modifiers for all input events

See also eb27724eb9c32a954c2f75b45307a207d0 which didn't fix
the reported issue. Not quite sure yet how to implement those
lock & modifiers for fake events (i.e. artificially created
by the app itself).

Fixes T5510

edje: Remove external param_type from EO

This is not exactly Legacy API but at the same time doesn't
belong to EO (see previous commits).

Revert "scroller : fix mirrored contents_pos_set call"

Summary:
This reverts commit de313d6296ac8a2d9a3c0f9762179ffd42a833ff.

Because x-coordination mirroring is called properly from wanted_regison_set.
There was a problem in scroller init process.
Before edje_obj was initialize in elm_interface_scrollable_objects_set, mirrored_set was being called.
So Move mirrored_set after elm_interface_scrollable_objects_set
it can fix this issue(https://phab.enlightenment.org/D4944)

Reviewers: raster, woohyun, SanghyeonLee, akanad, eagleeye, cedric, singh.amitesh, eunue

Subscribers: jpeg, cedric

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

docs: Fix typos and some wrong expressions in Ecore, Ecore_Audio, Ecore_Cocoa, Ecore_Con, and Ector API reference doxygen.

Summary: I had fixed some typos and wrong expressions, such as capital letters, $simbols in .eo and singulars in Ecore, Ecore_Audio, Ecore_Cocoa, Ecore_Con, and Ector API reference doxygen.

Test Plan: Doxygen Revision

Reviewers: Jaehyun_Cho, stefan, jpeg, cedric, raster

Reviewed By: Jaehyun_Cho

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

ephysics: set proper geometry when setting an object to a face

Summary:
It was missing a break statement in a switch.

Fixes T5547

@fix

Reviewers: cedric

Subscribers: jpeg

Maniphest Tasks: T5547

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

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

eina: enable fallthrough warning suppression only on GCC 7 and above.

ecore-drm2: Don't use Atomic functions to set DPMS levels

This was initially an experiment in trying to use Atomic properties to
set dpms on/off, however it does not turn off backlight support when
triggered so it is useless.

Fixes T5462

@fix

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

ecore-drm2: Fix output backlight value type definitions

When we go to set output backlight level we use doubles, so lets just
store these values as doubles in the structure.

@fix

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

ecore-drm2: Add internal functions to get backlight values

Small patch to add an internal function which can be used to retrieve
backlight values on output creation.

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

eina: fix clang compilation with EINA_FALLTHROUGH

btw, using clang 4.0.0, I can't produce fallthrough warnings using
-Weverything, -Wall or -Wimplicit-fallthrough

ecore_drm2: Fix error handling in _output_dpms_atomic_set

This was skipping the error path on failure and setting some state as if
it was successful.  Then the next attempt at a page flip was actually
setting this state.

So _output_dpms_atomic_set (which has always been broken) wasn't actually
the function that successfully disabled dpms.

This is confounding attempts to debug why dpms isn't coming back on
properly.

Now it won't turn *off* either, because it really never should have.

Ref T5462

scroller : fix mirrored contents_pos_set call

Summary:
Fixes that a scroller can not be moved normally even if it is changed to mirrored.
In elm_scroller, _mirrored_set is called when sizing_eval is called. because
When content size is changed, it should be scrolled based on mirrored coordinates.
Also In elm_interface_scrollable,
elm_interface_scrollable_content_pos_set of _elm_scrollable_mirrored_set
to be called regardless of mirrored state.

Test Plan: scroller test on elementary_test.

Reviewers: raster, woohyun, SanghyeonLee, akanad, eagleeye, cedric, singh.amitesh, eunue

Subscribers: cedric, jpeg

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

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

elc_multibuttonentry: role of MBE item, MBE updated, and state_set_get API added.

Summary:
Reason for role change:
MBE items do not  act like push buttons, they need to maintain their status and at a time only one item can be selected, they act like radio buttons.
Role change of MBE from entry to panel, because MBE is just a container and entry is one of the children, when entry gets access frame, it will work as how entry works, but making MBE role as entry does not work as expected and also MBE should not receive highlight as a whole but only its children should receive highlight one more reason why MBE role cannot be entry.
state_set_get API added to get items state.

Test Plan:
When atspi mode is enabled the access frame has to
navigate directly on item and not on MBE as a whole and state has to
be read out

Reviewers: kimcinoo, cedric

Reviewed By: cedric

Subscribers: cedric, govi, rajeshps, jpeg

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

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

ecore_evas: use strbuf instead of PATH_MAX.

elm_config: password mode change should reflect on already running applications.

Summary:
When password mode is changed, the change has to reflect on already running
apps as well. config_sub_apply will not be called for an already running app.

Test Plan:
1. 2 running process
2. Change the config value of password last show in one process
3. Check whether the change has reflected in previous process.

Signed-off By: Shilpa Singh<shilpa.singh@samsung.com>

Reviewers: jpeg, cedric, thiepha

Subscribers: govi, rajeshps

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

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

eldbus: add test suite for eldbus (eldbus_signal_handler*)

Summary:
add tests for next API:
eldbus_signal_handler_add()
eldbus_signal_handler_del()
eldbus_signal_handler_connection_get()
eldbus_signal_handler_interface_get()
eldbus_signal_handler_member_get()
eldbus_signal_handler_path_get()
eldbus_signal_handler_sender_get()
eldbus_signal_handler_ref()
eldbus_signal_handler_unref()
eldbus_signal_handler_free_cb_add()
eldbus_signal_handler_free_cb_del()

Reviewers: cedric, NikaWhite, myoungwoon, raster, artem.popov

Subscribers: jpeg

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

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

elementary: refactor day handling in calendar to just use unsigned char.

edje: clear up left over warning on fallthrough switch case statement.

edje: use strbuf instead of hard coded PATH_MAX.

ecore_evas: reuse code by relying on switch case statement fallthrough for key offset case.

evas: reuse code for colorspace logic by using switch case statement fallthrough.

elementary: reuse code by relying on switch case statement fallthrough.

evas: use switch case statement fallthrough to handle colorspace logic reuse.

evas: TGA does use switch case statement fallthrough to factorize code.

evas: silence warning for SSE3 optimization.

draw: silence warning in a tricky loop/switch statement.

edje: object have small difference and we use switch case fallthrough to reuse code.

eet: silence properly reused code through switch case statement fallthrough.

edje_cc: when first parameter is a string and the second one a range, just reuse code.

emile: SSL state should fallthrough to avoid duplication of logic.

emile: fallthrough is legal here as we handle fallback case when caller doesn't know about GRY8.

eina: add EINA_FALLTHROUGH to disable warning on useful case serie with no break.

GCC has started introducing a detection for series of case in a switch statement
without break for each case. We do use that trick a lot to reduce our code base.
Even if in most case we have documented this so that people using coverity don't
try to fix it. Now with GCC we need to silence it properly to avoid future
problem.

ecore_wl2: silence gcc warning.

Code is correct, but gcc can not follow the branch if and believe it
is worth triggering a warning.

eina: silence gcc warning.

Code is correct and won't lead to a case where size is > 0 and buffer
isn't set, but gcc can't see the relation between both variable.

evas/examples: add intro for evas-event-filter.c example

Reviewers: cedric

Reviewed By: cedric

Subscribers: jpeg

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

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

evas/examples: fix minor typo 'whe'

Reviewers: cedric

Subscribers: jpeg

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

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

examples/evas: add tutorial commentary for evas-init-shutdown

Summary:
This is the most basic of the Evas examples and serves as the starting
point for new Evas users.  Since this targets neophytes, we can afford
to be much more detailed in commentary.

Reviewers: cedric

Subscribers: jpeg

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

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

examples/evas: add a README

Summary:
Categorize the examples by topic, and identify certain examples as
introductory; these can be commented more verbosely than the other
examples.

Signed-off-by: Bryce Harrington <bryce@osg.samsung.com>
Reviewers: cedric

Reviewed By: cedric

Subscribers: jpeg

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

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

eldbus: add test suite for eldbus (eldbus_message*)

Summary:
add tests for next API
eldbus_message_arguments_vget()
eldbus_message_arguments_append()
eldbus_message_error_get()
eldbus_message_from_eina_value()
eldbus_message_iter_struct_like_to_eina_value()
eldbus_message_iter_arguments_vappend()
eldbus_message_iter_arguments_vget()
eldbus_message_signature_get()
eldbus_message_method_return_new()
eldbus_message_error_get()
eldbus_message_iter_container_new()
eldbus_message_iter_fixed_array_append()
eldbus_message_method_return_new()
eldbus_message_error_new()
eldbus_message_iter_del()
eldbus_message_iter_fixed_array_get()
eldbus_hello()

Reviewers: cedric, raster, NikaWhite, myoungwoon, artem.popov

Subscribers: jpeg

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

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

evas: revise grammar for memory allocator error status

Summary:
Wordsmith phrasings to be a bit more concise.  In example code,
distinguish better between the OOM and Low Mem conditions that it
is demonstrating.

Reviewers: cedric

Subscribers: jpeg

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

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

popup: fix unintentional object deletion when change scrollable

Summary:
If popup scrollable change from FALSE to TRUE after content or text is added,
content_area is deleted, so it cause null pointer problem.
main_layout has content_area in its CONTENT_PART, but in scrollable_set API,
tbl set into main_layout's CONTENT_PART, so content_area is deleted.

On the contrary, if scrollable change to FALSE,
content object in content_area is deleted, because tbl set into content_area's CONTENT_PART.

So if some object set into other object,
unset previous object and set it into properly part after that.

A case using item_append should be fixed by other patch.

@fix

Test Plan: Change scrollable repeatedly after adding content or text to popup.

Reviewers: jpeg, singh.amitesh, conr2d, cedric, raster

Reviewed By: cedric

Subscribers: herb

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

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