evas gl: Add a hack for GLES 3.1 with NVIDIA + GLVND

I believe there's a limitation in libglvnd where it doesn't
propagate the calls to eglGetProcAddress() properly to the vendor
library.

See also 265c851a8f2ac3cd1f93d523736d4cce0454fe2c

evas: Fix crash with events on the canvas

Since ecore now uses efl events to feed input events to the
canvas, anyone can now listen to any event on the evas. But
when using the legacy API the event info needs to be the legacy
struct, and not the eo event info otherwise crashes will happen.

While this is a new use of events, I consider it valid and it's
better to fix it rather than disallowing it. Fixed by wrapping
evas events the same way evas object events were handled.

Fixes T5266

evas gl: Fix usage of OSMesa

It seems OSMesa was recently updated to not expose symbols statically,
so dlsym() returns invariably NULL. GetProcAddress must be used. Note
though that the extension "EGL_KHR_get_all_proc_addresses" is not
present (OSMesa is OpenGL, not GLES), and there is anyway no list
of extensions in OSMesa (at the WSI level, glGetString() returns a
ton of GL extensions as expected).

My OSMesa version is 11.2.0 (mesa 17.0.1).

This fixes make check.

@fix

edje: Move prev_description out of ephysics ifdef

prev_description was used when HAVE_EPHYSICS is set, which is the
default, but I also added a use in 7072fbc2bfe25881 where the map
was not properly reset.

This removes an ugly #ifdef and opens the door to other fixes
similar to that map one.

genlist: Fix some more decorate mode issues

The item, after having been unswallowed from its decorate
item, becomes unclipped and unparented. The parent was well
reset, but the clip wasn't.

Test case:
  elementary_test -to "Genlist Decorate Item Mode"

I'm sure some bugs are still lurking. Genlist is so lovely.

Fixes T1551

edje: Fix rare issue with map

In the following sequence, the swallowed object map property is
never reset as it should have been:

- swallow object
- start program, change state to have a map
- do something
- start program, change state to have no map
    but before render, unswallow the object

At this point, the object will never be un-mapped. This is weird.
Somehow edje_calc avoids calling evas_object_map_[enable_]_set
excessively, but I believe the issue is that the object does not
need recalc. Its container needed recalc, not the child (which is
mapped). I'm not 100% sure.

Test case:
  elementary_test -to "Genlist Decorate Item Mode"
  Click on rotate, select a few items, scroll up and down. Enjoy.

Ref T1551

@fix

genlist: Make sure only one item is decorated

In "Genlist Decorate Item Mode" after decorating a few items
(rotate or slide, doesn't matter), only one item or none should
be decorated. Scrolling up and down the genlist should work just
fine. This fixes massive render issues and inconsistent states
of the items in this test case.

"rotate" mode is still going nuts.

Ref T1551

genlist: Make logic more readable

genlist: Fix invalid call to stack above/below

Test process:
  elementary_test -to "Genlist Decorate Item Mode"

Slide any item to the right, observe error logs.

Ref T1551

evas: Add more debug info to an ERR message (stack)

Also use eina_safety as those are safety checks. Helps with GDB
(single breakpoint in eina_safety).

Revert "ecore-evas reduce modifier modification on every event to cut cpu"

This reverts commit 3a9d54085b8751cbd0bca4698a8ff702d9bdc7b4.

I got crash and a lot of valgrind warning with this patch. All in all, I
think we can just wait for next release and do a proper cleanup of our API
to not rely on strings at all.

For references this is the first valgrind warning I get (Not going to past the 100 following one) :

==11860== Invalid write of size 4
==11860==    at 0xB10DDD1: _ecore_event_evas_modifier_lock_update (ecore_input_evas.c:432)
==11860==    by 0xB10E3DD: ecore_event_evas_mouse_move (ecore_input_evas.c:725)
==11860==    by 0x5D5115D: _ecore_call_handler_cb (ecore_private.h:317)
==11860==    by 0x5D5115D: _ecore_event_call (ecore_events.c:518)
==11860==    by 0x5D5CC47: _ecore_main_loop_iterate_internal (ecore_main.c:2381)
==11860==    by 0x5D5D42E: ecore_main_loop_begin (ecore_main.c:1289)
==11860==    by 0x5D5D490: _efl_loop_begin (ecore_main.c:2831)
==11860==    by 0x5D59555: efl_loop_begin (efl_loop.eo.c:32)
==11860==    by 0x14F275: main (test.c:1188)
==11860==  Address 0x19a36828 is 7 bytes after a block of size 1 alloc'd
==11860==    at 0x4C2AACE: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==11860==    by 0x4C2CC81: realloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==11860==    by 0xB10DDA8: _ecore_event_evas_modifier_lock_update (ecore_input_evas.c:425)
==11860==    by 0xB10E3DD: ecore_event_evas_mouse_move (ecore_input_evas.c:725)
==11860==    by 0x5D5115D: _ecore_call_handler_cb (ecore_private.h:317)
==11860==    by 0x5D5115D: _ecore_event_call (ecore_events.c:518)
==11860==    by 0x5D5CC47: _ecore_main_loop_iterate_internal (ecore_main.c:2381)
==11860==    by 0x5D5D42E: ecore_main_loop_begin (ecore_main.c:1289)
==11860==    by 0x5D5D490: _efl_loop_begin (ecore_main.c:2831)
==11860==    by 0x5D59555: efl_loop_begin (efl_loop.eo.c:32)
==11860==    by 0x14F275: main (test.c:1188)

eina: Fix more typo in doxygen

The eina_(rw)_slice_startswith functions both incorrectly describe how
the 'prefix' parameter is used, so fix those also

@fix

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

eina: Fix typo in doxygen

eina_(rw)_slice_startswith functions both incorrectly describe the
return value as 'slice ends with', when clearly the function is used
to find if a slice 'starts with' a prefix

@fix

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

ecore-evas reduce modifier modification on every event to cut cpu

this only modifiers modifiers if the modifier mask changed. it stores
it per seat and matches up before deciding to actually modifier the
modifiers. this SHOULD fix T5252

@fix

elementary: refactor code to avoid duplication in elm_code_widget.

ecore_evas: keep VNC API BETA until multi evas output is in, just in case.

elm_code: Don't leak rows when we resize down

Thanks Cedric :)
@fix

elementary: force elm_code line fill on object resize.

This fix elementary_test use of elm_code where the object needed
to get clicked to have their content displayed. The reason was that
the line where appended while the object size was 0 and once it was
resized the line where not displayed.

luncher: Smoothing engage theme

edje/examples: don't alter source directory

We would have removed files from the source directory before... oops...

elementary: fix elm_code widget tests by actually setting a font.

luncher: Update theme to ensure we're sending scale reset signls

ecore-evas-wayland: update no-op configure commit case for new acking method

ref 1b9970658f17d6b130a24cf9a960e1a43b696160
ref 5e7456a3150ddaed135f702c70800c7ed3d4f16f

Revert "evas_table: abort when there is already the option struct"

This reverts commit 008711b3b0f0b33e47e6b2fae7f925a0a9fb76ee.

this breaks elm_table_pack_set() as well as other valid usage and is
not a valid change to the code

ecore-evas-x: set flag for maximized state

fixes initial maximize state

ecore-evas-wayland: commit on no-op configure events

this handles the case of a window configure not triggering any visible change

fix T5192

elm_code: Fix position of cursor in selection

Make sure cursor is always at the left of our selection
@fix

Gengrid: Correct double comparision.

Summary:
 Use the more accurate EINA_DBL_EQ for double comparision.

@fix

Signed-off-by: Umesh Tanwar <umesh.tanwar@samsung.com>
Reviewers: singh.amitesh, cedric, jpeg, raster

Reviewed By: singh.amitesh

Subscribers: atulfokk, cedric, jpeg

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

Signed-off-by: Amitesh Singh <amitesh.sh@samsung.com>

ecore_animator: use EINA_DBL_EQ for comparing double values

Summary:
EINA_FLT_EQ was used to compare double values. replace it with EINA_DBL_EQ

@fix
Signed-off-by: Umesh Tanwar <umesh.tanwar@samsung.com>
Reviewers: raster, cedric, singh.amitesh

Reviewed By: singh.amitesh

Subscribers: atulfokk

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

Signed-off-by: Amitesh Singh <amitesh.sh@samsung.com>

Revert "Genlist: Calculate the min size of content before item realize"

This reverts commit 60566ca34d0036d21e5d134ba299b6f0fa26ce86.

This broke the layout in "Genlist Full Widget"

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: cedric, SanghyeonLee, jpeg

Reviewed By: jpeg

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

genlist: Fix invalid state of reused content

If an item is marked as disabled it should be re-enabled
before being put in the reusable contents cache. Otherwise
a following use of this object may result in a disabled
item being used, making the UI effectively disfunctional.

Also modify the test case to show and test this behaviour.

Add an efl_isa() to protect calls to elm_widget APIs.

Fixes T5236

@fix

evas - gl - remove warning for unused var with opengl vs gles

emotion: avoid crash during shutdown due to Ecore_Event queue.

elocation: avoid crash during shutdown due to Ecore_Event queue.

elput: avoid crash during shutdown due to Ecore_Event queue.

elementary: avoid crash during shutdown due to Ecore_Event queue.

eio: avoid crash during shutdown due to Ecore_Event queue.

efreet: avoid crash during shutdown due to Ecore_Event queue.

eeze: avoid crash during shutdown due to Ecore_Event queue.

ecore_x: avoid crash during shutdown due to Ecore_Event queue.

ecore_wl2: avoid crash during shutdown due to Ecore_Event queue.

ecore_win32: avoid crash during shutdown due to Ecore_Event queue.

ecore_wayland: avoid crash during shutdown due to Ecore_Event queue.

ecore_sdl: avoid crash during shutdown due to Ecore_Event queue.

ecore_psl1ght: avoid crash during shutdown due to Ecore_Event queue.

ecore_ipc: avoid crash during shutdown due to Ecore_Event queue.

ecore_input: avoid crash during shutdown due to Ecore_Event queue.

ecore_imf: avoid crash during shutdown due to Ecore_Event queue.

ecore_evas: avoid crash during shutdown due to Ecore_Event queue.

ecore_drm: avoid crash during shutdown due to Ecore_Event queue.

ecore_cocoa: avoid crash during shutdown due to Ecore_Event queue.

ecore_con: avoid potential crash during shutdown due to Ecore_Event queue.

ecore: avoid potential crash on shutdown in Ecore_Exe

ecore: add ecore_event_type_flush.

During shutdown it is possible that some event are still in ecore events
queue and get processed after the shutdown of the module that did emit them.
This would lead to crash in some case. The answer to this problem is to
normally manually track all ecore event in the queue and destroy them
before shutdown... Of course that make the API difficult to use and
basically nobody got it right.

This new API do actually as it says remove all the ecore event of a
certain type from ecore events queue. It is to be called on shutdown.

@fix

ecore_cocoa: fix windows initial rendering

We need to refresh the Cocoa's content view when showing the window.
In some particular cases (including the elementary_test initial window),
the content view is redrawn because of external events (the entry
animator). When no action at all is performed, the window and the
content view are resized... nice... but not drew...

Such a simple fix for this hell of a bug!

Fixes T5210

ecore_evas_cocoa: better debugging log

ecore_cocoa: factorize a little access to the cocoa window

eldbus: don't abs() an unsigned value

clang complained about abs() being used on an unsigned integer.
Calling abs() is actually unnecessary.

eio: fix printf format error

edje: better error message when a po file does not exist

edje: make sure the po files are well-copied once

edje: use portable cp options

The -u option is GNU-specific.

ecore_wl2: Fix up xdgv6 ack_configures

We've been immediately acking configure with the correct serial number, then
later at commit time sending an incorrect serial (generating a new one).

Remove the extra ack, and save that serial for later, and don't overwrite
it with a current serial when we get a toplevel configure.

Oddly, compositors were letting us get away with this behaviour, so this
probably looks functionally the same as before.

ecore_wl2: Fix initial maximized/fullscreen state

This defers the fullscreen/maximized states if we don't have a shell
surface, and sets them in the first shell surface commit.

ref T5044

ecore_wl2: Rename badly named API

Oops, I broke naming convention with this, but it's unreleased and beta so
I hope nobody notices as long as I change it now...

ecore_wl2: Remove wl_shell client support

We're never going to encounter a compositor without at least xdg v5
support, so remove wl_shell support.

Revert "ecore-evas-wayland: Fix issue of apps not starting up in fullscreen mode"

This reverts commit dfb18775002c5130b60bdc03ed60b7de356e6012.

This did fix the problem of "rage -f" not starting properly, but it
broke toggling between fullscreen and !fullscreen for other apps.

They'd resize to a large, but not quite fullscreen, size, then render with decor
present, and big black bars on the right/bottom edges (which were offscreen due
to the client size and position)

This isn't really the right place to fix this - it's a protocol usage bug, not
a canvas bug.

ecore-anim: Remove unused variable

Gcc reports 'i' as unused here and checking the code confirms that, so
remove unused variable.

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

Revert "ecore_input - dont try update modifiers and locks on mouse events"

This reverts commit f654714d75afdd07ad851bf278a39ddf8de6caba.

Modifiers do influence mouse events, though a mouse input can't change them...

This commit broke modifer+drag on windows in E, so I'm reverting it.

evas filters: Save selected render op of the final object

If an object is meant to use COPY mode (for evil purposes,
most likely), then its filtered output should also be using
COPY mode.

@fix

win32: removed duplicate winsock2.h and added WIN32_LEAN_AND_MEAN in Ecore.h

Summary: winsock2.h is dublicate in src/lib/ecore/ecore_main.c

Reviewers: NikaWhite, an.kroitor, cedric, raster, rimmed, vtorri

Subscribers: cedric, jpeg

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

popup: hides the popup when hide effect is finished

Summary:
When popup hides and shows fast, sometimes popup is not shown
because of the difference visible value between popup and notify
(case: popup visible state: 1, popup_notify visible state: 0)
To sync this value I added the visible set code in elc_popup.c

@fix

Test Plan:
1. run elementary_text -to popup
2. shows the hide popup
3. hides popup using the hide button
4. repeat 2,3 step
5. popup will be shown well

Reviewers: jpeg, cedric, Hermet, raster

Subscribers: Blackmole, woohyun

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

ecore_input - dont try update modifiers and locks on mouse events

mouse events have nothing to do with modifiers or locks, so dont try
modify them on mouse events. a total waste of cpu and time.

this should also fix T5251

multibuttonentry : fix box indicated item correctly in item_insert_before and item_insert_after

Summary:
If user added item with item_insert_before and item_insert_after, item was not displayed.
For this reason, fixed box indicated item correctly from reference->button to VIEW(reference).
Additionally, remove the unused variable button in _Multibuttonentry_Item struct.

Reviewers: woohyun, CHAN, cedric, jpeg

Reviewed By: CHAN

Subscribers: cedric, jpeg

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

evas: Check for null pointer deref

Summary:
Earlier places in the function are testing if obj->layer is null, so
should be checked here as well before it's used.

@fix CID1371826

Reviewers: jpeg

Reviewed By: jpeg

Subscribers: cedric, jpeg

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

evas image cache - add lots more locking to try stamp out any thread bug

this definitely fixes T5223 because it adds an engine lock around the
code segemtnt that does compare time stamps... and hopefulyl a few
more things too.

@fix

Luncher theme: Prepare to size engage correctly.

gl_cocoa: use triple buffering

Interestingly enough, this makes some graphical artifacts vanish...

ecore_evas_cocoa: fix invalid initial requested width

ecore_evas_cocoa: use ecore_event to match windows

ecore_cocoa: pass the right window to ecore_evas

ecore_evas_cocoa: remove more pointless code

eina: force copy of not copied virtualized file while doing an eina_file_dup.

The other way around is pretty much impossible as you don't know who does
an eina_file_dup and for how long they keep there reference.

T5234

ecore-wl2: Clarify post-condition for global lookup in _ecore_wl2_shell_bind

Summary:
The !global test is a post-condition that _ecore_wl2_global_find
provided valid data, so the code is clearer if this is kept separate
from the subsequent if chain for looking up the shell.  Since this
post-condition returns from the function on error, it can stand alone.

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

Reviewed By: ManMower

Subscribers: jpeg, cedric

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

evas: clean up GL images for emojis when GL context is free'd

If GL context is free'd before processing font shutdown,
textures for emoji glyph's GL images will be free'd without clean
up its GL images. It causes eina mempool infinite loop issue when
emoji's GL images are free'd in shutdown process.

So, the patch will make a list for emoji's GL images in context and
clean up them when the context is free'd. Just like font textures in
context.

@fix

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

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

elm_test: Add a test case for Emojis

Yay to full color characters. The list of selected emojis is
quite random. Some more could be added.

Just two remarks:
- Aren't they supposed to scale down?
- Why is my grinning face always ugly?

evas gl: Add missing GLES 3.1 definitions

Evas_GL_API supports the GLES 3.1 functions but the required
macros were not present. Thanks Brokenshakles for the tip.

naviframe: keep unfocusable state of item before finishing item push/pop

If item pop was started without focus at a naviframe object
and the naviframe object got focus before finising item pop,
the naviframe object could give focus to its first object of prev item.
It could ruin focus orders and failed to restore previous focused
object in the item.
To keep consistency of focus policy, same logic will be applied to
item push scenario.
@fix

Change-Id: Ia74bdce3620bd59622ef32a0cedf5fbd84815400

evas: Print CRI message in case of bad event usage

This will abort E when using Bryce and EINA_LOG_ABORT is enabled
(it is enabled by default when running a non-release version of E).
That's on purpose, as that code needs fixing :)

Ref T3144

ecore_evas_cocoa: remove a lot of useless code

I still don't know what's going on with the rendering problems, but
at least now  I have less dead code in my way...

ecore_evas_cocoa: set the render func at initialization

ecore_evas: process mouse move even when there is no cursor

tests: Don't print garbage

GLView tends to print some random garbage when using the
GLView Gears test. That's because we're allocating and printing
a 0 byte buffer. Don't do that.

evas gl: Fix version detection for GLES 3.1

It was assumed that GLES 3 would only work with EGL but in fact
OpenGL 4.3 & 4.5 are supersets of GLES 3.0 & 3.1 respectively. So
GLX should also support GLES 3.0 or GLES 3.1 for evas gl, if the
driver supports it, of course.

Of course while doing this patch things didn't go like they were
supposed to go. I'm currently using NVIDIA's proprietary driver,
that conveniently provides EGL with GLES 3.2. But wait, there's
a catch: GL_VERSION is "OpenGL ES 3.2 NVIDIA" except that none
of the functions of GLES 3.1 or GLES 3.2 are actually supported.
Those functions are only present in the GLX/OpenGL variant of the
driver. Thanks so much for making my life easier...

So yeah, this patch contains a hack for those invalid versions
of GLES 3.x. What was supposed to be a small fix became a huge
mess.

Also add a comment about the possibly invalid auto-upgrade from
GLES 2 to GLES 3.

This adds a test case in elm_test, but only to verify that
elm_glview_version_add(3) actually works. We need a proper GLES 3
test case, eventually (and 3.1, 3.2 of course).

evas: Set thread name for preload

This can be useful during debugging.

ecore_evas/x: don't mark as unused a used parameter

ecore_evas/cocoa: fix cursor's visibility

Fixes T5238

ecore: switch back to use ecore_thread_wait.

We need to make sure that we wait properly on the animator thread
otherwise we end up with the animator finish callback being called
later on during the shutdown process which means trouble.

docgen: include C usage snippet on event pages