fixed coverity issue

Change-Id: I14e7ced7fa0c1581454cafb46c404f191f163bd4

Merge "Removing spurious ; from shader macro code" into devel/master

Merge "Removed unused macro invocation from shader" into devel/master

Merge "Let Anti-Alias consider scale factor" into devel/master

Merge "Make reference count as signed integer" into devel/master

Merge "Remove build waring [-Wpessimizing-move] (don't need to std::move at return value)" into devel/master

Let Anti-Alias consider scale factor

Some days ago, we assume that the view which is smaller than 100x100 doesn't use
the range of anti-alias as 1.0.

(See https://review.tizen.org/gerrit/c/platform/core/uifw/dali-toolkit/+/279801)

But it make some visual defect when we use 56x56 size of view and scale it up.

Basically, the problem of these issues comes due to visuald don't know the
scale value.
So let we get actor's scale value as default shader property,
and then use it as anti-alias calculation parameter.

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

Make reference count as signed integer

Since the reference count can be negative if CANCEL cases, we should keep it
as signed integer value

(The patch https://review.tizen.org/gerrit/c/platform/core/uifw/dali-toolkit/+/300487 change as wrong integer type)

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

Specify operator as deleted as the implementation does not exist

Change-Id: Iee57165de6ee8c683b78459719035c4328d2afd6

(Scene3D) Cleaned up Doxygen tags in loader files

Change-Id: Ic59dced14afde154a8c8fa389ae26df063c00f23

Removed unused macro invocation from shader

Change-Id: I38953dd21347753de90b567f427b60dde0489637

Removing spurious ; from shader macro code

Change-Id: I0477a2791d4e56b80ebc1a24ef2b047daa4da901

Remove build waring [-Wpessimizing-move] (don't need to std::move at return value)

We don't need to use std::move for return value.

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

(Scene3D) Fix all Public API release versions

Change-Id: Ia47c4a603341400777ccdd37be788eb04a91a93c

Merge "Test harness sync" into devel/master

DALi Version 2.2.53

Change-Id: I0f978817151cd9e2e739c7c36a21b3021c47b63d

Merge "3D Scene Hit Testing" into devel/master

3D Scene Hit Testing

- New functions creates mesh using list of vertices and face indices to work with NUI.
- Added the functionality to detect which mesh is hit & emit signal

Requires C# binding and NUI Model.cs update

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

Merge "Added 2.3.0 support in the doxy file" into devel/master

Check missing accessible checker code

Fix svace issue

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

Allow bitmap 6byte utf8 glyph + Ignore build warning

The new warning option -Wstringop-overflow occured when we convert
from std::string to utf8 buffer.

Also, current dali support maximum 6 bytes utf8 glyph.
Let we also support them for BitmapFont case.

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

Added 2.3.0 support in the doxy file

Change-Id: I2cb3471159b559bd1eb5729d7b7d5a9fabd200bb

Test harness sync

Change-Id: I9044b20f53b2a13fa87eca58d84dd719d7958337

(A11y) Let we make some way to do not create new Accessible

Add some API that we will not create new Accessible() object anymore.

It will be useful when we ignore some Disposed view's accessible action.

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

Fix gcc-13 compile error (std::uint64_t not defined)

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

Fix coverity issue (invalid iterator access)

We can setup ImageType freely. If some user set invalid image type into
EncodedImageBuffer, it will be crash.

To avoid this situation, let we allow to set invalid image type, and
it will works like regular image type.

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

Merge "Remove Atlas parameter for TextureManager cache system" into devel/master

Merge "Implement AsyncTaskManager for toolkit UTC" into devel/master

Merge "(AnimatedVector) Cache lottie file's layer / marker info" into devel/master

Merge "Minor optimization for image visaul and texture manager" into devel/master

Merge "There is no function of the namespace defined in cpp, so it don't need the flag DALI_TOOLKIT_API" into devel/master

Merge "WayPointData is a struct defined in dali-toolkit/dali-scene3d/internal/algorithm/path-finder-waypoint-data.h" into devel/master

DALi Version 2.2.52

Change-Id: I1b00dd7c940c418fbeba87c71b93bd82ef5192f0

Implement AsyncTaskManager for toolkit UTC

Let we ignore Process() doesn't registered when we
check AsyncTask completed.

There was a lots of UTCs that randomly failed
due to the task completed and excuted at Process() timing.

Now we make async-task-manager triggered by event thread callback,
not include Process(). It will make sure that UTC who need to wait
async tasks completed is ensured.

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

Remove Atlas parameter for TextureManager cache system

If some visual attempt to atlas, they will use AtlasManager.
So there is no reason to keep atlas relative informations for
TextureInfo struct, what we are caching now.

+

This patch touch a lots of codes in texture-manager.
So, I also change something useless lvalue usages
Something like, const bool&

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

(AnimatedVector) Cache lottie file's layer / marker info

GetLayerInfo and GetMarkerInfo can be called usually if we use NUI system.
(Since we call Visual::Base::CreatePropertyMap() every times we change some
properties.)

But the LayerInfo / MarkerInfo never be changed if the file is fixed.
So let we cache that information, and use cached value.

It can also reduce mutex block at plugin system internally.

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

There is no function of the namespace defined in cpp, so it don't need the flag DALI_TOOLKIT_API

Change-Id: If6ff6ee18d4c6edc5573c9c473471b0f67c7742d

WayPointData is a struct defined in dali-toolkit/dali-scene3d/internal/algorithm/path-finder-waypoint-data.h

Change-Id: I61da87b6d9e792ba1bef80b87bb61bac359b7d71

Merge changes I1b1abc2c,I47083821,Icbe72d36,I6292beb0,I14b750ac, ... into devel/master

* changes:
  Test harness sync
  Limited joint/weight attributes to 4 sets
  Added vertex color test to model primitive
  Add macro defs to shader regen
  Adding debug to shader defs
  Updated model loader to handle multiple attr sets
  Switched joints to use UBO with larger size in PBR shader

Minor optimization for image visaul and texture manager

- Set texture index ordered bigger to smaller

Since we reserve the size of textureset internally,
set index bigger to smaller is more faster than else.

- Disconnect ObserverDestroyed what loadqueue connected

To avoid useless duplicated connection, let we disconnect
observer destroyed signal before process load queue.

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

Merge "(Vector) Support EncodedImageBuffer can use for vector image" into devel/master

Test harness sync

Change-Id: I1b1abc2c020803afec166955fd52b928f57530d8

Limited joint/weight attributes to 4 sets

Total number of supported joints per vertex = 16.

Change-Id: I4708382167d98f94b8d8f9fdf534972b85213c33

Added vertex color test to model primitive

Modifying model loading to reduce shader churn
Changed ShaderManager to return shader option
Added debug across shader generation

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

Add macro defs to shader regen

Change-Id: I6292beb044c8f634e5d528f465a8e16444bc58e4

Adding debug to shader defs

Change-Id: I14b750acdabb4ba6d96d9b1ab1ab259998e7e15f

Updated model loader to handle multiple attr sets

Modified GLTF parser for attributes to create attribute sets
for joints/weights/texcoords/colors

Added macro definitions to ShaderOptions, fixed hash to account for
macro/def strings.

Changed shader manager to create shader with extra attributes

Change-Id: Iabe2772eaf767ca3055dd415752c5a612942f0a4

Switched joints to use UBO with larger size in PBR shader

Change-Id: I816ff07f89cbbc1983dac65e6472892692f8d138

(AnimatedVector) Get marker informations

Let we make API to get marker informations from json file.
Marker information contains name, and pair of frame, start and end.

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

Merge "Do not trace image decoding process" into devel/master

Do not trace image decoding process

It will be re-open when we can control the log level of trace filter.

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

Remove observer from mLoadQueue when we request to remove observer

That might be happen during ResourceReady signal observing

- Add
- Remove

If than, the added request added in mLoadQueue 2 times.
and Remove request try to remove observe there (but will be failed actually.)

So, the visual get LoadCompleted signal twice.

It might have some logical problem in AnimatedImageVisual.
So let we keep ensure that 1-observer can only emit only 1-time.

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

(Vector) Support EncodedImageBuffer can use for vector image

Support to append extension when we use imagetype

For example, if EncodedImageBuffer has VECTOR_IMAGE type, we will generate url
"enbuf://3.svg" So we can use it directly by ImageView -> SvgVisual

And also, let we support Remote animated vector image.
(Since we can load animated vector image by raw buffer.)

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

Print image decoding process log level as release

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

Merge "Support glTF extention: KHR_mesh_quantization" into devel/master

Support glTF extention: KHR_mesh_quantization

Change-Id: I23fd7f3f481590f5716452813666cb31161bc8ae

Merge "DALi Version 2.2.51" into devel/master

DALi Version 2.2.51

Change-Id: Ib5f22d0063657d6992e74e19ee15c34234e216cc

Register some text property one time. + Shrink custom property count

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

Merge "Apply premultiply when external texture use mask" into devel/master

Merge "(AnimatedVector) Sync AnimatedVectorRenderer for UTC" into devel/master

Apply premultiply when external texture use mask

when external texture has alpha value and use mask, premultiply is not
applied.
add that patch to fix this issue

Change-Id: I9103c7a6185ea8c044a3fdfa00a5fc5a5376fe87

Merge "Fix unclear naming of member variable and function of FixedImageCache" into devel/master

(AnimatedVector) Sync AnimatedVectorRenderer for UTC

Apply newly added API - Load(const Dali::Vector<uint8_t>&) in toolkit utc

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

Fix unclear naming of member variable and function of FixedImageCache

 - Some naming of member variable and function have not changed after some logic is changed.
 - In current implementation, FixedImageCache not use a concept of front frame.
 - This patch fixed such a unclear naming and fixed parameters to match the naming too.

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

Add some more logs for images

 - Print the size of loaded image
 - Print url for SVG and Lottie
 - Change the name of VisualUrl property (for line coverage)

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

Merge "Remove the duplicated call with Focused call" into devel/master

Merge "Support NativeImageInterface can generate url" into devel/master

DALi Version 2.2.50

Change-Id: I1ccc9f25a084dbefb87acfd8325bb9b338606a57

Remove the duplicated call with Focused call

- To call `Activate()` of IME is already called in `OnKeyInputFocusGained()`
 and `OnKeyInputFocusGained()` is called by `OnTap()`.
- So, IME activate call is called twice unnecessarily.

- Also, IME properties did not be set in `OnTap`,
  so the normal layout of input panel shows first,
 and then another type of input panel shows again. (Issue)
- Fix this issue to erase to call `Activate` in `OnTap()`.

Change-Id: Ie18b35042b0b7878efd0a8fe3fbbd5da1d3e6def
Signed-off-by: Seoyeon Kim <seoyeon2.kim@samsung.com>

Support NativeImageInterface can generate url

Previously, we only support to generate url only by native image source.
But now, there was some usage to generate url from NativeImageQueue.

Unfortunatly, NativeImageQueue is devel api. So, let we just support
NativeImageInterface instead of NativeImageQueue directly.

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

Ensure blendshape weight between [0..1]

Let we ensure the blendshape weight is between 0 and 1.
So ignore cases when weight value is bigger than 1, less than 0.

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

Fix minor coverity issues (phase 2)

 - Move std::string
 - Move std::function
 - Move some Property::Map if possible

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

Fix randomly failed UTC (2)

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

Fix randomly failed UTC

- UtcDaliModelResourceCacheCheck
There was some logical error for that UTC.

- UtcDaliAnimatedImageVisualMultiImage01
It was possible that some AsyncTask can be processed
without event trigger. It is impossible to resolved now.
Until we make toolkit-async-task-manager, just block this UTC.

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

Merge "Remove Rasterize task when svg load failed" into devel/master

DALi Version 2.2.49

Change-Id: Iff66d95be44e2194a361751108e0173bfd691f2d

Remove Rasterize task when svg load failed

Since RasterizeTask's IsReady return false; that task will not be removed
forever.

To avoid this useless task keeping, let we remove that task when svg load
failed.

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

Merge "Fix ZWJ issue" into devel/master

Fix ZWJ issue

Added ZWJ sequence to solve the issue of ZWJ not working.
this patch processes ZWJ as one sequnce.
and this condition has been merged with the previous emoji logic.

Added exception handling in editable environment.
Removed unnecessary variables related to emoji.

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

Let we use default white IBL image even environment info not exist

When we use ModelNode without model url, we don't follow
environment map load task. So default IBL texture can be 'empty'.

In this case, the PBR shader what dali use default, might do some
strange behaviour.

To guard this cases, let we make all Dali::Scene3D::Model use
non-empty default textures.

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

Fix minor coverity issues

1. Remove std::string copy
2. Remove Property::Array copy

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

Merge "DALi Version 2.2.48" into devel/master

DALi Version 2.2.48

Change-Id: If68564f4b8690d7fdff4f5c3d59e586586c3b07e

Merge "Add color tag for text markup anchor" into devel/master

Add color tag for text markup anchor

"<a color='blue' clicked-color='red' href='https://www.tizen.org'>TIZEN</a>"

user can set color and clicked color in the anchor tag.
if not set, default color is applied.

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

Fix gcc-13 compile error (std::vector include)

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

Merge "Destroy removal visuals after idler" into devel/master

Merge "Keep reference when member callback excute + Make NPatchData as RefObject" into devel/master

Destroy removal visuals after idler

There was some bug when visual destructor called during
it's emit ResourceReady.

To avoid this case, Let we keep visuals more long term,
and Discard + Destroy after some idler called.

To avoid multiple Idler callback register,
let we make that idler callback in VisualFactory.

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

Merge "Add TextFitArray to text label" into devel/master

Keep reference when member callback excute + Make NPatchData as RefObject

Some CallbackBase didn't hold the reference of itself.
So it was possible to call destructor of itself during it's
API was running.

It might makes some unknown issues. So let we keep reference
for some issue-comes known APIs : SvgVisual, NPatchData, and lottie

It future, we should discard visuals rather than delate it directly.

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

Merge "Update test harness to use matching build system" into devel/master

Add TextFitArray to text label

Add a new function to satisfy the UX that
operates TextFit by considering the PointSize and MinLineSize of the text.

For example, TextFit should be able to find the one that
fits among [PointSize 24 + MinLineSize 40] or [PointSize 28 + MinLineSize 44].

But the previous TextFit only considers PointSize.

TextFitArray can solve this problem,
and this implemented it to respond to additional requirements without modifying the API.

Simple usage:
std::vector<DevelTextLabel::FitOption> fitOptions;
fitOptions.push_back(DevelTextLabel::FitOption(24, 40));
fitOptions.push_back(DevelTextLabel::FitOption(28, 44));
DevelTextLabel::SetTextFitArray(textLabel, true, fitOptions);

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

Safety check for control set/get property

Since some workerthread can access and try to get/set control's visual,
we might need to assert if worker thread try to use them.

Note : Current logic might got error if someone try to change API before
app create, but less care for now.

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

Merge "(WebView) Set display area and Change Visual when WebView relayout" into devel/master

DALi Version 2.2.47

Change-Id: I2b52489372e6b17b9cca2a4b6d587d2b91ac60cd

Update test harness to use matching build system

Change-Id: Ie78bf75496f7fb097afe487ad3a9650d9765ccf3

Merge "(visual-base-impl) Reduce cyclomatic complexity" into devel/master

DALi Version 2.2.46

Change-Id: I1a50e964f0a5bcb1a6d383e1a4cfe505976124e6