Destory the semaphore when Vulkan render target dies

Change-Id: I57dba6f941caa54f04c6f55a3b79d91094aee300

Merge "Replace the exception case in tizen backend." into devel/master

Replace the exception case in tizen backend.

Replace the excepption case in tizen backend when display server does not work.
When display server does not work, then ecore_wl2_display_connect() does not work and crash occurs in application size.
To handle the failure case where the display server does not work in the Tizen backend, you can modify the ecore_wl2_display_connect() function to avoid crashing the application.
Specifically, replace the exit() call with a more graceful error handling mechanism.

Change-Id: Ice83d181fe468bc0b9bdcca48f2fcc6496d855f1

Fix the bug about window resize/move completed signal.

Fix the bug about window resize/move completed signal.
When the window resize/move completed signal work,
application can not get the changed window geometry in the related signal callbacks.
This patch is to fix the related bug.

Change-Id: I3d0a6cc9eb7a97ee92a17d63b68512d651959e8f

Merge "Ensuring shader cache on android is not /home/owner" into devel/master

DALi Version 2.4.20

Change-Id: Ia8cfa9b75e0905a7e96bc43d5bc1967b508dba57

Make sure the handle of RenderPass is properly destroyed

Change-Id: I49aeda5049319cb0d9b8d45fa8c4a7415a35f576

DALi Version 2.4.19

Change-Id: I1718b7c2e12535af237ba62bb30649f1faa4aed8

Ensuring shader cache on android is not /home/owner

Change-Id: I0f4613f2c9a28cd582b41d10cd1dadcc85a20662

DALi Version 2.4.18

Change-Id: I32617bd9bea554a14fabd394c688f00bd3b2c245

Hover events should also be interrupted when reset.

Change-Id: I4fbccfaa358afe41255b3c41934e92d319336537

Merge "Fix svace issue in face manager" into devel/master

Add version checker

Change-Id: I71a54ecac503d77f7653ef9cc24dbd51a297a103

Fix svace issue in face manager

Change-Id: I5cc87b24e04b8052a3fa3220192d1afd92b8a2fd
Signed-off-by: Bowon Ryu <bowon.ryu@samsung.com>

Merge "Fix clear cache bug in face manager" into devel/master

Merge "Ensure render target is nullptr or not, and print logs" into devel/master

Fix clear cache bug in face manager

If we want to use remain count in ClearCache,
we should also properly clear face, size, and index.
But it is difficult to support it now.
And since there is no use for remain count, we will remove this feature for now.

Change-Id: I67bddd7ec695b5b724d45c1e6886492c3cd4eb7d
Signed-off-by: Bowon Ryu <bowon.ryu@samsung.com>

Merge "(Android) Updated polling technique as previous way was deprecated" into devel/master

DALi Version 2.4.17

Change-Id: I20805006fb59026d06f68e0ef4ce9a1dd6d00dd7

Ensure render target is nullptr or not, and print logs

Change-Id: I04f0278065c0356cf56590220b0d558d936aa735
Signed-off-by: Eunki Hong <eunkiki.hong@samsung.com>

Merge "Removed unnecessary fomat file as it breaks some clang builds" into devel/master

(Android) Updated polling technique as previous way was deprecated

Change-Id: I9442d36fd1e8513c51792bd8f8857ec75d03f1a3

Removed unnecessary fomat file as it breaks some clang builds

Change-Id: I5862009434dfe734583ea943a5c1db8f7b51289a

Merge "Fix memory leak and validaiton errors caused by Vulkan::Program" into devel/master

Merge "Reduce the memory allocation per draw call during Vulkan resource binding" into devel/master

Reduce the memory allocation per draw call during Vulkan resource binding

Change-Id: I604d9ccdb204dc9a6ed7bc9b773988e28553c682

Merge "Refactor: font face cache" into devel/master

Fix memory leak and validaiton errors caused by Vulkan::Program

Change-Id: I6d0e609f80ab7dec342c696e6093f50963bff664

Fix the error of link test with Vulkan library

Change-Id: Ie96fa47c3ab29378cf3c0b25fa3dd71e96aad00f

Refactor: font face cache

The core change in this patch is to create 1 freetype face per font file.
Previously, we created as many faces as font-face-cache-items.

This was fine for static (seldom changing) apps, but when using a wide variety of point sizes,
it would create a face every time, which used too much memory.

Now we reuse 1 face.
Instead of faces, freetype sizes are cached and activated on demand,
which uses significantly less memory than the previous face cache.

In static apps, the memory usage difference may not be significant,
but in extreme cases where point sizes exceed thousands, it is incomparably efficient.

And now, we can use pre-loaded memory fonts in async text as well via the font file manager!

TODO: Optimize font-face-cache-item.

Change-Id: I38f44e641e00ae7bfcf16891f92ba14d74c9bff9
Signed-off-by: Bowon Ryu <bowon.ryu@samsung.com>

Merge changes Ieb7e9b87,I017b729c into devel/master

* changes:
  Make gl interface pointer use case with DALI_LIKELY
  Cache BindBufferRange call + Reduce cache clear call if we can

Merge "Adds to get window rotation's angle and screen rotation." into devel/master

Make gl interface pointer use case with DALI_LIKELY

Since most of cases will use valid gl objects.
So let we ensure the gl validation by DALI_LIKELY markers.

Change-Id: Ieb7e9b8734ab0ac353b8b33422b776dd88007972
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Cache BindBufferRange call + Reduce cache clear call if we can

Change-Id: I017b729ca8f6cc93214165ecf47e4f22697c6ec5
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Merge "Add OverScrolled in WebView" into devel/master

DALi Version 2.4.16

Change-Id: I22ac7002af4ab1a89d902cb38846ca364dcb5e22

Adds to get window rotation's angle and screen rotation.

Adds to get window rotation's angle and screen rotation for Dnd.

Change-Id: I8d95c4ab6e981875ebbf93ac9cd7ff2d134397e6

Add OverScrolled in WebView

Change-Id: Ie4b3b4d33c42d19507a620f48876279e3a534157

Merge "(GLES) Ensure gles framebuffer bound at shared context" into devel/master

Merge "Ignore UniformBlocks for GLES3" into devel/master

Merge "(GLES) Match the binded target and matched target" into devel/master

Merge "Add flag  for precompile shader" into devel/master

(GLES) Ensure gles framebuffer bound at shared context

gles-context have framebuffer dependency cache objects.
But now, it looks that we only allow to Create/Bind the framebuffers
at shared context.

Since we cant destroy the framebuffer at another context,
we should ensure to remove the cache objects at shared conetext.

Change-Id: I4da09e30644488dcb2f217cee1fcf2d7213b1202
Signed-off-by: Eunki Hong <eunkiki.hong@samsung.com>

(GLES) Match the binded target and matched target

Until now, we use GL_COPY_WRITE_BUFFER for mapping the buffers.
This target usually be used for glCopyBufferSubData.

Since we have specific destination target per each GLES::Buffer, let we
ensure the matched buffer target and generate them.

Change-Id: I6cf2a9ff41b84dd82499bdba57792da2c9aa87d6
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Ignore UniformBlocks for GLES3

Since ResolveUniformBuffers() time doubled after we use UniformBlock at shader,
let we don't use it and use standalone block instead.

It will be changed after we can keep ther performance.

Change-Id: I3adde5ed295f0a9b621f94f4ebcab5b4bf6f64d2
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Add flag  for precompile shader

Change-Id: If48a07ac3943fd82f8c316bb68ff4d49701ff405

(Gles) Minor optimize UBOBinding

1.
std::vector<bool> is specialize for bitfields, which has less memory
and might have overhead at get/set. Let we avoid this panalty if possible.

2.
Remove unused member values for UniformBufferBindingDescriptor.

Change-Id: Ic09d0130cb103e9e6cbde843486c752c3e7abb1a
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

DALi Version 2.4.15

Change-Id: Icdb0a705fff5d39fb100868bb841b91cd0246324

Use ResizeUnitialize for UBO and texture bindings

Let we reserve the memory for some bindings - texture and UBO.

At ClearState() phase, we always clear all container, and re-allocate next draw.

It will less allocate, and less touch the count area memory. frequency.

Change-Id: I50f677b8ab530f03e0b22343fa4d18cad958f31c
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

DALi Version 2.4.14

Change-Id: I46ebe5569848b07c79abdcdee2899a0c407009bb

Fix coverity issue : Do not use FILE* after fclose()

Change-Id: I31caf37588300bf1ec2096b1bc7a01d82635a3ca
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Merge "Reserving descriptor write vector to known size" into devel/master

Merge "Reduce memory consumption by tweaking VMA preferred heap block size" into devel/master

Merge "(MacOS) Fix for startup error on recent mac builds" into devel/master

Merge "Use swap for some cases instead move assign. + UpdateOnce for TextureManager" into devel/master

(MacOS) Fix for startup error on recent mac builds

Change-Id: I8f78ff9c8ec81a1169f46b89af236e0ff7603dcb

Use swap for some cases instead move assign. + UpdateOnce for TextureManager

Change-Id: Ib2de780fb1bc3348a1f3a6984818869ca1628d98
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Add more SystemError print logs + Do not use DALI_LOG_WARNING

Print some more system error logs where using fread / fseek / ftell.

+

DALI_LOG_WARNINGS print some logs only for debug modes.
Let we print some critical logs as error level, and verbose logs as debug level.

Change-Id: I3e7c87882ee1bd72a123b71435e8861c6ad3949b
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Merge "(Vulkan) Make we get UniformBlock's type_name for vulkan" into devel/master

Reserving descriptor write vector to known size

Change-Id: I3c10c5a5415eb0ac6a78f8901fb43479e13c777c

Merge "(Vulkan) Vulkan setup documents for Ubuntu at README.md" into devel/master

(WebEngine) Fix doxygen 1.9.3 error

Change-Id: I98cb83592dc06b8ee4c2c2b4979dc646e9733c72
Signed-off-by: Eunki Hong <eunkiki.hong@samsung.com>

Merge "(WebView) Update and sync all the latest patches related to WebView that have been modified so far" into devel/master

Merge "Check key is null for UngrabKeyList" into devel/master

Merge "Inverted samplers/bound textures in descriptor binding" into devel/master

(WebView) Update and sync all the latest patches related to WebView that have been modified so far

- Fix for UserMediaPermissionRequest object to be maintained by user
- Add WebView FeedMouseWheel as hidden api
- Add SetVideoHole() for EFL window
- Add WebView DeviceListGet,SetDefaultAudio
- Add APIs for setting IME position, cursor theme name.

Change-Id: I2670d74b9f04a620119bd4fd38ee7fcca0be650c

ClearScene if previous frame presented + Present if dirty rect not empty

Their was some error if rendererAdded, but render instruction is empty.

If then, we don't swap the presented buffer. So latest rendered scene
don't cleared.

To fix this issue, let we acquire next frame if previous scene had some render
instructions to scene, but not now.

Also (for gles case) if we call eglSetDamageRegionKHR, we should call eglSwapBuffer.
If not, 0x3002 error occured.

Change-Id: Icf55be6fe92ab98def0e631b0205939c16a8f4e6
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Merge "Let we allow to throw exception at LRUCacheContainer::InternalReserve" into devel/master

Check key is null for UngrabKeyList

We check key is null or not at GrabKeyList API.
Let's follow the logic pairwise.

Change-Id: I5d4b8a5d92dfe9d1c5a3d9b61a418d243209c0c6
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

(Vulkan) Make we get UniformBlock's type_name for vulkan

Until now, due to some unknown reason, the name of uniform block didn't loaded well.

To fix this issue, let vulkan-reflection use type_name instead of it's name.

Change-Id: Iefc677abaa1832ed579797a3b2e69691151f6408
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Reduce memory consumption by tweaking VMA preferred heap block size

Change-Id: I53d21b0e758de494b9eb9f1c97951bda945e59d0

DALi Version 2.4.13

Change-Id: I3bd709990da9dd42903a4f5e1413f438e858f61e

Let we allow to throw exception at LRUCacheContainer::InternalReserve

Coverity said std::vector constructor can throw exception.
We don't need to control those exception cases detail but,
let we just avoid useless exception check for it.

Change-Id: I1adec4c01849b4800ba8227b63e6e70cbbc1ffa3
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

(Vulkan) Vulkan setup documents for Ubuntu at README.md

Change-Id: I6fdbe2b44737dbf2bac0f935caba60dfe403d99d
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Revert the previous SVACE fix for VMA null termination issue

The previous fix is try to fix the SVACE error to ensure null
termination in VmaStringBuilder::Add by resizing the buffer
and appending '\0' after copying. It's noticed that VMA
statistics string is broken after this fix.

We realised that such fix is wrong, because the data is only
used as an input to the function VmaCreateStringCopy, which
takes explicit length and adds null terminator to the created
copy.

We should ignore this SVACE error, as advised by the author
of VMA:
https://github.com/GPUOpen-LibrariesAndSDKs/VulkanMemoryAllocator/pull/463

Change-Id: I5027a4ebe07a102c05a07108cc0108bcf1d73fc1

Merge "DALi Version 2.4.12" into devel/master

DALi Version 2.4.12

Change-Id: If2031bb3a6e791aae30925bd97f0ea568dd970c2

Add appname and path to Dumptree

This reverts commit e47f563929f6b85fb81b96091a87a746073316d4.

Change-Id: Idfd37fb4ca3dee896693ae9111435e677ae2d5b3

Inverted samplers/bound textures in descriptor binding

Change-Id: I1c3d57b0a79d11d6118271ba38b1d5948b1dafe3

Merge "Skip rendering if renderer never been added" into devel/master

Merge "Fix Svace issue about font variations." into devel/master

Fix Svace issue about font variations.

Change-Id: Ib022243ffd5d8d56e062ab0e1e58e65580f25f5d

Skip rendering if renderer never been added

Change-Id: Ia499b8b8f040503be1f47f913bb4910916f4a8fd
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Merge "Consume exception during DBusServer destruct" into devel/master

Merge changes I549d2fd5,I5348dcc1,Ic3fb46fa,I05c8b768,I8e0b2555 into devel/master

* changes:
  Ensure that RenderScene actually renders something.
  Fixing swapchain buffer count and cmd buffer allocs
  Reworking pre-render/post-render
  Debug extra
  Ensure empty render instruction doesn't swap buffers

Merge "Add Font variations to font client" into devel/master

Consume exception during DBusServer destruct

Change-Id: Ifd513a2f38975268b33611ebcfe0923e943084ed
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Ensure that RenderScene actually renders something.

RenderScene should only get called if there is something to
present and nothing prevents a renderer being drawn.

The only early-out that's allowed now is null scene handle.

ClearScene will only get called if RenderScene did not present
when we expected it to have actually drawn something.

If there are no renderers, but we set the background color of
the window; currently, FullSwapNextFrame is set. If this is not
desired behaviour, it should be changed in
SceneHolder::SetBackgroundColor.

Change-Id: I549d2fd5a25f6a182d89c8c2cc5abfaa07f985c6
Signed-off-by: David Steele <david.steele@samsung.com>

Fixing swapchain buffer count and cmd buffer allocs

Number of swapchain buffers should be based on minImageCount
from surfaceCapabilities, rather than the current number of
images that getSwapchainImagesKHR returns.

Currently, it is possible for the swapchain to get invalidated,
and the new swapchain can be generated with more images than
min. We only need to sync the min number; so we double or triple
buffer based on this count.

(number of CommandBuffers / SwapchainBuffers is based on this min count,
not total image count)

Changed this to only reallocate command buffers in Reset(), rather than
breaking any outstanding sequence.

Change-Id: I5348dcc1762426e5370a35cd7f48d4612a3b5114

Reworking pre-render/post-render

Changed main render loop to check if there is a full-swap
without any rendering (usually caused by having no render-task
yet, but resizing or setting the background color of the window).

If so, we need to ensure that an empty scene can be correctly
rendered. Added a ClearScene to Core API.

This is more pertinent to Vulkan renderer, which has to ensure
that the window's render surface is properly acquired _before_
drawing to it, and presenting that surface needs the right
sync primitives.

Modified VulkanGraphicsController::PresentRenderTarget() to also call
Surface->PostRender(), in the same way that EglGraphicsController does.
(Prelude to other re-work for PartialRendering).

Change-Id: Ic3fb46faa17b3ec67053cfc4fbd48f6a31cc414c

Debug extra

Change-Id: I05c8b768acfdebfce8fa661ff69e6e13d6412a53

Ensure empty render instruction doesn't swap buffers

If we aren't going to draw anything to the surface, then
don't acquire next image or render anything.

SwapBuffers is handled through Swapchain::Present, which maintains
it's own framecount; The current buffer index is retrieved from the
main window's swapchain.

Updated VulkanGraphicsController::PresentRenderTarget to also call
WindowSurface::PostRender; this has a load of handling for partial
rendering that we may need in the future.

Will keep rendering framebuffers, regardless of whether they are drawn
or not.

Change-Id: I8e0b2555647e2ce74d425c9632246f19ea295470

Merge "DALi Version 2.4.11" into devel/master

DALi Version 2.4.11

Change-Id: Ic63a2f9489f166a4d2640f32a881da674223df6d

Fix UNSAFE DirectRendering flush EndRenderPass well

Since Context::ResetGLESState() reset current renderpass and render target info,
it might not worsk well for EndRenderPass stage.

Change-Id: If3223d784fa865565c457986acdfc1fdb40b92fb
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>

Add Font variations to font client

Change-Id: Ia48dc13f08135650f29fcb811b77e52ffddbd6c7

Merge "Make move operation as noexcept" into devel/master

Merge "Fix ISOLATE DirectRenderingGlView didn't show texture" into devel/master

Revert "Add appname and path to Dumptree"

This reverts commit 928399d10fbc0f1bc32248a6ab907f7b8a8ff9f8.

Change-Id: I3f395482ef9bdd09dfa45ad18473c31322c82713

Fix ISOLATE DirectRenderingGlView didn't show texture

We didn't use native renderer's context even we use ISOLATE mode.

It just breakdown normal dali pipeline codes and textures

Change-Id: I8d3c2f7e9f88a186a8f24c6beaccc4d1a42a21ed
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>