evas image updates - reduce overhead with bordered images not scaled

this fixes over-rendering (where we render the whole image) if it has
a border set. do proper region updates if the image has a border but
is not scaled and the fill is at 0 0 and sams size as the object etc.
etc. so this is simple and doable.

this fixes T4123

evas objkect image - make opaque rect getting code more readable

this just makes it more obvious where its handling the opaque middle
fill case.  see T4123 - i was just reding the code and this seemingly
does the right thing.

ignore generated shaders .x file

edje: fix ambient color

Summary: change specular color to ambient int the function of ambient color

Reviewers: cedric, raster, Hermet

Subscribers: jpeg, artem.popov

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

elm_win: handle elm,action,menu signal for wl clients

trigger the xdg-shell method for this when the signal is received

fix T3460

theme: copy border icon click -> elm,action,signal program

elm_win: simplify opaque region updating during fullscreening

already handled during _elm_win_frame_del

elm_win: update opaque region when frame is deleted

fix T4092

config: increase base image cache to fit two 4K background in.

This is not ideal, as it would be more accurate to dynamically set
the cache to be a function of the number of screens and their size.

elementary: as screen get bigger, we need more cache to store uncompressed image.

edje_pick: copy dependent group

Parts can use enother groups as source, for example textblock. If source
groups does not exist - group is not valid. This patch extend the group
copy process and copy dependend group too.

@fix

elua: enable module lookups in apps location

This will allow for better organization of the docgen script.

ecore_wl2: dnd: fix leak of event resource when not used

In the else branch the allocated event is never used. Make sure we free the
resource before returning.

CID: 1357753

Revert "elementary: Fix issue of toggling border state causing visual jumps"

This reverts commit fb24955782b9242beb9447ebb6631b2b63921988.

While this patch did fix the issue of toggling window Borderless state
in EFl-Wl client applications, it is "Enlightenment's" stance that it
will not change to match this fix, so this has to be reverted else
internal E dialogs show up with no border (due to them using SSD).

This is going to re-open T4092 sadly :( and now leave that bug with no
possible fix.

ref T4092

Fixes T4126

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

Ui text: remove unused variables

evas: Fix call to NULL object

textblock: Prevent crash in evas_object_textblock_cursor_new(NULL)

This happened in expedite

tests: Add ifdef clang to avoid warning on GCC and elsewhere

Avoids warning about unsupported pragma

eo: Change eo_override to avoid struct passing and GCC construct

This is a (minor) API & ABI break in Eo.h!

I say minor as eo_override shouldn't be used yet (EO is unstable
and this patch includes all the use cases in EFL).

I'm not very happy about the new form of the macro, but it avoids
two things:
- passing in a struct (valid in C, but never used in EFL)
- using a GCC construct to create structs on the fly

It was inspired by the event array define, but I don't think
we need the runtime memcpy here.

See also:
https://gcc.gnu.org/onlinedocs/gcc/Compound-Literals.html

drm2: Fix shadow variable

int ret is not even needed here.

edje lang set - fix coverity harmless complaint

fix CID 1357755

efl: Add support for efl_event_flags() to hold events

This fixes the following ERR message:
ERR<30400>:eo /home/jpeg/e/core/efl/src/lib/eo/eo.c:462 _eo_call_resolve()
 in ../src/lib/evas/canvas/efl_event_input.eo.c:3: func 'efl_event_flags_get'
 (1483) could not be resolved for class 'Efl_Event_Hold'.

This is an EO-API only fix.

efl: Hold event (EO API) should implement instance_get

This fixes the following ERR:

ERR<23969>:eo /home/jpeg/e/core/efl/src/lib/eo/eo.c:462
  _eo_call_resolve() in ../src/lib/efl/interfaces/efl_event.eo.c:5:
  func 'efl_event_instance_get' (1479) could not be resolved for class 'Efl_Event_Hold'.

evas - on shutdown avoid potential invalid memory access

i don't know for sure if this fixes T4103 but in theory i think it
might given a reading of the backtrace and a guess at what might
happen, so try this fix. it doesn't hurt and can only help.

@fix

eina rbtree iterator - fix leak found while debugging other problems

there was a leak of elements on  the itrator stack when the stack was
flushed still having some items on it, thus losing their pointers and
never freeing them.

@fix

evas: Add a textblock to the image proxy example

This is to show that proxies can proxy more than just other
images, and also this was done to test D4159 (alternatively
changed into the previous commit).

evas: Avoid proxy subrender if size is 0

See also D4159. This is an alternative to the proposed patch.

Fixes T3949 (I hope!)

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

ecore_con: fix lookup of libcurl on OpenBSD.

@fix T4130

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

edje: fix definition of viewport in edje-3d

We now use eo_do() and EFL_CANVAS_SCENE3D_CLASS macro to create viewport

Reviewers: raster, Hermet, cedric

Subscribers: jpeg, artem.popov

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

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

docs: generate a keyword list for use by codelink plugin

tests: eina_cxx: make sure simple.eo gets disted in all cases

If we turn on release mode simple.eo did not land in the tarball as it was
behind the EFL_ENABLE_TESTS condition. We now dist it in all cases.

win: Set window alpha when changing theme

This sets the window alpha back to 0 when switching from a theme
with alpha to one without. Thanks @bu5hm4n for asking about this.

tests: Show use of eo_override for custom layout

This requires the previous fix in eo to actually work.

eo: Fix leak in eo_override and allow NULL to reset

eo_override would leak the vtable if called multiple times, this
fixes that. Also, it is now possible to revert back to the original
class' vtable by passing in { NULL, 0 }

I believe it is thus possible to incrementally override more
functions on an object. Absolutely not recommended, but should work.
But it is not possible to selectively revert back to the original
class implementation on a single method. Use eo_super for that,
or revert back the entire object overrides.

PS: Is it normal that we pass in a struct? We never do that in EFL...

Evas GL: Use [u]int64_t directly in all APIs

This removes the need for khronos_[u]int64_t as well as the special
typdef EvasGL[u]int64.

Hopefully this should work on all platforms (note: [u]int64_t is
used in Eina APIs, so it is already required for EFL apps).

Fixes T3200

evas: Also fix was_opaque for rect & image

I believe the function is not used, so it doesn't really
matter.

evas: Some more opacity check fixes

This should fix T3309

Snapshot objects are image objects, so the function is_opaque
exists. No need to bypass it. Also, alpha rectangles are not
opaque. Assume that anything with a filter is not opaque.

All of this fixes T3309 but the main point was on snapshot
objects (probably because the only point of a snapshot is
to apply a filter on it).

evas: Fix cutout regions with transparent images

This should fix rendering issues with E theme and
transparent borders.

Fixes T3309

win: Set proper flag when EDC specifies win "alpha"

elementary: only call ecore_wl2 if wayland is enabled

fix T4125

eio: handle failure due to lack of access right.

eio: make it possible to force fallback monitor for testing purpose.

eio: avoid access after death of eio handler due to inotify triggering IN_IGNORED event.

eio: cleanup thread pointer on thread finish.

elocation: properly generate pkgconfig file.

@fix T4120

elementary: Fix issue of toggling border state causing visual jumps

This commit partially fixes a phab ticket where toggling window border
state (elm_test -to "Window States") would cause the frame to be
destroyed thus making the window contents appear to jump around

ref T4092

NB: I call this a 'partial' fix because while it does fix the issue of
efl client apps, there is still some breakage in Enlightenment that
needs investigating.

@fix

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

ecore_drm2: Allow enabling outputs in drm events

Previously a drm event could create a new output or disconnect an output,
but could not re-enable an existing output.

This results in failure to properly wake from screen blanker.

edje: add required attribute in Edje_Part

Summary:
When using EDC written by another person, developer usually
does not need to know all specified parts in EDC but editable
like swallow, text or textblock parts.
The required flag tells whether the part is intended to be used outside EDC.

Reviewers: jpeg, cedric, raster, Hermet

Reviewed By: jpeg, raster

Subscribers: raster, NikaWhite, kimcinoo, jpeg

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

ecore_drm2: Try to mmap gbm buffers

We're currently doing screenshots in E under wayland by copying data
out of the framebuffer, mmaping gbm buffers makes screenshots work
again when rendering with GL.

ecore_drm2: Fix page flipping

Properly handle the ticker racing ahead of the refresh rate by once again
triple buffering.

Fixes T3819

ecore_drm2: Don't disable custom ticker if flip fails

When using a mix of vblank and pageflip events like we're stuck with now
a failed flip is going to happen sooner or later.  We fix this by triple
buffering.  We shouldn't turn off the custom ticker.

Evas font: Fix width query for OpenType fonts. (rewrite)

This is essentially a cleaner redo of ef817f15f0eaec9704ec25d9468c2c8497a5bc13.
Logic should be exactly the same as there, the different is that this
one shares the code between OT and non OT.

Please refer to that commit for more information.

Revert "Evas font: fix width query for OT"

This was not done correctly. This split the code, which is essentially
the same for both OT and non OT. It's the same logic with some minor
additions for OT, so most of the code should be together.

This reverts commit ef817f15f0eaec9704ec25d9468c2c8497a5bc13.

docs: more detailed stats (print intermediate percentages)

docs: generate separate doc pages for events

elm_cnp: use defined charset before falling back to default

the chance is very high that some glyphs are not fitting into the
default (US-ASCII) so better falling back here on specified utf8.

@ref T3972

Evas font: fix width query for OT

Fixes T4068.
Simply querying the last glyph to determine the width of the glyph sequence
won't always work, as OT can have negative offsets (adjusts the placement of a
specific glyph better).

The solution is to calculate the "max width" of some sequence that will
guarantee us proper width results. The worst solution would be to iterate on all
the glyphs and sum up the max width. This is a bit impractical. Instead, we will
inspect just the "cluster" of the last glyph, if one exists.

This should have no performance impact on trivial cases, and very little impact
on the others.

@fix

evas generic loaders fix timeout on windows where no alarm exists

this uses a thread - to do the same. based on code vtorri put in T3790

this should fix T3790

evas headers - remove duplicate declarations of the same function

this should fix T3027

elementary tooltip: --printf

ecore - add more ecore main loop checks that were missing in some places

so 1 ecore_exe func was missing a main loop check... just to be sure.

eina thread create - use pthread_sigmask as this can be called from thread

@fix

eo: fix tests after renaming.

elm_colorselector: selected item is updated when color is changed.

Summary:
 - Previously, mode change, color change updated
   sd->selected, but did not send signals to edje of the item.
 - Also, mode change in colorselector set sd->selected to
   the first item even when palette is visible in previous mode.
 - Now, when mode is changed, sd->selected is set NULL and send
   unselected signal if palette is invisible, and keep sd->selected
   if palette is visible.
 - sd->selected is set NULL when color is changed in picker and
   color bars because previous selected item is not current color of
   colorselector

Test Plan:
1.
    1) launch elementary_test colorselector.
    2)Select any item and check the item is selected when palette is visible.
    3) Change mode to Palette, Both, All, and check selected item is not changed.
    (Previously, selected item is changed to the first item of the palette, but previous item was seen as selected.)

2.
    1) launch elementary_test colorselector.
    2) Select any item and check the item is selected when palette is visible.
    3) Change mode to Components, Picker.
    4) Change mode to Palette, and check none of items is selected.
    (Previously, selected item is changed to the first item of the palette, but previous item was seen as selected.)

3.
    1) launch elementary_test colorselector.
    2) Select any item and check the item is selected when palette is visible.
    3) Change mode to All.
    4) Click arrows in Picker and color bars.
    5) Observe selected item is not unselected when color is not changed, and selected item is unselected when color is changed.
4.
    1) launch elementary_test colorselector.
    2) Select any item and check the item is selected when palette is visible.
    3) Press direction key and check selected item is changed.
    (Previously, selected item's edje is not updated.)

Reviewers: woohyun, Hermet, jpeg, cedric

Subscribers: cedric, jpeg

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

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

elementary: switch code accordingly to use new ref version.

eo: rationalize naming to follow other use of ref and wref in our API.

ecore: add test case for tick begin/end.

ecore: early destruction of animator allow for tick end to always be triggered.

T4043

elementary: Make wl frame borders above content

This commit just readds the layer setting code which made the frame
border be above content. This does make sense in some contexts

@fix

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

evas-wayland-shm: Create buffers with ARGB by default

We need to be creating buffers with ARGB by default so that things
like Alpha/Transparency work when toggled. If we always create with
XRGB then toggling Alpha/Transparency state fails.

@fix

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

ecore_wl2: clear out read_data and len

they are passed to the event struct, and later freed in there.

ethumb: tell the user if the connection to the deamon failed

otherwise the user just gets no thumbnails, and no error message what
actually happened.

elput: Cleanup erroneous errors when trying to get output name

When we make calls to get an Input device's output name, if the device
does not have an output name it's not actually an error so just return
NULL with less noise

@fix

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

evas-generic: Fix gcc wanring

Gcc suggets parens around comparison value. Silence gcc warning.

@fix

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

evas-generic: Fix gcc warning of incorrect format

Gcc warns us about using %d here when the argument is unsigned long.
Fix format params.

@fix

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

elementary: Fix issue of minimum resizing in wayland

This patch fixes an issue where if you tried to resize a window to
it's minimum size, the contents of the window would draw outside the
window frame. Basically, when we are setting min/max size hints to the
window object we need to account for framespace.

@fix

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

[Bug] EFL memory leak on Windows(handler continuously increasing)

Summary:
Fix: event need to clese  when create event:
  event = WSACreateEvent();

Reviewers: bowonryu, herb, Jaehyun, thiepha, Hermet, jaehwan, cedric, raster

Reviewed By: raster

Subscribers: NikaWhite, reutskiy.v.v, artem.popov

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

ecore_evas_extn: Check whether server_data sender is client's server.

Summary:
Sometimes, In ipc_server_data, extn->ipc.server's data is different with e->server's data.

The case is as follows.

Process'A' has a server.
Process'B' has 'A's client(ee address : 0xB0).
Process'B's client die, and 'B's server created. and server's ee address is same with destroyed client's ee(0xB0).
At the same time, 'A's server send the message to 'B's client.
but 'B's client is died! so _ipc_server_data would manipulate 'B's server data.

Test Plan: Tizen Mobile Text.

Reviewers: raster, spacegrapher, jpeg, wonsik, dkdk

Subscribers: cedric

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

Revert "edje_cc: Fix minor coverity defect"

This reverts commit 0106455588a7390f36bae9ac6d789e94410a5216.

breaks edje_cc.... :( spin 100% cpu building terminology themes

elm tooltips - fix positioning and more to be sane and have less bugs

the tooltip geometry code was extra complex and iffy with certain
situations. this cleans it up and also fixes the below bug

this fixes T3980

@fix

eina_safepointer: Fix coverity warning

I guess the overflow was badly handled. Fixing it by using
explicit int intermediate value.

Fixes CID 1356616 and 1356619:

Operands don't affect result
Logically dead code

gfx: Add NULL check to silence coverity

realloc() can return NULL if size is 0. It's like free().
So, the usage here is correct, and there are probably no
points to interpolate between anyway. I wonder if there
can be commands without points, though.

Fixes CID 1293004

edje: Fix coverity warning

CID 1355234 Logically dead code

The indicated dead code may have performed some action;
that action will never occur.
In eet_data_descriptor_element_add: Code can never be
reached because of a logical contradiction (CWE-561)

Solution: use explicit range within valid values with <= and >=
rather than excluded values with > and <

edje: Add missing 'break' in edje_embryo switch

Fixes CID: 1352521 1352522 1352523 1352524 1352525

See 4ed9b8325

evas: Fix some CID in generic loaders

Check the return value of fwrite. There is nothing
sensible to do as the receiver needs to handle the error
gracefully.

CID 1356613, 1356614, 1356615, 1356200

pdf: Tentative implementation of mono support

No idea if it's correct, since I have no sample.
Also, simplify ARGB code.

pdf: Fix page index and

The module was not able to load any PDF with a single page (since
the index starts from 0, not 1 as it was assumed).

Also, fix a CID where Coverity was very very right.

Fixes CID 1356608:

The operaton may have an undefined behavior or yield to an
unexpected result.

In poppler_load_image(int, int): A bit shift operation has
a shift amount which is too large or has a negative value.

embro_cc: Shut up some coverity warning

CID 1039677 1039678 1039679

The expression's value does not depend on the operands; often,
this represents an inadvertent logic error.

In doarg: An operation with non-constant operands that
computes a result with constant value (CWE-569)

cell is defined as an int, not a long, so it can't be > INT_MAX.

elm test: Shut up coverity warning

CID 1354839

It was non-sensical as there would be no iterator
if there are no children.

elm test: Fix non sense check

CID 1353720 1353721

edje_cc: Fix minor coverity defect

CID 1261439

An assigned value that is never used may represent unnecessary computation,
an incorrect algorithm, or possibly the need for cleanup or refactoring.

In source_fetch_file: A value assigned to a variable is never used. (CWE-563)

Tested compilation of a few EDC files with no breakage.

edje_pick: Fix use after free

Fixes CID 1267458 (trying again)

eo-cxx: Fix C++ use of new _eo_add_end signature

elementary: Properly swallow wayland clients

This patch fixes an issue where wayland windows would not swallow
clients properly. Basically, we should be swallowing the elm window
layout directly rather than use a rectangle (as was previous). This
commit also waits until the internal elm theme has been applied to the
sd->edje object before adding the wayland window frame.

This commit has the added benefit of being able to remove an additional
Evas_Object from the internal data structure.

@fix

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

elementary: Clip window swallow clients

This commit sets up a client_clip for the windows client being
swallowed so that during resize, any window contents do not draw
outside the window.

@fix

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

Revert "efl theme for wayland borders - dont accept events on client swallow"

This reverts commit fdaefdbf9a8f518930b2d5f71372d1c52f8b2ee7. This is
being reverted because the issue of wayland borders not swallowing
clients has now been fixes and thus mouse_events work as expected wrt
client swallow.

efl theme for wayland borders - dont accept events on client swallow

docs: generate more info about events

Eo add ref: Fix a bug causing leaks and wrong refcount in some cases.

When using eo_add_ref, it was increasing the refcount before the user
context in the addition has fully ended. This means the object had its
reference increased while still not finalized, which means it was
sometimes passed with an increased refcount to unsuspecting class code.
The correct behaviour is to increase the reference count just before
returning the object to the user at the end of eo_add so the reference
count is only increased for whoever asked for it.

Breaks ABI!

@fix