Use Dali::Integration::TimeChecker feature instead duplicate macro (phase 2)

Let we use Dali::Integration::TimeChecker feature for ecore wl2

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

Merge "DALi Version 2.3.51" into devel/master

DALi Version 2.3.51

Change-Id: Ic03990fe054b43633bb70df3aa61f8f3c1826921

Merge "Shader Parser and SPIRV generator changes" into devel/master

Merge "Skip GPU that doesn't support Vulkan 1.3 or above" into devel/master

Merge "Ensured clear color vector is the right size" into devel/master

Skip GPU that doesn't support Vulkan 1.3 or above

Change-Id: I61964212e086c848160be3e477444a36ca42a808

Make ShaderPreCompiler as DALI_ADAPTOR_API

ShaderPreCompiler defined at dali-adaptor now.
Let we make matched keyword.

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

Merge "Merged descriptor sets for shared UBOs" into devel/master

Merge "Vulkan Tizen Build" into devel/master

Vulkan Tizen Build

Change-Id: Ia119b4914d1f958a6e9ab726dd96934f7f3bb93d

Shader Parser and SPIRV generator changes

- Assigning locations to in/out between stages when type is larger than 4 floats (mat2, mat3, mat4)
Example:

Vertex shader:
out (location=0) highp mat3 vMatrix;
out (location=3) highp float vNumber; // Location must be 3 not 1!

Fragment shader:
in (location=0) highp mat3 vMatrix;
in (location=3) highp float vNumber; // Location must be 3 not 1!

This patch doesn't deal with arrays and doesn't deal with input attributes in
vertex shader. It only resolves problem of default PBR shader.

- SPIRVGenerator::Generate() sates 'valid' flag correctly now (false on error)

Change-Id: I19ee9ac639baf30ef988a43c06ad6295f042c424

(ShaderParser) Fix coverity issue : don't copy std::string

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

Let we get ArrayElementStride by GL_UNIFORM_ARRAY_STRIDE

Since GL_UNIFORM_BLOCK_DATA_SIZE is implement-dependent value,
we'd better use more belief result from GetActiveUniformsiv with GL_UNIFORM_ARRAY_STRIDE

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

(ShaderParser) Fix compile error for #extension

There was compile error when #extension used before som other codes.

To avoid this kind of issue,
let we collect all extension prefix codes, and then define it after
version directly.

Relative logs :

  623.248 E/DALI    (P  854, T 6612): gles-graphics-shader.cpp: Compile(120) > Code: #version 320 es
  623.248 E/DALI    (P  854, T 6612): #ifdef GL_KHR_blend_equation_advanced
  623.248 E/DALI    (P  854, T 6612): #extension GL_KHR_blend_equation_advanced : enable
  623.248 E/DALI    (P  854, T 6612): #endif
  623.248 E/DALI    (P  854, T 6612): #if defined(GL_KHR_blend_equation_advanced) || __VERSION__>=320
  623.248 E/DALI    (P  854, T 6612):   layout(blend_support_all_equations) out;
  623.248 E/DALI    (P  854, T 6612): #endif
  623.248 E/DALI    (P  854, T 6612): //@name image-visual-shader.frag
  623.248 E/DALI    (P  854, T 6612): #define TEXTURE texture
  623.248 E/DALI    (P  854, T 6612): #define TEXTURE_CUBE texture
  623.248 E/DALI    (P  854, T 6612): #define TEXTURE_LOD textureLod
  623.248 E/DALI    (P  854, T 6612): #define TEXTURE_CUBE_LOD textureLod
  623.248 E/DALI    (P  854, T 6612): #define textureCube texture
  623.248 E/DALI    (P  854, T 6612): #define texture2D texture
  623.248 E/DALI    (P  854, T 6612): #define texture2DLod textureLod
  623.248 E/DALI    (P  854, T 6612): #define textureCubeLod textureLod
  623.248 E/DALI    (P  854, T 6612):
  623.248 E/DALI    (P  854, T 6612): //@version 100
  623.248 E/DALI    (P  854, T 6612): #extension GL_OES_EGL_image_external_essl3:require
  623.248 E/DALI    (P  854, T 6612):
  623.248 E/DALI    (P  854, T 6612): in mediump vec2 vTexCoord;
  623.248 E/DALI    (P  854, T 6612): #if defined(IS_REQUIRED_DEBUG_VISUAL_SHADER) || defined(IS_REQUIRED_ROUNDED_CORNER) || defined(IS_REQUIRED_BORDERLINE)
  623.248 E/DALI    (P  854, T 6612): in highp vec2 vPosition;
  623.248 E/DALI    (P  854, T 6612): flat in highp vec2 vRectSize;
  623.248 E/DALI    (P  854, T 6612): flat in highp vec2 vOptRectSize;
  623.248 E/DALI    (P  854, T 6612): flat in highp float vAliasMargin;
  623.248 E/DALI    (P  854, T 6612): #ifdef IS_REQUIRED_ROUNDED_CORNER
  623.248 E/DALI    (P  854, T 6612): flat in highp vec4 vCornerRadius;
  623.248 E/DALI    (P  854, T 6612): #endif
  623.248 E/DALI    (P  854, T 6612): #endif
  623.248 E/DALI    (P  854, T 6612): #ifdef IS_REQUIRED_DEBUG_VISUAL_SH
  623.248 E/DALI    (P  854, T 6612): gles-graphics-shader.cpp: Compile(121) > glCompileShader() failed:
  623.248 E/DALI    (P  854, T 6612): 0:19: P0001: Extension directive must occur before any non-preprocessor tokens

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

(ShaderParser) Optimize TokenizeLine function

1. regex is slow. Let we tokenize without regex.
2. Let we use std::string_view for GetToken() instead of string-creation.

After this patch, preprocess color-visual shader spend time reduced 95%
8.000 ms --> 0.372 ms (release mode)

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

Merge changes I179cfcd6,I6c702db7,I6994521e,I424c7a02 into devel/master

* changes:
  Print some log after preprocess finished
  Fix pre-compile works well after Preprocess.
  Make preprocessed shader include null-terminate keyword
  (ShaderPaser) Make flat qualifier define write ordering

Merge "Fix the memory leak of input panel wayland resource." into devel/master

Fix the memory leak of input panel wayland resource.

Fix the memory leak of wayland resource when input panel is used.
It is for tizen wayland backend.

Change-Id: I7e9bfc18e41dbf2dba9a6fa0a6b73b3ce30960e4

Print some log after preprocess finished

Let we print that preprocess finished to check the duration of preprocess

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

Fix pre-compile works well after Preprocess.

Since we don't insert the version of shader for pre-compile case,
Preprocess result of precompiled shader is not avalid.

To fix this issue, we should attach the version of shader.
For now, most of shaders in dali-toolkit use 100 version.
So let we use it forcibly.

TODO : We need to consider that Open ShaderData API,
to avoid duplicate implementation.
For now, just keep these duplicated logic.

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

Make preprocessed shader include null-terminate keyword

It will fix some kind of glCompileShader failed case.

ERROR: DALI: gles-graphics-shader.cpp: Compile(121) > glCompileShader() failed:
0:16(9): error: syntax error, unexpected NEW_IDENTIFIER

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

(ShaderPaser) Make flat qualifier define write ordering

Since flat keyword should be exist earlier than in / out keyword,
let we change the parsing method, and ensure
FLAT keyword works well.

Appendix:

Flat keyword spec for gles : https://docs.nvidia.com/drive/drive-os-5.2.0.0L/drive-os/DRIVE_OS_Linux_SDK_Development_Guide/baggage/GLSL_ES_Specification_3.20.withchanges.pdf
SPIRV don't use flat keyword at vertex shader (reddit search result) : https://www.reddit.com/r/vulkan/comments/yizy4i/whats_the_best_way_to_pass_an_index_from_a_vertex/?rdt=64516

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

Merge "Fix the leak of wayland resource." into devel/master

Merge "DALi Version 2.3.50" into devel/master

Fix the leak of wayland resource.

Fix the leak of wayland resource in tizen backend.
The related wayland resource is tizen_display_policy, tizen_policy and wl_registry.
It should be free before wl_event_queue is free.

Change-Id: I20bd835169154457c7f00f3d85bf4d7635fb639d

DALi Version 2.3.50

Change-Id: I9b404d4c5b883dedb6499ac173ff6e8a3fc1c9de

Fix compile error with thorvg by gcc-14

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

Merge "[AT-SPI] Add offset to GetNodeInfo API." into devel/master

Merge "Delete sync object at DiscardResource()" into devel/master

Ensured clear color vector is the right size

Change-Id: I461cebeb34b1c8ea04fef2febbd8d35b8c3b05d6

Merged descriptor sets for shared UBOs

If a uniform block is shared between shader stages, then
the descriptor set should have the StageFlags set to
both stage bits. There should only be 1 such descriptor set
in the descriptor set binding.

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

ShaderParser: Removed single line comments in tokenizer

Change-Id: I6b2bb9e8bf479eeb177781aec20ef885695a71c9

[AT-SPI] Add offset to GetNodeInfo API.

The offset is added to every GetExtents() result.
So add offset to GetExtents() result in GetNodeInfo.

Change-Id: Ifa903d8af8d8bfdcd2a191a99b556e1c2d290594

Delete sync object at DiscardResource()

Since we don't queueing Resource of sync object,
we should call delete for Discard cases.

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

Advanced blending prefix for GLES shaders

- Prefix is being added now when parsing GLES shaders
- Added shader prefix strings to ShaderParserInfo - prefixes are added
  after setting the #version.

Change-Id: If25ec5232054a713a6c8965c37a6091710689aa6

Merge "Associate DefaultLabel with actor" into devel/master

(Capture) Allow to re-start capture during finished signal

Some user wants to re-start capture inside of
CaptureFinishedSignal.

We already reset whole resources of source items at Start() API.
So it could be allowed to re-start the capture.

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

Associate DefaultLabel with actor

Change-Id: I9239f6ecf4ad619df83aa19c0dc0027a89e8b8b0

Applying native image to GLES shader

- Embedding extension string outside legacy-prefix block in modern shader
- Keeping current behaviour for legacy shaders
- Extension string depends on GLSL version returned by GL not on #version

Change-Id: I5db0a2d8688ec9614d80848df72fdc9e9fef2708

DALi Version 2.3.49

Change-Id: Ib8af59835a9315df83bfeb1b4ff7b82e057056a6

(AddOn Manager) dlclose for addon manage failed cases

Let we ensure to close dynamic library handles if we don't control.

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

Add KeyboardEventReceivedSignal to InputMethodContext

KeyboardEventReceivedSignal is the same as EventReceivedSignal.
The EventReceivedSignal combines different kinds of events triggered by the IMF into a single signal, which it then emits.
These events often happen during text entry through S/W keyboard and Applications depend on this signal to perform various actions.
There exists a situation where further emissions take place inside the emitted signal.
Unfortunately, these events are already exposed publicly and cannot be segregated separately.
As a workaround, we keep the original signal intact but separate signals meant exclusively for usage within dali.

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

Merge "Add for thorvg build guide at Ubuntu" into devel/master

Add for thorvg build guide at Ubuntu

Let we add some guideline to build & install thorvg at Ubuntu.

Also, check the version of thorvg to prepare API changeness

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

(Addon Manager) Add dlclose & show error logs instead of asserting

Change-Id: I151a341cb65a27b716eb4e97cc9d3fe92330dd4e

Merge "Shader parser changes" into devel/master

Shader parser changes

- GLSL dialect is verified with simple shader compilation
- if compilation fails we fall back to basic 300 es
- Parser has now 100, 300, 310 and 320 versions separated
- Version of GLSL is pulled from the GraphicsInterface (and cache)

Change-Id: I41875180db18dab511a3b7780910cf98b6d739c6

Merge "DALi Version 2.3.48" into devel/master

DALi Version 2.3.48

Change-Id: I8ba192ee716dfccf52671769f0ccf86d7182ee11

Return PixelBuffer::IsAlphaPreMultiplied() true if pixel format don't have alpha.

Let we make pixelbuffer is premultiplied if it doesn't have alpha channel
after request MultiplyColorByAlpha().

It will increase speed of rendering near 2% if we render RGB <-> RGBA format image repeatly.

See : https://confluence.sec.samsung.net/pages/viewpage.action?pageId=925598862

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

Consider that strdup failed case

Let we print some log and ignore shader parsing when strdup failed, due to
memory limits.

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

Merge "(AddOnManager) Add more error check for addon manager" into devel/master

Merge "Added matrix row stride to reflection" into devel/master

Merge "Cached depth/stencil state in command buffer" into devel/master

Merge "Make Capture Timer Start at Processor stage." into devel/master

(AddOnManager) Add more error check for addon manager

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

Merge "Skip SwitchBridge when dbus client is not ready." into devel/master

Merge changes Ib3e93f24,I2ea1ba1a,Icf43e46b into devel/master

* changes:
  Sending preprocessed code when fishing for samplers
  Shader parser supports duplicated uniform blocks
  ShaderParser changes

DALi Version 2.3.47

Change-Id: I6e1160785d4e5c1ee440b99a7f68f475e604d199

Added matrix row stride to reflection

The shader reflection needs to specify the row stride
for matrix3 types. This ensures that when mat3 is in
a uniform block that the padding is taken into account
when writing.

Rowstride is number of bytes that a row takes in memory.

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

Skip SwitchBridge when dbus client is not ready.

If DBusClient is not ready, don't remove initialize timer.

Change-Id: I24cf2993f80770ae137359e4c50454874235a8cd

Sending preprocessed code when fishing for samplers

Change-Id: Ib3e93f244860308549791e1b5d1bc8c8a6f24464

Shader parser supports duplicated uniform blocks

GLSL may redefine the same uniform block across multiple shader stages.
The GLSL for Vulkan needs to consider it and assign binding points correctly
to those blocks, otherwise the data won't be shared. GLES3 does that implicitly.

Change-Id: I2ea1ba1afd3fd4fb7578e53549535a317121c855

ShaderParser changes

- Support for textureCube() and textureLod() calls
- Old GLSL2 semantics can be used in modern shader same as gl_FragColor
- TEXTURE_CUBE, TEXTURE_LOD, TEXTURE_CUBE_LOD macros added

Change-Id: Icf43e46b5302d0f3ef284f38bbdb7ba73c39e432

Merge "Avoid print eglSwapBuffer trace logs for TV profile" into devel/master

(AddOn Manager) New API to load addons/libraries rather than via an ENV variable

Change-Id: I8cdf8ff46f78991214990dc26537209b4e40001d

Make Capture Timer Start at Processor stage.

 - If there is many works after Capture is requested, then it can emit false negative.
 - This patch is Cheery-picked patch from tizen patch,
   (https://review.tizen.org/gerrit/#/c/platform/core/uifw/dali-adaptor/+/319312/)

Change-Id: I424b1da8f57dfcbe94ee781e48d404c6b881b581
Signed-off-by: Seungho Baek <sbsh.baek@samsung.com>

Cached depth/stencil state in command buffer

Change-Id: I1bcfbea332ed381ba1850e6618d03bf0f36e87e9

Avoid print eglSwapBuffer trace logs for TV profile

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

Merge changes I82d8d719,I7ac1df5a into devel/master

* changes:
  Fixes to GLES3 shader processing pipeline
  Bug fixes and improvements.

DALi Version 2.3.46

Change-Id: I3fa01da7ffb853d74bcb428271166247aee542fe

Fixes to GLES3 shader processing pipeline

- Reflection assigns binding points implicitly
- GLES shader doesn't mix up @version and legacy #version

Change-Id: I82d8d719773263e4e567e606b5644aaf6b7e9132

Bug fixes and improvements.

- [Vulkan] fixed memory corruption when binding textures
- [Vulkan] fixed memory corruption when binding UBOs
- [Vulkan/GLES] Added API for obtaining device limitations
- [Vulkan/GLES] Added 'flat' keyword support to the parser

Change-Id: I7ac1df5abad042901eaefe41c1fdb0b19d835ac2

Merge "Adding depth buffer to swapchain" into devel/master

Prevent crash when a11y bridge is disabled

Change-Id: Iba2bd83d2623ead9e907cd1a6d0ffe4f9e4548db

Adding depth buffer to swapchain

Changed stencil funcs in graphics-api to better match
the api calls required in both GLES and VUlkan. (Rolled
SetStencilFunc and SetStencilOp into a single SetStencilState).

Added depth buffer creation to swapchain. This should invalidate
all pipelines, as the render pass will change, but we don't have
a pipeline cache yet...

Ensured pipeline counts color attachments of render pass when
setting up blending

Changed clip space matrix to not invert Z.

Change-Id: I5a461b86a8decb7148089b722bef4d226a1c1051

Fix crash on destroying static bridge obj

Change-Id: I3f7ab0fcf280dca3ddae9a7c553c82885d2a1299

DALi Version 2.3.45

Change-Id: I87b1e7df8b32d9518c30f317deb7ddef00ca979e

Merge "(RenderSurface/Graphics) Added Resize method to manage different GLES & Vulkan behaviour" into devel/master

Merge "Fix wrong variable name for window target" into devel/master

(RenderSurface/Graphics) Added Resize method to manage different GLES & Vulkan behaviour

Change-Id: I08d44294d303a487cc0d2306910d080f5582fb91

Merge "Changed vulkan startup order" into devel/master

Fix wrong variable name for window target

Change-Id: I54a74ef5effb473b83e7175041234335ab578876

Ensure we don't touch shutdowned EglGraphicsController

Let we make IsShuttingDown() API as static,
so we can ask it without raw pointer of graphics controller.

Most of API in GLES::Resource hold the eglController as l-value.
So if someone try to get graphics controller after shutting down,
it might make some crash.

To avoid these kind of issue, let we ensure to call DiscardResource
only if graphics controller is not in shutting down.

TODO : native image should be called DiscardResource() even if it is shutting
downed. How can we ensure it?

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

Changed vulkan startup order

Instead of creating a physical device, then creating a surface
and asserting if that device can't support the surface,
creates the vulkan surface, and checks against each device to see
if that surface can be presented.

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

(Build) Remove EGL/GLES library check if Vulkan is enabled

Change-Id: Id4a3e108b0b4ac76d5336159bcbfa121ee1a55c5

DALi Version 2.3.44

Change-Id: I495122c5270aac65be7c0ce7b254f612f5f40278

Merge changes I16ce7eff,Ic103c573,I651ed4d0 into devel/master

* changes:
  Changed vulkan-fence to use unique_ptr
  Vulkan render pass uses ref counted handle
  Vulkan resource lifecycle management

Merge "Add WebAuthDisplayQR, Response and MediaPermission" into devel/master

Changed vulkan-fence to use unique_ptr

Change-Id: I16ce7eff8ed73ab3395c1521d516a9edb3449b8f

Vulkan render pass uses ref counted handle

As we are now sharing render pass between framebuffers
of a swapchain, using a ref-counted handle will ensure
that we can clean this up safely.

Change-Id: Ic103c57314c39c2141883d688175664c8d30939d

Vulkan resource lifecycle management

For Vulkan implementation of graphics resources:
  Added discard queue, that's cleaned down during Flush().

Added ResourceWithDeleter as a secondary interface on top of ResourceBase, to allow
Program/Pipeline to inherit solely from ResourceBase, and not have to utilize the
same resource deleters that the main graphics objects require.

Removed VkResource inheritance completely from Impl classes.
Kept VkResource as VkSharedResource, as we may still use this on RenderPassImpl

All Impl classes now have a public Destroy method which is called from their
destructor, but may also be called earlier if needed. Note, CommandBuffers should
not be deleted directly, but instead returned to the command pool for re-use.

Implemented owned containers for some Impl objects using
std::unique_ptr. Framebuffer owns it's attachments, Swapchain owns
it's framebuffers.

Change-Id: I651ed4d041cd2fb3e7698304eeede66625031945

Make ShaderPrecompiler move shader list + Store shader prefix list as string instead string_view

Until now, Precompiler hold only string_view.
It mean, the lifecycle of shader code is not depend on that object.

To make sure that we could control the lifecycle of source code string,
let we keep it as std::string, instead of std::string_view.

And also, make we use std::move instead of copy them.
It will make less performance overhead.

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

Merge "Prevent revisiting nodes in GetMatches*" into devel/master

Prevent revisiting nodes in GetMatches*

Change-Id: I55c93244b1c4e1804c7e00baaca4fcf44d404718

Merge "Added an option to select the web engine type in the WebView constructor" into devel/master

Add WebAuthDisplayQR, Response and MediaPermission

Change-Id: Ia034489abd004c0080d772e84e0c797f5b30e985

DALi Version 2.3.43

Change-Id: I56fa73fb16275720c3717e6c30b5dd692720e3b3

Added an option to select the web engine type in the WebView constructor

- Previously, the web engine was fixed and selected for all DALi/NUI APPs in dali.sh, but now it can be chosen in the constructor.

Change-Id: I481e5ee48553c29141ecf3ef78ce7cd30f743267