Fix code comments

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

Fix many svace issues at dali-toolkit (integer overflow + etc)

This is a combination of 3 commits.

Let we remove many cases of dali-toolkit when we can be call 0u - 1u, which is undefined behavior.

Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>
(Scene3D) Fix svace issue : convert uint32_t to std::streamoff + minor

Fix svace issue when we try to convert from 32bit unsigned integer
to (might) 64bit signed integer.

+

Fix the name of function collision issue

Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>
Fix svace issue : Avoid to divide by zero cases

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

Do not call malloc during terminate

There was some strange crash issue during application terminate.

That crash comes at malloc.c during destruct JsonParser.

So, let we just remove some potential of mallocate memory during terminate,
what might not neccessary.

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

Assume memory allocation failed cases

Let we consider malloc return null.

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

ImageVisualShader debug script make to add custom attribute / varying / uniform

Let we allow to set some extra uniform and attribute , varying if user need to debug.

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

DALi Version 2.3.13

Change-Id: I694a7fa17d2953cb73dea911f332f3276de1352d

(Automated Tests) Use github link instead of w3 for SVG image

Change-Id: I801f86b265f0101c5c906ccfcb221c477aee9484

(Vector) Change ConditionalWait as Mutex at lottie task

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

Change  precision of corner radius relative shader codes as highp

There was some visual defect when view size is near 20,000 in target.

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

Merge "Change precision of shader variable" into devel/master

Merge "DALi Version 2.3.12" into devel/master

DALi Version 2.3.12

Change-Id: I853019d9e13691b62c471ac789e010c101fca2d1

Fix Shadow crash issue during SceneOff

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

Change precision of shader variable

Change-Id: I4138a170c0ba9bea240a0fd0fcfab36afc2d182b
Signed-off-by: seungho baek <sbsh.baek@samsung.com>

Merge "Support Scene3D on gles 2.0" into devel/master

Merge "Fix bug when we try to load two image s.t. premultyOnLoad difference" into devel/master

Merge "Emit ResourceReady if we call Reload even if it was ready visual." into devel/master

Merge changes Ic9dba03a,Id5ec87c1 into devel/master

* changes:
  Print image process duration when we trace it
  Reduce the length of log during image trace

Support Scene3D on gles 2.0

Let we support Scene3D::Model under gles 2.0 devices.

Note : Since glsl doesnt support textureLod, we cannot support roughness.

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

Added UNSAFE_RENDERING_DIRECT backend mode for GLView.

UNSAFE_RENDERING_DIRECT mode allows injecting the GL calls
directly into window context. This method is considered unsafe
and should be used cautiously as it may alter GL state of the
main DALi pipeline (ie. affect UI rendering).

Change-Id: Idd36a26541f9af56657c1eef72ee7e83e2a45581
Signed-off-by: Adam Bialogonski <adam.b@samsung.com>

Print image process duration when we trace it

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

Reduce the length of log during image trace

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

Fix bug when we try to load two image s.t. premultyOnLoad difference

Let we seperate cache between
"Could-be-premultiplied, but not loaded yet" VS "Do not premultiplied"

Currently, we check both of them as same item.

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

Emit ResourceReady if we call Reload even if it was ready visual.

If the visual was already ready state, and if we call Reload, the visual state
still ResourceReady. So ResourceReady signal was not be emitted.

Since someone might need to know the timing of resource loaded after Reload(),
Make we reset status as PREPARE, and wait callback again.

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

Merge "DALi Version 2.3.11" into devel/master

DALi Version 2.3.11

Change-Id: Ifb9447b21d0f483c31d2658660f19c15694704a5

Merge "Ignore corner blur radius if glsl version is low" into devel/master

Test harness sync

Change-Id: I480e0fb3b1f3a475ea02ec7be1599f9af36a8f18

Merge "Let we make WebView invisible if window is hide" into devel/master

Merge "Make uShadowBias highp" into devel/master

Merge changes Iafce95d1,I701627ff,Ic2b33c53 into devel/master

* changes:
  Apply some missing shader names
  Add precompiled shader names
  Add name of shader what we generate as default

DALi Version 2.3.10

Change-Id: I86045f8c3be35213371476eac93fcc52d3b3b615

Make uShadowBias highp

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

Apply some missing shader names

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

Add precompiled shader names

Let we allow to use precompiled shader also use shader name

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

Add name of shader what we generate as default

Let we use the feature of shader name for our dali side default shaders.

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

Let we make WebView invisible if window is hide

Since we didn't check window visibility changed during webview is on scene
+
We don't check whether webview is on scene or not,

it could be make WebEngine keep rendering infinitly.

Let we make web engine visible only if we can assume that it is visible now.

TODO : We cannot check parent visibility changed for current system correctly,
without any overhead. Todo in future

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

Merge "Make model load task priority as low" into devel/master

Merge "Create Shadow RenderTask only shadow is required." into devel/master

Ignore corner blur radius if glsl version is low

Let we make to not blur rounded corner if gles version is too low

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

Make model load task priority as low

Since gltf model load spend a lots of time, let we make it priority low

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

Create Shadow RenderTask only shadow is required.

 - By using OrderIndex, Shadow RenderTask can be created only it is required.

Change-Id: I65a14ff4db167b1aa546be81f80238008322e7b5
Signed-off-by: seungho baek <sbsh.baek@samsung.com>

Fix wrong order of linear enumeration

Since Fitting mode have FIT_HEIGHT = 4 and FIT_WIDTH = 5, we need to follow up the order of enum

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

Merge "Test harness sync" into devel/master

DALi Version 2.3.9

Change-Id: I1800c69f074d795cb1252698c9aebb1fcd98f71e

Test harness sync

Change-Id: Ic3887917d576dc81ab4d252128dc832f2ecdfbc9

Merge "Fix decorator cursor size didn't changed issue" into devel/master

Fix decorator cursor size didn't changed issue

Since dali don't change the size during Relayout callback working,
the cursor size was not applied at 'current' relayout result.

So there was some issue that the cursor size applied well after 1 event loop working.

To avoid this issue, let we 'add' some relayout required actor to container.
It will call Relayout internally, at this loop.

TODO : Is there any other actor to be changed during relayout?

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

Fix editable text font size update issue

There is an issue that the font runs of the editable text area are separated
when the font size is updated.

If the editable text is an active state and the font size is updated,
the input font size should also be updated.

And due to the feature that changes the selected area's font size when the input font size is updated,
unintended font run separation and font size change occur if there is selected text when updating the font size.

This patch fixed so that the selection-related logic is not executed
when SetInputFontPointSize is called due to an update of the default font size.

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

Add editable text's idler function at PostProcessor time, instead of whenever we want

Since we need to remove whole queued style change signals even if idler add failed,
let we collect duplicated codes into single class, and make TextEditor / TextField just use it.

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

DALi Version 2.3.8

Change-Id: I0c69335ab01b58af356a99eb053d524d6cb75cef

Make we use valid geometry if broken image was n-patch and reloaded

It is possible that we use broken image not matched with given visual,
and then Reload API call it, and successed.

For this case, we need to change to use valid geometry again.

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

Merge "Call idle functions what we might be failed to add" into devel/master

Merge "Make ModelNode and Light as Control" into devel/master

Call idle functions what we might be failed to add

Since AddIdle could be failed at ecore side, we need to call given callback
synchronously, or need to print some error logs for some cases.

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

Merge "Make method to get VisualUrl shorter." into devel/master

Merge "(Scene3D) Guard cache impl will not be re-created at terminate case" into devel/master

Merge "(Scene3D) Optimize AnimatedProperty's KeyFrame values after load complete" into devel/master

Merge "(Vector) Use mutex instead of ConditionalWait for EventhThreadCallback" into devel/master

(Scene3D) Guard cache impl will not be re-created at terminate case

Since LifecycleController could be called first before Model / ModelCache destroy,
re-create the image resource cache at terminate time was possible.

Since we don't check the validation of this item, it will make crash.

So let we ensure that we call those API under same thread with adaptor.

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

Merge "Fix decorator color update issue" into devel/master

Make method to get VisualUrl shorter.

Let we allow to get only the last 20 character of URL name
when we don't need to print full URL.

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

Fix decorator color update issue

After a text decorator is created,
changing the related color causes only the property to be changed.
This patch fixes an issue where the actual actor's color was not changed.

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

Make ModelNode and Light as Control

Since we need to use some control features (like KeyboardFocus) let we make
ModelNode and Light as Control, instead of CustomActor.

TODO : We'd better make some universion Control for Scene3D.
For example, we should not allow to add Visual into ModelNode or Light.

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

(Vector) Use mutex instead of ConditionalWait for EventhThreadCallback

Since worker thread can call AddEventTriggerCallback freely,

we should make some lock between workerthread and
main thread(RemoveEventTriggerCallbacks, GetNextEventCallback, ~VectorAnimationThread).

ConditionalWait used only for AsyncTask rasterization, and sleep thread.
We'd better seperate those job locker.

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

(Scene3D) Optimize AnimatedProperty's KeyFrame values after load complete

Let we reduce the total key frame of MotionValue what we parse.

Since both json and bvh store the data by framerate.
So, there might be useless values are included.

If the number of keyframe is large, it might give some overhead to core.
So let we reduce useless keyframe for MotionValue and Model's internal animation.

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

Merge "DALi Version 2.3.7" into devel/master

DALi Version 2.3.7

Change-Id: I7a03929416d3d91b37b6c350885fd96bfb1c85b2

Test harness update.

"uColor" uniform added as first.
Custom uniforms are added before default uniforms in the reflection.

Change-Id: Ifb837b5e18c80afb94f24cf9372c5c60d840b477
Signed-off-by: Adam Bialogonski <adam.b@samsung.com>

Merge "Remove cubemap cache temperary + Fix bug when Uploaded PixelData resue after Texture GC" into devel/master

Merge "Change Property name USE_FIXED_CACHE to ENABLE_FRAME_CACHE" into devel/master

Remove cubemap cache temperary + Fix bug when Uploaded PixelData resue after Texture GC

If we create NewPixelDataWithReleaseAfterUpload and Upload to texture,
and if that texture reference count become 1 and GC,
the key of PixelData is not valid anymore.

To avoid this cases, Let we collect all PixelData container if some texture removed.
And also, since PixelData could be key of Texture cache, we should check the
reference count is less than 2.

+

Since current cubemap cache not works well reasonable, just remove it.
We will reopen it after we implement PixelDataList cache system again.

+

Print more log information if we need.

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

Merge "DALi Version 2.3.6" into devel/master

DALi Version 2.3.6

Change-Id: Ifc29588cd2df64ed6acb4412eeb9449a2e41d4ab

Merge "Fix bug that IBLFactor / Roughness not works well when we change material" into devel/master

Change Property name USE_FIXED_CACHE to ENABLE_FRAME_CACHE

existing name is ambiguous in meaning, so we change it to a more clear name.

Change-Id: I8a797dee0378ffbc6ab2547f63d9360f8a2da2b5

Merge "Fix Text Anchor don't work properly" into devel/master

Fix Text Anchor don't work properly

When the text component is vertical aligned as center or bottom,
The y offset was not calculated at anchor event.
Now, it works properly.

Change-Id: I31f2530122d28e07a32c8a75875a77bcc69353ee

Fix bug that IBLFactor / Roughness not works well when we change material

There was some bug when we change Material info, uMaxLOD and uIblIntensity does not applied what primitive has.

This is because Material's SetUniform API overwrite the value of those uniform.

This patch make we ensure the order of uniform register, so let we use correct LOD and IBL intensity.

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

Remove build warning at gcc-13

Remove build warning when we build dali-csharp-binder by gcc-13

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

Merge "Fix the 3D model rendering issue after the KHR_texture_transform extension support" into devel/master

Merge "Improve Textfit performance" into devel/master

Improve Textfit performance

We can assume point size of text and size of text's layout
are directly proportional.
So the size of the largest text point size
can be calculated by O(1) operations.
Previously, this was O(logn) operations.

Change-Id: Ie5aae28e949738ce0f67e389f3e920663f4e19bc

Fix svace issue

Change-Id: I2646aae8db36a5471915e6ee4101beef4a51c9ba

Merge "To prevent segmentation fault in AnimatedVectorImageVisual" into devel/master

Fix the 3D model rendering issue after the KHR_texture_transform extension support

Change-Id: I659fa20f8f18e6f6765c70c9c9ef031dd7e18ccb

[Scene3D] Let we allow to release PixelData memory after upload for 3D cache

Since we use PixelData as key of cached Texture, it might make some
CPU memory allocated even if we upload finished.

To avoid this behavior, let we make PixelData release memory
after upload to Texture for Scene3D case.

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

To prevent segmentation fault in AnimatedVectorImageVisual

Change-Id: I4c5c94eb9faeffd4727b5879254527692fe93be2
Signed-off-by: seungho baek <sbsh.baek@samsung.com>

Merge "DALi Version 2.3.5" into devel/master

DALi Version 2.3.5

Change-Id: I36cae7ee3679bc38ffacbcd0fce956920943dd3f

Merge "Make SceneView's SetResolution works well" into devel/master

Make SceneView's SetResolution works well

Let we change codes that might need to check whether the size of render target
is inputed by App side, by the API SetResolution.

We usually use it to reduce memory of SceneView FBO usage. So let we also make
ShadowMap size will relative with resolution.

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

Fix Rasterize not works when SetTransform is first time

It is possible that SVG load complete, and after than layout executed.

If then, mDefaultSize of SVG is not zero, and rasterize task not requested.

Unfortunatly, if fitting mode settup into image view, the required size
of resterize might not correct. So, ImageView try to re-layout once again
at it's OnResourceReady signal.

This patch make we send rasterize signal at least one time, So let we ensure
the ImageView resource ready signal comes at least one time.

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

Fix reorder warning on text-label

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

Fix TextFit do not work properly

When check for textfit, it doesn't distinguish between cases
where the layout does not change because text is too large
and the layout does not change because text is too small.

Now CheckForTextFit returns false when text is too large.

Change-Id: I03a414406ee336ec7bf24c8d067fc73379d119f7

Merge "Fix cursor position issue" into devel/master

Merge "Fixed Cache for AnimatedVectorImageVisual" into devel/master

Fix cursor position issue

This patch fixes an issue where the cursor position is incorrectly positioned
when padding is set for the textfield text editor.
It is a problem caused by the wrong location of the cursor layer.

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

Merge "Clear font cache when locale changed" into devel/master

Fixed Cache for AnimatedVectorImageVisual

Change-Id: I22bab282652f5377b95ce1d34db4564398be0ae8
Signed-off-by: seungho baek <sbsh.baek@samsung.com>