Merge "Add early return condition when there is no fontconfig handle" into devel/master

Merge "Changed command buffer static default initializer" into devel/master

DALi Version 2.4.7

Change-Id: Iac39adf3e326a38aefdf0ba522cb3f7b721e6cfd

Merge "Ensured swapchain/surface cleanup on device destruction" into devel/master

Merge "(NativeImageSource) Return nullptr if NativeImageSource::New() failed" into devel/master

Merge "Load the shader using glProgramBinary" into devel/master

Merge "Fix fontclient coverity issue" into devel/master

Add early return condition when there is no fontconfig handle

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

Merge "Separate FontConfig handle" into devel/master

Fix fontclient coverity issue

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

(NativeImageSource) Return nullptr if NativeImageSource::New() failed

For vulkan backend case, NativeImageSource's mImpl return nullptr.

But NativeImageSource object itself is not empty, we can call
NativeImageSource API freely, and got crash.

To avoid these kind of issue, let we return nullptr if NativeImageSource
creation failed. (Similar with NativeImageSourceQueue)

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

Separate FontConfig handle

This patch changes FontClient to create and use a handle to FontConfig.
Previously we always used FontConfig globally.
There was no problem because DALi had already recognized and developed this internally.

However, since FontConfig is a library open to the system,
FontConfig may be used by other libs that dali does not know about in the same process,
which may cause Thread safe problems.

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

Merge "(Vulkan::Reflection) Append uniform blocks only 1 times per each programs + Clean up some logic" into devel/master

Merge "(Vulkan) Fix minor coverity issues" into devel/master

Ensured swapchain/surface cleanup on device destruction

VulkanDevice should take ownership of vulkan surface / swapchain pair.
For the moment; have ensured that these are properly disposed of
in the VulkanDevice destructor.

[Could change SurfaceSwapchainPair to use UniquePtr instead of raw ptr,
using Destroy() as deallocator fn, then less work!]

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

Merge "Fixing swapchain for use with triple buffering" into devel/master

Merge "Add locale api to text abstraction" into devel/master

(Vulkan) Fix minor coverity issues

1. Non-initalized structs (spriv.h, command-buffer.h)
2. Make mMemory nullptr if allocation failed. (image-impl, buffer-impl)
3. Avoid nullptr copy, or negative length string for some case (texture, shader-impl)

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

(Vulkan::Reflection) Append uniform blocks only 1 times per each programs + Clean up some logic

Since we create relfections per each shader at vulkan side,
there was some problem if we try to use same uniform block at both
vertex and fragment shader.

Since current spriv don't know the uniform block's name,
let we just seperate it by binding of layout.

+

Their might be issue if VertexShader code have sampler2D (e.g. Scene3D blendshape)
It is code side bug, and we don't need to append sampler informations
whenever we call BuildReflection().

To make ensure code clean, let we clear mUniformBlocks and mUniformOpaques
at the begin of BuildReflection(), and sort + unique each of them.

TODO : Shouldn't we need to consider multi-set case?

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

Add locale api to text abstraction

Prevents the setlocale system call from being called repeatedly.

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

Load the shader using glProgramBinary

If there is a saved shader, load the shader using glProbramBinary

Change-Id: I5db0118d5bcc2f45537172d8b5039afe1c5fc309

Fixing swapchain for use with triple buffering

Some platforms are using triple buffering, but the
SwapchainBuffers only allocate 2. Changed to ensure that
the buffer count matches at least the minimum number of
images returned on swapchain creation.

Fixed coverity error in CreateSwapchainForSurface /
ReplaceSwapchainForSurface.

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

Merge "(Webp) Do not load file again if we know size and frame count" into devel/master

Merge "Collect strerror_r relative error log printing logs" into devel/master

Merge "DALi Version 2.4.6" into devel/master

DALi Version 2.4.6

Change-Id: Idafa6b6b3ec9f545e8351df04313664f2c3a8bd5

Collect strerror_r relative error log printing logs

Let we collect all cases to file description error cases.

Until now, we use strerror_r return type value to print error logs.
But it will not works at XSI base kernel.
And also, strerror_r need a lots of parameters so the line of codes increased
wherever we want to print errno value.

So we can control both GNU and XSI case,
and easly check the error occured point.

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

(Webp) Do not load file again if we know size and frame count

For single webp case, we release the buffer data after load finished.
But user can request the GetImageSize() and GetImageCount().
In this case, we re-load the webp buffer again, and
fail to release the buffers until loader desturcted.

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

Merge "Fontconfig: Replace FcInitReinitialize with FcInitLoadConfigAndFonts" into devel/master

Fix build error when we use tizen_9.0 repo

Let we add new macro OVER_TIZEN_VERSION_10, and use it for latest APIs

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

Fontconfig: Replace FcInitReinitialize with FcInitLoadConfigAndFonts

FcInitReinitialize() causes global font cache and config reload.
which may lead to race conditions and crashes.
Replaced it with FcInitLoadConfigAndFonts() to safely reload config within the current process without affecting others.

Change-Id: Ic5c911f399de141310e99711ae72db64c9dad3cd

Changed command buffer static default initializer

Change-Id: I21e31265e1f6d9604762057c0f7385cd02f0fa60

Merge "Changing swapchain acquisition at frame start" into devel/master

Merge "Fixing coverity errors" into devel/master

Merge "[Vulkan] Reduced queue submissions" into devel/master

Merge "Removing DeviceWaitIdle from Vk::SwapBuffers" into devel/master

Changing swapchain acquisition at frame start

Need to ensure that we don't create end of frame fence if we're not
going to render anything.

But, we do want to have same pipelines for fbos and scene; so we need
to sync swapchain _before_ drawing to fbos.

Ensured that Swapchain owns it's external images.

Change-Id: I4cc64607ad964362cb793f435324110027b45740

Fix compile error after add -Werror flag added

ADD_COMPILE_OPTIONS(-Werror) option added after dynamic graphics backend applied.

Since we never make compile error for build warnings until now, it breakdown
compile error when we build Release mode.

To fix this issue, let we just keep -Wall option, and just fix
build warnings, for more stable codes.

And, for ~SwapchainBuffer(), let we don't throw exception
at destructor. Instead, just print log and skip assert.

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

Merge "Add FullScreenMode to InputMethodContext" into devel/master

Merge "Add Combining Diacritical Marks to text script" into devel/master

Add Combining Diacritical Marks to text script

Combining Diacritical Marks is an inherited script.
This is not used alone; it basically follows the previous script.
On the dali side, this is handled as a COMMON script.

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

Fixing coverity errors

Change-Id: I84d9de999dc95efc0e2e740ba406aa4507ac0bbf

[Vulkan] Reduced queue submissions

Rather than individually queueing each command buffer, instead
rolled up all cmd buffers (for framebuffers) into a single submission.

(Still require separate queue submission for scene, as swapchain
cmd buffer is the final submission, and generates end of frame
fence)

(Also fixed EGL debug build for target)

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

Removing DeviceWaitIdle from Vk::SwapBuffers

Rather than a hard wait each frame, should instead
leverage the exising end of frame fence that's owned
by the swapchain buffer.

Moved the AcquireNextImage functions out of BeginRenderPass, and
into the VulkanGraphicsController::FrameStart method; this will
now wait on the current buffer's end of frame fence before trying
to get the next image (for this buffer index).

SwapBuffers() now just increases the buffer index modulo the buffer
count (aka the number of swapchain images that are available for this
device).

Changed CommandBuffer to contain up to BufferCount() Impls; the relevant
impl is selected for each command by getting the current buffer index
from the device.

Changed DescriptorPool implementation to use the current buffer index
rather than trying to manage it's own pool index. This was quite broken!
It now grows the poolList to match the buffer index, which will never
go past the BufferCount. (usually 2 or 3).

Change-Id: I994046805bfb38285063e87f8409e856f14b3459

DALi Version 2.4.5

Change-Id: I0f41c1bd2f75c12674dae76624018e0256fd5cc4

Fix coverity issue : Convert uint32_t to float first, then minus operate

Since it is possible that widthIn is zero, we should consider
the code with negavie cases.

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

Merge "Fix memory leak for depth and stencil buffer in Vulkan backend" into devel/master

The range of touch coordinates is [0:width - 1] and [0:height - 1]

So, modify coordinate calculation when rotating.

Change-Id: I1ce7788a098104e1a729e2c78a7cf3fb49b7e31a

Merge "Do not copy same UBODescriptor and TextureBinding at GLES::Context + Remove unused function : BindSamplers" into devel/master

Merge "Fixed coverage build" into devel/master

DALi Version 2.4.4

Change-Id: I734fddaea896fa0f5cbf1c8d7723ed6a230ed748

Fixed coverage build

Change-Id: Ib458a5d1f748e4787ef616ed41ef963ca7fa35ff

Fix memory leak for depth and stencil buffer in Vulkan backend

Change-Id: Iba74096c4ff2e1213a293a5fffcd8ee9b96ab7dd

Dynamic Graphics Backend

Change-Id: I013ff62b54f2e8badee2c10e91b40d178b6f213c

Do not copy same UBODescriptor and TextureBinding at GLES::Context + Remove unused function : BindSamplers

Some struct call std::fill_n internally when we call resize - for example : UniformBufferBindingDescriptor
But, we really don't need to use initialized value.

Let we make the container as Dali::Vector, so can skip useless initialize value.

Moreover, let we remove unused function, BindSamplers.

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

Merge changes I05449137,Ie4334fb8,I12dad280,Iaf63b42f into devel/master

* changes:
  Change render target semaphore handling
  Inverting framebuffer viewport for Vulkan
  Implementing framebuffer in vulkan backend
  Allowing arbitrary dumping of multi-frame command buffers

DALi Version 2.4.3

Change-Id: Ic2044196624e2cdba2ee02949d4fda02c93db600

Change render target semaphore handling

Fixed bug with dependency graph adding same dependency more than once to render target

Changed queue submission to handle vector of wait stage masks
(as a render task may depend on more than one offscreen to finish rendering)

Changed render task dependence so that only the "earliest" dependent needs to
wait for offscreen - (2nd dependant Submit can't wait on already signalled sema, apparently!)
Fixes a lot of lockups in demos.

Changed swapchain Submit() to pass in any offscreen dependencies.
  Swapchain submit adds own dependence on acquireNextImageSemaphore.

Test harness:
  Some tests skip initial "SendNotification" call. This means that scene-setup done by the test harness is ignored until the next frame.
  Changed the TestApplication InitilizeCore method to ensure that the messages are processed after setup.
  (So it doesn't matter if the test doesn't first call app.SendNotification()).

Change-Id: I05449137fa14ce1b00d3ecd152f4d0cf2409228d

Inverting framebuffer viewport for Vulkan

Change-Id: Ie4334fb8c5894d9ec080c82b7203f335cff4cc7b

Implementing framebuffer in vulkan backend

High level:
----------
Added render pass constructors for fbo.

Added offscreen texture dependency graph.

FBO command submission requires texture dependencies to ensure that
syncing semaphores are correctly setup.

Added submission semaphore to RenderTarget

Detail
------
vulkan-command-buffer.h/.cpp
  - Added mRenderTarget member which is stored when Begin() cmd is issued on this buffer.
  - Added call to check texture dependencies when BindTextures() cmd is issued.
  - Added call to add texture dependencies when BeginRenderPass() cmd is issued.
  - Replaced GetLastSwapchain with GetRenderTarget() (More generic)

vulkan-framebuffer.h/.cpp
  - Added render pass generation

vulkan-framebuffer-attachment.h
  - Added AttachmentDescription to constructor/New

vulkan-framebuffer-impl.h/cpp
  - Add AddRenderPass() method
  - Changed Attachment constructor to copy AttachmentDescription if not null, or use "Clear"/"Store"
    for load / store ops, respectively.

vulkan-graphics-controller.h/cpp
  - Changed FrameStart() to reset texture dependency graph
  - Changed SubmitCommandBuffers() to submit for all render targets (not just swapchain)
  - Changed PresentRenderTarget() to only present swapchain target
  - Changed CreateRenderTarget() to add render target to dependency checker
  - Added CreateFramebuffer() implementation
  - Added texture dependency methods
    - AddTextureDependencies() goes thru each attachment of a render target and adds it to the
        dependency checker
    - CheckTextureDependencies() goes through each bound texture to determine if it needs to wait
        on a render target submission
    - RemoveRenderTarget() - removes render target from dependency graph

vulkan-render-pass-impl.h/cpp
  - Added new New factory fn to pass in CreateInfo
  - Separated out sub-pass dependency generation - different setup for offscreen/swapchain.
  - Storing attachment handles in CreateInfo

vulkan-render-target.h/.cpp
  - Added mSubmitSemaphore. This is used when Submitting offscreen command buffers to the graphics
    queue to handle synchronization between render targets - the submission can wait on dependent
    targets to finish drawing to offscreens.
  - Added Submit() method to submit for either offscreen / swapchain.

vulkan-swapchain-impl.h/.cpp
  - Changed to use Queue::Submit method

vulkan-texture.h
  - Addition of CreateImageView() which ensures the image is allocated, and creates
    separate image view using existing parameters.

vulkan-queue.h/cpp
  - Moved Submit() function from Device class to Queue as method.

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

Allowing arbitrary dumping of multi-frame command buffers

Updated to pass out the render target information at the end of
the json.

Change-Id: Iaf63b42f89ab014c99a8635a1288b4a9ea9c9f23

[AT-SPI] Fix deputy finding logic in GetNavigableAtPoint to include root

Change-Id: I42fb2eed656443c1e6ebf68f8ff9570968fce631

Merge "[AT-SPI] Fix GetNavigableAtPoint when the embedded widget has no accessibile object" into devel/master

Merge "Fill empty UniformBufferBindingDescriptor only required" into devel/master

Merge "[AT-SPI] Do not subtract extentOffset if it's a sub window of Widget" into devel/master

[AT-SPI] Fix GetNavigableAtPoint when the embedded widget has no accessibile object

Change-Id: I690114db34985912c996b9e9a8b2c8196011bc3e

Merge "[AT-SPI] Implement missing DeputyAccessible feature" into devel/master

Merge "Fix potential memory leak in bidirectional support." into devel/master

Merge "Fix Unicode Byte Order Mark was misinterpreted as arabic character." into devel/master

Add FullScreenMode to InputMethodContext

to support ECORE_IMF_INPUT_HINT_FULLSCREEN_MODE
/**< The input panel should be shown in fullscreen mode @since 1.28 */

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

[AT-SPI] Implement missing DeputyAccessible feature

This feature helps determine the result of a recursive GetNavigableAtPoint
operation. The recursion is done by the at-spi2-core library.

Change-Id: I86ca04cc1e20496703635cb5fd8293f84c677be2

Merge "[AT-SPI] Check if child is visible in scrollable parent" into devel/master

Fill empty UniformBufferBindingDescriptor only required

Until now, we try to reset 64 * 32 byte whenever we bind uniform buffers.
But usually we don't fully use 64 bindings for uniforms.

And also, NULL_DESCRIPTOTR could be replaced as memset(0), what we already doing
for standalone uniform buffer bindings.

To reduce those useless std::fill_n and memset,
let we call memset for binding only if we cannot assume the data will
be comes

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

[AT-SPI] Do not subtract extentOffset if it's a sub window of Widget

Change-Id: Ic377579fc9b45510b37e1612baf9b7925308f584

Fix potential memory leak in bidirectional support.

the leak occurs when DestroyInfo() called more than once with same index.

Change-Id: I103d0e74963b104a76e9a22695b992df3d9ae752

Fix Unicode Byte Order Mark was misinterpreted as arabic character.

/ufeff is Unicode BOM character.

Change-Id: I0cdcb738c7cefe259125bf481fc4cbb10baf8ec3

DALi Version 2.4.2

Change-Id: I9e3ae62f63668027fa043dd210c8da112c0bfe8f

[AT-SPI] Check if child is visible in scrollable parent

 Fix an issue where GetNeighbor function would incorrectly find
the next item in a gridview, even if child item was not actually visible.
 In a grid view, the next item may be located to the right of
the current item, even if it is not visible on the screen due to scrolling.
 Add IsChildVisibleInScroll to checking the next item is actually visible or not.
 This change improves the user experience by ensuring that the highlight function
always finds the correct next item, even in grid views.

Change-Id: Id9cbe3f97d29e7a8d5458a6b60e6d7f9ab0f27a6

Fix moving a reference with std::move in LRUCacheContainer

Change-Id: I2ffe44ab4d75bc64fc7903a303d60ed00e4f3db5

Merge "Prepare NativeTexture only once per each frames" into devel/master

Merge "Add AutoRotation, LetterBox" into devel/master

Prepare NativeTexture only once per each frames

Let we make ensure that GLES::Texture::Prepare() called
only 1 times per each frames.

It will guard to update native texture target multiple times
so EGLImage confused.

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

Merge "Fix coverity: Division by zero" into devel/master

Fix coverity: Division by zero

Change-Id: I4d59433bdce554f3625a58f326c63aa859f5c1ef

Fix coverity : Do not call move to const & and & type

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

Fix coverity : need to call bundle_free what we created by bundle_decode

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

Add forceChildSearch a11y attribute to enable searching for out-of-bounds child

Change-Id: Iaa66022ec2edecfd9600f11f3af78f0e68c58cb1

Merge "Fix font client coverity issue" into devel/master

Fix font client coverity issue

Remove unique_lock unlock to prevent double unlocking.

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

Add AutoRotation, LetterBox

Change-Id: If8b6a28a0249133f42dc28c3575c44f35097960c

DALi Version 2.4.1

Change-Id: Ibadcdb5dc654a303f7135bf3d987bea6b186f436

Update emoji variation sequences to Version 16.0

Change-Id: I86f91ac9fb539187e148580232cc25bab8f2c459

Print infomation if we create eglSurface from Native if need

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

a11y refactor: Remove mKnownObjects

mKnownObjects is redundant as we already store accessbles in a map in bridge-impl.
Removig the data and update the path calculation logic.wq

Change-Id: I40899257ab293f2afe0bcdf9c3a7791dbc4e88bc

Do not call RequestUpdateOnce, instead Deiconify

We can only assume that Deiconify signal can assume that window is valid.
Other API is just kind of request to window server, so we could not send
rendering forcibly.

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

Skip Rendering if native texture not prepared

We should not render the program if texture was not prepared.
Also, if CreateResource failed, mEglImageExtensions might be nullptr.
So we need to make a nullptr check if CreateResource failed, and try to
call other API, like PrepareTexture().

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

Merge "Add DumpTree api" into devel/master

Merge "DALi Version 2.4.0" into devel/master

DALi Version 2.4.0

Change-Id: I9a8de8ca55a3702fae786ddbe1f1d947db8c20d6

Support CURLOPT_MAXREDIRS as environment + Print downloaded buffer if image load failed

Let we add environment value to control CURLOPT_MAXREDIRS.
Sometimes the redirection count 5 might too small to some applications.
Let we allow to them use more deep depth.

+

Sometimes, we failed to get valid image buffer from given url.

To check whether that url download real-image or not,
let we print prefix of buffers.

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